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

extract stream decoding to Decoder classes #1321

Merged
merged 39 commits into from
Oct 4, 2024
Merged

Conversation

farindk
Copy link
Contributor

@farindk farindk commented Oct 1, 2024

Move actual bitstream decoding into Decoder classes that can decode data irrespective where the data is located in the file.

CMakeLists.txt Outdated
@@ -243,7 +243,7 @@ endif()
plugin_option(OpenJPEG_ENCODER "OpenJPEG J2K encoder" OFF ON)
plugin_option(OpenJPEG_DECODER "OpenJPEG J2K decoder" OFF ON)
if (WITH_OpenJPEG_ENCODER OR WITH_OpenJPEG_DECODER)
find_package(OpenJPEG 2)
find_package(OpenJPEG)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know why you do this (I do too), but probably not wanted in this PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I'll have to put that on .gitignore. No matter how much care I take, sooner or later it slips through :-)

farindk added 27 commits October 1, 2024 22:42
…image data anymore and the locking should be in Box_iloc::read_data() anyway
@farindk farindk marked this pull request as ready for review October 4, 2024 21:51
@farindk farindk merged commit 1a9e030 into master Oct 4, 2024
34 of 36 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.

2 participants