From: Sven Wischnowsky <wischnow@informatik.hu-berlin.de>
To: zsh-workers@math.gatech.edu
Subject: Re: PATCH: Re: Two more (possible) problems with the new completion...
Date: Thu, 5 Nov 1998 14:21:40 +0100 (MET) [thread overview]
Message-ID: <199811051321.OAA03361@beta.informatik.hu-berlin.de> (raw)
In-Reply-To: Peter Stephenson's message of Thu, 05 Nov 1998 13:44:27 +0100
Peter Stephenson wrote:
>
> Sven Wischnowsky wrote:
> > `-W' behaves like `-k', you only need `compctl -W cdpath -/ cd'. There
> > is some extra hacklet that ensures that the old style (giving one
> > directory) still works (in most cases): if the string isn't an
> > parameter name, it is used directly as a directory name. So, unless
> > you have a directory named `$cdpath' (*not* the value of `cdpath') it
> > will produce no matches.
>
> Fine, but ... :-)
>
> In this case the return value of get_user_var() is a pointer to the
> stored array itself, so the array shouldn't be modified. The present
> code does a mischief to the value of $cdpath after the completion is
> finished. The patch below fixes it (I can even do this one myself).
Thanks. The patch below goes on top of this, just to keep gcc quiet.
BTW, in the Peter's glob patch he added some braces to keep gcc from
emitting lots of `ambiguous else' warnings and I've similiar things in
zle_tricky.c. We should remember to fix all of these (and the `char
used as subscript' warnings) before the next official release.
all th
Bye
Sven
*** os/Zle/zle_tricky.c Thu Nov 5 14:17:03 1998
--- Src/Zle/zle_tricky.c Thu Nov 5 14:17:08 1998
***************
*** 3848,3854 ****
if ((pp = get_user_var(cc->withd))) {
dirs = npp =
! (char*) halloc(sizeof(char *)*(arrlen(pp)+1));
while (*pp) {
pl = strlen(*pp);
tp = (char *) halloc(strlen(*pp) + tl);
--- 3848,3854 ----
if ((pp = get_user_var(cc->withd))) {
dirs = npp =
! (char**) halloc(sizeof(char *)*(arrlen(pp)+1));
while (*pp) {
pl = strlen(*pp);
tp = (char *) halloc(strlen(*pp) + tl);
--
Sven Wischnowsky wischnow@informatik.hu-berlin.de
next reply other threads:[~1998-11-05 13:31 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
1998-11-05 13:21 Sven Wischnowsky [this message]
-- strict thread matches above, loose matches on Subject: below --
1998-11-05 12:05 Sven Wischnowsky
1998-11-05 12:44 ` PATCH: " Peter Stephenson
1998-11-05 12:56 ` Peter Stephenson
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=199811051321.OAA03361@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).