[PATCH] D25171: clang-format: Add two new formatting options
Steve O'Brien via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 3 13:39:02 PDT 2016
elsteveogrande accepted this revision.
elsteveogrande added a reviewer: elsteveogrande.
elsteveogrande added a comment.
This revision is now accepted and ready to land.
Thought I'd take a look at the list of diffs and take a stab at a few (a few easy-looking and short ones :) ) to unblock some ppl, and unless @djasper has strong opinions about it I'd say why not.
Looking at other rules "nearby" I see spaces inside and outside square-brackets and angle-brackets, so it makes sense to allow this for conditions, especially if someone thinks this helps make it more readable, and for `not` as well (I personally like that one, it's easy to eyeball right past that `!`.
Also it appears to be off by default and I assume the unit tests work :)
https://reviews.llvm.org/D25171
More information about the cfe-commits
mailing list