[PATCH] D13899: Fix bug in suggested fix that truncated variable names to 1 character.

Samuel Benzaquen via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 11 07:07:54 PST 2015


sbenza added inline comments.

================
Comment at: clang-tidy/misc/UnusedParametersCheck.cpp:25
@@ +24,3 @@
+template <typename T>
+static CharSourceRange removeNode(const MatchFinder::MatchResult &Result,
+                                  const T *PrevNode, const T *Node,
----------------
aaron.ballman wrote:
> Are we preferring anonymous namespaces over static functions these days?
I didn't know there was a preference.
I usually do anon namespaces, but these functions were already here as static.


http://reviews.llvm.org/D13899





More information about the cfe-commits mailing list