fix up new lints and warnings

buffer/src/lib.rs

@@ -9,18 +9,22 @@ Rather than conditionally compile these methods, this library stubs
 out functionality when an allocator isn't available.
 */
 
-#![cfg_attr(not(test), no_std)]
+#![no_std]
 #![deny(missing_docs)]
 
 mod error;
 
 #[cfg(feature = "std")]
+#[macro_use]
+#[allow(unused_imports)]
 extern crate std as libstd;
 
 #[cfg(not(feature = "alloc"))]
 extern crate core as std;
 
 #[cfg(any(test, feature = "alloc"))]
+#[macro_use]
+#[allow(unused_imports)]
 extern crate alloc;
 #[cfg(feature = "alloc")]
 extern crate core;

buffer/src/value.rs

@@ -1790,6 +1790,8 @@ mod tests {
 mod alloc_tests {
     use super::*;
 
+    use crate::std::string::String;
+
     use sval::Stream as _;
     use sval_derive_macros::*;
 

derive/test/lib.rs

@@ -501,6 +501,7 @@ mod derive_tuple {
 
     #[test]
     fn skip() {
+        #[allow(dead_code)]
         #[derive(Value)]
         struct Tuple(#[sval(skip)] i32, i32);
 
@@ -1045,6 +1046,7 @@ mod derive_enum {
     #[test]
     fn skip() {
         #[derive(Value)]
+        #[allow(dead_code)]
         enum Enum {
             Record {
                 #[sval(skip)]

flatten/src/map.rs

@@ -357,7 +357,6 @@ impl<'sval, S: Stream<'sval>> Stream<'sval> for PassThru<S> {
 
 #[cfg(test)]
 mod tests {
-    use alloc::borrow::ToOwned;
     use sval_derive_macros::*;
 
     use super::*;

fmt/src/lib.rs

@@ -6,10 +6,10 @@ that formats it using the same output that you'd get if you
 derived [`core::fmt::Debug`].
 */
 
-#![cfg_attr(not(test), no_std)]
+#![no_std]
 #![deny(missing_docs)]
 
-#[cfg(feature = "alloc")]
+#[cfg(any(test, feature = "alloc"))]
 extern crate alloc;
 
 mod writer;

fmt/src/token_write.rs

@@ -535,6 +535,8 @@ impl<'a, W: TokenWrite + ?Sized> TokenWrite for &'a mut W {
 mod tests {
     use super::*;
 
+    use alloc::string::String;
+
     #[test]
     fn escape_debug() {
         let cases = [

src/lib.rs

@@ -79,7 +79,7 @@ The [`Value`] and [`Stream`] traits aren't object-safe themselves, but object-sa
 wrappers are provided by the `sval_dynamic` crate. This wrapper works in no-std.
 */
 
-#![cfg_attr(not(test), no_std)]
+#![no_std]
 #![deny(missing_docs)]
 
 #[cfg(feature = "std")]