zsh-workers
 help / color / mirror / code / Atom feed
* Re: PATCH: was: Re: zsh bug
@ 1999-05-26  6:26 Sven Wischnowsky
  0 siblings, 0 replies; 2+ messages in thread
From: Sven Wischnowsky @ 1999-05-26  6:26 UTC (permalink / raw)
  To: zsh-workers; +Cc: Glenn Trigg


I wrote this patch:

> --- Src/zle_tricky.c	Tue May 25 10:32:35 1999
> +++ Src/zle_tricky.c.old	Mon May 10 13:21:55 1999
> @@ -1069,14 +1069,13 @@
>  	    we = ll;
>  	tt = (char *)line;
>  	if (tmp) {
> -	    linptr += tmp - (char *) line;
>  	    line = (unsigned char *)tmp;
>  	    ll = strlen((char *)line);
> -	    addedx = 0;
>  	}
>  	if (t0 != STRING && inwhat != IN_MATH) {
>  	    if (tmp) {
>  		tmp = NULL;
> +		linptr = (char *)line;
>  		lexrestore();
>  		goto start;
>  	    }

Err, Bart, Greg: sorry, but this patch is reversed, dunno how this
happened. Here is it the right way round.

Bye
 Sven

--- Src/zle_tricky.c.old	Mon May 10 13:21:55 1999
+++ Src/zle_tricky.c	Tue May 25 10:32:35 1999
@@ -1069,13 +1069,14 @@
 	    we = ll;
 	tt = (char *)line;
 	if (tmp) {
+	    linptr += tmp - (char *) line;
 	    line = (unsigned char *)tmp;
 	    ll = strlen((char *)line);
+	    addedx = 0;
 	}
 	if (t0 != STRING && inwhat != IN_MATH) {
 	    if (tmp) {
 		tmp = NULL;
-		linptr = (char *)line;
 		lexrestore();
 		goto start;
 	    }

--
Sven Wischnowsky                         wischnow@informatik.hu-berlin.de


^ permalink raw reply	[flat|nested] 2+ messages in thread

* PATCH: was: Re: zsh bug
@ 1999-05-25  8:39 Sven Wischnowsky
  0 siblings, 0 replies; 2+ messages in thread
From: Sven Wischnowsky @ 1999-05-25  8:39 UTC (permalink / raw)
  To: zsh-workers; +Cc: Glenn Trigg


Glenn Trigg wrote:

> The wierd behaviour happens when I do the following:
> 
> kill -1 $(cat /var/run/ine
> 
> when I hit tab there, the line completes as:
> 
> kill -1 $(cat /var/run/ineetd.pid
> 
> I.e I get the extra letter which is the letter at the end when I hit
> tab.

This patch is for my modified 3.0.6. The problem was that 1) it didn't 
remember that on the first run through the loop in get_comp_string()
the addedx was removed and that it didn't use the linptr it had
modified for the second run.

This isn't needed for 3.1.5 because there I cleaned up handling of
$(...) some time ago (and for other reasons).

Bye
 Sven

--- Src/zle_tricky.c	Tue May 25 10:32:35 1999
+++ Src/zle_tricky.c.old	Mon May 10 13:21:55 1999
@@ -1069,14 +1069,13 @@
 	    we = ll;
 	tt = (char *)line;
 	if (tmp) {
-	    linptr += tmp - (char *) line;
 	    line = (unsigned char *)tmp;
 	    ll = strlen((char *)line);
-	    addedx = 0;
 	}
 	if (t0 != STRING && inwhat != IN_MATH) {
 	    if (tmp) {
 		tmp = NULL;
+		linptr = (char *)line;
 		lexrestore();
 		goto start;
 	    }

--
Sven Wischnowsky                         wischnow@informatik.hu-berlin.de


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~1999-05-26  6:26 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1999-05-26  6:26 PATCH: was: Re: zsh bug Sven Wischnowsky
  -- strict thread matches above, loose matches on Subject: below --
1999-05-25  8:39 Sven Wischnowsky

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).