xbolva00 added a comment.

Removed macro support (again...).

I agree with Nico, this warning is on by default so should be as exact as 
possible without many false positives. While we all know that the Chromium's 
false positive case could be rewritten for better code readability, on the 
other side Clang's default warnings shouldn't be annoying.

Nico, PTAL and possibly approve.


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

https://reviews.llvm.org/D66397



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

Reply via email to