From: Sven Wischnowsky <wischnow@informatik.hu-berlin.de>
To: zsh-workers@math.gatech.edu
Subject: Re: bug in 3.1.4 completion
Date: Thu, 5 Nov 1998 09:02:07 +0100 (MET) [thread overview]
Message-ID: <199811050802.JAA31133@beta.informatik.hu-berlin.de> (raw)
In-Reply-To: "Bart Schaefer"'s message of Wed, 4 Nov 1998 09:59:09 -0800
"Bart Schaefer" wrote:
>
> On Nov 4, 10:04am, Sven Wischnowsky wrote:
> } Subject: Re: bug in 3.1.4 completion
> }
> } greg@alphatech.com wrote:
> }
> } > % ls f ; now type TAB
> } > -> foo/ ; now type TAB
> } > -> foo/bar ; now type SPACE
> } > -> foo/ba ; notice the "r" got removed.
> } >
> } > This only seems to happen if you complete "foo" then
> } > immediately hit TAB again. If you type part of "bar" and
> } > then TAB it functions correctly.
> }
> } There was a missing fixsuffix() when inserting the unambiguous string
> } for normal completion. It's fixed by the second hunk in the patch
> } below (this is for 3.1.5, but you can easily insert the call to
> } fixsuffix() in 3.1.4 in do_ambiguous() in the else-branch of the `if(p)').
>
> Two points: (1) This patch is NOT for 3.1.5. It's for 3.1.5 *after*
> one of Sven's other patches has been applied. (2) I don't see the bug
> that Greg reported in my copy of 3.1.5, so it presumably was addressed
> by Sven's patch in zsh-workers/4412.
>
> I don't think BOTH the 4412 patch and this one are needed; so which is
> better, Sven? (I suspect Sven's other completion patches may have
> inadvertently backed out 4412, and therefore this fix got introduced
> in its place, but the 4412 one looks better to me.)
>
I prefer the one from message 4528. The patch from 4412 just wanted to
be absolutely sure and if do_single() was called, fixsuffix() got
called twice. The new patch avoids that.
Bye
Sven
--
Sven Wischnowsky wischnow@informatik.hu-berlin.de
next reply other threads:[~1998-11-05 8:10 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
1998-11-05 8:02 Sven Wischnowsky [this message]
-- strict thread matches above, loose matches on Subject: below --
1998-11-05 16:34 Sven Wischnowsky
1998-11-05 10:47 Sven Wischnowsky
1998-11-04 9:04 Sven Wischnowsky
1998-11-04 17:31 ` Bart Schaefer
1998-11-04 17:59 ` Bart Schaefer
1998-11-05 0:32 ` Greg Klanderman
1998-11-05 16:12 ` Greg Klanderman
1998-11-03 17:28 Greg Klanderman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=199811050802.JAA31133@beta.informatik.hu-berlin.de \
--to=wischnow@informatik.hu-berlin.de \
--cc=zsh-workers@math.gatech.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).