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

Brent Christian brent.christian at oracle.com
Fri Dec 21 18:25:04 UTC 2012


On 12/21/12 7:33 AM, Alan Bateman wrote:
> 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.

Definitely agree.

> 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.

I wasn't quite sure where to put it.  Moved to java/lang/System 
(assuming you didn't actually mean java/util/System).

 > In ExpectedEncoding
> then it looks it is missing an exit in the bad usage case.

Thanks, fixed.

> Also as an
> alternative to the failed flag then you could just throw an exception.

When practical, I like to test and report all tested conditions (instead 
of failing fast).  I think it helps provide scope to a failure, just 
from looking at the test report.  Bit of a personal preference, I guess. 
  I can change it to fail fast if that's preferred.

I do think an exception stack trace is easier to spot to someone just 
glancing through test results.  So I now have the flag and an exception 
- hope that's okay. :)

Updated webrev:
http://cr.openjdk.java.net/~bchristi/8003228/webrev.02/

Thanks,
-Brent



More information about the core-libs-dev mailing list