3 issues

  1. the current implementation in AbstractNode is protected, it would be useful to make it public, and to put the getNodeName into the Node interface
  2. the String[] in AbstractNode has incorrect values. I think that the values in NODE_TYPE_NAMES should be

    "Node", "Element", "Attribute", "Text", "CDATA", "Entity", "", “Processing Instruction",
    "Comment", "Document", "Document Type","","","Namespace","Unknown"
  3. It would be useful to have a Node.MAX_NODE_TYPE for sizing purposes

 

Mike Skells

 

Reply via email to