Re: how to deal with variable set but not used warnings?

From: Warner Losh <imp_at_bsdimp.com>
Date: Sun, 3 Jun 2018 15:33:02 -0600
On Sun, Jun 3, 2018 at 3:28 PM, Rick Macklem <rmacklem_at_uoguelph.ca> wrote:

> mmacy has sent me a bunch of warnings of the "variable set but not used"
> kind
> generated by gcc8.
>
> When I've looked at the code, these are for RPC arguments I parse but do
> not
> use at this time.
> I'd  like to leave the code in place, since these arguments may be needed
> in the
> future and it is hard to figure out how to get them years from now, when
> they
> might be needed.
> I can think of 3 ways to handle this:
> 1 - Get rid of the code. (As above, I'd rather not do this.)
> 2 - Wrap the code with "#if 0"/"#endif" or similar. I'll admit that I find
> this rather
>       ugly and tends to make the code harder to follow.
> 3 - Leave the code and add a comment w.r.t. why the variables are set but
> not used.
>
> So, what do others think is the preferable alternative?
> (Or maybe you have a #4 that seems better than any of these.)
>

4. Disable the stupid warning in the Makefile / build system. If you don't
care, and there's a good reason for what you are doing (sounds like there
is), better to just disable the warning as so much useless noise.

Warner
Received on Sun Jun 03 2018 - 19:33:04 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:16 UTC