Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

struct RefMvsFrame::{rp,rp_ref}: Remove raw ptr fields and pass through the callstack #983

Merged
merged 2 commits into from
Apr 19, 2024

Conversation

kkysen
Copy link
Collaborator

@kkysen kkysen commented Apr 18, 2024

…_dav1d`, as this is the only callsite and we'll need to store stuff on the stack.
@kkysen kkysen force-pushed the kkysen/struct-RefMvsFrame-rp_ref-field-remove branch from 233f0ae to d3f252e Compare April 19, 2024 01:40
@randomPoison
Copy link
Collaborator

Looks like this needs a rebase, but I reviewed the last commit which seems to be the only one for this PR. Let me know if I missed any commits that need review though.

@kkysen
Copy link
Collaborator Author

kkysen commented Apr 19, 2024

Looks like this needs a rebase, but I reviewed the last commit which seems to be the only one for this PR. Let me know if I missed any commits that need review though.

Oops. It's the last two actually.

@kkysen kkysen force-pushed the kkysen/struct-Rav1dRefmvsDSPContext-wrapper-methods branch from 349218b to 6b4ef97 Compare April 19, 2024 19:44
Base automatically changed from kkysen/struct-Rav1dRefmvsDSPContext-wrapper-methods to main April 19, 2024 19:46
@kkysen kkysen merged commit 9ee877e into main Apr 19, 2024
@kkysen kkysen deleted the kkysen/struct-RefMvsFrame-rp_ref-field-remove branch April 19, 2024 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

struct refmvs_frame: Make fields safer without changing layout
2 participants