Re: [COGITO PATCH] Optimized print_help()

From: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Date: Jun 15, 2005, 18:58 UTC
Message-ID: <42B07A6F.3000001@lsrfire.ath.cx>
In-Reply-To: <1118795583.3890.27.camel@dv>

Pavel Roskin schrieb:

Show 11 quoted lines

>>> +	cat $toolpath | sed -n '3,/^$/s/^# *//p'
>> 
>> What about also removing UUOC from this line ...
> 
> 
> UUOC?  I had too look it up :-)
> 
> It's not just "useless use of cat", but also useless use of
> redirection in both cases.  I guess sed can exit after it finishes
> processing the range (I don't know if it actually does), so feeding
> it the whole file is not need.

It doesn't matter if the shell opens the file or if sed does it itself, sed's ability to close the file and quit when done doesn't depend on that. So this call is equivalent and has the advantage of being resistant against filenames starting with a "-":

   sed -n '3,/^$/s/^# *//p' <"$toolpath"

Rene