Make fields of Atom private

atom!() macros create values through a `const fn` constructor

Author: SimonSapin

Cargo.toml

@@ -22,7 +22,6 @@ lazy_static = "1"
 serde = "1"
 phf_shared = "0.8"
 new_debug_unreachable = "1.0"
-string_cache_shared = {path = "./shared", version = "0.3"}
 
 [[test]]
 name = "small-stack"

integration-tests/src/lib.rs

@@ -157,14 +157,14 @@ macro_rules! assert_eq_fmt (($fmt:expr, $x:expr, $y:expr) => ({
 #[test]
 fn repr() {
     fn check(s: &str, data: u64) {
-        assert_eq_fmt!("0x{:016X}", Atom::from(s).unsafe_data, data);
+        assert_eq_fmt!("0x{:016X}", Atom::from(s).unsafe_data(), data);
     }
 
     fn check_static(s: &str, x: Atom) {
-        assert_eq_fmt!("0x{:016X}", x.unsafe_data, Atom::from(s).unsafe_data);
-        assert_eq!(0x2, x.unsafe_data & 0xFFFF_FFFF);
+        assert_eq_fmt!("0x{:016X}", x.unsafe_data(), Atom::from(s).unsafe_data());
+        assert_eq!(0x2, x.unsafe_data() & 0xFFFF_FFFF);
         // The index is unspecified by phf.
-        assert!((x.unsafe_data >> 32) <= TestAtomStaticSet::get().atoms.len() as u64);
+        assert!((x.unsafe_data() >> 32) <= TestAtomStaticSet::get().atoms.len() as u64);
     }
 
     // This test is here to make sure we don't change atom representation
@@ -182,7 +182,7 @@ fn repr() {
     check("xyzzy01", 0x3130_797A_7A79_7871);
 
     // Dynamic atoms. This is a pointer so we can't verify every bit.
-    assert_eq!(0x00, Atom::from("a dynamic string").unsafe_data & 0xf);
+    assert_eq!(0x00, Atom::from("a dynamic string").unsafe_data() & 0xf);
 }
 
 #[test]