Review request for 6381464: "SimpleFormatter should use one single line format"

Alan Bateman Alan.Bateman at
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.


More information about the core-libs-dev mailing list