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

Reply via email to