Re: [PATCH] test defs: add subroutine for input unindenting

[Ralf Wildenhues]

* Stefano Lattarini wrote on Wed, Feb 16, 2011 at 12:29:02AM CET:
> On Tuesday 15 February 2011, Ralf Wildenhues wrote:
> > The space after : is not portable.  The sed script could be shortened to
> >   sed -n "/[^ $tab].*$/{
> >             s///p
> >             d
> >           }"
> >
> Did you mean 'q' instead of 'd' here, right?

Yes.  Sorry about that.

> > It's a bit of a shame this uses a temp file though.  awk should be able
> > to cope without
> >
> Yes, if I could assume "new" awk.  But traditional awk lacks the 'sub()'
> and 'gsub()' builtins, so I really really don't want to go down that road.
> Moreover, the use of a tempfile is not a big issue here, because we can
> already assume the existence of a temporary working directory (i.e., the
> one the test runs in) and of a cleanup trap (installed by `tests/defs').

True.  One would assume that it's not common to use more than one
function invocation in parallel:
  deindent foo | deindent > ...

or:
  deindent foo > ... & deindent bar ...

> But if you truly truly dislike the use of the tempfile, I could try to
> use perl instead...  WDYT?

That'll slow things down more than using a temp file, so I'm reasonably
happy with sed+sed right now.

Thanks,
Ralf