Merge pull request #807 from epage/clippy

style: Make clippy happy
toml-rs · Nov 4, 2024 · d93f0e6 · d93f0e6
2 parents 17b0c36 + 36fb1ef
commit d93f0e6
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/crates/toml/src/map.rs b/crates/toml/src/map.rs
@@ -135,7 +135,14 @@ impl Map<String, Value> {
         String: Borrow<Q>,
         Q: Ord + Eq + Hash + ?Sized,
     {
-        self.map.remove(key)
+        #[cfg(not(feature = "preserve_order"))]
+        {
+            self.map.remove(key)
+        }
+        #[cfg(feature = "preserve_order")]
+        {
+            self.map.shift_remove(key)
+        }
     }
 
     /// Retains only the elements specified by the `keep` predicate.
@@ -501,7 +508,14 @@ impl<'a> OccupiedEntry<'a> {
     /// Takes the value of the entry out of the map, and returns it.
     #[inline]
     pub fn remove(self) -> Value {
-        self.occupied.remove()
+        #[cfg(not(feature = "preserve_order"))]
+        {
+            self.occupied.remove()
+        }
+        #[cfg(feature = "preserve_order")]
+        {
+            self.occupied.shift_remove()
+        }
     }
 }
 

diff --git a/crates/toml_edit/src/ser/map.rs b/crates/toml_edit/src/ser/map.rs
@@ -1,6 +1,7 @@
 use super::{Error, KeySerializer, SerializeValueArray, ValueSerializer};
 
 #[doc(hidden)]
+#[allow(clippy::large_enum_variant)]
 pub enum SerializeMap {
     Datetime(SerializeDatetime),
     Table(SerializeInlineTable),