commons-imaging icon indicating copy to clipboard operation
commons-imaging copied to clipboard

IMAGING-310 JpegImageParser.getImageInfo() Refactor colorType logic to first look at numberOfComponents

Open jephillips34 opened this issue 3 years ago • 3 comments

fix for IMAGING-310 JpegImageParser: Grayscale JPEG file with app14Segment returns ColorType.UNKNOWN

jephillips34 avatar Aug 17 '21 22:08 jephillips34

Thanks for the pull request and sorry for taking so long to review it @jephillips34 . Will try to get on it in the next days :+1:

kinow avatar Sep 04 '21 13:09 kinow

Coverage Status

Coverage remained the same at 76.839% when pulling fcfafb6efebb52d5f4566344970ba551350b5c69 on jephillips34:IMAGING-310-JpegImageParser-colorType into b6b4e9c739ffc22fa0cdad3d54dd203682a47960 on apache:master.

coveralls avatar Sep 12 '21 21:09 coveralls

Rebased onto master and squashed commits. Let's start reviewing it :)

kinow avatar Sep 12 '21 21:09 kinow

Closing: No reply.

garydgregory avatar Dec 23 '23 16:12 garydgregory