Make it a renamed lint instead

Author: russelltg

clippy_lints/src/deprecated_lints.rs

@@ -123,13 +123,3 @@ declare_deprecated_lint! {
     pub UNSAFE_VECTOR_INITIALIZATION,
     "the replacement suggested by this lint had substantially different behavior"
 }
-
-
-/// **What it does:** Nothing, this lint has been deprecated.
-///
-/// **Deprecation reason:** This lint was merged with new_without_default
-declare_deprecated_lint! {
-    pub NEW_WITHOUT_DEFAULT_DERIVE,
-    style,
-    "`fn new() -> Self` without `#[derive]`able `Default` implementation"
-}

clippy_lints/src/lib.rs

@@ -1030,6 +1030,7 @@ pub fn register_plugins(reg: &mut rustc_plugin::Registry<'_>, conf: &Conf) {
 
 pub fn register_renamed(ls: &mut rustc::lint::LintStore) {
     ls.register_renamed("clippy::stutter", "clippy::module_name_repetitions");
+    ls.register_renamed("clippy::new_without_default_derive", "clippy::new_without_default");
 }
 
 // only exists to let the dogfood integration test works.

tests/ui/rename.rs

@@ -11,3 +11,10 @@
 
 #[warn(clippy::stutter)]
 fn main() {}
+
+#[warn(clippy::new_without_default_derive)]
+struct Foo;
+
+impl Foo {
+    fn new() -> Self { Foo }
+}