A way to break visitor about ast-types HOT 7 CLOSED

Do you mean return false; (and not calling this.traverse), which would not descent further at that node? Or do you want to completely stop traversing the AST?

from ast-types.

Yes, i mean to completely stop traversing

from ast-types.

Yep, throwing an exception is currently the best way to abort not just the current subtree (return false for that) but all remaining nodes. I like the semantics of throwing an exception, because you could potentially catch it in an ancestor visitor if you wanted to.

I'm curious if your hesitance to use exceptions has more to do with performance, or the burden of writing your own try-catch block to intercept the exception. If the latter, I'd be happy to consider a special exception type that could be caught in the top-level PathVisitor.prototype.visit method.

from ast-types.

Published support for PathVisitor.prototype.abort just now (v0.6.13): https://www.npmjs.com/package/ast-types.

Thanks for the idea!

from ast-types.

Wow, that's great! Thank you a lot, that really simplifies things in two upcoming projects - ast-test and ast-replace.

from ast-types.

Nice, I'm glad to hear it, and looking forward to seeing what you do with those projects :)

from ast-types.

Yes, here they are: ast-replace, ast-test.

from ast-types.