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
Content-Type: text/plain; charset=us-ascii
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/