[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [patch] /* XXX - possibly run Coproc->name through word expansion? *

From: Chet Ramey
Subject: Re: [patch] /* XXX - possibly run Coproc->name through word expansion? */
Date: Wed, 20 Apr 2016 08:12:09 -0700
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:38.0) Gecko/20100101 Thunderbird/38.7.2

On 4/19/16 11:03 AM, Piotr Grzybowski wrote:

>     > - if it expands to a legal identifier create the coproc
>     This isn't necessary; there's no reason to restrict a coproc name to
>     something stricter than the set of valid executable names.  I suppose
>     I could see restricting it to the same set of valid names as shell
>     functions. 
> ‚Äčok, but I thought along these lines: if the name is not a valid
> identifier, how are you going to refer by this name to the array with the
> descriptors? but maybe checking legal_identifier is too much.

That's a really good point.  We can let people do unwise things (creating
coprocs with weird names) and not let them pollute the shell's namespace
at the same time.


``The lyf so short, the craft so long to lerne.'' - Chaucer
                 ``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, ITS, CWRU    address@hidden    http://cnswww.cns.cwru.edu/~chet/

reply via email to

[Prev in Thread] Current Thread [Next in Thread]