Skip to content

Commit

Permalink
Use swap_remove on IndexMap
Browse files Browse the repository at this point in the history 
Plain remove is deprecated.
  • Loading branch information
@dtolnay
dtolnay committed Sep 9, 2019
1 parent d941bfa commit b3d33fe
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions src/map.rs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,10 @@ impl Map<String, Value> {
String: Borrow<Q>,
Q: Ord + Eq + Hash,
{
self.map.remove(key)

This comment has been minimized.

Sign in to view

Copy link
@davidgraeff

davidgraeff Nov 4, 2019 •
edited
Loading

Hm. I'm getting:

error[E0599]: no method named `swap_remove` found for type `indexmap::map::OccupiedEntry<'a, std::string::String, value::Value>` in the current scope
   --> /home/david/.cargo/registry/src/github.com-1ecc6299db9ec823/serde_json-1.0.41/src/map.rs:699:30
    |
699 |         return self.occupied.swap_remove();
    |                              ^^^^^^^^^^^ method not found in `indexmap::map::OccupiedEntry<'a, std::string::String, value::Value>`

error: aborting due to previous error

since updating. indexmap is at 1.1.0. Is there a reason for this change, because of this comment in indexmap:

/// NOTE: Same as .swap_remove
    ///
    /// Computes in **O(1)** time (average).
    pub fn remove<Q: ?Sized>(&mut self, key: &Q) -> Option<V>

Edit: Tagged the wrong line. This comment is meant to be for line 695
#[cfg(feature = "preserve_order")]
return self.map.swap_remove(key);
#[cfg(not(feature = "preserve_order"))]
return self.map.remove(key);
}

/// Gets the given key's corresponding entry in the map for in-place
Expand Down Expand Up @@ -692,7 +695,10 @@ impl<'a> OccupiedEntry<'a> {
/// ```
#[inline]
pub fn remove(self) -> Value {
self.occupied.remove()
#[cfg(feature = "preserve_order")]
return self.occupied.swap_remove();
#[cfg(not(feature = "preserve_order"))]
return self.occupied.remove();
}
}

Expand Down

0 comments on commit b3d33fe

Please sign in to comment.