RFR : 8003228 : (props) sun.jnu.encoding should be set to UTF-8 [macosx]

Alan Bateman Alan.Bateman at oracle.com
Fri Dec 21 15:33:02 UTC 2012


On 20/12/2012 22:20, Brent Christian wrote:
> Forgot a regtest - now included w/ v.01:
>
> http://cr.openjdk.java.net/~bchristi/8003228/webrev.01/
>
I think the change is okay, it would nice to have got an authoritative 
reference to give more confidence that this is the right thing but that 
doesn't seem to be possible.

I also think it's important that the folks on macosx-port-dev that have 
been complaining about this help us test it out, particularly folks that 
work in Asian locales. I don't know how many of them build the jdk so 
you might need to send a reminder once there is a jdk8 build available 
with this change. I don't think we should consider backporting this 
change to jdk7u until there is strong evidence that the change in jdk8 
is good.

Thanks for creating a test. I'm not sure that test/java/util/Properties 
is right place as that directory is for tests of the 
java.util.Properties API. I guess I would move it to java/util/System as 
there isn't an obvious place in the sun/** tree. In ExpectedEncoding 
then it looks it is missing an exit in the bad usage case. Also as an 
alternative to the failed flag then you could just throw an exception.

-Alan.



More information about the core-libs-dev mailing list