MaybeUninit, serde StdError, unittests

rust-embedded-community / serde-json-core

`serde-json` for `no_std` programs

Apache License 2.0

161 stars 59 forks source link

Closed jordens closed 2 years ago

jordens commented 2 years ago

MaybeUninint instead of the deprecated mem::uninitialized (given the unstable maybe_uninit_slice feature, in the future this should use slice_assume_init_ref)
Two unittests
serde::*::StdError for our Errors
The bump in serde version further is not needed or helpful AFAICT (this is a library).
The reorganization of the code using MaybeUninit in #59 is not necessary

jordens commented 2 years ago

@eldruin thanks for handling the previous PRs! Could you have a look here as well? It would be great if we can land this and then release with #69