Mogball added inline comments.

================
Comment at: clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp:157
     // will clean this up afterwards.
-    MyDiag << FixItHint::CreateReplacement(
-        RemovalRange, (Twine(" /*") + Param->getName() + "*/").str());
+    if (Options.get("CommentOutUnusedParameters", true))
+      MyDiag << FixItHint::CreateReplacement(
----------------
Oh I was thinking that the option would disable editing the parameter at all. 
I.e. it would leave unused parameters untouched or remove the parameter from 
the function signature altogether (if it can).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116488/new/

https://reviews.llvm.org/D116488

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

Reply via email to