On Thu, 18 Jun 2026 18:07:30 GMT, Alan Bateman <[email protected]> wrote:

>> Do you recall how ImageReader is handling this situation right now? Does it 
>> ignore such inconsistencies or throw eagerly or fail latently like 
>> ExplodedImage?
>> 
>> I'm hesitant to change ExplodedImage when we can't ensure parity with 
>> ImageReader.
>
>> I would expect us to detect and handle this earlier.
> 
> The only files in META-INF/preview will be the class files for the wrapper 
> classes, optional and java.time. There aren't any non-class resources. A bug 
> in the JDK build or something making changes outside of build could create 
> these weird scenario but I don't think we should worry about that now. We can 
> do improvements on this once the code is in main line.

Created https://bugs.openjdk.org/browse/JDK-8386905 to further investigate this 
potential parity in both contexts.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2545#discussion_r3438018569

Reply via email to