Add clone_from to Map; Map::append now uses IndexMap::extend

serde-rs · Nov 17, 2022 · 3347248 · 3347248
1 parent ca41bdd
commit 3347248
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/map.rs b/src/map.rs
@@ -197,9 +197,8 @@ impl Map<String, Value> {
     #[inline]
     pub fn append(&mut self, other: &mut Self) {
         #[cfg(feature = "preserve_order")]
-        for (k, v) in mem::replace(&mut other.map, MapImpl::default()) {
-            self.map.insert(k, v);
-        }
+        self.map
+            .extend(mem::replace(&mut other.map, MapImpl::default()));
         #[cfg(not(feature = "preserve_order"))]
         self.map.append(&mut other.map);
     }
@@ -304,6 +303,11 @@ impl Clone for Map<String, Value> {
             map: self.map.clone(),
         }
     }
+
+    #[inline]
+    fn clone_from(&mut self, source: &Self) {
+        self.map.clone_from(&source.map)
+    }
 }
 
 impl PartialEq for Map<String, Value> {