Date: Thu, 14 Aug 1997 11:31:11 +0300 (IDT)
From: Eli Zaretskii <eliz AT is DOT elta DOT co DOT il>
To: Oberhumer Markus <k3040e4 AT c210 DOT edvz DOT uni-linz DOT ac DOT at>
cc: djgpp-workers <djgpp-workers AT delorie DOT com>
Subject: Re: c1args.c patches
In-Reply-To: <199708101940.PAA16531@delorie.com>
Message-ID: <Pine.SUN.3.91.970814113020.6734A-100000@is>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Precedence: bulk


On Sun, 10 Aug 1997, Oberhumer Markus wrote:

> + static __inline__ int c1_stackavail(void)

If we put this into the startup code, why make it static?  I think it
should be made global, so applications could use it.  I suggest we
call it `stackavail', export its prototype in, say, <stdlib.h>, and
add docs to advertise it.

> !     /* quoting is only recognized at the beginning of an argument */
> !     else if (ep == bp && !quote && (*ep == '\'' || *ep == '"'))
>       {

I didn't actually test this change, but if it indeed only allows
quotes at the beginning of an argument, I think this is bad idea: it
breaks wildcard expansion on LFN platforms for pathnames with embedded
spaces.  Consider this:

		  ls /*/'Program Files'

If you only allow the quote at the beginning, you cannot say such
things, because the wildcard will have to be quoted as well.  The fact
that the current version allows this is a feature, not a bug.

Can you explain what's the bug with names like i'vebeen.org is?
Doesn't i\'vebeen.org work?

(For the record: Markus has sent me these patches a long time ago
asking for my comments, but they got buried in my backlog.)