Re: fix some -Wunused-but-set-variable warnings in vmd

2022-01-04 Thread Sebastian Benoit
Claudio Jeker(cje...@diehard.n-r-g.com) on 2022.01.04 14:12:02 +0100: > On Tue, Jan 04, 2022 at 10:58:41AM +0100, Claudio Jeker wrote: > > This are obvious and easy to fix unused but set variables. > > There are more in vioscsi.c but those are actually used if compiled with > > DEBUG set. > > The

Re: fix some -Wunused-but-set-variable warnings in vmd

2022-01-04 Thread Dave Voutila
Claudio Jeker writes: > On Tue, Jan 04, 2022 at 10:58:41AM +0100, Claudio Jeker wrote: >> This are obvious and easy to fix unused but set variables. >> There are more in vioscsi.c but those are actually used if compiled with >> DEBUG set. > > The changes in loadfile_elf.c, vioqcow2.c and vmd.c

Re: fix some -Wunused-but-set-variable warnings in vmd

2022-01-04 Thread Claudio Jeker
On Tue, Jan 04, 2022 at 08:42:29AM -0500, Dave Voutila wrote: > > Claudio Jeker writes: > > > This are obvious and easy to fix unused but set variables. > > There are more in vioscsi.c but those are actually used if compiled with > > DEBUG set. > > ok dv@, but maybe just nuke that fd variable i

Re: fix some -Wunused-but-set-variable warnings in vmd

2022-01-04 Thread Dave Voutila
Claudio Jeker writes: > This are obvious and easy to fix unused but set variables. > There are more in vioscsi.c but those are actually used if compiled with > DEBUG set. ok dv@, but maybe just nuke that fd variable in vioqcow2 instead? Index: vioqcow2.c ==

Re: fix some -Wunused-but-set-variable warnings in vmd

2022-01-04 Thread Claudio Jeker
On Tue, Jan 04, 2022 at 10:58:41AM +0100, Claudio Jeker wrote: > This are obvious and easy to fix unused but set variables. > There are more in vioscsi.c but those are actually used if compiled with > DEBUG set. The changes in loadfile_elf.c, vioqcow2.c and vmd.c are trivial and can be committed o

fix some -Wunused-but-set-variable warnings in vmd

2022-01-04 Thread Claudio Jeker
This are obvious and easy to fix unused but set variables. There are more in vioscsi.c but those are actually used if compiled with DEBUG set. -- :wq Claudio Index: loadfile_elf.c === RCS file: /cvs/src/usr.sbin/vmd/loadfile_elf.c,v