rename to rc_clone_in_vec_init

Author: yonip23

CHANGELOG.md

@@ -3642,6 +3642,7 @@ Released 2018-09-13
 [`range_step_by_zero`]: https://rust-lang.github.io/rust-clippy/master/index.html#range_step_by_zero
 [`range_zip_with_len`]: https://rust-lang.github.io/rust-clippy/master/index.html#range_zip_with_len
 [`rc_buffer`]: https://rust-lang.github.io/rust-clippy/master/index.html#rc_buffer
+[`rc_clone_in_vec_init`]: https://rust-lang.github.io/rust-clippy/master/index.html#rc_clone_in_vec_init
 [`rc_mutex`]: https://rust-lang.github.io/rust-clippy/master/index.html#rc_mutex
 [`recursive_format_impl`]: https://rust-lang.github.io/rust-clippy/master/index.html#recursive_format_impl
 [`redundant_allocation`]: https://rust-lang.github.io/rust-clippy/master/index.html#redundant_allocation
@@ -3658,7 +3659,6 @@ Released 2018-09-13
 [`redundant_slicing`]: https://rust-lang.github.io/rust-clippy/master/index.html#redundant_slicing
 [`redundant_static_lifetimes`]: https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes
 [`ref_binding_to_reference`]: https://rust-lang.github.io/rust-clippy/master/index.html#ref_binding_to_reference
-[`ref_clone_in_vec_init`]: https://rust-lang.github.io/rust-clippy/master/index.html#ref_clone_in_vec_init
 [`ref_option_ref`]: https://rust-lang.github.io/rust-clippy/master/index.html#ref_option_ref
 [`regex_macro`]: https://rust-lang.github.io/rust-clippy/master/index.html#regex_macro
 [`repeat_once`]: https://rust-lang.github.io/rust-clippy/master/index.html#repeat_once

clippy_lints/src/lib.register_all.rs

