Right now it returns the node itself or null if iteration is to halt before entering the children. This is very counter intuitive, as it might be interpreted as promise to change the node, symmetrical to the return value of leaveXXXNode, which it definitely isn't.

Change the enter return value to "should I continue or not" - a boolean. I've shot myself in the foot twice now.