-
Notifications
You must be signed in to change notification settings - Fork 686
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 frobert@inf.u-szeged.hu
90e54e4 to
78c18af
Compare
|
@akosthekiss Thanks for the review, I've updated the PR. |
akosthekiss
left a comment
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
LaszloLango
left a comment
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
zherczeg
left a comment
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 frobert@inf.u-szeged.hu