[ 
https://issues.apache.org/jira/browse/JXPATH-114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12554347
 ] 

Michele Vivoda commented on JXPATH-114:
---------------------------------------

Sure, I forgot....

XPath: count(//node())
XML:
<root><a foo='34'>123.3</a><!-- com --><a attr='34.3'>34.3<!-- om --></a></root>

xalan and saxon result is 7, jxpath gives 3.


> node() implementation in DOM and JDOM model
> -------------------------------------------
>
>                 Key: JXPATH-114
>                 URL: https://issues.apache.org/jira/browse/JXPATH-114
>             Project: Commons JXPath
>          Issue Type: Bug
>            Reporter: Michele Vivoda
>            Priority: Minor
>
> I think that the code in DOMNodePointer.java, line 120 is wrong because 
> considers only element and document to be matched by node().
> while instead it matches any node that pass from there.
> case Compiler.NODE_TYPE_NODE :
>                     return nodeType == Node.ELEMENT_NODE
>                             || nodeType == Node.DOCUMENT_NODE;
> should be changed to 
> case Compiler.NODE_TYPE_NODE :
>                     return true;
> Same in JDOMNodePointer, line 391
>                   return true;//(node instanceof Element) || (node instanceof 
> Document);

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to