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

[Mandy Chung]

 On 3/28/11 9:45 PM, Alan Bateman wrote:

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() ?

That's a good suggestion.  I will do that.

Otherwise the updated webrev looks good to me.

Thanks
Mandy