Please review -XshowSettings a java launcher option.

[Xueming Shen]

On 11/11/2010 16:42, Kumar Srinivasan wrote:
>> line 176, 188, 190-191, 195, and other lines in printPrintLocales and 
>> printLocale methods:
>> - the assignment to the buf and out variable to itself (returned from 
>> StringBuffer.append() method) is not necessary.
>
>
> Yes fixed, I missed these.

The "intention" of returning the StringBuffer itself for those append() 
methods is that
you can then write the code like


private static void printLocale(PrintStream ostream) {
    ostream.println(new StringBuilder("\n" + LOCALE_SETTINGS + "\n")
                        .append(INDENT)
                       .append("default locale: ")
                       .append(Locale.getDefault().getDisplayLanguage())
                       .append(prettyPrintLocales())
                       .toString());
}


One more nit is that you might want to do something special for

   line.separator =

to make it "readable"

-Sherman