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

Review usage of .to_lowercase() #17777

Open
SimonSapin opened this issue Jul 18, 2017 · 0 comments
Open

Review usage of .to_lowercase() #17777

SimonSapin opened this issue Jul 18, 2017 · 0 comments
Labels

Comments

@SimonSapin
Copy link
Member

@SimonSapin SimonSapin commented Jul 18, 2017

It does Unicode case mapping, whereas some parts of the web platform that are case-insensitive are only so in the ASCII range. Where appropriate, switch to .to_ascii_lowercase() instead (with use std::ascii::AsciiExt;).

@KiChjang KiChjang added the I-refactor label Jul 18, 2017
@frewsxcv frewsxcv self-assigned this Jul 26, 2017
bors-servo added a commit that referenced this issue Jul 27, 2017
Audit usages of unicode case-changing methods.

Part of #17777.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17883)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Jul 28, 2017
Audit usages of unicode case-changing methods.

Part of #17777.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17883)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Jul 28, 2017
Audit usages of unicode case-changing methods.

Part of #17777.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17883)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Jul 29, 2017
Audit usages of unicode case-changing methods.

Part of #17777.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17883)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Aug 2, 2017
Audit usages of unicode case-changing methods.

Part of #17777.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17883)
<!-- Reviewable:end -->
@frewsxcv frewsxcv removed their assignment Aug 2, 2017
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Aug 2, 2017
…from frewsxcv:frewsxcxv-lowercase); r=SimonSapin

Part of servo/servo#17777.

Source-Repo: https://github.com/servo/servo
Source-Revision: 5fff90c73f77dd9ed41b692e6b3ea960a29ee8a8

--HG--
extra : subtree_source : https%3A//hg.mozilla.org/projects/converted-servo-linear
extra : subtree_revision : 4dc82b9feba039e3f300ec5884d04ecab21f13e9
JerryShih pushed a commit to JerryShih/gecko-dev that referenced this issue Aug 3, 2017
…from frewsxcv:frewsxcxv-lowercase); r=SimonSapin

Part of servo/servo#17777.

Source-Repo: https://github.com/servo/servo
Source-Revision: 5fff90c73f77dd9ed41b692e6b3ea960a29ee8a8
aethanyc pushed a commit to aethanyc/gecko-dev that referenced this issue Aug 3, 2017
…from frewsxcv:frewsxcxv-lowercase); r=SimonSapin

Part of servo/servo#17777.

Source-Repo: https://github.com/servo/servo
Source-Revision: 5fff90c73f77dd9ed41b692e6b3ea960a29ee8a8
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Oct 1, 2019
…from frewsxcv:frewsxcxv-lowercase); r=SimonSapin

Part of servo/servo#17777.

Source-Repo: https://github.com/servo/servo
Source-Revision: 5fff90c73f77dd9ed41b692e6b3ea960a29ee8a8

UltraBlame original commit: 374c0f030db82ebaff91dda56d02dfb0e1d1b2e7
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Oct 1, 2019
…from frewsxcv:frewsxcxv-lowercase); r=SimonSapin

Part of servo/servo#17777.

Source-Repo: https://github.com/servo/servo
Source-Revision: 5fff90c73f77dd9ed41b692e6b3ea960a29ee8a8

UltraBlame original commit: 374c0f030db82ebaff91dda56d02dfb0e1d1b2e7
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Oct 1, 2019
…from frewsxcv:frewsxcxv-lowercase); r=SimonSapin

Part of servo/servo#17777.

Source-Repo: https://github.com/servo/servo
Source-Revision: 5fff90c73f77dd9ed41b692e6b3ea960a29ee8a8

UltraBlame original commit: 374c0f030db82ebaff91dda56d02dfb0e1d1b2e7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.