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

Error callback function for RgbaInputFile #1951

Open
vmsrc opened this issue Jan 8, 2025 · 1 comment
Open

Error callback function for RgbaInputFile #1951

vmsrc opened this issue Jan 8, 2025 · 1 comment

Comments

@vmsrc
Copy link

vmsrc commented Jan 8, 2025

It would be nice if we could override the error callback handler for RgbaInputFile.
The default error callback dumps error messages on stderr and that might be undesirable.
For most of the InputFile classes it is already possible to set custom error callback by constructing with ContextInitializer, for example:
MultiPartInputFile ifile( fileName, ContextInitializer() .silentHeaderParse (true) .strictHeaderValidation (false) .disableChunkReconstruction(false) .setErrorHandler(customExrErrorHandlerCb) );
Such constructor is not implemented for RgbaInputFile, AcesInputFile and possibly other classes.

@kdt3rd
Copy link
Contributor

kdt3rd commented Jan 9, 2025

yes, now that we have replaced the lower level reading functions, the next release of openexr will swap out the writing classes, and then the "other" / secondary classes like rgba input and aces after that. Given we are in a mode of simplifying and modernising, we are making one layer of changes at a time to make sure we are able to quickly react if we break something. If you want to propose a patch to pass through the context to the multipart input file internally, happy to accept that, but otherwise just haven't got to it yet.

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

No branches or pull requests

2 participants