src: default --icu_case_mapping on as a v8 option#9454

Closed

srl295 wants to merge 1 commit intonodejs:masterfrom

srl295:default_icu_case_mapping

Member

srl295 commented Nov 3, 2016

make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
tests and/or benchmarks are included
commit message follows commit guidelines

Affected core subsystem(s)

src

Description of change

toLocaleUpperCase() and toLocaleLowerCase() do not function properly
without this flag.
basic test case. The test case would fail if --no_icu_case_mapping
was set.

Fixes: #9445

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ i18n-api semver-minor v8 engine