Only make warnings fatal when testing.

That way, a deprecation warning won’t break dependants.

Author: SimonSapin

plugin/src/lib.rs

@@ -12,7 +12,7 @@
 
 #![feature(plugin_registrar, quote, box_syntax)]
 #![feature(rustc_private, slice_patterns)]
-#![deny(warnings)]
+#![cfg_attr(test, deny(warnings))]
 #![allow(unused_imports)]  // for quotes
 
 extern crate syntax;

shared/lib.rs

@@ -11,7 +11,7 @@
 //! the macros crate and the run-time library, in order to guarantee
 //! consistency.
 
-#![deny(warnings)]
+#![cfg_attr(test, deny(warnings))]
 
 #[macro_use] extern crate debug_unreachable;
 extern crate phf;

src/lib.rs

@@ -10,7 +10,7 @@
 #![crate_name = "string_cache"]
 #![crate_type = "rlib"]
 
-#![deny(warnings)]
+#![cfg_attr(test, deny(warnings))]
 #![cfg_attr(test, feature(test, filling_drop))]
 #![cfg_attr(bench, feature(rand))]
 #![cfg_attr(feature = "unstable", feature(unsafe_no_drop_flag, plugin))]