yahoo / mdbm

MDBM a very fast memory-mapped key/value store.
BSD 3-Clause "New" or "Revised" License
992 stars 111 forks source link

Clang format? #70

Open zwoop opened 7 years ago

zwoop commented 7 years ago

Curious, have you thought about running clang-format over the source? I find it refreshingly useful to automate indentation, since it takes the subjective aspect out of it (just agree on a .clang-format config).

I tried it quickly, and I have to say it looks a lot nicer :-).

zwoop commented 7 years ago

Any more thought on this? I'd be willing to provide a .clang-format, as well as providing tools around getting clang-format binaries. Fwiw, for another project I work on, we use pre-built binaries for this, such that we all run the same version of clang-format (otherwise, there is chaos).

timrc-git commented 7 years ago

Please ignore my accidental thumbs up. I haven't had a chance to play with clang-format yet.