ksyx added inline comments.

================
Comment at: clang/lib/Format/DefinitionBlockSeparator.cpp:80
         return;
       Whitespaces.replaceWhitespace(*TargetToken, NewlineToInsert,
+                                    TargetToken->OriginalColumn,
----------------
HazardyKnusperkeks wrote:
> What if NewlineToInsert is 0? Does that explain @MyDeveloperDay s observation?
Then the line break would be removed (concatenate with last line), which is not 
supposed to happen with Always style but possible at first line of Never style. 

I have located the problem and roughly the solution but it takes some time to 
find a neat fix :)

Thanks for the feedback


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117520/new/

https://reviews.llvm.org/D117520

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to