Re: Regression in braces completion

[Peter Stephenson <p.w.stephenson@ntlworld.com>]

On Sun, 12 Oct 2008 00:10:51 -0700
Bart Schaefer <schaefer@brasslantern.com> wrote:
> (That took *way* longer to track down than it should have, because I
> didn't realize just how *many* calls to "compadd -U" there are, and I
> missed the crucial one the first time and ended up chasing a different
> red herring for an hour.  Oy.)

I don't know what they're all doing, either.

> Removing the -U *only* from the compadd on line 634 of _path_files is
> sufficient to fix this particular example, but I wonder if it should be
> removed from all three of the compadd calls in the "we are listing, or
> something" branch (see comment line 605), or if it needs to be omitted
> based on some other condition ... or if we need to know when spelling
> correction has been done so we can only add -U when we have to, or some
> even more complicated set of conditions.

There will be an underlying reason why this particular case inserts the
brace in the wrong place.  It will take many hours of work to find out.

-- 
Peter Stephenson <p.w.stephenson@ntlworld.com>
Web page now at http://homepage.ntlworld.com/p.w.stephenson/