Skip to content

Commit 55b4d30

Browse files
committed
HashBag: Remove unused get helper
1 parent c92f1e3 commit 55b4d30

File tree

1 file changed

+4
-15
lines changed

1 file changed

+4
-15
lines changed

src/hash_bag.rs

+4-15
Original file line numberDiff line numberDiff line change
@@ -70,19 +70,6 @@ where
7070
self.table.len()
7171
}
7272

73-
pub fn get<Q>(&self, k: &Q) -> Option<&V>
74-
where
75-
K: Borrow<Q>,
76-
Q: Hash + Eq + ?Sized,
77-
{
78-
let hash = make_hash(&self.build_hasher, &k);
79-
self.table.find(hash, |(p, _v)| p.borrow() == k).map(|b| {
80-
// Here we tie the lifetime of self to the value.
81-
let r = unsafe { b.as_ref() };
82-
&r.1
83-
})
84-
}
85-
8673
pub fn get_all<'map, 'key, Q>(&'map self, k: &'key Q) -> GetAllIter<'map, 'key, Q, K, V>
8774
where
8875
K: Borrow<Q>,
@@ -240,11 +227,12 @@ mod test {
240227
assert!(map.len() == 2);
241228
map.insert(1, 2);
242229
assert!(map.len() == 3);
243-
assert!(map.get(&5) == Some(&5));
244230

245231
let mut vals: Vec<_> = map.get_all(&1).map(|kv| kv.1).copied().collect();
246232
vals.sort_unstable();
247233
assert_eq!(&[1, 2], vals.as_slice());
234+
let vals = map.get_all(&5).map(|kv| kv.1).copied().collect::<Vec<_>>();
235+
assert_eq!(&[5], vals.as_slice());
248236
}
249237

250238
#[test]
@@ -258,7 +246,8 @@ mod test {
258246
hellos.sort_unstable();
259247
assert_eq!(&["bob", "world"], hellos.as_slice());
260248

261-
assert_eq!(Some(&"bob"), map.get("bye"));
249+
let vals: Vec<_> = map.get_all("bye").map(|kv| kv.1).copied().collect();
250+
assert_eq!(&["bob"], vals.as_slice());
262251
}
263252

264253
#[test]

0 commit comments

Comments
 (0)