[PATCH] D60139: [clang-tidy] Add bugprone-placement-new-target-type-mismatch check

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 9 11:45:47 PDT 2019


Eugene.Zelenko added inline comments.


================
Comment at: clang-tidy/bugprone/PlacementNewTargetTypeMismatch.cpp:38
+  const CastExpr *Cast = dyn_cast<CastExpr>(PlacementExpr);
+  if (nullptr == Cast) {
+    return;
----------------
JonasToth wrote:
> braces, above the comment does not follow the correct-sentence style
I would put nullptr as second argument for == operator.


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

https://reviews.llvm.org/D60139





More information about the cfe-commits mailing list