> This adds support for parsing thumbnails in an APP1 Exif marker.
> 
> This builds on an unfinished proposal by Brian Burkhalter (around 2016). In 
> that previous work the only additional meta info he parsed was the image 
> creation time; this PR similarly includes the same property. (I can't speak 
> to why he included that property, but it looks like he has a lot of 
> experience with ImageIO so I trust his judgment.)
> 
> The test addresses the original images attached to the ticket plus a few 
> extra images I found on my computer that include unusual properties. 
> (Possibly those images are malformed, but if they exist in the wild and other 
> platforms support them then I'd prefer to support them too.)

Jeremy has updated the pull request incrementally with one additional commit 
since the last revision:

  8160327: fixing typo so `thumbnailPos` can be zero
  
  The `thumbnailLength` cannot be zero, but the position can be.

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/22898/files
  - new: https://git.openjdk.org/jdk/pull/22898/files/05ee149c..20d44cfb

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=22898&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22898&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/22898.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22898/head:pull/22898

PR: https://git.openjdk.org/jdk/pull/22898

Reply via email to