erik.pilkington added a comment.

> but don't want to annotate each unused ObjC method parameters with 
> `__attribute__((unused))` or insert `(void)unused_param` into the body of the 
> method to silence the warning since, unlike C/C++ functions, it's not 
> possible to comment out the unused parameters of ObjC methods.

Thats also true of function parameters in C (and Objective-C), so I don't think 
that argument is very convincing. I think it is true that this is a much less 
useful diagnostic for methods though, because they're more likely to be 
overriding something that requires a certain selector. Maybe we should just 
suppress this warning in general for methods (and even virtual functions in 
C++)?


Repository:
  rC Clang

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

https://reviews.llvm.org/D61147



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

Reply via email to