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

feat: add CLI flag to disable loading of gainmaps #264

Merged
merged 3 commits into from
Mar 9, 2025
Merged

Conversation

Tom94
Copy link
Owner

@Tom94 Tom94 commented Mar 9, 2025

Also overhauls loading of images: instead of checking for format compatibility in a separate step from loading, both are now handled in the same (ImageLoader::load) function. A special type of exception is instead thrown if the format is deemed to be unsupported, in which case the next loader in will be tried in order. This doesn't just simplify a whole bunch of logic but also avoids loading image data into memory twice when loading Ultra HDR JPEG images.

@Tom94 Tom94 force-pushed the optional-gainmap branch 3 times, most recently from 67289d0 to 1b39dee Compare March 9, 2025 09:24
Tom94 added 3 commits March 9, 2025 10:36
Also overhauls loading of images: instead of checking for format
compatibility in a separate step from loading, both are now handled in
the same (ImageLoader::load) function. A special type of exception is
instead thrown if the format is deemed to be unsupported, in which case
the next loader in will be tried in order. This doesn't just simplify a
whole bunch of logic but also avoids loading image data into memory
twice when loading Ultra HDR JPEG images.
@Tom94 Tom94 force-pushed the optional-gainmap branch from d686f61 to 86d503e Compare March 9, 2025 09:36
@Tom94 Tom94 merged commit dde8864 into master Mar 9, 2025
4 checks passed
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.

1 participant