[LLVMdev] [cfe-dev] Any objections to turning on -Wunused-parameter? (and any other -Wunused-* that are off?)

[Renato Golin]

On 18 July 2015 at 10:48, Chandler Carruth <chandlerc at gmail.com> wrote:
> But currently -Wunused-parameter is hard disabled. I've not checked to see
> if there are any other disabling of -Wunused-* variants, but I'd like to
> move the project toward firmly enabling them and being clean with them.

+1


> The technique I'd like to use is leaving out names of unused parameters,
> using a /*CommentedName*/ if there is a useful name, otherwise just omitting
> the name.

I'd just get rid of everything. Often, outdated comments can be more
harmful than no comments.

cheers,
--renato