Refactor serializer

Replace rmp_serde::Serializer with a simpler implementation derived from it. Signed-off-by: Emanuele Giaquinta <[email protected]>
aviramha · Nov 26, 2024 · 7746bc6 · 7746bc6
1 parent 2d55c1e
commit 7746bc6
Show file tree

Hide file tree

Showing 4 changed files with 488 additions and 19 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -46,7 +46,6 @@ itoa = { version = "1", default-features = false }
 once_cell = { version = "1", default-features = false, features = ["race"] }
 pyo3 = { version = "^0.22.6", default-features = false, features = ["extension-module"] }
 rmp = { version = "^0.8.14", default-features = false, features = ["std"] }
-rmp-serde = { version = "1", default-features = false }
 serde = { version = "1", default-features = false }
 serde_bytes = { version = "0.11.15", default-features = false, features = ["std"] }
 simdutf8 = { version = "0.1.5", default-features = false, features = ["std"] }

diff --git a/src/serialize/ext.rs b/src/serialize/ext.rs
@@ -31,9 +31,6 @@ impl Serialize for Ext {
         let length = unsafe { PyBytes_GET_SIZE((*ext).data) as usize };
         let data = unsafe { std::slice::from_raw_parts(buffer, length) };
 
-        serializer.serialize_newtype_struct(
-            rmp_serde::MSGPACK_EXT_STRUCT_NAME,
-            &(tag as i8, ByteBuf::from(data)),
-        )
+        serializer.serialize_newtype_variant("", tag as u32, "", &ByteBuf::from(data))
     }
 }