Skip to content

Stdlib case mapping differs from latest stable node #1195

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
Closed
dcodeIO opened this issue Mar 27, 2020 · 1 comment · Fixed by #1206
Closed

Stdlib case mapping differs from latest stable node #1195

dcodeIO opened this issue Mar 27, 2020 · 1 comment · Fixed by #1206
Labels
enhancement

Comments

@dcodeIO
Copy link
Member

dcodeIO commented Mar 27, 2020

Affected code points are

origLowerCode != expectLowerCode 42951
  origLowerCode = 18446744069414561735
  expectLowerCode = 18446744069414561736

origUpperCode != expectUpperCode 42952
  origUpperCode = 18446744069414561736
  expectUpperCode = 18446744069414561735

origLowerCode != expectLowerCode 42953
  origLowerCode = 18446744069414561737
  expectLowerCode = 18446744069414561738

origUpperCode != expectUpperCode 42954
  origUpperCode = 18446744069414561738
  expectUpperCode = 18446744069414561737

origLowerCode != expectLowerCode 42997
  origLowerCode = 18446744069414561781
  expectLowerCode = 18446744069414561782

origUpperCode != expectUpperCode 42998
  origUpperCode = 18446744069414561782
  expectUpperCode = 18446744069414561781

Looks like something changed in node's v8 version recently. Tested with node v13.12.0, while earlier versions appear to be fine. cc @MaxGraey
@dcodeIO dcodeIO added the bug label Mar 27, 2020
@dcodeIO dcodeIO mentioned this issue Mar 27, 2020
Merged
@MaxGraey
Copy link
Member

MaxGraey commented Mar 27, 2020 •
edited
Loading

Yeah, node.js updated unicode to latest 13.0.0 (we use 12.1.0 currently): nodejs/node@74e7e72

@MaxGraey MaxGraey added enhancement and removed bug labels Mar 27, 2020
@MaxGraey MaxGraey mentioned this issue Apr 3, 2020
Merged
@ekmixon ekmixon mentioned this issue Nov 5, 2022
Open
@MaxMood96 MaxMood96 mentioned this issue Nov 5, 2022
Open
@MaxMood96 MaxMood96 mentioned this issue Dec 25, 2022
Open
@ekmixon ekmixon mentioned this issue Dec 26, 2022
Open
@MaxMood96 MaxMood96 mentioned this issue Jun 23, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update to Unicode 13.0.0
2 participants
@dcodeIO @MaxGraey