Display is now closer behavior to clojure's pr-str of edn.

naomijub / edn-rs

[DEPRECATED]: Crate to parse and emit EDN

https://crates.io/crates/edn-rs

MIT License

81 stars 14 forks source link

Display is now closer behavior to clojure's pr-str of edn. #142

Closed Grinkers closed 7 months ago

Grinkers commented 8 months ago

Mostly cosmetic, as having extra , and such is within spec.

I think before there was concern about iter().peekable()? Or maybe misunderstandings from a bunch of other commits. Here's a small example of this pattern, but it gets completely optimized out. This is actually faster because there's no String being allocated, and it's going straight into the fmt buffer. https://godbolt.org/z/n1qfbTWs8

Grinkers commented 8 months ago

Probably easier to view the two commits in isolation. One is just a rename, but makes the text diff large.