On Mon, 21 Jul 2025 20:48:12 GMT, simon <d...@openjdk.org> wrote:

>>> Maybe it can be reused in another points?
>> 
>> ClassFileFormatVersion and Optional are more user-oriented classes; internal 
>> code don't really ever use these two types, so I don't think you need to 
>> extract this logic specifically.
>
> I mean about the rule the version - similar to isSupportedClassFileVersion 
> method

I still don't think so. I intentionally proposed the original RFE so we have 
one place where we can consolidate this handling, and that place should be here 
instead of some other place.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26406#discussion_r2220377268

Reply via email to