[PATCH] D43581: [clang-tidy/google] Fix the Objective-C global variable declaration check 🔧
Yan Zhang via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 22 12:55:27 PST 2018
Wizard added inline comments.
================
Comment at: clang-tidy/google/GlobalVariableDeclarationCheck.cpp:92
+ "an appropriate prefix (see "
+ "http://google.github.io/styleguide/objcguide#constants).")
<< Decl->getName() << generateFixItHint(Decl, true);
----------------
aaron.ballman wrote:
> We don't usually put hyperlinks in the diagnostic messages, so please remove this.
>
> My suggestion about describing what constitutes an appropriate prefix was with regards to the style guide wording itself. For instance, that document doesn't mention that two capital letters is good. That's not on you to fix before this patch goes in, of course.
Btw it is actually "2 or more" characters for prefix. I think it makes sense because we need at least 2 or more characters to call it a "prefix" :-)
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D43581
More information about the cfe-commits
mailing list