nokogiri 1.7.2-java → 1.8.0-java

data/ext/java/nokogiri/internals/NokogiriNamespaceContext.java

@@ -33,11 +33,11 @@
 package nokogiri.internals;
 
 import java.util.ArrayList;
-import java.util.Hashtable;
+import java.util.HashMap;
 import java.util.Iterator;
-import java.util.List;
-import java.util.Set;
+import java.util.Map;
 import java.util.Map.Entry;
+import java.util.Set;
 
 import javax.xml.XMLConstants;
 import javax.xml.namespace.NamespaceContext;
@@ -48,28 +48,20 @@ import javax.xml.namespace.NamespaceContext;
  * @author Yoko Harada <yokolet@gmail.com>
  *
  */
+public final class NokogiriNamespaceContext implements NamespaceContext {
 
-public class NokogiriNamespaceContext implements NamespaceContext {
-    private static NokogiriNamespaceContext namespaceContext;
 	public static final String NOKOGIRI_PREFIX = "nokogiri";
     public static final String NOKOGIRI_URI = "http://www.nokogiri.org/default_ns/ruby/extensions_functions";
     public static final String NOKOGIRI_TEMPORARY_ROOT_TAG = "nokogiri-temporary-root-tag";
     
-    private Hashtable<String,String> register;
+    private final Map<String,String> register;
 
     public static NokogiriNamespaceContext create() {
-        if (namespaceContext == null) namespaceContext = new NokogiriNamespaceContext();
-        try {
-            NokogiriNamespaceContext clone = (NokogiriNamespaceContext) namespaceContext.clone();
-            return clone;
-        } catch (CloneNotSupportedException e) {
-            NokogiriNamespaceContext freshContext = new NokogiriNamespaceContext();
-            return freshContext;
-        }
+        return new NokogiriNamespaceContext();
     }
     
     private NokogiriNamespaceContext() {
-        this.register = new Hashtable<String,String>();
+        register = new HashMap<String, String>(6, 1);
         register.put(NOKOGIRI_PREFIX, NOKOGIRI_URI);
         register.put("xml", "http://www.w3.org/XML/1998/namespace");
         register.put("xhtml", "http://www.w3.org/1999/xhtml");
@@ -95,21 +87,19 @@ public class NokogiriNamespaceContext implements NamespaceContext {
     public String getPrefix(String uri) {
     	if (uri == null) {
             throw new IllegalArgumentException("uri is null");
-        } else {
-            Set<Entry<String, String>> entries = register.entrySet();
-            for (Entry<String, String> entry : entries) {
-                if (uri.equals(entry.getValue())) {
-                    return entry.getKey();
-                }
+        }
+        Set<Entry<String, String>> entries = register.entrySet();
+        for (Entry<String, String> entry : entries) {
+            if (uri.equals(entry.getValue())) {
+                return entry.getKey();
             }
         }
         return null;
     }
 
     public Iterator<String> getPrefixes(String uri) {
-        if (register == null) return null;
         Set<Entry<String, String>> entries = register.entrySet();
-        List<String> list = new ArrayList<String>();
+        ArrayList<String> list = new ArrayList<String>(entries.size());
         for (Entry<String, String> entry : entries) {
             if (uri.equals(entry.getValue())) {
                 list.add(entry.getKey());
@@ -119,12 +109,12 @@ public class NokogiriNamespaceContext implements NamespaceContext {
     }
     
     public Set<String> getAllPrefixes() {
-        if (register == null) return null;
         return register.keySet();
     }
 
     public void registerNamespace(String prefix, String uri) {
-        if("xmlns".equals(prefix)) prefix = "";
-        this.register.put(prefix, uri);
+        if ("xmlns".equals(prefix)) prefix = "";
+        register.put(prefix, uri);
     }
+
 }

data/ext/java/nokogiri/internals/NokogiriXPathFunction.java

@@ -32,21 +32,20 @@
 
 package nokogiri.internals;
 
-import static nokogiri.internals.NokogiriHelpers.getNokogiriClass;
-
 import java.util.List;
 
 import javax.xml.xpath.XPathFunction;
 import javax.xml.xpath.XPathFunctionException;
 
-import nokogiri.NokogiriService;
 import nokogiri.XmlNode;
 import nokogiri.XmlNodeSet;
 
 import org.jruby.Ruby;
 import org.jruby.RubyArray;
 import org.jruby.RubyBoolean;
+import org.jruby.RubyFixnum;
 import org.jruby.RubyFloat;
+import org.jruby.RubyInteger;
 import org.jruby.RubyString;
 import org.jruby.javasupport.JavaUtil;
 import org.jruby.javasupport.util.RuntimeHelpers;
@@ -61,81 +60,64 @@ import org.w3c.dom.NodeList;
  * @author Yoko Harada <yokolet@gmail.com>
  */
 public class NokogiriXPathFunction implements XPathFunction {
-    private static NokogiriXPathFunction xpathFunction;
-    private IRubyObject handler;
-    private String name;
-    private int arity;
+
+    private final IRubyObject handler;
+    private final String name;
+    private final int arity;
     
     public static NokogiriXPathFunction create(IRubyObject handler, String name, int arity) {
-        if (xpathFunction == null) xpathFunction = new NokogiriXPathFunction();
-        try {
-            NokogiriXPathFunction clone = (NokogiriXPathFunction) xpathFunction.clone();
-            clone.init(handler, name, arity);
-            return clone;
-        } catch (CloneNotSupportedException e) {
-            NokogiriXPathFunction freshXpathFunction = new NokogiriXPathFunction();
-            freshXpathFunction.init(handler, name, arity);
-            return freshXpathFunction;
-        }
+        return new NokogiriXPathFunction(handler, name, arity);
     }
 
-    private void init(IRubyObject handler, String name, int arity) {
+    private NokogiriXPathFunction(IRubyObject handler, String name, int arity) {
         this.handler = handler;
         this.name = name;
         this.arity = arity;
     }
 
-    private NokogiriXPathFunction() {}
-
     public Object evaluate(List args) throws XPathFunctionException {
-        if(args.size() != this.arity) {
+        if (args.size() != this.arity) {
             throw new XPathFunctionException("arity does not match");
         }
         
-        Ruby ruby = this.handler.getRuntime();
-        ThreadContext context = ruby.getCurrentContext();
+        final Ruby runtime = this.handler.getRuntime();
+        ThreadContext context = runtime.getCurrentContext();
 
-        IRubyObject result = RuntimeHelpers.invoke(context, this.handler,
-                this.name, fromObjectToRubyArgs(args));
+        IRubyObject result = RuntimeHelpers.invoke(context, this.handler, this.name, fromObjectToRubyArgs(runtime, args));
 
-        return fromRubyToObject(result);
+        return fromRubyToObject(runtime, result);
     }
 
-    private IRubyObject[] fromObjectToRubyArgs(List args) {
+    private static IRubyObject[] fromObjectToRubyArgs(final Ruby runtime, List args) {
         IRubyObject[] newArgs = new IRubyObject[args.size()];
         for(int i = 0; i < args.size(); i++) {
-            newArgs[i] = fromObjectToRuby(args.get(i));
+            newArgs[i] = fromObjectToRuby(runtime, args.get(i));
         }
         return newArgs;
     }
 
-    private IRubyObject fromObjectToRuby(Object o) {
+    private static IRubyObject fromObjectToRuby(final Ruby runtime, Object obj) {
         // argument object type is one of NodeList, String, Boolean, or Double.
-        Ruby runtime = this.handler.getRuntime();
-        if (o instanceof NodeList) {
-            XmlNodeSet xmlNodeSet = (XmlNodeSet)NokogiriService.XML_NODESET_ALLOCATOR.allocate(runtime, getNokogiriClass(runtime, "Nokogiri::XML::NodeSet"));
-            xmlNodeSet.setNodeList((NodeList) o);
+        if (obj instanceof NodeList) {
+            XmlNodeSet xmlNodeSet = XmlNodeSet.create(runtime);
+            xmlNodeSet.setNodeList((NodeList) obj);
             return xmlNodeSet;
-        } else {
-            return JavaUtil.convertJavaToUsableRubyObject(runtime, o);
         }
+        return JavaUtil.convertJavaToUsableRubyObject(runtime, obj);
     }
 
-    private Object fromRubyToObject(IRubyObject o) {
-        Ruby runtime = this.handler.getRuntime();
-        if(o instanceof RubyString) {
-            return o.toJava(String.class);
-        } else if (o instanceof RubyFloat) {
-            return o.toJava(Double.class);
-        } else if (o instanceof RubyBoolean) {
-            return o.toJava(Boolean.class);
-        } else if (o instanceof XmlNodeSet) {
-            return (NodeList)o;
-        } else if (o instanceof RubyArray) {
-            XmlNodeSet xmlNodeSet = XmlNodeSet.newXmlNodeSet(runtime.getCurrentContext(), (RubyArray)o);
-            return (NodeList)xmlNodeSet;
-        } else /*if (o instanceof XmlNode)*/ {
-            return ((XmlNode) o).getNode();
+    private static Object fromRubyToObject(final Ruby runtime, IRubyObject obj) {
+        if (obj instanceof RubyString) return obj.asJavaString();
+        if (obj instanceof RubyBoolean) return obj.toJava(Boolean.class);
+        if (obj instanceof RubyFloat) return obj.toJava(Double.class);
+        if (obj instanceof RubyInteger) {
+            if ( obj instanceof RubyFixnum ) return RubyFixnum.fix2long(obj);
+            return obj.toJava(java.math.BigInteger.class);
+        }
+        if (obj instanceof XmlNodeSet) return obj;
+        if (obj instanceof RubyArray) {
+            return XmlNodeSet.newXmlNodeSet(runtime, (RubyArray) obj);
         }
+        /*if (o instanceof XmlNode)*/ return ((XmlNode) obj).getNode();
     }
 }

data/ext/java/nokogiri/internals/NokogiriXPathFunctionResolver.java

@@ -44,25 +44,22 @@ import org.jruby.runtime.builtin.IRubyObject;
  * @author sergio
  * @author Yoko Harada <yokolet@gmail.com>
  */
-public class NokogiriXPathFunctionResolver implements XPathFunctionResolver {
-    private static NokogiriXPathFunctionResolver resolver;
+public final class NokogiriXPathFunctionResolver implements XPathFunctionResolver {
+
     private IRubyObject handler;
     
     public static NokogiriXPathFunctionResolver create(IRubyObject handler) {
-        if (resolver == null) resolver = new NokogiriXPathFunctionResolver();
-        try {
-            NokogiriXPathFunctionResolver clone = (NokogiriXPathFunctionResolver) resolver.clone();
-            clone.setHandler(handler);
-            return clone;
-        } catch (CloneNotSupportedException e) {
-            NokogiriXPathFunctionResolver freshResolver = new NokogiriXPathFunctionResolver();
-            freshResolver.setHandler(handler);
-            return freshResolver;
-        }
+        NokogiriXPathFunctionResolver freshResolver = new NokogiriXPathFunctionResolver();
+        freshResolver.setHandler(handler);
+        return freshResolver;
     }
     
     private NokogiriXPathFunctionResolver() {}
-    
+
+    public final IRubyObject getHandler() {
+        return handler;
+    }
+
     public void setHandler(IRubyObject handler) {
         this.handler = handler;
     }

data/ext/java/nokogiri/internals/NokogiriXPathVariableResolver.java

@@ -42,18 +42,11 @@ import javax.xml.xpath.XPathVariableResolver;
  * @author Yoko Harada <yokolet@gmail.com>
  */
 public class NokogiriXPathVariableResolver implements XPathVariableResolver {
-    private static NokogiriXPathVariableResolver resolver;
-    private HashMap<QName,String> variables = new HashMap<QName,String>();
+
+    private final HashMap<QName,String> variables = new HashMap<QName,String>();
 
     public static NokogiriXPathVariableResolver create() {
-        if (resolver == null) resolver = new NokogiriXPathVariableResolver();
-        try {
-            NokogiriXPathVariableResolver clone = (NokogiriXPathVariableResolver) resolver.clone();
-            return clone;
-        } catch (CloneNotSupportedException e) {
-            NokogiriXPathVariableResolver freshResolver = new NokogiriXPathVariableResolver();
-            return freshResolver;
-        }
+        return new NokogiriXPathVariableResolver();
     }
     
     private NokogiriXPathVariableResolver() {}

data/ext/java/nokogiri/internals/ParserContext.java

@@ -98,8 +98,7 @@ public class ParserContext extends RubyObject {
           return;
 
         RubyString stringData = null;
-        if (invoke(context, data, "respond_to?",
-                   ruby.newSymbol("to_io").to_sym()).isTrue()) {
+        if (invoke(context, data, "respond_to?", ruby.newSymbol("to_io")).isTrue()) {
             RubyIO io =
                 (RubyIO) TypeConverter.convertToType(data,
                                                      ruby.getIO(),
@@ -107,20 +106,17 @@ public class ParserContext extends RubyObject {
             // use unclosedable input stream to fix #495
             source.setByteStream(new UncloseableInputStream(io.getInStream()));
 
-        } else if (invoke(context, data, "respond_to?",
-                          ruby.newSymbol("read").to_sym()).isTrue()) {
+        } else if (invoke(context, data, "respond_to?", ruby.newSymbol("read")).isTrue()) {
             stringData = invoke(context, data, "read").convertToString();
 
-        } else if (invoke(context, data, "respond_to?",
-                          ruby.newSymbol("string").to_sym()).isTrue()) {
+        } else if (invoke(context, data, "respond_to?", ruby.newSymbol("string")).isTrue()) {
             stringData = invoke(context, data, "string").convertToString();
 
         } else if (data instanceof RubyString) {
             stringData = (RubyString) data;
 
         } else {
-            throw ruby.newArgumentError(
-                "must be kind_of String or respond to :to_io, :read, or :string");
+            throw ruby.newArgumentError("must be kind_of String or respond to :to_io, :read, or :string");
         }
 
         if (stringData != null) {

data/ext/java/nokogiri/internals/ReaderNode.java

@@ -69,19 +69,20 @@ import org.w3c.dom.Document;
  */
 public abstract class ReaderNode {
 
-    Ruby ruby;
+    final Ruby ruby;
     public ReaderAttributeList attributeList;
     public Map<String, String> namespaces;
     public int depth, nodeType;
     public String lang, localName, xmlBase, prefix, name, uri, value, xmlVersion = "1.0";
     public int startOffset, endOffset;
     public boolean hasChildren = false;
-    public abstract String getString();
     private Document document = null;
 
-    private static ElementNode elementNode = null;
-    private static ClosingNode closingNode = null;
-    private static TextNode textNode = null;
+    protected ReaderNode(final Ruby runtime) {
+        this.ruby = runtime;
+    }
+
+    public abstract String getString();
 
     public IRubyObject getAttributeByIndex(IRubyObject index){
         if(index.isNil()) return index;
@@ -134,13 +135,13 @@ public abstract class ReaderNode {
     }
 
     public IRubyObject getAttributes(ThreadContext context) {
-        if(attributeList == null) return context.getRuntime().getNil();
-        RubyHash hash = RubyHash.newHash(context.getRuntime());
+        final Ruby runtime = context.runtime;
+        if (attributeList == null) return runtime.getNil();
+        RubyHash hash = RubyHash.newHash(runtime);
         for (int i=0; i<attributeList.length; i++) {
-            IRubyObject k = stringOrBlank(context.getRuntime(), attributeList.names.get(i));
-            IRubyObject v = stringOrBlank(context.getRuntime(), attributeList.values.get(i));
-            if (context.getRuntime().is1_9()) hash.op_aset19(context, k, v);
-            else hash.op_aset(context, k, v);
+            IRubyObject k = stringOrBlank(runtime, attributeList.names.get(i));
+            IRubyObject v = stringOrBlank(runtime, attributeList.values.get(i));
+            hash.fastASetCheckString(runtime, k, v); // hash.op_aset(context, k, v)
         }
         return hash;
     }
@@ -162,15 +163,13 @@ public abstract class ReaderNode {
     }
 
     public IRubyObject getNamespaces(ThreadContext context) {
-        if(namespaces == null) return ruby.getNil();
-        RubyHash hash = RubyHash.newHash(ruby);
-        Set<String> keys = namespaces.keySet();
-        for (String key : keys) {
-            String stringValue = namespaces.get(key);
-            IRubyObject k = stringOrBlank(context.getRuntime(), key);
-            IRubyObject v = stringOrBlank(context.getRuntime(), stringValue);
-            if (context.getRuntime().is1_9()) hash.op_aset19(context, k, v);
-            else hash.op_aset(context, k, v);
+        final Ruby runtime = context.runtime;
+        if (namespaces == null) return runtime.getNil();
+        RubyHash hash = RubyHash.newHash(runtime);
+        for (Map.Entry<String, String> entry : namespaces.entrySet()) {
+            IRubyObject k = stringOrBlank(runtime, entry.getKey());
+            IRubyObject v = stringOrBlank(runtime, entry.getValue());
+            hash.fastASetCheckString(runtime, k, v); // hash.op_aset(context, k, v)
         }
         return hash;
     }
@@ -215,7 +214,7 @@ public abstract class ReaderNode {
     }
 
     public void setLang(String lang) {
-        lang = (lang != null) ? lang : null;
+        this.lang = lang;
     }
 
     public IRubyObject toSyntaxError() { return ruby.getNil(); }
@@ -253,27 +252,15 @@ public abstract class ReaderNode {
     }
 
     public static ClosingNode createClosingNode(Ruby ruby, String uri, String localName, String qName, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-        if (closingNode == null) closingNode = new ClosingNode();
-        ClosingNode clone;
-        try {
-            clone = (ClosingNode) closingNode.clone();
-        } catch (CloneNotSupportedException e) {
-            clone = new ClosingNode();
-        }
-        clone.init(ruby, uri, localName, qName, depth, langStack, xmlBaseStack);
-        return clone;
+        return new ClosingNode(ruby, uri, localName, qName, depth, langStack, xmlBaseStack);
     }
 
     public static class ClosingNode extends ReaderNode {
 
-        public ClosingNode() {}
-
-        public ClosingNode(Ruby ruby, String uri, String localName, String qName, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-            init(ruby, uri, localName, qName, depth, langStack, xmlBaseStack);
-        }
+        // public ClosingNode() {}
 
-        public void init(Ruby ruby, String uri, String localName, String qName, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-            this.ruby = ruby;
+        ClosingNode(Ruby runtime, String uri, String localName, String qName, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
+            super(runtime);
             nodeType = ReaderNodeType.END_ELEMENT.getValue();
             this.uri = "".equals(uri) ? null : uri;
             this.localName = localName.trim().length() > 0 ? localName : qName;
@@ -296,35 +283,20 @@ public abstract class ReaderNode {
 
         @Override
         public String getString() {
-            StringBuffer sb = new StringBuffer();
-            sb.append("</").append(name).append(">");
-            return new String(sb);
+            return "</" + name + '>';
         }
     }
 
     public static ElementNode createElementNode(Ruby ruby, String uri, String localName, String qName, XMLAttributes attrs, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-        if (elementNode == null) elementNode = new ElementNode();
-        ElementNode clone;
-        try {
-            clone = (ElementNode) elementNode.clone();
-        } catch (CloneNotSupportedException e) {
-            clone = new ElementNode();
-        }
-        clone.init(ruby, uri, localName, qName, attrs, depth, langStack, xmlBaseStack);
-        return clone;
+        return new ElementNode(ruby, uri, localName, qName, attrs, depth, langStack, xmlBaseStack);
     }
 
     public static class ElementNode extends ReaderNode {
-        private final List<String> attributeStrings = new ArrayList<String>();
-
-        public ElementNode() {}
 
-        public ElementNode(Ruby ruby, String uri, String localName, String qName, XMLAttributes attrs, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-            init(ruby, uri, localName, qName, attrs, depth, langStack, xmlBaseStack);
-        }
+        // public ElementNode() {}
 
-        public void init(Ruby ruby, String uri, String localName, String qName, XMLAttributes attrs, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-            this.ruby = ruby;
+        ElementNode(Ruby runtime, String uri, String localName, String qName, XMLAttributes attrs, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
+            super(runtime);
             this.nodeType = ReaderNodeType.ELEMENT.getValue();
             this.uri = "".equals(uri) ? null : uri;
             this.localName = localName.trim().length() > 0 ? localName : qName;
@@ -354,7 +326,6 @@ public abstract class ReaderNode {
                     if (xmlBase == null) xmlBase = resolveXmlBase(n, v, xmlBaseStack);
                 }
                 attributeList.add(u, n, v);
-                attributeStrings.add(n + "=\"" + v + "\"");
             }
         }
 
@@ -408,32 +379,32 @@ public abstract class ReaderNode {
 
         @Override
         public String getString() {
-            StringBuffer sb = new StringBuffer();
-            sb.append("<").append(name);
+            StringBuffer sb = new StringBuffer(24);
+            sb.append('<').append(name);
             if (attributeList != null) {
                 for (int i=0; i<attributeList.length; i++) {
-                    sb.append(" ").append(attributeStrings.get(i));
+                    String n = attributeList.names.get(i);
+                    String v = attributeList.values.get(i);
+                    sb.append(' ').append(n).append('=')
+                      .append('"').append(v).append('"');
                 }
             }
-            if (hasChildren) sb.append(">");
+            if (hasChildren) sb.append('>');
             else sb.append("/>");
-            return new String(sb);
+            return sb.toString();
         }
     }
 
-    public static class ReaderAttributeList {
-        List<String> namespaces  = new ArrayList<String>();
-        List<String> names  = new ArrayList<String>();
-        List<String> values = new ArrayList<String>();
+    private static class ReaderAttributeList {
+        final List<String> namespaces  = new ArrayList<String>();
+        final List<String> names  = new ArrayList<String>();
+        final List<String> values = new ArrayList<String>();
         int length = 0;
 
         void add(String namespace, String name, String value) {
-            namespace = namespace != null ? namespace : "";
-            namespaces.add(namespace);
-            name = name != null ? name : "";
-            names.add(name);
-            value = value != null ? value : "";
-            values.add(value);
+            namespaces.add(namespace != null ? namespace : "");
+            names.add(name != null ? name : "");
+            values.add(value != null ? value : "");
             length++;
         }
 
@@ -449,8 +420,8 @@ public abstract class ReaderNode {
 
     public static class EmptyNode extends ReaderNode {
 
-        public EmptyNode(Ruby ruby) {
-            this.ruby = ruby;
+        public EmptyNode(Ruby runtime) {
+            super(runtime);
             this.nodeType = ReaderNodeType.NODE.getValue();
         }
 
@@ -473,10 +444,10 @@ public abstract class ReaderNode {
     public static class ExceptionNode extends EmptyNode {
         private final XmlSyntaxError exception;
 
-        // Still don't know what to do with ex.
         public ExceptionNode(Ruby runtime, Exception ex) {
             super(runtime);
-            exception = (XmlSyntaxError) NokogiriService.XML_SYNTAXERROR_ALLOCATOR.allocate(runtime, getNokogiriClass(ruby, "Nokogiri::XML::SyntaxError"));
+            exception = XmlSyntaxError.createXMLSyntaxError(runtime); // Nokogiri::XML::SyntaxError
+            exception.setException(ex);
         }
 
         @Override
@@ -491,26 +462,15 @@ public abstract class ReaderNode {
     }
 
     public static TextNode createTextNode(Ruby ruby, String content, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-        if (textNode == null) textNode = new TextNode();
-        TextNode clone;
-        try {
-            clone = (TextNode) textNode.clone();
-        } catch (CloneNotSupportedException e) {
-            clone = new TextNode();
-        }
-        clone.init(ruby, content, depth, langStack, xmlBaseStack);
-        return clone;
+        return new TextNode(ruby, content, depth, langStack, xmlBaseStack);
     }
 
     public static class TextNode extends ReaderNode {
-        public TextNode() {}
 
-        public TextNode(Ruby ruby, String content, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-            init(ruby, content, depth, langStack, xmlBaseStack);
-        }
+        // public TextNode() {}
 
-        public void init(Ruby ruby, String content, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-            this.ruby = ruby;
+        TextNode(Ruby runtime, String content, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
+            super(runtime);
             this.value = content;
             this.localName = "#text";
             this.name = "#text";
@@ -531,4 +491,4 @@ public abstract class ReaderNode {
             return value;
         }
     }
-}
+}