[PATCH] test: optionally print subtest number
Tomi Ollila
tomi.ollila at iki.fi
Tue Dec 13 13:23:31 PST 2011
On Wed, 14 Dec 2011 00:58:35 +0400, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> On Tue, 13 Dec 2011 22:55:18 +0200, Tomi Ollila <tomi.ollila at iki.fi> wrote:
> > On Wed, 14 Dec 2011 00:15:43 +0400, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> > >
> > > Looks good to me. Except for tabs taking too much space. Perhaps the
> > > following would be better?
> > >
> > > printf "%-4s" "[$(($test_count - 1))]"
> >
> > I attempted the same size reduction. Therefore I know that
> > this should to be either
> >
> > printf " %-4s" "[$(($test_count - 1))]"
> > or
> > printf "\t%-4s" "[$(($test_count - 1))]"
> >
> > (or something similar) so that there is space betweem BROKEN and [num]
> >
>
> Oh, apparently, not enough testing on my side.
>
> I vote for the first version with a space.
Have to agree, Now that I checked tabs were not used in output lines
before this addition.
>
> Regards,
> Dmitry
Tomi
>
> > This takes 4 bytes out from width (and drops tab as field separator)
> > (and only few lines goes over 80 char width (some goes even with this
> > reduction). So ...
> >
> >
> > > Regards,
> > > Dmitry
> >
> > Tomi
More information about the notmuch
mailing list