From c29a9438e6a56e507b034347c82b73591b6414e8 Mon Sep 17 00:00:00 2001 From: Alex Saveau Date: Fri, 2 Feb 2024 10:23:58 -0800 Subject: [PATCH] Make HashSet::insert return OccupiedEntry Signed-off-by: Alex Saveau --- src/set.rs | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/src/set.rs b/src/set.rs index fdff46d88..2f4a97c9d 100644 --- a/src/set.rs +++ b/src/set.rs @@ -1003,7 +1003,7 @@ where /// match singles.entry(ch) { /// Vacant(single_entry) => { /// // We found a new character for the first time. - /// single_entry.insert() + /// single_entry.insert(); /// } /// Occupied(single_entry) => { /// // We've already seen this once, "move" it to dupes. @@ -2211,7 +2211,7 @@ impl fmt::Debug for OccupiedEntry<'_, T, S, A> { /// /// // Nonexistent key (insert) /// match set.entry("b") { -/// Entry::Vacant(view) => view.insert(), +/// Entry::Vacant(view) => { view.insert(); }, /// Entry::Occupied(_) => unreachable!(), /// } /// assert!(set.contains("b") && set.len() == 2); @@ -2247,7 +2247,7 @@ impl<'a, T, S, A: Allocator> Entry<'a, T, S, A> { { match self { Entry::Occupied(entry) => entry, - Entry::Vacant(entry) => entry.insert_entry(), + Entry::Vacant(entry) => entry.insert(), } } @@ -2442,16 +2442,7 @@ impl<'a, T, S, A: Allocator> VacantEntry<'a, T, S, A> { /// assert!(set.contains("poneyland")); /// ``` #[cfg_attr(feature = "inline-more", inline)] - pub fn insert(self) - where - T: Hash, - S: BuildHasher, - { - self.inner.insert(()); - } - - #[cfg_attr(feature = "inline-more", inline)] - fn insert_entry(self) -> OccupiedEntry<'a, T, S, A> + pub fn insert(self) -> OccupiedEntry<'a, T, S, A> where T: Hash, S: BuildHasher,