Make serde dependency optional, but enabled by default

Author: Thom Chiovoloni

.travis.yml

@@ -8,6 +8,7 @@ rust:
 os:
   - linux
 script:
+  - cargo build --no-default-features
   - cargo build
   - cargo test --all
   - "cd string-cache-codegen && cargo build && cd .."

Cargo.toml

@@ -17,10 +17,14 @@ edition = "2018"
 [lib]
 name = "string_cache"
 
+[features]
+serde_support = ["serde"]
+default = ["serde_support"]
+
 [dependencies]
 precomputed-hash = "0.1"
 lazy_static = "1"
-serde = "1"
+serde = { version = "1", optional = true }
 phf_shared = "0.8"
 new_debug_unreachable = "1.0"
 

src/trivial_impls.rs

@@ -8,6 +8,7 @@
 // except according to those terms.
 
 use crate::{Atom, StaticAtomSet};
+#[cfg(feature = "serde_support")]
 use serde::{Deserialize, Deserializer, Serialize, Serializer};
 use std::borrow::Cow;
 use std::fmt;
@@ -69,6 +70,7 @@ impl<Static: StaticAtomSet> AsRef<str> for Atom<Static> {
     }
 }
 
+#[cfg(feature = "serde_support")]
 impl<Static: StaticAtomSet> Serialize for Atom<Static> {
     fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
     where
@@ -79,6 +81,7 @@ impl<Static: StaticAtomSet> Serialize for Atom<Static> {
     }
 }
 
+#[cfg(feature = "serde_support")]
 impl<'a, Static: StaticAtomSet> Deserialize<'a> for Atom<Static> {
     fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
     where