Skip to content
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.

Sign up for GitHub

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

Jump to bottom

deps: backport fix to v8 bug in toUpper('ç') #9828

Closed
wants to merge 2 commits into from
Closed

deps: backport fix to v8 bug in toUpper('ç') #9828

wants to merge 2 commits into from

Commits on Dec 30, 2016

  1. deps: cherry-pick 2f5da9a from V8 upstream 

    Original commit message:
  Fix the uppercasing of U+00E7(ç) and U+00F7(÷)
  Due to a typo in runtime-i18n.js, 'ç'(U+00E7) was not uppercased
  while '÷'(U+00F7) was incorrectly uppercased to '×'(U+00D7).

  Add a comprehensive test for Latin-1 supplemental block
  (U+00A0 ~ U+00FF). (they're special-cased for speed-up and
  needs to have a test for the range.).

  TEST=intl/general/case-mapping
  BUG=v8:5681

  Review-Url: https://codereview.chromium.org/2533033003
  Cr-Commit-Position: refs/heads/master@{nodejs#41331}

PR-URL: nodejs#9828
Fixes: nodejs#9785
    @srl295
    srl295 committed Dec 30, 2016
    Configuration menu
    Copy the full SHA
    32882e5 View commit details
    Browse the repository at this point in the history

  2. deps: add test for v8 bug in toUpper('ç') 

    * add test for ç/Ç in various locales

Reviewed-by: TBD
PR-URL: nodejs#9828
Fixes: nodejs#9785
    @srl295
    srl295 committed Dec 30, 2016
    Configuration menu
    Copy the full SHA
    f9687b4 View commit details
    Browse the repository at this point in the history