-
Notifications
You must be signed in to change notification settings - Fork 684
Fix super keyword parsing for arrow functions #2664
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
5884d5e
to
3c30071
Compare
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
3c30071
to
c754d95
Compare
What happened with the test case? |
Dismiss my previous review, because the PR changed too much since then.
c754d95
to
e37b414
Compare
@zherczeg I've added the missing test case. |
e37b414
to
e94fd6e
Compare
Note: #2686 must land first. |
e94fd6e
to
90e54e4
Compare
The patch is updated to the latest master after #2686 has landed. |
This patch fixes jerryscript-project#2657. JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik [email protected]
90e54e4
to
78c18af
Compare
@akosthekiss Thanks for the review, I've updated the PR. |
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.
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.
LGTM
This patch fixes #2657.
JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik [email protected]