@@ -263,12 +263,12 @@ store.register_group(true, "clippy::all", Some("clippy_all"), vec![
     LintId::of(ranges::MANUAL_RANGE_CONTAINS),
     LintId::of(ranges::RANGE_ZIP_WITH_LEN),
     LintId::of(ranges::REVERSED_EMPTY_RANGES),
+    LintId::of(rc_clone_in_vec_init::RC_CLONE_IN_VEC_INIT),
     LintId::of(redundant_clone::REDUNDANT_CLONE),
     LintId::of(redundant_closure_call::REDUNDANT_CLOSURE_CALL),
     LintId::of(redundant_field_names::REDUNDANT_FIELD_NAMES),
     LintId::of(redundant_slicing::REDUNDANT_SLICING),
     LintId::of(redundant_static_lifetimes::REDUNDANT_STATIC_LIFETIMES),
-    LintId::of(ref_clone_in_vec_init::REF_CLONE_IN_VEC_INIT),
     LintId::of(reference::DEREF_ADDROF),
     LintId::of(regex::INVALID_REGEX),
     LintId::of(repeat_once::REPEAT_ONCE),

clippy_lints/src/lib.register_lints.rs

@@ -446,6 +446,7 @@ store.register_lints(&[
     ranges::RANGE_PLUS_ONE,
     ranges::RANGE_ZIP_WITH_LEN,
     ranges::REVERSED_EMPTY_RANGES,
+    rc_clone_in_vec_init::RC_CLONE_IN_VEC_INIT,
     redundant_clone::REDUNDANT_CLONE,
     redundant_closure_call::REDUNDANT_CLOSURE_CALL,
     redundant_else::REDUNDANT_ELSE,
@@ -454,7 +455,6 @@ store.register_lints(&[
     redundant_slicing::DEREF_BY_SLICING,
     redundant_slicing::REDUNDANT_SLICING,
     redundant_static_lifetimes::REDUNDANT_STATIC_LIFETIMES,
-    ref_clone_in_vec_init::REF_CLONE_IN_VEC_INIT,
     ref_option_ref::REF_OPTION_REF,
     reference::DEREF_ADDROF,
     regex::INVALID_REGEX,

clippy_lints/src/lib.register_suspicious.rs

@@ -26,7 +26,7 @@ store.register_group(true, "clippy::suspicious", Some("clippy_suspicious"), vec!
     LintId::of(methods::SUSPICIOUS_MAP),
     LintId::of(mut_key::MUTABLE_KEY_TYPE),
     LintId::of(octal_escapes::OCTAL_ESCAPES),
-    LintId::of(ref_clone_in_vec_init::REF_CLONE_IN_VEC_INIT),
+    LintId::of(rc_clone_in_vec_init::RC_CLONE_IN_VEC_INIT),
     LintId::of(suspicious_trait_impl::SUSPICIOUS_ARITHMETIC_IMPL),
     LintId::of(suspicious_trait_impl::SUSPICIOUS_OP_ASSIGN_IMPL),
 ])

clippy_lints/src/lib.rs

@@ -345,14 +345,14 @@ mod ptr_offset_with_cast;
 mod pub_use;
 mod question_mark;
 mod ranges;
+mod rc_clone_in_vec_init;
 mod redundant_clone;
 mod redundant_closure_call;
 mod redundant_else;
 mod redundant_field_names;
 mod redundant_pub_crate;
 mod redundant_slicing;
 mod redundant_static_lifetimes;
-mod ref_clone_in_vec_init;
 mod ref_option_ref;
 mod reference;
 mod regex;
@@ -891,7 +891,7 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
     let max_include_file_size = conf.max_include_file_size;
     store.register_late_pass(move || Box::new(large_include_file::LargeIncludeFile::new(max_include_file_size)));
     store.register_late_pass(|| Box::new(strings::TrimSplitWhitespace));
-    store.register_late_pass(|| Box::new(ref_clone_in_vec_init::RefCloneInVecInit));
+    store.register_late_pass(|| Box::new(rc_clone_in_vec_init::RcCloneInVecInit));
     // add lints here, do not remove this comment, it's used in `new_lint`
 }
 

clippy_lints/src/ref_clone_in_vec_init.rs clippy_lints/src/rc_clone_in_vec_init.rsclippy_lints/src/ref_clone_in_vec_init.rs renamed to clippy_lints/src/rc_clone_in_vec_init.rs

@@ -36,13 +36,13 @@ declare_clippy_lint! {
     /// }
     /// ```
     #[clippy::version = "1.62.0"]
-    pub REF_CLONE_IN_VEC_INIT,
+    pub RC_CLONE_IN_VEC_INIT,
     suspicious,
     "initializing `Arc` or `Rc` in `vec![elem; len]`"
 }
-declare_lint_pass!(RefCloneInVecInit => [REF_CLONE_IN_VEC_INIT]);
+declare_lint_pass!(RcCloneInVecInit => [RC_CLONE_IN_VEC_INIT]);
 
-impl LateLintPass<'_> for RefCloneInVecInit {
+impl LateLintPass<'_> for RcCloneInVecInit {
     fn check_expr(&mut self, cx: &LateContext<'_>, expr: &Expr<'_>) {
         let Some(macro_call) = root_macro_call_first_node(cx, expr) else { return; };
         let Some(VecArgs::Repeat(elem, _)) = VecArgs::hir(cx, expr) else { return; };
@@ -57,7 +57,7 @@ fn emit_lint(cx: &LateContext<'_>, symbol: Symbol, macro_call: &MacroCall) {
 
     span_lint_and_then(
         cx,
-        REF_CLONE_IN_VEC_INIT,
+        RC_CLONE_IN_VEC_INIT,
         macro_call.span,
         &format!("calling `{symbol_name}::new` in `vec![elem; len]`"),
         |diag| {

…ef_clone_in_vec_init/arc/complex_case.rs …rc_clone_in_vec_init/arc/complex_case.rstests/ui/ref_clone_in_vec_init/arc/complex_case.rs renamed to tests/ui/rc_clone_in_vec_init/arc/complex_case.rs tests/ui/ref_clone_in_vec_init/arc/complex_case.rs renamed to tests/ui/rc_clone_in_vec_init/arc/complex_case.rs

@@ -1,4 +1,4 @@
-#![warn(clippy::ref_clone_in_vec_init)]
+#![warn(clippy::rc_clone_in_vec_init)]
 use std::sync::Mutex;
 
 fn main() {

…lone_in_vec_init/arc/complex_case.stderr …lone_in_vec_init/arc/complex_case.stderrtests/ui/ref_clone_in_vec_init/arc/complex_case.stderr renamed to tests/ui/rc_clone_in_vec_init/arc/complex_case.stderr tests/ui/ref_clone_in_vec_init/arc/complex_case.stderr renamed to tests/ui/rc_clone_in_vec_init/arc/complex_case.stderr

@@ -11,7 +11,7 @@ LL | |         2
 LL | |     ];
    | |_____^
    |
-   = note: `-D clippy::ref-clone-in-vec-init` implied by `-D warnings`
+   = note: `-D clippy::rc-clone-in-vec-init` implied by `-D warnings`
    = note: each element will point to the same `Arc` instance
    = help: if this is intentional, consider extracting the `Arc` initialization to a variable
    = help: or if not, initilaize each element individually

…/ref_clone_in_vec_init/arc/custom_arc.rs …i/rc_clone_in_vec_init/arc/custom_arc.rstests/ui/ref_clone_in_vec_init/arc/custom_arc.rs renamed to tests/ui/rc_clone_in_vec_init/arc/custom_arc.rs tests/ui/ref_clone_in_vec_init/arc/custom_arc.rs renamed to tests/ui/rc_clone_in_vec_init/arc/custom_arc.rs

@@ -1,4 +1,4 @@
-#![warn(clippy::ref_clone_in_vec_init)]
+#![warn(clippy::rc_clone_in_vec_init)]
 
 #[derive(Clone)]
 struct Arc;

…ref_clone_in_vec_init/arc/simple_case.rs …/rc_clone_in_vec_init/arc/simple_case.rstests/ui/ref_clone_in_vec_init/arc/simple_case.rs renamed to tests/ui/rc_clone_in_vec_init/arc/simple_case.rs tests/ui/ref_clone_in_vec_init/arc/simple_case.rs renamed to tests/ui/rc_clone_in_vec_init/arc/simple_case.rs

@@ -1,4 +1,4 @@
-#![warn(clippy::ref_clone_in_vec_init)]
+#![warn(clippy::rc_clone_in_vec_init)]
 use std::sync::Arc;
 
 fn main() {