Mirrors/rust-analyzer
2
0
Fork 0
mirror of https://github.com/rust-lang/rust-analyzer synced 2024-12-25 20:43:21 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix feature name

Browse source
This commit is contained in:
Aleksey Kladov 2020-10-05 20:22:44 +02:00
parent e5f252ade7
commit bff812ddfe
2 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
crates/ide/src/completion/complete_postfix/format_like.rs
View file

@ -1,4 +1,4 @@
// Feature: Postfix completion for `format`-like strings. // Feature: Format String Completion.
// //
// `"Result {result} is {2 + 2}"` is expanded to the `"Result {} is {}", result, 2 + 2`. // `"Result {result} is {2 + 2}"` is expanded to the `"Result {} is {}", result, 2 + 2`.
// //

12
xtask/src/codegen/gen_feature_docs.rs
View file

@ -38,7 +38,9 @@ impl Feature {
for block in comment_blocks { for block in comment_blocks {
let id = block.id; let id = block.id;
assert!(is_valid_feature_name(&id), "invalid feature name: {:?}", id); if let Err(msg) = is_valid_feature_name(&id) {
panic!("invalid feature name: {:?}:\n {}", id, msg)
}
let doc = block.contents.join("\n"); let doc = block.contents.join("\n");
let location = Location::new(path.clone(), block.line); let location = Location::new(path.clone(), block.line);
acc.push(Feature { id, location, doc }) acc.push(Feature { id, location, doc })
@ -49,7 +51,7 @@ impl Feature {
} }
} }
fn is_valid_feature_name(feature: &str) -> bool { fn is_valid_feature_name(feature: &str) -> Result<(), String> {
'word: for word in feature.split_whitespace() { 'word: for word in feature.split_whitespace() {
for &short in ["to", "and"].iter() { for &short in ["to", "and"].iter() {
if word == short { if word == short {
@ -58,14 +60,14 @@ fn is_valid_feature_name(feature: &str) -> bool {
} }
for &short in ["To", "And"].iter() { for &short in ["To", "And"].iter() {
if word == short { if word == short {
return false; return Err(format!("Don't capitalize {:?}", word));
} }
} }
if !word.starts_with(char::is_uppercase) { if !word.starts_with(char::is_uppercase) {
return false; return Err(format!("Capitalize {:?}", word));
} }
} }
true Ok(())
} }
impl fmt::Display for Feature { impl fmt::Display for Feature {