Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sources.redhat.com/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
Date:	Tue, 19 Feb 2002 09:05:01 -0500
From:	Chet Ramey <chet AT nike DOT ins DOT cwru DOT edu>
To:	tbisp AT yahoo DOT com
Subject:	Re: Proposed patch for bash (completion spec problem related)
Cc:	cygwin AT cygwin DOT com, chet AT po DOT cwru DOT edu
Reply-To:	chet AT po DOT cwru DOT edu
Message-ID:	<020219140501.AA76103.SM@nike.ins.cwru.edu>
Read-Receipt-To:	chet AT po DOT CWRU DOT Edu
MIME-Version:	1.0
X-In-Reply-To:	Message from tbisp AT yahoo DOT com of Mon, 18 Feb 2002 17:06:36 -0800 (PST) (id <20020219010636 DOT 21380 DOT qmail AT web21206 DOT mail DOT yahoo DOT com>)

> Possible patch for bash to fix the problem noticed by Jason Tishler in msg:
> http://sources.redhat.com/ml/cygwin/2002-02/msg00581.html
> 
> 
> --- subst.c.old	Mon Nov  5 07:19:50 2001
> +++ subst.c	Mon Feb 18 16:58:50 2002
> @@ -1339,7 +1339,7 @@
>  	break;
>  
>        i = te /* + member (string[te], d) */;
> -      while (member (string[i], d) && whitespace(string[i]))
> +      while (member (string[i], d)/* && whitespace(string[i])*/)
>  	i++;

The right fix is to replace the call to whitespace() with one to spctabnl().
There's another fix needed for skip_to_delim(), but my current sources look
sufficiently different from bash-2.05a (multibyte character support) that
I'm not going to release a patch.

Chet

-- 
``The lyf so short, the craft so long to lerne.'' - Chaucer
( ``Discere est Dolere'' -- chet)

Chet Ramey, CWRU    chet AT po DOT CWRU DOT Edu    http://cnswww.cns.cwru.edu/~chet/

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

- Raw text -