about summary refs log tree commit diff
path: root/compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2021-06-30 18:14:16 +0000
committerbors <bors@rust-lang.org>2021-06-30 18:14:16 +0000
commitcadb93b6de1b5ba5aca53c5c4c7987f3c09962a8 (patch)
tree62005f68115f070d9ed6a3cf0a5a42da78d5df95 /compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp
parent3525a6b61acb442af5b08d70d1ced3c3ea3cf1c0 (diff)
parent018be41deedd086191b8ce45895164e0aa7046b0 (diff)
downloadrust-cadb93b6de1b5ba5aca53c5c4c7987f3c09962a8.tar.gz
rust-cadb93b6de1b5ba5aca53c5c4c7987f3c09962a8.zip
Auto merge of #7400 - popzxc:restrict-locales, r=Manishearth
New lint: `disallowed_script_idents`

This PR implements a new lint to restrict locales that can be used in the code,
as proposed in #7376.

Current concerns / unresolved questions:

- ~~Mixed usage of `script` (as a Unicode term) and `locale` (as something that is easier to understand for the broad audience). I'm not sure whether these terms are fully interchangeable and whether in the current form it is more confusing than helpful.~~ `script` is now used everywhere.
- ~~Having to mostly copy-paste `AllowedScript`. Probably it's not a big problem, as the list of scripts is standardized and is unlikely to change, and even if we'd stick to the `unicode_script::Script`, we'll still have to implement custom deserialization, and I don't think that it will be shorter in terms of the amount of LoC.~~ `unicode::Script` is used together with a filtering deserialize function.
- Should we stick to the list of "recommended scripts" from [UAX #31](http://www.unicode.org/reports/tr31/#Table_Recommended_Scripts) in the configuration?

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: ``[`disallowed_script_idents`]``

r? `@Manishearth`
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp')
0 files changed, 0 insertions, 0 deletions