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

[Alan Barrett <apb%netbsd.org@localhost>]

On Sun, 05 Oct 2014, Alistair Crooks wrote:
> On Sun, Oct 05, 2014 at 02:13:15PM +0000, Alan Barrett wrote:
> >  #ifdef PUFFSDEBUG
> >  extern int puffsdebug; /* puffs_subr.c */
> > -#define DPRINTF(x) if (puffsdebug > 0) printf x
> > -#define DPRINTF_VERBOSE(x) if (puffsdebug > 1) printf x
> > +#define DPRINTF(x) do { \
> > +		if (puffsdebug > 0) printf x; \
> > +	while (/*CONSTCOND*/0)
>
> I think it'd be even more safe to close the block with a '}' before the
> while.

Good idea.

--apb (Alan Barrett)