JonasToth added inline comments.

================
Comment at: clang-tidy/bugprone/TooSmallLoopVariableCheck.cpp:20
+
+static const char LoopName[] = "forLoopName";
+static const char loopVarName[] = "loopVar";
----------------
JonasToth wrote:
> ztamas wrote:
> > JonasToth wrote:
> > > Please move these variable in the matcher function and make them 
> > > `StringRef` instead of `const char[]`.
> > These variables are used not only inside the matcher function but also in 
> > the check() function.
> > 
> I didn't see that, but they should still be `StringRef` as type.
One more nit i forgot: these variables should be `static` for linkage and be 
CamelCase to match the naming conventions.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53974



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to