"James C. McPherson" wrote:
> 
> ...
> 
>   68 STDHEAD=$'<meta http-equiv="cache-control" conStent="no-cache"></meta>
> 
> I think you've got an extra "S" there :-)

Fixed.

> I'm pretty sure that these lines won't work with /usr/bin/sed:

Note that the code now uses /usr/xpg4/bin/sed if it's available (since
/usr/bin/sed is not CSI-conformant and "gsed"'s multibyte support is...
erm... scetchy...).

>   2947  $SED -e '/^#/d' -e $'/^[ \t]*$/d' $REGFILE | while read LINE; do
>   2978  $SED -e '/^#/d' -e $'/^[ \t]*$/d' $cf | while read LINE; do

Why ? $'...' literals handle stuff like '\n' or '\t' like C string
literals, e.g. the literal $'/^[ \t]*$/d' will expand to
'/^[<space><tab>]*$/d' (<space> means the space charatcer, <tab> means
the <tab> character). The idea behind the change was to avoid that the
source has "invisible" tab characters around.

> Apart from that it looks ok to me.

Thanks! :-)

----

Bye,
Roland

-- 
  __ .  . __
 (o.\ \/ /.o) roland.ma...@nrubsig.org
  \__\/\/__/  MPEG specialist, C&&JAVA&&Sun&&Unix programmer
  /O /==\ O\  TEL +49 641 3992797
 (;O/ \/ \O;)
_______________________________________________
tools-discuss mailing list
tools-discuss@opensolaris.org

Reply via email to