search

kanidm / concread

Concurrently Readable Data Structures for Rust
Mozilla Public License 2.0
343 stars 15 forks source link

RFC: Add optional Serde support for HashMap #106

Closed adamreichold closed 8 months ago

adamreichold commented 8 months ago

The implementation of Deserialize is based on the implementation of Extend.

I can of course remove the first commit as it is admittedly unrelated and unnecessary for Serde support, it just felt better working in a code base that starts without Clippy warnings and stays that way.

adamreichold commented 8 months ago

There's a lot of clippy-calming in there as well

Indeed. I did put it into it a separate commit and can drop it if you prefer.

yaleman commented 8 months ago

Oh no need, just calling it out. I like keeping 📎 happy

Firstyear commented 8 months ago

This is excellent @adamreichold, thank you very much!