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

mod obu: Cleanup error handling #611

Merged
merged 4 commits into from
Dec 12, 2023
Merged

mod obu: Cleanup error handling #611

merged 4 commits into from
Dec 12, 2023

Conversation

kkysen
Copy link
Collaborator

@kkysen kkysen commented Dec 7, 2023

No description provided.

@kkysen kkysen requested a review from randomPoison December 7, 2023 23:59
@kkysen kkysen force-pushed the kkysen/cleanup-fn-tile_log2 branch from 6bae1c3 to 6255e91 Compare December 8, 2023 00:04
@kkysen kkysen force-pushed the kkysen/mod-obu-error-handling branch from 4e2b18d to 109c9f5 Compare December 8, 2023 00:04
… except the on-error cleanup/logging, which is done in `fn rav1d_parse_obus`.

This is treated differently from `fn parse_{seq,frame}_hdr`,
as those had only single call sites and only did logging on error,
not semantic changes (the `fn rav1d_data_props_copy` call).
@kkysen kkysen force-pushed the kkysen/cleanup-fn-tile_log2 branch from 6255e91 to afa7c05 Compare December 8, 2023 02:52
@kkysen kkysen force-pushed the kkysen/mod-obu-error-handling branch from 109c9f5 to 3bf1344 Compare December 8, 2023 02:52
Base automatically changed from kkysen/cleanup-fn-tile_log2 to main December 12, 2023 21:30
src/obu.rs Show resolved Hide resolved
@kkysen kkysen merged commit 9f69208 into main Dec 12, 2023
18 checks passed
@kkysen kkysen deleted the kkysen/mod-obu-error-handling branch December 12, 2023 21:48
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.

2 participants