> 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: replacing the "sony-d700" image
  
  The origins of that image were unknown, so we weren't sure if we had the 
rights to store it in the OpenJDK repo.
  
  I couldn't figure out how to create this kind of uncompressed thumbnail from 
an image editing app, so I spliced this new file together manually in a hex 
editor using the sony-d700 image as a blueprint.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/22898/files
  - new: https://git.openjdk.org/jdk/pull/22898/files/366a8c37..24b6feea

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

  Stats: 4 lines in 3 files changed: 1 ins; 0 del; 3 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