Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implement String.prototype.normalize #4903

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rerobika
Copy link
Member

JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik robert.fancsik@h-lab.eu

@rerobika rerobika added the ES.next Related to ES2015+ features label Dec 13, 2021
@rerobika rerobika force-pushed the string_normalize branch 3 times, most recently from 5a35a80 to 35ea705 Compare December 13, 2021 16:36
JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik robert.fancsik@h-lab.eu
@@ -2805,6 +2805,58 @@ ecma_op_advance_string_index (ecma_string_t *str_p, /**< input string */
} /* ecma_op_advance_string_index */
#endif /* JERRY_ESNEXT */

#if JERRY_ICU
Copy link
Member

Choose a reason for hiding this comment

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

Should have a default value in jerry-core/config.h

@lygstate
Copy link
Contributor

I suggest move icu out of core by providing a port api.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ES.next Related to ES2015+ features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants