[ 
https://issues.apache.org/jira/browse/JXPATH-113?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Benedikt Ritter updated JXPATH-113:
-----------------------------------
    Description: 
In ChildContext I get a NullPointerException, the error appears in method 
prepare()

{code:java}
    iterator = (startFromParentLocation ? parent.getParent() : 
parent).childIterator(nodeTest,
                reverse, startFromParentLocation ? parent : null);
{code}
    
should be changed in my opinion to

{code:java}
final NodePointer actualParent = startFromParentLocation ? parent.getParent() : 
parent;
        iterator = actualParent==null ? null : 
actualParent.childIterator(nodeTest,
                reverse, startFromParentLocation ? parent : null);
{code}

The tests triggering the error are testIssue172_CountFollowingSiblingNode and 
testIssue172_CountPrecedingSiblingNode

  was:
In ChildContext I get a NullPointerException, the error appears in method 
prepare()

    iterator = (startFromParentLocation ? parent.getParent() : 
parent).childIterator(nodeTest,
                reverse, startFromParentLocation ? parent : null);
    
should be changed in my opinion to

final NodePointer actualParent = startFromParentLocation ? parent.getParent() : 
parent;
        iterator = actualParent==null ? null : 
actualParent.childIterator(nodeTest,
                reverse, startFromParentLocation ? parent : null);

The tests triggering the error are testIssue172_CountFollowingSiblingNode and 
testIssue172_CountPrecedingSiblingNode


> NullPointerException in ChildContext
> ------------------------------------
>
>                 Key: JXPATH-113
>                 URL: https://issues.apache.org/jira/browse/JXPATH-113
>             Project: Commons JXPath
>          Issue Type: Bug
>            Reporter: Michele Vivoda
>            Priority: Minor
>             Fix For: 1.3
>
>
> In ChildContext I get a NullPointerException, the error appears in method 
> prepare()
> {code:java}
>     iterator = (startFromParentLocation ? parent.getParent() : 
> parent).childIterator(nodeTest,
>                 reverse, startFromParentLocation ? parent : null);
> {code}
>     
> should be changed in my opinion to
> {code:java}
> final NodePointer actualParent = startFromParentLocation ? parent.getParent() 
> : parent;
>         iterator = actualParent==null ? null : 
> actualParent.childIterator(nodeTest,
>                 reverse, startFromParentLocation ? parent : null);
> {code}
> The tests triggering the error are testIssue172_CountFollowingSiblingNode and 
> testIssue172_CountPrecedingSiblingNode



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to