Re: RFR(10): 8181147: JNU_GetStringPlatformChars should have a fast path for UTF-8

2017-06-15 Thread Claes Redestad
On 2017-06-15 18:50, Alan Bateman wrote: On 15/06/2017 17:26, Chris Hegarty wrote: : Claes, This is the first test in the core area that will now use a test specific native library, which will need to be built ( by the build system, which it does ), and pointed to when executing jtreg from

Re: RFR(10): 8181147: JNU_GetStringPlatformChars should have a fast path for UTF-8

2017-06-15 Thread Alan Bateman
On 15/06/2017 17:26, Chris Hegarty wrote: : Claes, This is the first test in the core area that will now use a test specific native library, which will need to be built ( by the build system, which it does ), and pointed to when executing jtreg from the command line, e.g. jtreg ... -nativepa

Re: RFR(10): 8181147: JNU_GetStringPlatformChars should have a fast path for UTF-8

2017-06-15 Thread Chris Hegarty
> On 15 Jun 2017, at 14:29, Claes Redestad wrote: > ... > http://cr.openjdk.java.net/~redestad/8181147/jdk.06/ Claes, This is the first test in the core area that will now use a test specific native library, which will need to be built ( by the build system, which it does ), and pointed to when

Re: RFR(10): 8181147: JNU_GetStringPlatformChars should have a fast path for UTF-8

2017-06-15 Thread Xueming Shen
On 6/15/17, 6:29 AM, Claes Redestad wrote: Hi Sherman, One nick picking is that it might be better to initialize the constant "LATIN1" from the String class when intializing String_coder_ID? recently I got a "naked constants" complain #JDK-8156530, so I guess it might be better to do so at y

Re: RFR(10): 8181147: JNU_GetStringPlatformChars should have a fast path for UTF-8

2017-06-15 Thread Claes Redestad
Hi Sherman, On 06/15/2017 05:27 AM, Xueming Shen wrote: Hi Claes, The change looks fine. Yes, encoding the 2-byete latin1 at native looks reasonable. Thanks for reviewing! One nick picking is that it might be better to initialize the constant "LATIN1" from the String class when intializi