RFR: 8323815: Source launcher should find classes with $ in names [v2]

Christian Stein cstein at openjdk.org
Wed Jan 24 15:33:56 UTC 2024


On Wed, 24 Jan 2024 08:01:09 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

>> Christian Stein has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Include more intere'$'ting compilation units
>
> test/langtools/tools/javac/launcher/MultiFileSourceLauncherTests.java line 78:
> 
>> 76:                       public static void main(String... args) throws Exception {
>> 77:                         System.out.println(Class.forName("World$Core"));
>> 78:                         System.out.println(Class.forName("p.q.Unit$123$Fir$t$$econd"));
> 
> I think I would suggest two more tests:
> - one that would (in a separate run) lookup `p.q.Unit$123$Fir$t` (to ensure the search is broad enough)
> - one with a package name with `$`.

Added both tests a suggested.

The first is not test in the separate run, but a separate top-level compilation unit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17526#discussion_r1465095187


More information about the compiler-dev mailing list