mathstuf / rust-keyutils

Rust interface to the Linux keyring
BSD 3-Clause "New" or "Revised" License
17 stars 9 forks source link

formatted source code according to the rust guidelines using rustfmt #6

Closed derekstraka closed 8 years ago

mathstuf commented 8 years ago

Thanks. Is there a rationale document somewhere? The rustfmt repo seems to just point to a forest of "FIXME" notes from a cursory check.

Some things I have nits with:

[1]In fact, IMO, strict line length limits tend to lead to worse style because how to break a line is not usually that well covered.

jcreekmore commented 8 years ago

So, rustfmt does accept a rustfmt.toml file in your top-level that lets you specify different options (sadly, this is the best reference I have found for the accepted options: https://github.com/rust-lang-nursery/rustfmt/blob/3ce2b840cf73253cedf86aaf3b6b7a71e27d6b79/src/config.rs#L259-L304)

derekstraka commented 8 years ago

It looks like I can create a toml file for 1 and 3. I'm not sure about the vertical alignment preservation though.