Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/fs/puffs



>   | Make this build again without debugging enabled; DPRINTF() can end up
>   | as empty, and in an if conditional, you then need braces if that's the
>   | only potential body.
>
> That change makes no sense to me - the original code was
>
> 	if (error)
> 		DPRINTF((....));
>
> and even if DPRINTF() could produce nothing, the result would be
>
> 	if (error)
> 		;
>
> which is perfectly good C (useless, but acceptable).  Adding { } around
> the ';' changes nothing.

Well...

    compile  puffs/puffs_vnops.o
/u/build/HEAD/src/sys/fs/puffs/puffs_vnops.c: In function 'zerofill_lastpage':
/u/build/HEAD/src/sys/fs/puffs/puffs_vnops.c:1160:70: error: suggest braces around empty body in an 'if' statement [-Werror=empty-body]
   DPRINTF(("zero-fill 0x%lx@0x%" PRIx64 " => %d\n", len, off, error));
                                                                      ^
cc1: all warnings being treated as errors

*** Failed target:  puffs_vnops.o

> But in any case, apb changed it yesterday, so DPRINTF() never produces
> nothing (personally I'm not sure that should have been needed, but that's
> beside the point).

Ah.  It's quite possible that my latest update was from before
that commit.

> What was the error that resulted in the addition of the {} ?

See above.

Regards,

- Håvard


Home | Main Index | Thread Index | Old Index