-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Update script detection for Unicode 10 #9401
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one tiny nit that will maybe avoid a bug in a future where we enable Zanabazar Square...
src/mbgl/util/i18n.cpp
Outdated
@@ -239,9 +240,12 @@ DEFINE_IS_IN_UNICODE_BLOCK(HalfwidthandFullwidthForms, 0xFF00, 0xFFEF) | |||
// DEFINE_IS_IN_UNICODE_BLOCK(Takri, 0x11680, 0x116CF) | |||
// DEFINE_IS_IN_UNICODE_BLOCK(Ahom, 0x11700, 0x1173F) | |||
// DEFINE_IS_IN_UNICODE_BLOCK(WarangCiti, 0x118A0, 0x118FF) | |||
// DEFINE_IS_IN_UNICODE_BLOCK(ZanabazarSquare, 0x11A00, 0x11A00) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't this block actually end at 0x11a4f
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow, good eye! Fixed in 3d50fd902e0ce480e2d12f7a583ffc53c4a004e0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also fixed in mapbox/mapbox-gl-js#4940.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Long may we remember Zanabazar
Updated script detection code to reflect changes in Unicode 10 and UTR 50 revision 17.
This change updates the script detection code to reflect changes in Unicode 10 and UTR #50 revision 17.
This is a port of mapbox/mapbox-gl-js#4919.
/cc @ChrisLoer