On Tue, 27 Jan 2026 14:00:43 GMT, Jorn Vernee <[email protected]> wrote:

>> The `this.md` field is initialized (by `checkModuleDescriptor(MODULE_INFO);` 
>> from within the constructor) to the "base" compiled module descriptor. It 
>> also includes this comment:
>> 
>>> ```
>>>                // Initialize the base md if it's not yet. A "base" md can 
>>> be either the
>>>                // root module-info.class or the first versioned 
>>> module-info.class
>>> ```
>> 
>> With that in place, and the consistency checks being performed in 
>> `checkModuleDescriptor(MODULE_INFO);`, we are good.
>
> I mean for cases where there are multiple module-info.class files. The 
> automatic module name check only checks the first one, right?

Right.

> I mean for cases where there are multiple `module-info.class` files.

Which are checked for a single name separately. Thus, comparing an automatic 
module name with the name from the first compiled module is sufficient.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29316#discussion_r2732246538

Reply via email to