Remove all mention and testing of #[plugin(clippy)] and warn if used

Author: oli-obk

CONTRIBUTING.md

@@ -14,10 +14,11 @@ extern crate clippy_lints;
 pub fn plugin_registrar(reg: &mut Registry) {
     reg.sess.lint_store.with_read_lock(|lint_store| {
         for (lint, _, _) in lint_store.get_lint_groups() {
+            reg.sess
+                .struct_warn("the clippy plugin is being deprecated, please use cargo clippy or rls with the clippy feature")
+                .emit();
             if lint == "clippy" {
-                reg.sess
-                    .struct_warn("running cargo clippy on a crate that also imports the clippy plugin")
-                    .emit();
+                // cargo clippy run on a crate that also uses the plugin
                 return;
             }
         }

README.md

@@ -1,6 +1,3 @@
-#![feature(plugin)]
-#![plugin(clippy)]
-
 #[allow(dead_code)]
 enum Baz {
     One,
@@ -12,7 +9,9 @@ struct Test {
     b: Baz,
 }
 
-fn main() {
+fn main() { }
+
+pub fn foo() {
     use Baz::*;
     let x = Test { t: Some(0), b: One };
 

src/lib.rs

@@ -1,5 +1,3 @@
-#![feature(plugin)]
-#![plugin(clippy)]
 #![deny(clippy)]
 
 #[allow(dead_code)]
@@ -15,3 +13,5 @@ impl Iterator for Foo {
 }
 
 impl ExactSizeIterator for Foo {}
+
+fn main() {}

tests/conf_whitelisted.rs

@@ -1,5 +1,3 @@
-#![feature(plugin)]
-#![plugin(clippy)]
 #![allow(warnings)]
 
 // this should compile in a reasonable amount of time

tests/cc_seme.rs renamed to tests/run-pass/cc_seme.rs

@@ -1,12 +1,11 @@
-#![feature(plugin)]
-#![plugin(clippy)]
 #![deny(mut_mut, zero_ptr, cmp_nan)]
 #![allow(dead_code)]
 
-#[macro_use]
-extern crate lazy_static;
+// compiletest + extern crates doesn't work together
+//#[macro_use]
+//extern crate lazy_static;
 
-use std::collections::HashMap;
+//use std::collections::HashMap;
 
 // ensure that we don't suggest `is_nan` and `is_null` inside constants
 // FIXME: once const fn is stable, suggest these functions again in constants
@@ -17,6 +16,7 @@ static mut BUH: bool = 42.0 < std::f32::NAN;
 
 #[allow(unused_variables, unused_mut)]
 fn main() {
+    /*
     lazy_static! {
         static ref MUT_MAP : HashMap<usize, &'static str> = {
             let mut m = HashMap::new();
@@ -26,6 +26,7 @@ fn main() {
         static ref MUT_COUNT : usize = MUT_MAP.len();
     }
     assert_eq!(*MUT_COUNT, 1);
+    */
     // FIXME: don't lint in array length, requires `check_body`
     //let _ = [""; (42.0 < std::f32::NAN) as usize];
 }

tests/ice_exacte_size.rs renamed to tests/run-pass/ice_exacte_size.rs

@@ -1,5 +1,5 @@
-#![feature(plugin)]
-#![plugin(clippy)]
+
+
 
 #[macro_use]
 extern crate serde_derive;
@@ -16,3 +16,5 @@ struct MacroAttributesTest {
 fn macro_attributes_test() {
     let _ = MacroAttributesTest { _foo: 0 };
 }
+
+fn main() {}

tests/issue-825.rs renamed to tests/run-pass/issue-825.rs

@@ -0,0 +1 @@
+fn main() {}

tests/mut_mut_macro.rs renamed to tests/run-pass/mut_mut_macro.rs

@@ -1,6 +1,6 @@
 // error-pattern: error reading Clippy's configuration file
 
 
-#![plugin(clippy(conf_file="../ui/conf_bad_toml.toml"))]
+
 
 fn main() {}

tests/used_underscore_binding_macro.rs renamed to tests/run-pass/used_underscore_binding_macro.rs

@@ -1,6 +1,6 @@
 // error-pattern: error reading Clippy's configuration file: `blacklisted-names` is expected to be a `Vec < String >` but is a `integer`
 
 
-#![plugin(clippy(conf_file="../ui/conf_bad_type.toml"))]
+
 
 fn main() {}

tests/auxiliary/conf_whitelisted.toml renamed to tests/run-pass/whitelist/clippy.toml

@@ -1,4 +1,4 @@
-#![feature(plugin, custom_attribute)]
+#![feature(custom_attribute)]
 
 #![allow(clippy)]
 #![warn(cyclomatic_complexity)]

tests/run-pass/whitelist/conf_whitelisted.rs

@@ -1,4 +1,4 @@
-#![feature(plugin, custom_attribute)]
+#![feature(custom_attribute)]
 
 #![warn(cyclomatic_complexity)]
 #![warn(unused)]

tests/ui/conf_bad_toml.toml renamed to tests/ui/bad_toml/clippy.toml

@@ -1,4 +1,4 @@
-#![feature(plugin, never_type)]
+#![feature(never_type)]
 
 #![warn(diverging_sub_expression)]
 #![allow(match_same_arms, logic_bug)]

tests/ui/conf_bad_toml.rs renamed to tests/ui/bad_toml/conf_bad_toml.rs

@@ -1,4 +1,4 @@
-#![feature(plugin, alloc)]
+#![feature(alloc)]
 #![feature(associated_type_defaults)]
 
 

tests/ui/bad_toml/conf_bad_toml.stderr

@@ -1,4 +1,4 @@
-#![feature(plugin, non_ascii_idents)]
+#![feature(non_ascii_idents)]
 
 #![warn(clippy, pub_enum_variant_names)]
 

tests/ui/conf_bad_type.toml renamed to tests/ui/bad_toml_type/clippy.toml

@@ -1,4 +1,4 @@
-#![feature(plugin, box_syntax)]
+#![feature(box_syntax)]
 
 #![allow(warnings, clippy)]
 

tests/ui/conf_bad_type.rs renamed to tests/ui/bad_toml_type/conf_bad_type.rs

@@ -1,4 +1,4 @@
-#![feature(plugin, custom_attribute)]
+#![feature(custom_attribute)]
 #![warn(excessive_precision)]
 #![allow(print_literal)]
 

tests/ui/bad_toml_type/conf_bad_type.stderr

@@ -1,4 +1,4 @@
-#![feature(plugin, custom_attribute)]
+#![feature(custom_attribute)]
 
 
 use std::collections::*;

tests/ui/conf_bad_arg.rs

@@ -4,8 +4,8 @@
 #![allow(unused, no_effect, unnecessary_operation)]
 #![warn(mut_mut)]
 
-//#![plugin(regex_macros)]
-//extern crate regex;
+
+
 
 fn fun(x : &mut &mut u32) -> bool {
     **x > 0

tests/ui/conf_bad_arg.stderr

@@ -1,4 +1,4 @@
-#![feature(plugin, const_fn)]
+#![feature(const_fn)]
 
 
 #![allow(dead_code)]

tests/ui/conf_bad_toml.stderr

@@ -1,4 +1,4 @@
-#![feature(plugin, box_syntax)]
+#![feature(box_syntax)]
 
 
 #![warn(no_effect, unnecessary_operation)]

tests/ui/conf_bad_type.stderr

@@ -1,5 +1,5 @@
 
-#![plugin(clippy(conf_file="../auxiliary/conf_french_blacklisted_name.toml"))]
+
 
 #![allow(dead_code)]
 #![allow(single_match)]

tests/ui/conf_french_blacklisted_name.stderr

@@ -1,6 +1,6 @@
 // error-pattern: error reading Clippy's configuration file: unknown key `foobar`
 
 
-#![plugin(clippy(conf_file="../auxiliary/conf_unknown_key.toml"))]
+
 
 fn main() {}

tests/ui/conf_path_non_string.rs

@@ -1,4 +1,4 @@
-#![feature(plugin, custom_attribute, stmt_expr_attributes)]
+#![feature(custom_attribute, stmt_expr_attributes)]
 
 #![allow(unused_parens)]