[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-23 Thread David Gatwood via Phabricator via cfe-commits
dgatwood marked an inline comment as done. dgatwood added inline comments. Comment at: clang-tools-extra/clang-tidy/google/RequireCategoryMethodPrefixesCheck.cpp:23 + Finder->addMatcher(objcMethodDecl(hasDeclContext( +

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-20 Thread David Gatwood via Phabricator via cfe-commits
dgatwood updated this revision to Diff 221142. dgatwood added a comment. Updated the configuration key in the test file. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65917/new/ https://reviews.llvm.org/D65917 Files: clang-tools-extra/clang-tidy/google/CMakeLists.txt

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-20 Thread David Gatwood via Phabricator via cfe-commits
dgatwood updated this revision to Diff 221128. dgatwood marked 5 inline comments as done. dgatwood added a comment. Renamed to ExemptClassPrefixes. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65917/new/ https://reviews.llvm.org/D65917 Files:

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-20 Thread David Gatwood via Phabricator via cfe-commits
dgatwood added inline comments. Comment at: clang-tools-extra/clang-tidy/google/RequireCategoryMethodPrefixesCheck.cpp:23 + Finder->addMatcher(objcMethodDecl(hasDeclContext( + objcCategoryDecl())).bind(CustomCategoryMethodIdentifier), this); +}

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-13 Thread David Gatwood via Phabricator via cfe-commits
dgatwood updated this revision to Diff 220123. dgatwood added a comment. Switched to objcMethodDecl(hasDeclContext(objcCategoryDecl())). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65917/new/ https://reviews.llvm.org/D65917 Files: clang-tools-extra/clang-tidy/google/CMakeLists.txt

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-06 Thread David Gatwood via Phabricator via cfe-commits
dgatwood added inline comments. Comment at: clang-tools-extra/clang-tidy/google/RequireCategoryMethodPrefixesCheck.cpp:57 + } + std::string method_name = method_declaration->getNameAsString(); + auto owning_objc_class_interface = method_declaration->getClassInterface();

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-06 Thread David Gatwood via Phabricator via cfe-commits
dgatwood updated this revision to Diff 219151. dgatwood marked 9 inline comments as done. dgatwood added a comment. Tweaked documentation, and replaced the one remaining instance of getNameAsString(). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65917/new/

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-05 Thread David Gatwood via Phabricator via cfe-commits
dgatwood added a comment. Ah. I expected my comments to be submitted as part of uploading a new revision. Still learning this UI. :-/ Comment at: clang-tools-extra/clang-tidy/google/RequireCategoryMethodPrefixesCheck.cpp:57 + } + std::string method_name =

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-05 Thread David Gatwood via Phabricator via cfe-commits
dgatwood updated this revision to Diff 218953. dgatwood marked 7 inline comments as done. dgatwood added a comment. Fixed a couple of variable names in the .h file, renamed kCustomCategoryMethodIdentifier to CustomCategoryMethodIdentifier, and switched to llvm::any_of. CHANGES SINCE LAST

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-04 Thread David Gatwood via Phabricator via cfe-commits
dgatwood updated this revision to Diff 218812. dgatwood marked 22 inline comments as done. dgatwood added a comment. Incorporated additional feedback. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65917/new/ https://reviews.llvm.org/D65917 Files:

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-03 Thread David Gatwood via Phabricator via cfe-commits
dgatwood marked 17 inline comments as done. dgatwood added inline comments. Comment at: clang-tools-extra/clang-tidy/google/RequireCategoryMethodPrefixesCheck.cpp:57 + } + std::string method_name = method_declaration->getNameAsString(); + auto owning_objc_class_interface =

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-09-03 Thread David Gatwood via Phabricator via cfe-commits
dgatwood updated this revision to Diff 218478. dgatwood added a comment. Incorporated feedback. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65917/new/ https://reviews.llvm.org/D65917 Files: clang-tools-extra/clang-tidy/google/CMakeLists.txt

[PATCH] D65917: [clang-tidy] Added check for the Google style guide's category method naming rule.

2019-08-30 Thread David Gatwood via Phabricator via cfe-commits
dgatwood added a comment. Whoops. I was expecting to get emailed about review comments and never got anything, so I thought it was just not getting reviewed. I'll work on this again. Sorry about that. :-) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D65917: Added clang-tidy module for the Google style guide's category method naming rule.

2019-08-07 Thread David Gatwood via Phabricator via cfe-commits
dgatwood created this revision. dgatwood added a reviewer: stephanemoore. Herald added subscribers: cfe-commits, mgorny. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D65917 Files: clang-tools-extra/clang-tidy/google/CMakeLists.txt