RFR: 8160327: Support for thumbnails present in APP1 marker for JPEG [v10]

Brian Burkhalter bpb at openjdk.org
Mon Mar 10 17:13:12 UTC 2025


On Tue, 4 Mar 2025 03:41:10 GMT, Jeremy Wood <duke at openjdk.org> wrote:

>> 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.)~~
>> 
>> The images used in this test are contributed by Brian and me.
>
> Jeremy Wood has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8160327: adding "Reading Thumbnail Images" for Exif marker changes
>   
>   This is in response to:
>   https://github.com/openjdk/jdk/pull/22898#issuecomment-2695754262

> the full name on your profile does not match the author name in this pull requests' [HEAD](https://git.openjdk.org/jdk/pull/22898/commits/eb2283c671c08e08a0f9e65c8c78f61e2dd01215) commit

@mickleness I think this PR is ready to be integrated after which one of us will sponsor it. I don't know whether you want to fix the aforementioned conflict in author name.

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

PR Comment: https://git.openjdk.org/jdk/pull/22898#issuecomment-2711282281


More information about the client-libs-dev mailing list