[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] test defs: add subroutine for input unindenting
From: |
Stefano Lattarini |
Subject: |
Re: [PATCH] test defs: add subroutine for input unindenting |
Date: |
Thu, 17 Feb 2011 19:18:52 +0100 |
User-agent: |
KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; ) |
Hello Peter.
On Thursday 17 February 2011, Peter Rosin wrote:
> Den 2011-02-15 21:19 skrev Ralf Wildenhues:
> > It's a bit of a shame this uses a temp file though. awk
> > should be able to cope without (but be sure to try old
> > Solaris awk if you go for it).
>
> Ahh, who needs awk? :-)
>
> [CUT]
>
Wow, this seems really clever! (I say "seems" because I must
admit I haven't tried to read it in detail, nor to really
understand it).
However ...
> Too much of a maintenance hassle? You tell me.
>
... I do think it's really too much of a maintenance hassle
*for the use of the function we expect to do*. If we were
writing a library of general-purpose shell functions, I'd
definitely choose your implementation over mine, but in the
present situation I see no real advatange. Sorry.
Anyway, Ralf has the final word here, so let's wait to hear
from him too.
> Cheers,
> Peter
>
>
Regards,
Stefano
- Re: [Ping PATCHES] {master} Optimize tests `instspc-*.test' for speed., Ralf Wildenhues, 2011/02/14
- Re: [Ping PATCHES] {master} Optimize tests `instspc-*.test' for speed., Stefano Lattarini, 2011/02/15
- [PATCH] test defs: add subroutine for input unindenting (was: Re: [Ping PATCHES] {master} Optimize tests `instspc-*.test' for speed.), Stefano Lattarini, 2011/02/15
- Re: [PATCH] test defs: add subroutine for input unindenting, Ralf Wildenhues, 2011/02/15
- Re: [PATCH] test defs: add subroutine for input unindenting, Stefano Lattarini, 2011/02/15
- Re: [PATCH] test defs: add subroutine for input unindenting, Ralf Wildenhues, 2011/02/16
- Re: [PATCH] test defs: add subroutine for input unindenting, Stefano Lattarini, 2011/02/16
- Re: [PATCH] test defs: add subroutine for input unindenting, Ralf Wildenhues, 2011/02/17
- Re: [PATCH] test defs: add subroutine for input unindenting, Peter Rosin, 2011/02/17
- Re: [PATCH] test defs: add subroutine for input unindenting, Peter Rosin, 2011/02/17
- Re: [PATCH] test defs: add subroutine for input unindenting,
Stefano Lattarini <=
- Re: [PATCH] test defs: add subroutine for input unindenting, Peter Rosin, 2011/02/18
- Re: [PATCH] test defs: add subroutine for input unindenting, Peter Rosin, 2011/02/18
- Re: [PATCH] test defs: add subroutine for input unindenting, Ralf Wildenhues, 2011/02/21
- Re: [PATCH] test defs: add subroutine for input unindenting, Stefano Lattarini, 2011/02/21
- Re: [PATCH] test defs: add subroutine for input unindenting, Ralf Wildenhues, 2011/02/21
- Re: [PATCH] test defs: add subroutine for input unindenting, Peter Rosin, 2011/02/28
- Re: [PATCH] test defs: add subroutine for input unindenting, Peter Rosin, 2011/02/28
- Re: [PATCH] test defs: add subroutine for input unindenting, Stefano Lattarini, 2011/02/28
- Re: [PATCH] test defs: add subroutine for input unindenting, Ralf Wildenhues, 2011/02/28
- Re: [PATCH] test defs: add subroutine for input unindenting, Peter Rosin, 2011/02/28