[PATCH] D63623: [clang-tidy] new check: bugprone-posix-return

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 28 06:35:33 PDT 2019


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





More information about the cfe-commits mailing list