rymiel added a comment.

This patch also fixes https://github.com/llvm/llvm-project/issues/52911 (which 
is probably a duplicate anyway)



================
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:583-584
+              ProbablyBracedList ||
+              NextTok->is(tok::l_brace) && LBraceStack.back().PrevTok &&
+                  LBraceStack.back().PrevTok->is(tok::identifier);
 
----------------
    llvm-project/clang/lib/Format/UnwrappedLineParser.cpp:583:71: warning: '&&' 
within '||' [-Wlogical-op-parentheses]
              NextTok->is(tok::l_brace) && LBraceStack.back().PrevTok &&
              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
    llvm-project/clang/lib/Format/UnwrappedLineParser.cpp:583:71: note: place 
parentheses around the '&&' expression to silence this warning
              NextTok->is(tok::l_brace) && LBraceStack.back().PrevTok &&
                                                                      ^
              (
1 warning generated.



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

https://reviews.llvm.org/D150403

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

Reply via email to