Restrict nohash visibility explicitly

Signed-off-by: Liam Gray <[email protected]>

Author: hoxxep

benches/counter.rs

@@ -12,8 +12,8 @@
 // limitations under the License.
 
 use criterion::{black_box, criterion_group, criterion_main, Criterion};
-use prometheus::{Counter, CounterVec, IntCounter, Opts};
 use fnv::FnvBuildHasher;
+use prometheus::{Counter, CounterVec, IntCounter, Opts};
 use std::collections::HashMap;
 use std::sync::{atomic, Arc};
 use std::thread;
@@ -25,7 +25,11 @@ fn bench_counter_with_label_values(c: &mut Criterion) {
     )
     .unwrap();
     c.bench_function("counter_with_label_values", |b| {
-        b.iter(|| counter.with_label_values(&black_box(["eins", "zwei", "drei"])).inc())
+        b.iter(|| {
+            counter
+                .with_label_values(&black_box(["eins", "zwei", "drei"]))
+                .inc()
+        })
     });
 }
 
@@ -52,7 +56,7 @@ fn bench_counter_with_mapped_labels_fnv(c: &mut Criterion) {
         Opts::new("benchmark_counter", "A counter to benchmark it."),
         &["one", "two", "three"],
     )
-        .unwrap();
+    .unwrap();
 
     c.bench_function("counter_with_mapped_labels_fnv", |b| {
         b.iter(|| {

src/nohash.rs

@@ -2,9 +2,9 @@ use std::hash::{BuildHasherDefault, Hasher};
 
 /// Inspired by nohash-hasher, but we avoid the crate dependency because it's in public archive.
 #[derive(Copy, Clone, Debug, Default)]
-pub struct NoHashHasher(u64);
+pub(crate) struct NoHashHasher(u64);
 
-pub type BuildNoHashHasher = BuildHasherDefault<NoHashHasher>;
+pub(crate) type BuildNoHashHasher = BuildHasherDefault<NoHashHasher>;
 
 impl Hasher for NoHashHasher {
     #[inline]

src/vec.rs

@@ -157,7 +157,10 @@ impl<T: MetricVecBuilder> MetricVecCore<T> {
         Ok(h.finish())
     }
 
-    fn get_label_values<'a, V, S: BuildHasher>(&'a self, labels: &'a HashMap<&str, V, S>) -> Result<Vec<&'a str>>
+    fn get_label_values<'a, V, S: BuildHasher>(
+        &'a self,
+        labels: &'a HashMap<&str, V, S>,
+    ) -> Result<Vec<&'a str>>
     where
         V: AsRef<str> + std::fmt::Debug,
     {