On Tue, 4 Mar 2025 03:41:10 GMT, Jeremy Wood <d...@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
