Review request for 6381464: "SimpleFormatter should use one single line format"
Alan Bateman
Alan.Bateman at oracle.com
Tue Mar 29 04:45:04 UTC 2011
Mandy Chung wrote:
>
> '\n' is a character added in the string created in L156 but not in a
> format string in which %n is used to represents a newline.
I'm not sure about this one as \n might not be the line separator on the
platform. After creating the PrintWriter then could you use pw.println() ?
Otherwise the updated webrev looks good to me.
-Alan
More information about the core-libs-dev
mailing list