Bug 188928

Summary: YARR: Update UCS canonicalization tables for Unicode 11
Product: WebKit Reporter: Michael Saboff <msaboff>
Component: JavaScriptCoreAssignee: Michael Saboff <msaboff>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ews-watchlist, keith_miller, mark.lam, saam, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Michael Saboff 2018-08-24 13:30:12 PDT
The generated Yarr file YarrCanonicalizeUCS2.cpp haven't been updated for over 2 years.  They need to be updated for Unicode 11.
Comment 1 Michael Saboff 2018-08-24 13:39:29 PDT
Created attachment 348036 [details]
Patch
Comment 2 EWS Watchlist 2018-08-24 13:41:29 PDT
Attachment 348036 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/yarr/YarrCanonicalizeUCS2.cpp:57:  UCS2_CANONICALIZATION_SETS is incorrectly named. Don't use underscores in your identifier names.  [readability/naming/underscores] [4]
ERROR: Source/JavaScriptCore/yarr/YarrCanonicalizeUCS2.cpp:84:  UCS2_CANONICALIZATION_RANGES is incorrectly named. Don't use underscores in your identifier names.  [readability/naming/underscores] [4]
Total errors found: 2 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Mark Lam 2018-08-24 13:43:15 PDT
Comment on attachment 348036 [details]
Patch

rs=me.
Comment 4 WebKit Commit Bot 2018-08-24 14:15:08 PDT
Comment on attachment 348036 [details]
Patch

Clearing flags on attachment: 348036

Committed r235333: <https://trac.webkit.org/changeset/235333>
Comment 5 WebKit Commit Bot 2018-08-24 14:15:09 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2018-08-24 14:16:16 PDT
<rdar://problem/43696830>