================
@@ -183,6 +183,17 @@ static void insertIfFunction(const Decl &D,
     Funcs.insert(FD);
 }
 
+static void insertIfParamOfNotThePrimaryFunction(
----------------
ymand wrote:

This name is hard to parse (long and a bit confusing). Could a simpler name 
suffice (even if less precise)? E..g `insertIfSecondaryFunctionParam`? or even 
better just `insertIfSecondaryParam`? The reader will have to look at the 
function definition comments to understand the name, but I think that will be 
necessary regardless.

https://github.com/llvm/llvm-project/pull/170311
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to