[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