On Wed, 1 May 2024 11:18:36 GMT, Jan Lahoda <jlah...@openjdk.org> wrote:

> When writing the `ModuleMainClass` attribute, `CreateSymbols` uses an UTF8 
> entry (incorrect), instead of a Class_info (correct). This patch fixes that.

Looks good to me.

test/langtools/tools/javac/platform/createsymbols/CreateSymbolsTestImpl.java 
line 1104:

> 1102:         CreateSymbols.EXTENSION = ".class";
> 1103: 
> 1104:         deleteRecursively(ctSym);

Is double deletion needed for some kind of race condition safety?

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

Marked as reviewed by asotona (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19031#pullrequestreview-2034978645
PR Review Comment: https://git.openjdk.org/jdk/pull/19031#discussion_r1587160139

Reply via email to