On Wed, 30 Oct 2024 12:49:16 GMT, Jan Lahoda <jlah...@openjdk.org> wrote:

> Do I miss something?

Okay, so this must have been modified at some point to add/use 
isSupportedModuleDescriptorVersion, in which case readModuleAttribute won't be 
called. In that case, what you have is okay.

Minor nit, can you fix up the code style in the modified expression as it has 
ended up with && at the start and end of lines which looks a bit messy.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21431#discussion_r1822617486

Reply via email to