search

jai-imageio / jai-imageio-core

JAI ImageIO Core (without javax.media.jai dependencies)
Other
234 stars 87 forks source link

Add support for LSB-encoded TIFF images. #39

Closed lbellonda closed 6 years ago

lbellonda commented 7 years ago

Fix and tests for issue #37. Please, review the changes, since the impact on the LZW decoder can be huge.

SethChampagneNRL commented 6 years ago

Does this merge fully resolve issue #37? Is there a reason it has been ignored for almost a full year? I am currently experiencing the same issue with the TIFFLZWDecompressor.

stain commented 6 years ago

@SethChampagneNRL I don't know, but it looks like it, as it seems to add full support for LSB.

This issue has been more overseen than ignored. As the pull request only adds new code in the case of (previously broken) LSB-encoded files I think it's better to merge and see if it fully resolves #37.

SethChampagneNRL commented 6 years ago

I should mention that I used a local build of this branch and it resolved the NullPointerException issue I was having. I was able to read the compressed tiff. I appreciate the merge.