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.
dt_imageio_j2k_read_profile()
apparently needs to decode whole image. even after going through openjpeg sources, i see no way to replaceparameters.cp_limit_decoding = LIMIT_TO_MAIN_HEADER;
Caveat 2: at least in debian,Seems to be fixed by the last commitopenjpeg-1.5
installs<openjpeg.h>
into/usr/include
too.I think i have workarounded that, but if
libopenjpeg-dev
is installed side-by-side with newlibopenjp2-7-dev
, there may be issues (works for me right now)...openjp2-2.1
api is different (obviously) from old v1.5, do we want to support old `openjpeg-1.5 too? I guess can be done, but will be ugly.As far as i can tell, this is feature complete.
Exporting to {jp2,j2k} works, reading from works, reading ICC works.
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=826810
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=826805