Abpostelnicu marked 9 inline comments as done.
Abpostelnicu added inline comments.


================
Comment at: lib/Format/TokenAnnotator.cpp:2398
 
+// Determine if the next token from the closing scope is an inheritance token
+static bool hasMultipleInheritance(const FormatToken &Tok) {
----------------
djasper wrote:
> I don't understand this comment or what the function is supposed to do. It 
> seems to search whether there is an inheritance comma somewhere in the rest 
> of the line.
I've corrected the comment, if we have single inheritance we don't want to 
break onto the next line before the inheritance operand, that's why i'm looking 
for a TT_InheritanceComma, because if there is one then we are dealing with 
multiple inheritance.  


Repository:
  rL LLVM

https://reviews.llvm.org/D30487



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

Reply via email to