Skip to content

Commit 5f353cb

Browse files
committed
clippy and fmt
1 parent 5d74eee commit 5f353cb

File tree

1 file changed

+14
-12
lines changed

1 file changed

+14
-12
lines changed

contrib/tools/config-docs-generator/src/extract_docs.rs

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -595,9 +595,7 @@ fn strip_type_suffix(value: &str) -> String {
595595
];
596596

597597
for suffix in &suffixes {
598-
if value.ends_with(suffix) {
599-
let without_suffix = &value[..value.len() - suffix.len()];
600-
598+
if let Some(without_suffix) = value.strip_suffix(suffix) {
601599
// Only strip if the remaining part looks like a numeric literal
602600
// (contains only digits, underscores, dots, minus signs, or quotes for string literals)
603601
if !without_suffix.is_empty()
@@ -1217,10 +1215,12 @@ mod tests {
12171215

12181216
let result = extract_config_docs_from_rustdoc(&invalid_rustdoc, &None);
12191217
assert!(result.is_err());
1220-
assert!(result
1221-
.unwrap_err()
1222-
.to_string()
1223-
.contains("Missing 'index' field"));
1218+
assert!(
1219+
result
1220+
.unwrap_err()
1221+
.to_string()
1222+
.contains("Missing 'index' field")
1223+
);
12241224
}
12251225

12261226
#[test]
@@ -1318,11 +1318,13 @@ and includes various formatting.
13181318
assert!(notes[1].contains("Only relevant"));
13191319
assert!(notes[2].contains("Units: milliseconds"));
13201320

1321-
assert!(result
1322-
.0
1323-
.deprecated
1324-
.unwrap()
1325-
.contains("Use `new_field` instead"));
1321+
assert!(
1322+
result
1323+
.0
1324+
.deprecated
1325+
.unwrap()
1326+
.contains("Use `new_field` instead")
1327+
);
13261328

13271329
let toml_example = result.0.toml_example.unwrap();
13281330
assert!(toml_example.contains("# This is a comment"));

0 commit comments

Comments
 (0)