svn commit: r1843867 - /velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/SimpleNode.java

[cb...@apache.org]

Author: cbrisson
Date: Sun Oct 14 23:23:25 2018
New Revision: 1843867

URL: http://svn.apache.org/viewvc?rev=1843867&view=rev
Log:
[engine] Ensure PrintWriter is flushed in SimpleNode.dump(), and escape newlines

Modified:
    velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/SimpleNode.java

Modified: velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/SimpleNode.java
URL: http://svn.apache.org/viewvc/velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/SimpleNode.java?rev=1843867&r1=1843866&r2=1843867&view=diff
==============================================================================
--- velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/SimpleNode.java (original)
+++ velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/SimpleNode.java Sun Oct 14 23:23:25 2018
@@ -252,7 +252,7 @@ public class SimpleNode implements Node
         toString(String), otherwise overriding toString() is probably all
         you need to do. */
     /**
-     * @param prefix
+     * @param prefix display prefix
      * @return String representation of this node.
      */
     public String toString(String prefix)
@@ -277,7 +277,8 @@ public class SimpleNode implements Node
      * <p>Override {@link #dump(String, PrintWriter} if you want to customize
      * how the node dumps out its children.
      *
-     * @param prefix
+     * @param prefix display prefix
+     * @param out output print stream
      */
     public final void dump(String prefix, PrintStream out)
     {
@@ -295,7 +296,9 @@ public class SimpleNode implements Node
         {
             charset = Charset.defaultCharset();
         }
-        dump(prefix, new PrintWriter(new OutputStreamWriter(out, charset)));
+        PrintWriter pw = new PrintWriter(new OutputStreamWriter(out, charset));
+        dump(prefix, pw);
+        pw.flush();
     }
     
     /**
@@ -303,8 +306,8 @@ public class SimpleNode implements Node
      * <p>Override this method if you want to customize how the node dumps
      * out its children.</p>
      *
-     * @param Print
-     * @param prefix
+     * @param prefix display prefix
+     * @param out output print writer
      */
     public void dump(String prefix, PrintWriter out)
     {
@@ -494,7 +497,7 @@ public class SimpleNode implements Node
 
         for (Token t = getFirstToken(); t != null; )
         {
-            tokens.append("[").append(t.image).append("]");
+            tokens.append("[").append(t.image.replace("\n", "\\n")).append("]");
             if (t.next != null)
             {
                 if (t.equals(getLastToken()))