Review (Updated) : 4884238 : Constants for Standard Charsets

Alan Bateman Alan.Bateman at oracle.com
Tue Apr 19 08:14:33 UTC 2011


Mike Duigou wrote:
> :
>
> The latest webrev : http://cr.openjdk.java.net/~mduigou/4884238/2/webrev/
>
> Any other remaining feeback?
>
> Mike
Given that it now only defines the constants for the standard charsets 
then I think the new name is better, although with static import then it 
might not matter too much.

I'm not sure whether we should use this in the zip code as we don't want 
this to causes the lesser used charsets to be loaded. It might be worth 
double checking the startup.

The test claims to have been brewed in 2010. The "infrastructure" pasted 
in at the end is another reminder that we should have had a standard 
Assert/equivalent class in the test suite.

-Alan






More information about the core-libs-dev mailing list