Skip to content

Accept an already parsed AST #10

Open
wants to merge 2 commits into from

5 participants

@Couto
Couto commented Dec 10, 2012

This is a very specific feature that I needed to avoid repetition.
Since it was made to solve a very particular problem, feel free to close this ticket without accepting it, if you feel so.

I needed to analyze the AST before walking through the nodes, since I had the source, and the esprima's generated ast, it fell redundant to let falafel generate the AST again, so this pull request is only a way of using the options to provide an AST to falafel.

@jcla1
jcla1 commented Apr 17, 2013

:+1:

@inian
inian commented on 05dcb4a Sep 30, 2013

The AST passed in must have range attached to each node. There must be a check done to see if there is range attached to the passed in AST.

@acgessler

+1

@jareware

I would need either this or #33. Any chance of getting a maintainer to merge either one in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.