Inconsistent iteration over Node children

[cos]

Node implements Iterable allowing iteration over its children.

The problem is that the functionality seems to be only implemented in very few of AstNode``s children. It works for list-like ones (e.g.,Block) but doesn't for many others (e.g.IfStatement). It makes navigating over the AST very difficult. Yes,Visitor` helps to some degree, but it forces programming in a certain style.

Is this behavior intended?

Considering the children are already identified by the Visitor, couldn't they also be exposed through the iterator (or maybe a separate iterator)?

Thanks!

[makc]

not to mention that this breaks methods like replaceChild

java.lang.NullPointerException
    at org.mozilla.javascript.Node.getChildBefore(Node.java:222)
    at org.mozilla.javascript.Node.replaceChild(Node.java:319)

because they don't set first/next children properties randomly