[PATCH] [clang-tidy] Make the named parameter check only warn on declarations if a definition is visible.

[Alexander Kornienko]

================
Comment at: clang-tidy/google/NamedParameterCheck.cpp:81
@@ +80,3 @@
+    // Fallback to an unused marker.
+    std::string NewName = " /*unused*/";
+
----------------
I'd make NewName a StringRef which stores the name or "unused", and add " /*" and "*/" when issuing a fixit.

http://reviews.llvm.org/D4772