Hi Claes, Sherman,
Thanks for the reviews:
I reordered as the suggested the checks for encoding type to put utf-8,
most common first
in JNU_NewStringPlatform() and JNU_GetStringPlatformChars().
Webrev updated in place:
http://cr.openjdk.java.net/~rriggs/webrev-simplify-jnu-8198697/
Thanks, Roger
On 3/2/2018 12:03 PM, Xueming Shen wrote:
+1
On 03/02/2018 07:49 AM, Roger Riggs wrote:
Please review...
On 2/28/2018 9:45 AM, Roger Riggs wrote:
Hi,
In an effort to untangle some of the issues with property
initialization I was looking
at the platform encoding initialization and found a simplification.
Currently, the initialization occurs as a side effect of the first
call to JNU_NewStringPlatform and
involves a upcall to get sun.jnu.encoding from the system
properties. The value is cached for later use.
The native System.initProperties determines the platform specific
encoding via java_props_md.c and does an upcall to set the
sun.jnu.encoding system property, taking care to do it before the
first string that needs platform encoding.
The change directly initializes the platform encoding fast path
before it is needed to encode platform strings.
And moves the setting sun.jnu.encoding system property after the
command line arguments are inserted
keeping it from being overridden by -D on the command line that can
only confuse confusion
with code that later reads the property.
Please review and comment.
webrev:
http://cr.openjdk.java.net/~rriggs/webrev-simplify-jnu-8198697/
Issue:
https://bugs.openjdk.java.net/browse/JDK-8198697
Thanks, Roger
