search

hoodie / icalendar-rs

📆 icalendar library, in Rust of course - for fun
Apache License 2.0
122 stars 32 forks source link

fix: remove ToString and implement fmt::Display instead #13

Closed hoodie closed 4 years ago

hoodie commented 4 years ago

As pointed out already in #11 (ref std::string::ToString) implementing ToString directly isn't such a great idea, especially since it may panic. Hence this replaces our ToString impl with a Display impl.

matthiasbeyer commented 4 years ago

Forget what I said above. ToString is implemented for all Display types. I'm dumb.