[PATCH] clang-format: Adding two new format options.

2016-09-17 Thread Robin Sommer via cfe-commits
**SpacesAroundConditions** (``bool``) If ``true``, spaces will be inserted around if/for/while conditions. **SpacesAfterNot** (``bool``) If ``true``, spaces will be inserted after ``!``. --- docs/ClangFormatStyleOptions.rst | 6 ++ include/clang/Format/Format.h| 8 lib/Form

[PATCH] D25171: clang-format: Add two new formatting options

2016-10-02 Thread Robin Sommer via cfe-commits
rsmmr created this revision. rsmmr added reviewers: djasper, lodato. rsmmr added a subscriber: cfe-commits. Herald added a subscriber: klimek. This patch adds two new options, feedback welcome. SpacesAroundConditions (bool) If true, spaces will be inserted around if/for/while conditions. Spac

[PATCH] D25171: clang-format: Add two new formatting options

2016-10-05 Thread Robin Sommer via cfe-commits
rsmmr added a comment. Cool, thanks Steve. What's the next step for me to get it committed? https://reviews.llvm.org/D25171 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D25171: clang-format: Add two new formatting options

2016-10-05 Thread Robin Sommer via cfe-commits
rsmmr added a comment. Sure, I'm aiming to use clang-format on a couple of open-source code bases using this style, with the main one being the Bro network security monitor, see www.bro.org and github.com/bro/bro (note the stars and forks :-) Bro is also featured on GitHub's list of security sh

[PATCH] D25171: clang-format: Add two new formatting options

2016-10-05 Thread Robin Sommer via cfe-commits
rsmmr added a comment. Well, last time I counted it was more like 100 contributors---Bro existed before GitHub (and before git). The style-guide is lacking, yes ... what can I say. I don't really want to argue about importance of the project. We would like to use clang-format here and elsewher