google / fleetspeak-rs

A Fleetspeak connector library for the Rust programming language.
MIT License
7 stars 7 forks source link

Format code using 'cargo fmt' #1

Closed Atul9 closed 4 years ago

panhania commented 4 years ago

I am sorry, but for me after applying these fixes the code looks worse and inconsistent (although I am sure it is consistent according to some set of rules that Rustfmt follows). This is why in general I don't believe in auto-formatters: they lack "the sense of beauty". I always make a very considerate decisions about whitespace placement to improve readability in a particular context, something that you cannot really formalize.

Of course, the sense of beauty is very subjective. I am sure that there are people who would strongly disagree with me here. However, since I am probably the most frequent reader of this code, I would prefer to keep it my way :)

So, unfortunately, I cannot merge this. But thank you for your effort, I am sure that there are projects that find it very useful!