Skip to content

Commit 039bb1e

Browse files
authored
Merge pull request #187 from max-heller/slice-schema-name
Switch schema name for [T] from "&[T]" to "[T]"
2 parents fae27b3 + 37c207c commit 039bb1e

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

source/postcard-schema/src/impls/builtins_nostd.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ impl<T: Schema + ?Sized> Schema for &'_ T {
9090

9191
impl<T: Schema> Schema for [T] {
9292
const SCHEMA: &'static NamedType = &NamedType {
93-
name: "&[T]",
93+
name: "[T]",
9494
ty: &DataModelType::Seq(T::SCHEMA),
9595
};
9696
}

source/postcard-schema/tests/schema.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ fn test_struct_serialize() {
136136
&NamedValue {
137137
name: "f",
138138
ty: &NamedType {
139-
name: "&[T]",
139+
name: "[T]",
140140
ty: &DataModelType::Seq(&NamedType {
141141
name: "u8",
142142
ty: &DataModelType::U8
@@ -156,7 +156,7 @@ fn test_slice_serialize() {
156156
ty: &DataModelType::Struct(&[&NamedValue {
157157
name: "x",
158158
ty: &NamedType {
159-
name: "&[T]",
159+
name: "[T]",
160160
ty: &DataModelType::Seq(&U8_SCHEMA)
161161
}
162162
},]),
@@ -204,7 +204,7 @@ fn owned_punning() {
204204

205205
// TODO: This is wildly repetitive, and likely could benefit from interning of
206206
// repeated types, strings, etc.
207-
assert_eq!(ser_borrowed_schema.len(), 269);
207+
assert_eq!(ser_borrowed_schema.len(), 268);
208208

209209
// Check that we round-trip correctly
210210
let deser_borrowed_schema =

0 commit comments

Comments
 (0)