[PATCH] D30487: ClangFormat - Add option to break before inheritance separation operator in class declaration
Daniel Jasper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 9 07:35:15 PST 2017
djasper added a comment.
I think the patch is fine, except for the name of the flag. It is not breaking inheritance ;).
Maybe BreakBeforeInhertianceColonAndComma, but that's pretty long still. I think maybe we can shorten this to BreakBeforeInhertianceComma, as it never makes sense to break before the comma if we keep the ":" on the old line. What do you think?
Repository:
rL LLVM
https://reviews.llvm.org/D30487
More information about the cfe-commits
mailing list