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.

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

Avoid surrogate math in HasRTLChars(). #9262

Merged
merged 1 commit into from Jan 30, 2018

Conversation

Projects
None yet
3 participants
@jgraham
Copy link
Contributor

jgraham commented Jan 30, 2018

Behavior change: Certain unpaired high surrogates will result in
the text being considered RTL where that wasn't the case before.

MozReview-Commit-ID: JL7wiRjhslC

Upstreamed from https://bugzilla.mozilla.org/show_bug.cgi?id=1428774 [ci skip]

Avoid surrogate math in HasRTLChars().
Behavior change: Certain unpaired high surrogates will result in
the text being considered RTL where that wasn't the case before.

MozReview-Commit-ID: JL7wiRjhslC

Upstreamed from https://bugzilla.mozilla.org/show_bug.cgi?id=1428774 [ci skip]
@jgraham

This comment has been minimized.

Copy link
Contributor Author

jgraham commented Jan 30, 2018

Code reviewed upstream.

@jgraham jgraham merged commit 6e1e0c9 into master Jan 30, 2018

@jgraham jgraham deleted the sync_b5c3d33557ea2fa4698295b07bb076eef47890e7 branch Jan 30, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.