On Mon, 30 Mar 2026 09:09:23 GMT, David Beaumont <[email protected]> wrote:

> it "Represents the module entries...". 

Do you mind looking for a different name? It is very confusing to have it be 
named ModuleReference when we already have a ModuleReference that is a 
reference a module's content. 

readNameOffsets is surprising to see here because it's a static method that 
doesn't have a ModuleReference its method signature.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29414#discussion_r3008564674

Reply via email to