[PATCH] D57787: [clang-tidy] modernize-avoid-c-arrays: avoid main function (PR40604)

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 6 11:09:01 PST 2019


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM aside from a nit.



================
Comment at: clang-tidy/modernize/AvoidCArraysCheck.cpp:36-38
+  if (!FD) // If not a FunctionDecl, then certainly not main entry function.
+    return false;
+  return FD->isMain();
----------------
`return FD ? FD->isMain() : false;` ?


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D57787





More information about the cfe-commits mailing list