dtolnay / serde-yaml

Strongly typed YAML library for Rust
Apache License 2.0
965 stars 164 forks source link

Replace Arc->Rc, AtomicUsize->Cell #254

Closed dtolnay closed 2 years ago

dtolnay commented 2 years ago

Deserializer's input field already contains a Box<dyn Read> so it is already not Send nor Sync regardless of the things touched here, and nobody has complained so far.

We can't easily switch to Box<dyn Read + Send + Sync> because serde_yaml::Deserializer::from_reader doesn't have those bounds, so I'm not sure what we would do if someone really needs this on multiple threads.