extfs audio.in & hp48.in patch
Roland Illig
roland.illig at gmx.de
Mon Oct 18 10:19:22 UTC 2004
Oswald Buddenhagen wrote:
>>-$KERMIT -C "SET EXIT WARNING OFF,REMOTE $*,QUIT"
>>+$KERMIT -C "SET EXIT WARNING OFF,REMOTE $@,QUIT"
>
> counter-productive:
>
> # set -- one two three
> # for i in "you $@ suck"; do echo $i; done
> you one
> two
> three suck
> #
Yes, you're right. Thanks for pointing this out. I have changed this to
$KERMIT -C "SET EXIT WARNING OFF,REMOTE $1,QUIT"
and quoted the places where hp48_cmd is called. I think this is easier
to understand. (And I don't like $*.)
[Offtopic:] Can you tell me a situation where you use $*?
> i suggest you read and internalize a [posix] shell documentation (ash/dash
> seems like a good candidate) before starting to fix shell scripts. btw,
> the autoconf manual gives a good overview of shell incompatibilites -
> for those of you who think one can't overdo quoting.
Well, I think that it is not easy to do overquoting the wrong way. The
approach that I like is to quote everything that I consider to be a
string. This includes arguments to `case'.
Roland
More information about the mc-devel
mailing list