daphnediane created this revision.
daphnediane added a reviewer: djasper.
daphnediane added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
Fix for the formatting options combination of BreakBeforeBinaryOperators: All,
AlignAfterOpenBracket: AlwaysBreak not handling long templates c
daphnediane added a comment.
In https://reviews.llvm.org/D24703#545706, @djasper wrote:
> I think, this is the wrong fix. Instead, a
>
> || (Left.is(TT_TemplateOpener) && !Right.is(TT_TemplateCloser))
>
>
> should be added to the last return statement of this function.
>
> Also, could you pl
daphnediane updated this revision to Diff 71767.
daphnediane added a comment.
Adds test case, changes to suggested fix allowing break after template opener
that is not immediately followed by template closer.
https://reviews.llvm.org/D24703
Files:
lib/Format/TokenAnnotator.cpp
unittests/Fo
daphnediane added a comment.
In https://reviews.llvm.org/D24703#545917, @djasper wrote:
> Looks good. Thank you!
Thanks. Are there additional steps I need to get it commited or do I just wait
for a commiter to notice?
https://reviews.llvm.org/D24703
___
daphnediane added a comment.
ping.
FYI I have found some additional similar cases when
AlignConsecutiveDeclarations is also true, that I'll see if I can document/test
case etc. at some point.
https://reviews.llvm.org/D24703
___
cfe-commits mailin
