search

RReverser / serde-xml-rs

xml-rs based deserializer for Serde (compatible with 1.0+)
https://crates.io/crates/serde-xml-rs
MIT License
272 stars 90 forks source link

Panic at serde_json's Value serialization #128

Open Niedzwiedzw opened 4 years ago

Niedzwiedzw commented 4 years ago

https://github.com/Niedzwiedzw/serde-xml-rs/blob/bfd38c52440cca274819b6f84f12d47ceb438fa3/tests/test.rs#L173 right so this is a showcase of a panic, @RReverser, is this behavior intentional?

punkstarman commented 4 years ago

@Niedzwiedzw, the serializer is quite simply lacking support for a lot of cases.

With the lockdown, I don't have much time to devote to much besides family and my day job.

I will see what I can do soon.

Niedzwiedzw commented 4 years ago

I've opened up a PR for my solution, I guess its good enough