Core changes for cpp rewrite #1724
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add additional API for querying information from the context for integration into the C++ code base. These should be reviewed prior to final release whether they need to persist or are only temporary Signed-off-by: Kimball Thurston <[email protected]>
Not sure when that got added to the C++ library, but wasn't in the core Signed-off-by: Kimball Thurston <[email protected]>
Long name attribute flags was not being set correctly and the rules around when there is a mismatch between the bit flags in the header and the type attribute were not being correctly handled Signed-off-by: Kimball Thurston <[email protected]>
Signed-off-by: Kimball Thurston <[email protected]>
Signed-off-by: Kimball Thurston <[email protected]>
Signed-off-by: Kimball Thurston <[email protected]>
meshula
reviewed
Apr 20, 2024
meshula
approved these changes
Apr 20, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These are changes to the core library needed for the C++ library to leverage the core more fully. Some of this might need to change prior to release, but should be fine to start
Website preview: https://openexr--1724.org.readthedocs.build/en/1724/