-
Notifications
You must be signed in to change notification settings - Fork 425
Support basic arrow function serialization #1650
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good so far. Not sure what else you are (going to) run into.
ArrowFunctionExpression: { | ||
enter(path: BabelTraversePath, state: ClosureRefVisitorState) { | ||
state.functionInfo.depth++; | ||
state.functionInfo.lexicalDepth++; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clever
Rebased and added some basic tests. @NTillmann would be good to review this now as it's a long standing PR that was actually relatively simple to implement for basic cases. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trueadm is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Release notes: adds basic ES2015 arrow function serialization
This allows Prepack to serialize back to arrow functions and also traverse them correctly when understanding
this
.