Hi Eric,

* Ralf Wildenhues wrote on Mon, Apr 10, 2006 at 01:34:01PM CEST:
> * Eric Blake wrote on Sun, Apr 09, 2006 at 03:17:36PM CEST:
> > 
> > > Erm, instead of the
> > >   :; $1
> > > 
> > > could we just have
> > >   m4_default([$1], [:])
> > > 
> > > please?  This avoids the size overhead incurred for the non-pathological
> > > case.

> I have a real argument against `:;': it clutters the trace output of
>   ./testsuite -x
> 
> by adding a line
>   + : 

Which is bogus, as the part changed is only invoked when not tracing.
D'oh.  I've applied your original patch as-is.  Sorry for the noise.

Cheers,
Ralf


Reply via email to