Skip to content

Commit 2ab3a9b

Browse files
committed
More fixes
1 parent b9036a4 commit 2ab3a9b

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

clippy_dev/src/update_lints.rs

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -249,6 +249,7 @@ pub fn rename(old_name: &str, new_name: &str, uplift: bool) {
249249
let lint = &mut lints[old_lint_index];
250250
let old_name_upper = old_name.to_uppercase();
251251
let new_name_upper = new_name.to_uppercase();
252+
lint.name = new_name.into();
252253

253254
if try_rename_file(
254255
Path::new(&format!("tests/ui/{}.rs", old_name)),
@@ -265,12 +266,14 @@ pub fn rename(old_name: &str, new_name: &str, uplift: bool) {
265266
}
266267

267268
let replacements;
268-
let replacements = if lint.module == lint.name
269+
let replacements = if lint.module == old_name
269270
&& try_rename_file(
270271
Path::new(&format!("clippy_lints/src/{}.rs", old_name)),
271272
Path::new(&format!("clippy_lints/src/{}.rs", new_name)),
272273
) {
273-
lint.module = new_name.into();
274+
for lint in lints.iter_mut().filter(|l| l.module == old_name) {
275+
lint.module = new_name.into();
276+
}
274277
replacements = [(&*old_name_upper, &*new_name_upper), (old_name, new_name)];
275278
replacements.as_slice()
276279
} else if !lint.module.contains("::")
@@ -279,15 +282,18 @@ pub fn rename(old_name: &str, new_name: &str, uplift: bool) {
279282
Path::new(&format!("clippy_lints/src/{}/{}.rs", lint.module, new_name)),
280283
)
281284
{
285+
let renamed_mod = format!("{}::{}", lint.module, old_name);
286+
for lint in lints.iter_mut().filter(|l| l.module == renamed_mod) {
287+
lint.module = format!("{}::{}", lint.module, new_name);
288+
}
282289
replacements = [(&*old_name_upper, &*new_name_upper), (old_name, new_name)];
283290
replacements.as_slice()
284291
} else {
285292
replacements = [(&*old_name_upper, &*new_name_upper), ("", "")];
286293
&replacements[0..1]
287294
};
288-
lint.name = new_name.into();
289295

290-
for (_, file) in clippy_lints_src_files() {
296+
for (_, file) in clippy_lints_src_files().filter(|(rel_path, _)| rel_path != OsStr::new("renamed_lints.rs")) {
291297
rewrite_file(file.path(), |s| replace_ident_like(s, replacements));
292298
}
293299

0 commit comments

Comments
 (0)