Proposed patch for bash (completion spec problem related)

Jason Tishler jason@tishler.net
Tue Feb 19 09:23:00 GMT 2002


On Tue, Feb 19, 2002 at 09:05:01AM -0500, Chet Ramey wrote:
> On Mon, Feb 18, 2002 at 05:06:36PM -0800, AJ Reins wrote:
> > Possible patch for bash to fix the problem noticed by Jason Tishler in msg:
> > http://sources.redhat.com/ml/cygwin/2002-02/msg00581.html
> > 
> > [snip]

AJ, thanks for tracking down this problem.

> The right fix is to replace the call to whitespace() with one to spctabnl().
> There's another fix needed for skip_to_delim(), ...

Chet, thanks for reviewing AJ's patch.

Corinna, attached is AJ's patch redone with a call to spctabnl() but
without the unknown fix to skip_to_delim().  Nevertheless, Cygwin bash
2.05a, with this patch, can now handle completion specs.  Would you be
willing to release a new bash 2.05a with this patch?

Thanks,
Jason
-------------- next part --------------
--- subst.c.orig	Tue Feb 19 09:29:06 2002
+++ subst.c	Tue Feb 19 09:30:01 2002
@@ -1339,7 +1339,7 @@ split_at_delims (string, slen, delims, s
 	break;
 
       i = te /* + member (string[te], d) */;
-      while (member (string[i], d) && whitespace(string[i]))
+      while (member (string[i], d) && spctabnl (string[i]))
 	i++;
 
       if (string[i])

-------------- next part --------------
--
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/


More information about the Cygwin mailing list