gribozavr added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/bugprone/PosixReturnCheck.cpp:61
+ if (const auto *AlwaysTrueOp =
Result.Nodes.getNodeAs<BinaryOperator>("atop")) {
+ diag(AlwaysTrueOp->getOperatorLoc(), "%0 returns zero on success and a
positive value on failure so this check always returns true") <<
getFunctionSpelling(Result, "atop");
+ return;
----------------
80 columns, please.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63623/new/
https://reviews.llvm.org/D63623
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
