Skip to content

Add support for function argument initializers. #2571

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

Merged

Conversation

zherczeg
Copy link
Member

EcmaScript 2015 14.1.
Note: arrow functions with default arguments are not supported yet.

@@ -76,6 +76,9 @@ typedef enum
#ifndef CONFIG_DISABLE_ES2015_CLASS
SCAN_STACK_CLASS, /**< class language element */
#endif /* !CONFIG_DISABLE_ES2015_CLASS */
#ifndef CONFIG_DISABLE_ES2015_FUNCTION_PARAMETER_INITIALIZER
SCAN_STACK_FUNCTION_PARAMETERS, /**< function parameter initializer */
#endif /* CONFIG_DISABLE_ES2015_FUNCTION_PARAMETER_INITIALIZER */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Closing comments of #ifndefs should be perfixed with ! (all over the commit).

@akosthekiss akosthekiss added development Feature implementation ES2015 Related to ES2015 features labels Oct 26, 2018
@zherczeg zherczeg force-pushed the argument_initializer branch from 8e96969 to 75ce74c Compare October 29, 2018 13:02
EcmaScript 2015 14.1.

Note: arrow functions with default arguments are not supported yet.

JerryScript-DCO-1.0-Signed-off-by: Zoltan Herczeg zherczeg.u-szeged@partner.samsung.com
@zherczeg zherczeg force-pushed the argument_initializer branch from 75ce74c to 5562a2a Compare October 29, 2018 14:29
Copy link
Member

@akosthekiss akosthekiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@rerobika rerobika left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (informal)

Copy link
Contributor

@LaszloLango LaszloLango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LaszloLango LaszloLango merged commit 83fc0aa into jerryscript-project:master Oct 30, 2018
@zherczeg zherczeg deleted the argument_initializer branch October 30, 2018 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development Feature implementation ES2015 Related to ES2015 features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants