RFR (XS) 8232734 [TESTBUG] avoid using JDK symbols in ExtraSymbols.symbols.txt

Ioi Lam ioi.lam at oracle.com
Thu Oct 31 21:56:15 UTC 2019


Hi Calvin,

Thanks for the review.

I'll add two more test cases:

             appendSymbol(fw, ""); // empty string
             appendSymbol(fw, "Hello_\uffff");


The generated config file looks like this:

5 -1: Hello
0 -1:
7 -1: Hello_\x01
8 -1: Hello_\xc3\xbf
9 -1: Hello_\xe1\x88\xb4
9 -1: Hello_\xef\xbf\xbf
10 -1: NewSymbol0
10 -1: NewSymbol1
10 -1: NewSymbol2
.....


Thanks
- Ioi

On 10/31/19 12:27 PM, Calvin Cheung wrote:
> Hi Ioi,
>
> The removed ExtraSymbols.symbols.txt contains a zero-length symbol as 
> follows:
>
> 0 -1:
>
> Maybe the above could be added in ExtraSymbols.makeLotsExtraSymbols()?
>
> Looks good otherwise.
>
> thanks,
>
> Calvin
>
> On 10/30/19 7:33 PM, Ioi Lam wrote:
>> https://bugs.openjdk.java.net/browse/JDK-8232734
>> http://cr.openjdk.java.net/~iklam/jdk14/8232734-avoid-jdk-symbols-in-extra-symbols-test.v01/ 
>>
>>
>> ExtraSymbols.symbols.txt contains symbols that were once
>> extracted from the JDK. This has caused confusion when
>> developers are removing APIs from the JDK, and they are not
>> sure if they should also remove the corresponding symbols in
>> this file.
>>
>> In this patch, ExtraSymbols.symbols.txt is removed altogether. Instead,
>> a text file is generated at run time to cover the necessary cases.
>>
>> Tested with hs-tier1/2.
>>
>> Thanks
>> - Ioi
>>



More information about the hotspot-runtime-dev mailing list