search

rabite0 / hunter

The fastest file manager in the galaxy!
Do What The F*ck You Want To Public License
1.31k stars 64 forks source link

Format code using 'cargo fmt' #9

Closed Atul9 closed 5 years ago

rabite0 commented 5 years ago

This is probably the right thing to do, although I must say I prefer some of the formatting as it is. Still thinking about this.

cedws commented 5 years ago

There's not much point in doing this unless CI is set up to do lint checking. The source will eventually stray from this formatting.

rabite0 commented 5 years ago

Yeah, I'm probably not going to merge this fully. Some things just look worse, idiomatic or not, I honestly don't really care too much about that. Some parts are an improvement though. Will pick the nuggets as I see fit some time.

The code in general is probably not all that idiomatic itself, I didn't really read a lot of Rust by others and for example I use way too many folds all over the place, but I just like the style. :D

Thanks for the contribution in any case!