nokogiri 1.11.0.rc3-java → 1.11.4-java

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

@@ -1,35 +1,3 @@
-/**
- * (The MIT License)
- *
- * Copyright (c) 2008 - 2012:
- *
- * * {Aaron Patterson}[http://tenderlovemaking.com]
- * * {Mike Dalessio}[http://mike.daless.io]
- * * {Charles Nutter}[http://blog.headius.com]
- * * {Sergio Arbeo}[http://www.serabe.com]
- * * {Patrick Mahoney}[http://polycrystal.org]
- * * {Yoko Harada}[http://yokolet.blogspot.com]
- *
- * Permission is hereby granted, free of charge, to any person obtaining
- * a copy of this software and associated documentation files (the
- * 'Software'), to deal in the Software without restriction, including
- * without limitation the rights to use, copy, modify, merge, publish,
- * distribute, sublicense, and/or sell copies of the Software, and to
- * permit persons to whom the Software is furnished to do so, subject to
- * the following conditions:
- *
- * The above copyright notice and this permission notice shall be
- * included in all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
- * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
- * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
- * IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
- * CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
- * TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
- * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
- */
-
 package nokogiri.internals;
 
 import static nokogiri.internals.NokogiriHelpers.rubyStringToString;
@@ -58,202 +26,229 @@ import org.xml.sax.InputSource;
  * @author Patrick Mahoney <pat@polycrystal.org>
  * @author Yoko Harada <yokolet@gmail.com>
  */
