Avoid intermediate String allocation when cloning Characters nodes

Add a getBuffer() accessor to CharBufferNode so that deepCloneNode()
can copy directly from char[] to char[] via the Characters constructor,
instead of going through toString() and toCharArray().

Author: sideshowbarker

src/nu/validator/htmlparser/sax/SAXTreeBuilder.java

@@ -236,8 +236,9 @@ private void deepCloneNode(Node node, ParentNode destination) throws SAXExceptio
                 break;
             case CHARACTERS:
                 // Clone the characters
-                String text = node.toString();
-                Characters cloneChars = new Characters(null, text.toCharArray(), 0, text.length());
+                Characters srcChars = (Characters) node;
+                char[] buf = srcChars.getBuffer();
+                Characters cloneChars = new Characters(null, buf, 0, buf.length);
                 destination.appendChild(cloneChars);
                 break;
             default:

src/nu/validator/saxtree/CharBufferNode.java

@@ -50,6 +50,14 @@ public abstract class CharBufferNode extends Node {
         System.arraycopy(buf, start, buffer, 0, length);
     }
 
+    /**
+     * Returns the buffer.
+     * @return the buffer
+     */
+    public char[] getBuffer() {
+        return buffer;
+    }
+
     /**
      * Returns the wrapped buffer as a string.
      *