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

[Brian Burkhalter]

On Sat, 22 Feb 2025 00:19:44 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> I don't know how to initiate a CSR. Is that something I can initiate, or does that require a more senior sponsor?
>> 
>> I'm not done with the test files yet. (I'll make a note in this PR when I am.)
>> 
>> Regarding sample files:
>> I replaced the jpeg file that embeds a JFIF *and* EXIF thumbnail. I still need to:
>> A. replace the jpeg file "exif-rgb-thumbnail" and
>> B. review this thread for other interesting test cases
>
>> I don't know how to initiate a CSR. Is that something I can initiate, or does that require a more senior sponsor?
> 
> I can do that, probably next week. We will need to update the JPEG metadata documentation.

> @bplb : 2. Do you still think a test case featuring an uncompressed RGB main image with a uncompressed RGB thumbnail is necessary?

I doubt it is necessary. I don't think the case is very common.

>     3. Should I add a test that includes an EXIF thumbnail followed by a JFIF thumbnail?

I do not think it is necessary. The JFIF-before-Exif case is already uncommon and I suspect that Exif-before-JFIF is even more so, if it is even possible.

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

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