-public abstract class ParserContext extends RubyObject {
-    protected InputSource source = null;
-    protected IRubyObject detected_encoding = null;
-    protected int stringDataSize = -1;
-    protected String java_encoding;
-
-    public ParserContext(Ruby runtime) {
-        // default to class 'Object' because this class isn't exposed to Ruby
-        super(runtime, runtime.getObject());
+public abstract class ParserContext extends RubyObject
+{
+  protected InputSource source = null;
+  protected IRubyObject detected_encoding = null;
+  protected int stringDataSize = -1;
+  protected String java_encoding;
+
+  public
+  ParserContext(Ruby runtime)
+  {
+    // default to class 'Object' because this class isn't exposed to Ruby
+    super(runtime, runtime.getObject());
+  }
+
+  public
+  ParserContext(Ruby runtime, RubyClass klass)
+  {
+    super(runtime, klass);
+  }
+
+  protected InputSource
+  getInputSource()
+  {
+    return source;
+  }
+
+  public void
+  setIOInputSource(ThreadContext context, IRubyObject data, IRubyObject url)
+  {
+    source = new InputSource();
+    ParserContext.setUrl(context, source, url);
+
+    source.setByteStream(new IOInputStream(data));
+    if (java_encoding != null) {
+      source.setEncoding(java_encoding);
     }
+  }
 
-    public ParserContext(Ruby runtime, RubyClass klass) {
-        super(runtime, klass);
-    }
+  public void
+  setStringInputSource(ThreadContext context, IRubyObject data, IRubyObject url)
+  {
+    source = new InputSource();
+    ParserContext.setUrl(context, source, url);
 
-    protected InputSource getInputSource() {
-        return source;
-    }
-
-    public void setIOInputSource(ThreadContext context, IRubyObject data, IRubyObject url) {
-        source = new InputSource();
-        ParserContext.setUrl(context, source, url);
+    Ruby ruby = context.getRuntime();
 
-        source.setByteStream(new IOInputStream(data));
-        if (java_encoding != null) {
-            source.setEncoding(java_encoding);
-        }
+    if (!(data instanceof RubyString)) {
+      throw ruby.newArgumentError("must be kind_of String");
     }
 
-    public void setStringInputSource(ThreadContext context, IRubyObject data, IRubyObject url) {
-        source = new InputSource();
-        ParserContext.setUrl(context, source, url);
-
-        Ruby ruby = context.getRuntime();
-
-        if (!(data instanceof RubyString)) {
-            throw ruby.newArgumentError("must be kind_of String");
-        }
-
-        RubyString stringData = (RubyString) data;
-
-        if (stringData.encoding(context) != null) {
-            RubyString stringEncoding = stringData.encoding(context).asString();
-            String encName = NokogiriHelpers.getValidEncodingOrNull(stringEncoding);
-            if (encName != null) {
-                java_encoding = encName;
-            }
-        }
-
-        ByteList bytes = stringData.getByteList();
+    RubyString stringData = (RubyString) data;
 
-        stringDataSize = bytes.length() - bytes.begin();
-        ByteArrayInputStream stream = new ByteArrayInputStream(bytes.unsafeBytes(), bytes.begin(), bytes.length());
-        source.setByteStream(stream);
-        source.setEncoding(java_encoding);
+    if (stringData.encoding(context) != null) {
+      RubyString stringEncoding = stringData.encoding(context).asString();
+      String encName = NokogiriHelpers.getValidEncodingOrNull(stringEncoding);
+      if (encName != null) {
+        java_encoding = encName;
+      }
     }
 
-    public static void setUrl(ThreadContext context, InputSource source, IRubyObject url) {
-        String path = rubyStringToString(url);
-        // Dir.chdir might be called at some point before this.
-        if (path != null) {
+    ByteList bytes = stringData.getByteList();
+
+    stringDataSize = bytes.length() - bytes.begin();
+    ByteArrayInputStream stream = new ByteArrayInputStream(bytes.unsafeBytes(), bytes.begin(), bytes.length());
+    source.setByteStream(stream);
+    source.setEncoding(java_encoding);
+  }
+
+  public static void
+  setUrl(ThreadContext context, InputSource source, IRubyObject url)
+  {
+    String path = rubyStringToString(url);
+    // Dir.chdir might be called at some point before this.
+    if (path != null) {
+      try {
+        URI uri = URI.create(path);
+        source.setSystemId(uri.toURL().toString());
+      } catch (Exception ex) {
+        // fallback to the old behavior
+        File file = new File(path);
+        if (file.isAbsolute()) {
+          source.setSystemId(path);
+        } else {
+          String pwd = context.getRuntime().getCurrentDirectory();
+          String absolutePath;
           try {
-            URI uri = URI.create(path);
-            source.setSystemId(uri.toURL().toString());
-          } catch (Exception ex) {
-            // fallback to the old behavior
-            File file = new File(path);
-            if (file.isAbsolute()) {
-              source.setSystemId(path);
-            } else {
-              String pwd = context.getRuntime().getCurrentDirectory();
-              String absolutePath;
-              try {
-                absolutePath = new File(pwd, path).getCanonicalPath();
-              } catch (IOException e) {
-                absolutePath = new File(pwd, path).getAbsolutePath();
-              }
-              source.setSystemId(absolutePath);
-            }
+            absolutePath = new File(pwd, path).getCanonicalPath();
+          } catch (IOException e) {
+            absolutePath = new File(pwd, path).getAbsolutePath();
           }
+          source.setSystemId(absolutePath);
         }
+      }
     }
-
-    protected void setEncoding(String encoding) {
-    	source.setEncoding(encoding);
+  }
+
+  protected void
+  setEncoding(String encoding)
+  {
+    source.setEncoding(encoding);
+  }
+
+  /**
+   * Set the InputSource to read from <code>file</code>, a String filename.
+   */
+  public void
+  setInputSourceFile(ThreadContext context, IRubyObject file)
+  {
+    source = new InputSource();
+    ParserContext.setUrl(context, source, file);
+  }
+
+  /**
+   * Set the InputSource from <code>stream</code>.
+   */
+  public void
+  setInputSource(InputStream stream)
+  {
+    source = new InputSource(stream);
+  }
+
+  /**
+   * Wrap Nokogiri parser options in a utility class.  This is
+   * read-only.
+   */
+  public static class Options
+  {
+    protected static final long STRICT = 0;
+    protected static final long RECOVER = 1;
+    protected static final long NOENT = 2;
+    protected static final long DTDLOAD = 4;
+    protected static final long DTDATTR = 8;
+    protected static final long DTDVALID = 16;
+    protected static final long NOERROR = 32;
+    protected static final long NOWARNING = 64;
+    protected static final long PEDANTIC = 128;
+    protected static final long NOBLANKS = 256;
+    protected static final long SAX1 = 512;
+    protected static final long XINCLUDE = 1024;
+    protected static final long NONET = 2048;
+    protected static final long NODICT = 4096;
+    protected static final long NSCLEAN = 8192;
+    protected static final long NOCDATA = 16384;
+    protected static final long NOXINCNODE = 32768;
+
+    public final boolean strict;
+    public final boolean recover;
+    public final boolean noEnt;
+    public final boolean dtdLoad;
+    public final boolean dtdAttr;
+    public final boolean dtdValid;
+    public final boolean noError;
+    public final boolean noWarning;
+    public final boolean pedantic;
+    public final boolean noBlanks;
+    public final boolean sax1;
+    public final boolean xInclude;
+    public final boolean noNet;
+    public final boolean noDict;
+    public final boolean nsClean;
+    public final boolean noCdata;
+    public final boolean noXIncNode;
+
+    protected static boolean
+    test(long options, long mask)
+    {
+      return ((options & mask) == mask);
     }
 
-    /**
-     * Set the InputSource to read from <code>file</code>, a String filename.
-     */
-    public void setInputSourceFile(ThreadContext context, IRubyObject file) {
-        source = new InputSource();
-        ParserContext.setUrl(context, source, file);
+    public
+    Options(long options)
+    {
+      strict = ((options & RECOVER) == STRICT);
+      recover = test(options, RECOVER);
+      noEnt = test(options, NOENT);
+      dtdLoad = test(options, DTDLOAD);
+      dtdAttr = test(options, DTDATTR);
+      dtdValid = test(options, DTDVALID);
+      noError = test(options, NOERROR);
+      noWarning = test(options, NOWARNING);
+      pedantic = test(options, PEDANTIC);
+      noBlanks = test(options, NOBLANKS);
+      sax1 = test(options, SAX1);
+      xInclude = test(options, XINCLUDE);
+      noNet = test(options, NONET);
+      noDict = test(options, NODICT);
+      nsClean = test(options, NSCLEAN);
+      noCdata = test(options, NOCDATA);
+      noXIncNode = test(options, NOXINCNODE);
     }
-
-    /**
-     * Set the InputSource from <code>stream</code>.
-     */
-    public void setInputSource(InputStream stream) {
-        source = new InputSource(stream);
-    }
-
-    /**
-     * Wrap Nokogiri parser options in a utility class.  This is
-     * read-only.
-     */
-    public static class Options {
-        protected static final long STRICT = 0;
-        protected static final long RECOVER = 1;
-        protected static final long NOENT = 2;
-        protected static final long DTDLOAD = 4;
-        protected static final long DTDATTR = 8;
-        protected static final long DTDVALID = 16;
-        protected static final long NOERROR = 32;
-        protected static final long NOWARNING = 64;
-        protected static final long PEDANTIC = 128;
-        protected static final long NOBLANKS = 256;
-        protected static final long SAX1 = 512;
-        protected static final long XINCLUDE = 1024;
-        protected static final long NONET = 2048;
-        protected static final long NODICT = 4096;
-        protected static final long NSCLEAN = 8192;
-        protected static final long NOCDATA = 16384;
-        protected static final long NOXINCNODE = 32768;
-
-        public final boolean strict;
-        public final boolean recover;
-        public final boolean noEnt;
-        public final boolean dtdLoad;
-        public final boolean dtdAttr;
-        public final boolean dtdValid;
-        public final boolean noError;
-        public final boolean noWarning;
-        public final boolean pedantic;
-        public final boolean noBlanks;
-        public final boolean sax1;
-        public final boolean xInclude;
-        public final boolean noNet;
-        public final boolean noDict;
-        public final boolean nsClean;
-        public final boolean noCdata;
-        public final boolean noXIncNode;
-
-        protected static boolean test(long options, long mask) {
-            return ((options & mask) == mask);
-        }
-
-        public Options(long options) {
-            strict = ((options & RECOVER) == STRICT);
-            recover = test(options, RECOVER);
-            noEnt = test(options, NOENT);
-            dtdLoad = test(options, DTDLOAD);
-            dtdAttr = test(options, DTDATTR);
-            dtdValid = test(options, DTDVALID);
-            noError = test(options, NOERROR);
-            noWarning = test(options, NOWARNING);
-            pedantic = test(options, PEDANTIC);
-            noBlanks = test(options, NOBLANKS);
-            sax1 = test(options, SAX1);
-            xInclude = test(options, XINCLUDE);
-            noNet = test(options, NONET);
-            noDict = test(options, NODICT);
-            nsClean = test(options, NSCLEAN);
-            noCdata = test(options, NOCDATA);
-            noXIncNode = test(options, NOXINCNODE);
-        }
+  }
+
+  /*
+  public static class NokogiriXInlcudeEntityResolver implements org.xml.sax.EntityResolver {
+      InputSource source;
+      public NokogiriXInlcudeEntityResolver(InputSource source) {
+          this.source = source;
+      }
+
+      @Override
+      public InputSource resolveEntity(String publicId, String systemId)
+              throws SAXException, IOException {
+          if (systemId != null) source.setSystemId(systemId);
+          if (publicId != null) source.setPublicId(publicId);
+          return source;
+      }
+  } */
+
+  public static abstract class ParserTask<T extends ParserContext> implements Callable<T>
+  {
+
+    protected final ThreadContext context; // TODO does not seem like a good idea!?
+    protected final IRubyObject handler;
+    protected final T parser;
+
+    protected
+    ParserTask(ThreadContext context, IRubyObject handler, T parser)
+    {
+      this.context = context;
+      this.handler = handler;
+      this.parser = parser;
     }
 
-    /*
-    public static class NokogiriXInlcudeEntityResolver implements org.xml.sax.EntityResolver {
-        InputSource source;
-        public NokogiriXInlcudeEntityResolver(InputSource source) {
-            this.source = source;
-        }
-
-        @Override
-        public InputSource resolveEntity(String publicId, String systemId)
-                throws SAXException, IOException {
-            if (systemId != null) source.setSystemId(systemId);
-            if (publicId != null) source.setPublicId(publicId);
-            return source;
-        }
-    } */
-
-    public static abstract class ParserTask<T extends ParserContext> implements Callable<T> {
-
-        protected final ThreadContext context; // TODO does not seem like a good idea!?
-        protected final IRubyObject handler;
-        protected final T parser;
-
-        protected ParserTask(ThreadContext context, IRubyObject handler, T parser) {
-            this.context = context;
-            this.handler = handler;
-            this.parser = parser;
-        }
-
-    }
+  }
 
 }

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

@@ -1,35 +1,3 @@
-/**
- * (The MIT License)
- *
- * Copyright (c) 2008 - 2011:
- *
- * * {Aaron Patterson}[http://tenderlovemaking.com]
- * * {Mike Dalessio}[http://mike.daless.io]
- * * {Charles Nutter}[http://blog.headius.com]
- * * {Sergio Arbeo}[http://www.serabe.com]
- * * {Patrick Mahoney}[http://polycrystal.org]
- * * {Yoko Harada}[http://yokolet.blogspot.com]
- *
- * Permission is hereby granted, free of charge, to any person obtaining
- * a copy of this software and associated documentation files (the
- * 'Software'), to deal in the Software without restriction, including
- * without limitation the rights to use, copy, modify, merge, publish,
- * distribute, sublicense, and/or sell copies of the Software, and to
- * permit persons to whom the Software is furnished to do so, subject to
- * the following conditions:
- *
- * The above copyright notice and this permission notice shall be
- * included in all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
- * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
- * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
- * IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
- * CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
- * TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
- * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
- */
-
 package nokogiri.internals;
 
 import java.util.ArrayList;
@@ -62,427 +30,534 @@ import static nokogiri.internals.NokogiriHelpers.*;
  * @author Yoko Harada <yokolet@gmail.com>
  *
  */
-public abstract class ReaderNode {
-
-    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;
-    private Document document = null;
-
-    protected ReaderNode(final Ruby runtime) {
-        this.ruby = runtime;
+public abstract class ReaderNode
+{
+
+  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;
+  private Document document = null;
+
+  protected
+  ReaderNode(final Ruby runtime)
+  {
+    this.ruby = runtime;
+  }
+
+  public abstract String getString();
+
+  public IRubyObject
+  getAttributeByIndex(IRubyObject index)
+  {
+    if (index.isNil()) { return index; }
+
+    long i = index.convertToInteger().getLongValue();
+    if (i > Integer.MAX_VALUE) {
+      throw ruby.newArgumentError("value too long to be an array index");
     }
 
-    public abstract String getString();
-
-    public IRubyObject getAttributeByIndex(IRubyObject index){
-        if(index.isNil()) return index;
-
-        long i = index.convertToInteger().getLongValue();
-        if(i > Integer.MAX_VALUE) {
-            throw ruby.newArgumentError("value too long to be an array index");
+    if (attributeList == null) { return ruby.getNil(); }
+    if (i < 0 || attributeList.length <= i) { return ruby.getNil(); }
+    return stringOrBlank(ruby, attributeList.values.get(((Long)i).intValue()));
+  }
+
+  public IRubyObject
+  getAttributeByName(IRubyObject name)
+  {
+    if (attributeList == null) { return ruby.getNil(); }
+    String value = attributeList.getByName(rubyStringToString(name));
+    return stringOrNil(ruby, value);
+  }
+
+  public IRubyObject
+  getAttributeByName(String name)
+  {
+    if (attributeList == null) { return ruby.getNil(); }
+    String value = attributeList.getByName(name);
+    return stringOrNil(ruby, value);
+  }
+
+  public IRubyObject
+  getAttributeCount()
+  {
+    if (attributeList == null) { return ruby.newFixnum(0); }
+    return ruby.newFixnum(attributeList.length);
+  }
+
+  public IRubyObject
+  getAttributesNodes()
+  {
+    RubyArray array = RubyArray.newArray(ruby);
+    if (attributeList != null && attributeList.length > 0) {
+      if (document == null) {
+        document = XmlDocument.createNewDocument(ruby);
+      }
+      for (int i = 0; i < attributeList.length; i++) {
+        if (!isNamespace(attributeList.names.get(i))) {
+          Attr attr = document.createAttributeNS(attributeList.namespaces.get(i), attributeList.names.get(i));
+          attr.setValue(attributeList.values.get(i));
+          XmlAttr xmlAttr = new XmlAttr(ruby, attr);
+          array.append(xmlAttr);
         }
-
-        if (attributeList == null) return ruby.getNil();
-        if (i<0 || attributeList.length <= i) return ruby.getNil();
-        return stringOrBlank(ruby, attributeList.values.get(((Long)i).intValue()));
+      }
     }
-
-    public IRubyObject getAttributeByName(IRubyObject name){
-        if(attributeList == null) return ruby.getNil();
-        String value = attributeList.getByName(rubyStringToString(name));
-        return stringOrNil(ruby, value);
+    return array;
+  }
+
+  public IRubyObject
+  getAttributes(ThreadContext context)
+  {
+    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(runtime, attributeList.names.get(i));
+      IRubyObject v = stringOrBlank(runtime, attributeList.values.get(i));
+      hash.fastASetCheckString(runtime, k, v); // hash.op_aset(context, k, v)
     }
-
-    public IRubyObject getAttributeByName(String name){
-        if(attributeList == null) return ruby.getNil();
-        String value = attributeList.getByName(name);
-        return stringOrNil(ruby, value);
+    return hash;
+  }
+
+  public IRubyObject
+  getDepth()
+  {
+    return ruby.newFixnum(depth);
+  }
+
+  public IRubyObject
+  getLang()
+  {
+    return stringOrNil(ruby, lang);
+  }
+
+  public IRubyObject
+  getLocalName()
+  {
+    return stringOrNil(ruby, localName);
+  }
+
+  public IRubyObject
+  getName()
+  {
+    return stringOrNil(ruby, name);
+  }
+
+  public IRubyObject
+  getNamespaces(ThreadContext context)
+  {
+    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)
     }
-
-    public IRubyObject getAttributeCount(){
-        if(attributeList == null) return ruby.newFixnum(0);
-        return ruby.newFixnum(attributeList.length);
+    return hash;
+  }
+
+  public IRubyObject
+  getXmlBase()
+  {
+    return stringOrNil(ruby, xmlBase);
+  }
+
+  public IRubyObject
+  getPrefix()
+  {
+    return stringOrNil(ruby, prefix);
+  }
+
+  public IRubyObject
+  getUri()
+  {
+    return stringOrNil(ruby, uri);
+  }
+
+  public IRubyObject
+  getValue()
+  {
+    return stringOrNil(ruby, value);
+  }
+
+  public IRubyObject
+  getXmlVersion()
+  {
+    return ruby.newString(xmlVersion);
+  }
+
+  public RubyBoolean
+  hasAttributes()
+  {
+    if (attributeList == null || attributeList.length == 0) { return ruby.getFalse(); }
+    return ruby.getTrue();
+  }
+
+  public abstract RubyBoolean hasValue();
+
+  public RubyBoolean
+  isDefault()
+  {
+    // TODO Implement.
+    return ruby.getFalse();
+  }
+
+  public boolean
+  isError() { return false; }
+
+  protected void
+  parsePrefix(String qName)
+  {
+    int index = qName.indexOf(':');
+    if (index != -1) { prefix = qName.substring(0, index); }
+  }
+
+  public void
+  setLang(String lang)
+  {
+    this.lang = lang;
+  }
+
+  public IRubyObject
+  toSyntaxError() { return ruby.getNil(); }
+
+  public IRubyObject
+  getNodeType() { return ruby.newFixnum(nodeType); }
+
+  public static enum ReaderNodeType {
+    NODE(0),
+    ELEMENT(1),
+    ATTRIBUTE(2),
+    TEXT(3),
+    CDATA(4),
+    ENTITY_REFERENCE(5),
+    ENTITY(6),
+    PROCESSING_INSTRUCTION(7),
+    COMMENT(8),
+    DOCUMENT(9),
+    DOCUMENT_TYPE(10),
+    DOCUMENTFRAGMENT(11),
+    NOTATION(12),
+    WHITESPACE(13),
+    SIGNIFICANT_WHITESPACE(14),
+    END_ELEMENT(15),
+    END_ENTITY(16),
+    XML_DECLARATION(17);
+
+    private final int value;
+    ReaderNodeType(int value)
+    {
+      this.value = value;
     }
 
-    public IRubyObject getAttributesNodes() {
-        RubyArray array = RubyArray.newArray(ruby);
-        if (attributeList != null && attributeList.length > 0) {
-            if (document == null) {
-                document = XmlDocument.createNewDocument(ruby);
-            }
-            for (int i=0; i<attributeList.length; i++) {
-                if (!isNamespace(attributeList.names.get(i))) {
-                    Attr attr = document.createAttributeNS(attributeList.namespaces.get(i), attributeList.names.get(i));
-                    attr.setValue(attributeList.values.get(i));
-                    XmlAttr xmlAttr = new XmlAttr(ruby, attr);
-                    array.append(xmlAttr);
-                }
-            }
-        }
-        return array;
+    public int getValue()
+    {
+      return value;
     }
-
-    public IRubyObject getAttributes(ThreadContext context) {
-        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(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;
+  }
+
+  public static ClosingNode
+  createClosingNode(Ruby ruby, String uri, String localName, String qName, int depth, Stack<String> langStack,
+                    Stack<String> xmlBaseStack)
+  {
+    return new ClosingNode(ruby, uri, localName, qName, depth, langStack, xmlBaseStack);
+  }
+
+  public static class ClosingNode extends ReaderNode
+  {
+
+    // public ClosingNode() {}
+
+    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 = ! isBlank(localName) ? localName : qName;
+      this.name = qName;
+      parsePrefix(qName);
+      this.depth = depth;
+      if (!langStack.isEmpty()) { this.lang = langStack.peek(); }
+      if (!xmlBaseStack.isEmpty()) { this.xmlBase = xmlBaseStack.peek(); }
     }
 
-    public IRubyObject getDepth() {
-        return ruby.newFixnum(depth);
+    @Override
+    public IRubyObject
+    getAttributeCount()
+    {
+      return ruby.newFixnum(0);
     }
 
-    public IRubyObject getLang() {
-        return stringOrNil(ruby, lang);
+    @Override
+    public RubyBoolean
+    hasValue()
+    {
+      return ruby.getFalse();
     }
 
-    public IRubyObject getLocalName() {
-        return stringOrNil(ruby, localName);
+    @Override
+    public String
+    getString()
+    {
+      return "</" + name + '>';
+    }
+  }
+
+  public static ElementNode
+  createElementNode(Ruby ruby, String uri, String localName, String qName, XMLAttributes attrs, int depth,
+                    Stack<String> langStack, Stack<String> xmlBaseStack)
+  {
+    return new ElementNode(ruby, uri, localName, qName, attrs, depth, langStack, xmlBaseStack);
+  }
+
+  public static class ElementNode extends ReaderNode
+  {
+
+    // public ElementNode() {}
+
+    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 = ! isBlank(localName) ? localName : qName;
+      this.name = qName;
+      parsePrefix(qName);
+      this.depth = depth;
+      parseAttributes(attrs, langStack, xmlBaseStack);
     }
 
-    public IRubyObject getName() {
-        return stringOrNil(ruby, name);
+    @Override
+    public RubyBoolean
+    hasValue()
+    {
+      return ruby.getFalse();
     }
 
-    public IRubyObject getNamespaces(ThreadContext context) {
-        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)
+    private void
+    parseAttributes(XMLAttributes attrs, Stack<String> langStack, Stack<String> xmlBaseStack)
+    {
+      if (attrs.getLength() > 0) { attributeList = new ReaderAttributeList(); }
+      String u, n, v;
+      for (int i = 0; i < attrs.getLength(); i++) {
+        u = attrs.getURI(i);
+        n = attrs.getQName(i);
+        v = attrs.getValue(i);
+        if (isNamespace(n)) {
+          if (namespaces == null) { namespaces = new HashMap<String, String>(); }
+          namespaces.put(n, v);
+        } else {
+          if (lang == null) { lang = resolveLang(n, v, langStack); }
+          if (xmlBase == null) { xmlBase = resolveXmlBase(n, v, xmlBaseStack); }
         }
-        return hash;
+        attributeList.add(u, n, v);
+      }
     }
 
-    public IRubyObject getXmlBase() {
-        return stringOrNil(ruby, xmlBase);
+    private String
+    resolveLang(String n, String v, Stack<String> langStack)
+    {
+      if ("xml:lang".equals(n)) {
+        return v;
+      } else if (!langStack.isEmpty()) {
+        return langStack.peek();
+      } else {
+        return null;
+      }
     }
 
-    public IRubyObject getPrefix() {
-        return stringOrNil(ruby, prefix);
+    private String
+    resolveXmlBase(String n, String v, Stack<String> xmlBaseStack)
+    {
+      if (isXmlBase(n)) {
+        return getXmlBaseUri(n, v, xmlBaseStack);
+      } else if (!xmlBaseStack.isEmpty()) {
+        return xmlBaseStack.peek();
+      } else {
+        return null;
+      }
     }
 
-    public IRubyObject getUri() {
-        return stringOrNil(ruby, uri);
+    private String
+    getXmlBaseUri(String n, String v, Stack<String> xmlBaseStack)
+    {
+      if ("xml:base".equals(n)) {
+        if (v.startsWith("http://")) {
+          return v;
+        } else if (v.startsWith("/") && v.endsWith("/")) {
+          String sub = v.substring(1, v.length() - 2);
+          String base = xmlBaseStack.peek();
+          if (base.endsWith("/")) {
+            base = base.substring(0, base.length() - 1);
+          }
+          int pos = base.lastIndexOf("/");
+          return base.substring(0, pos).concat(sub);
+        } else {
+          String base = xmlBaseStack.peek();
+          if (base.endsWith("/")) { return base.concat(v); }
+          else { return base.concat("/").concat(v); }
+        }
+      } else if ("xlink:href".equals(n)) {
+        if (v.startsWith("http://")) {
+          return v;
+        } else if (!xmlBaseStack.isEmpty()) {
+          String base = xmlBaseStack.peek();
+          return base;
+        }
+      }
+      return null;
     }
 
-    public IRubyObject getValue() {
-        return stringOrNil(ruby, value);
+    @Override
+    public String
+    getString()
+    {
+      StringBuffer sb = new StringBuffer(24);
+      sb.append('<').append(name);
+      if (attributeList != null) {
+        for (int i = 0; i < attributeList.length; 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('>'); }
+      else { sb.append("/>"); }
+      return sb.toString();
     }
-
-    public IRubyObject getXmlVersion() {
-        return ruby.newString(xmlVersion);
+  }
+
+  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)
+    {
+      namespaces.add(namespace != null ? namespace : "");
+      names.add(name != null ? name : "");
+      values.add(value != null ? value : "");
+      length++;
     }
 
-    public RubyBoolean hasAttributes() {
-        if (attributeList == null || attributeList.length == 0) return ruby.getFalse();
-        return ruby.getTrue();
+    String
+    getByName(String name)
+    {
+      for (int i = 0; i < names.size(); i++) {
+        if (name.equals(names.get(i))) {
+          return values.get(i);
+        }
+      }
+      return null;
     }
+  }
 
-    public abstract RubyBoolean hasValue();
+  public static class EmptyNode extends ReaderNode
+  {
 
-    public RubyBoolean isDefault(){
-        // TODO Implement.
-        return ruby.getFalse();
+    public
+    EmptyNode(Ruby runtime)
+    {
+      super(runtime);
+      this.nodeType = ReaderNodeType.NODE.getValue();
     }
 
-    public boolean isError() { return false; }
-
-    protected void parsePrefix(String qName) {
-        int index = qName.indexOf(':');
-        if(index != -1) prefix = qName.substring(0, index);
+    @Override
+    public IRubyObject
+    getXmlVersion()
+    {
+      return this.ruby.getNil();
     }
 
-    public void setLang(String lang) {
-        this.lang = lang;
+    @Override
+    public RubyBoolean
+    hasValue()
+    {
+      return ruby.getFalse();
     }
 
-    public IRubyObject toSyntaxError() { return ruby.getNil(); }
-
-    public IRubyObject getNodeType() { return ruby.newFixnum(nodeType); }
-
-    public static enum ReaderNodeType {
-        NODE(0),
-        ELEMENT(1),
-        ATTRIBUTE(2),
-        TEXT(3),
-        CDATA(4),
-        ENTITY_REFERENCE(5),
-        ENTITY(6),
-        PROCESSING_INSTRUCTION(7),
-        COMMENT(8),
-        DOCUMENT(9),
-        DOCUMENT_TYPE(10),
-        DOCUMENTFRAGMENT(11),
-        NOTATION(12),
-        WHITESPACE(13),
-        SIGNIFICANT_WHITESPACE(14),
-        END_ELEMENT(15),
-        END_ENTITY(16),
-        XML_DECLARATION(17);
-
-        private final int value;
-        ReaderNodeType(int value) {
-            this.value = value;
-        }
-
-        public int getValue() {
-            return value;
-        }
+    @Override
+    public String
+    getString()
+    {
+      return null;
     }
-
-    public static ClosingNode createClosingNode(Ruby ruby, String uri, String localName, String qName, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-        return new ClosingNode(ruby, uri, localName, qName, depth, langStack, xmlBaseStack);
+  }
+
+  public static class ExceptionNode extends EmptyNode
+  {
+    private final XmlSyntaxError exception;
+
+    public
+    ExceptionNode(Ruby runtime, Exception ex)
+    {
+      super(runtime);
+      exception = XmlSyntaxError.createXMLSyntaxError(runtime); // Nokogiri::XML::SyntaxError
+      exception.setException(ex);
     }
 
-    public static class ClosingNode extends ReaderNode {
-
-        // public ClosingNode() {}
-
-        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 = ! isBlank(localName) ? localName : qName;
-            this.name = qName;
-            parsePrefix(qName);
-            this.depth = depth;
-            if (!langStack.isEmpty()) this.lang = langStack.peek();
-            if (!xmlBaseStack.isEmpty()) this.xmlBase = xmlBaseStack.peek();
-        }
-
-        @Override
-        public IRubyObject getAttributeCount() {
-            return ruby.newFixnum(0);
-        }
-
-        @Override
-        public RubyBoolean hasValue() {
-            return ruby.getFalse();
-        }
-
-        @Override
-        public String getString() {
-            return "</" + name + '>';
-        }
+    @Override
+    public boolean
+    isError()
+    {
+      return true;
     }
 
-    public static ElementNode createElementNode(Ruby ruby, String uri, String localName, String qName, XMLAttributes attrs, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-        return new ElementNode(ruby, uri, localName, qName, attrs, depth, langStack, xmlBaseStack);
+    @Override
+    public IRubyObject
+    toSyntaxError()
+    {
+      return this.exception;
     }
-
-    public static class ElementNode extends ReaderNode {
-
-        // public ElementNode() {}
-
-        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 = ! isBlank(localName) ? localName : qName;
-            this.name = qName;
-            parsePrefix(qName);
-            this.depth = depth;
-            parseAttributes(attrs, langStack, xmlBaseStack);
-        }
-
-        @Override
-        public RubyBoolean hasValue() {
-            return ruby.getFalse();
-        }
-
-        private void parseAttributes(XMLAttributes attrs, Stack<String> langStack, Stack<String> xmlBaseStack) {
-            if (attrs.getLength() > 0) attributeList = new ReaderAttributeList();
-            String u, n, v;
-            for (int i = 0; i < attrs.getLength(); i++) {
-                u = attrs.getURI(i);
-                n = attrs.getQName(i);
-                v = attrs.getValue(i);
-                if (isNamespace(n)) {
-                    if (namespaces == null) namespaces = new HashMap<String, String>();
-                    namespaces.put(n, v);
-                } else {
-                    if (lang == null) lang = resolveLang(n, v, langStack);
-                    if (xmlBase == null) xmlBase = resolveXmlBase(n, v, xmlBaseStack);
-                }
-                attributeList.add(u, n, v);
-            }
-        }
-
-        private String resolveLang(String n, String v, Stack<String> langStack) {
-            if ("xml:lang".equals(n)) {
-                return v;
-            } else if (!langStack.isEmpty()) {
-                return langStack.peek();
-            } else {
-                return null;
-            }
-        }
-
-        private String resolveXmlBase(String n, String v, Stack<String> xmlBaseStack) {
-            if (isXmlBase(n)) {
-                return getXmlBaseUri(n, v, xmlBaseStack);
-            } else if (!xmlBaseStack.isEmpty()) {
-                return xmlBaseStack.peek();
-            } else {
-                return null;
-            }
-        }
-
-        private String getXmlBaseUri(String n, String v, Stack<String> xmlBaseStack) {
-            if ("xml:base".equals(n)) {
-                if (v.startsWith("http://")) {
-                    return v;
-                } else if (v.startsWith("/") && v.endsWith("/")) {
-                    String sub = v.substring(1, v.length() - 2);
-                    String base = xmlBaseStack.peek();
-                    if (base.endsWith("/")) {
-                        base = base.substring(0, base.length() - 1);
-                    }
-                    int pos = base.lastIndexOf("/");
-                    return base.substring(0, pos).concat(sub);
-                } else {
-                    String base = xmlBaseStack.peek();
-                    if (base.endsWith("/")) return base.concat(v);
-                    else return base.concat("/").concat(v);
-                }
-            } else if ("xlink:href".equals(n)) {
-							if (v.startsWith("http://")) {
-								return v;
-							} else if (!xmlBaseStack.isEmpty()) {
-                String base = xmlBaseStack.peek();
-								return base;
-							}
-            }
-            return null;
-        }
-
-        @Override
-        public String getString() {
-            StringBuffer sb = new StringBuffer(24);
-            sb.append('<').append(name);
-            if (attributeList != null) {
-                for (int i=0; i<attributeList.length; 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('>');
-            else sb.append("/>");
-            return sb.toString();
-        }
+  }
+
+  public static TextNode
+  createTextNode(Ruby ruby, String content, int depth, Stack<String> langStack, Stack<String> xmlBaseStack)
+  {
+    return new TextNode(ruby, content, depth, langStack, xmlBaseStack);
+  }
+
+  public static class TextNode extends ReaderNode
+  {
+
+    // public TextNode() {}
+
+    TextNode(Ruby runtime, String content, int depth, Stack<String> langStack, Stack<String> xmlBaseStack)
+    {
+      super(runtime);
+      this.value = content;
+      this.localName = "#text";
+      this.name = "#text";
+      this.depth = depth;
+      if (!isBlank(content)) { nodeType = ReaderNodeType.TEXT.getValue(); }
+      else { nodeType = ReaderNodeType.SIGNIFICANT_WHITESPACE.getValue(); }
+      if (!langStack.isEmpty()) { this.lang = langStack.peek(); }
+      if (!xmlBaseStack.isEmpty()) { this.xmlBase = xmlBaseStack.peek(); }
     }
 
-    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) {
-            namespaces.add(namespace != null ? namespace : "");
-            names.add(name != null ? name : "");
-            values.add(value != null ? value : "");
-            length++;
-        }
-
-        String getByName(String name) {
-            for (int i=0; i<names.size(); i++) {
-                if (name.equals(names.get(i))) {
-                    return values.get(i);
-                }
-            }
-            return null;
-        }
+    @Override
+    public RubyBoolean
+    hasValue()
+    {
+      return ruby.getTrue();
     }
 
-    public static class EmptyNode extends ReaderNode {
-
-        public EmptyNode(Ruby runtime) {
-            super(runtime);
-            this.nodeType = ReaderNodeType.NODE.getValue();
-        }
-
-        @Override
-        public IRubyObject getXmlVersion() {
-            return this.ruby.getNil();
-        }
-
-        @Override
-        public RubyBoolean hasValue() {
-            return ruby.getFalse();
-        }
-
-        @Override
-        public String getString() {
-            return null;
-        }
-    }
-
-    public static class ExceptionNode extends EmptyNode {
-        private final XmlSyntaxError exception;
-
-        public ExceptionNode(Ruby runtime, Exception ex) {
-            super(runtime);
-            exception = XmlSyntaxError.createXMLSyntaxError(runtime); // Nokogiri::XML::SyntaxError
-            exception.setException(ex);
-        }
-
-        @Override
-        public boolean isError() {
-            return true;
-        }
-
-        @Override
-        public IRubyObject toSyntaxError() {
-            return this.exception;
-        }
-    }
-
-    public static TextNode createTextNode(Ruby ruby, String content, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-        return new TextNode(ruby, content, depth, langStack, xmlBaseStack);
-    }
-
-    public static class TextNode extends ReaderNode {
-
-        // public TextNode() {}
-
-        TextNode(Ruby runtime, String content, int depth, Stack<String> langStack, Stack<String> xmlBaseStack) {
-            super(runtime);
-            this.value = content;
-            this.localName = "#text";
-            this.name = "#text";
-            this.depth = depth;
-            if (!isBlank(content)) nodeType = ReaderNodeType.TEXT.getValue();
-            else nodeType = ReaderNodeType.SIGNIFICANT_WHITESPACE.getValue();
-            if (!langStack.isEmpty()) this.lang = langStack.peek();
-            if (!xmlBaseStack.isEmpty()) this.xmlBase = xmlBaseStack.peek();
-        }
-
-        @Override
-        public RubyBoolean hasValue() {
-            return ruby.getTrue();
-        }
-
-        @Override
-        public String getString() {
-            return value;
-        }
+    @Override
+    public String
+    getString()
+    {
+      return value;
     }
+  }
 
 }