zsh-workers
 help / color / mirror / code / Atom feed
* Src/exec.c:3961: BUG: q = 13 != queue_in = 14
@ 2017-06-27  8:38 ` Daniel Shahaf
  2017-06-27  9:07   ` Peter Stephenson
  0 siblings, 1 reply; 3+ messages in thread
From: Daniel Shahaf @ 2017-06-27  8:38 UTC (permalink / raw)
  To: zsh-workers

The error is reproducible, but my current recipe requires z-sy-h.
Haven't tried yet to minimize that, sorry.

Recipe:
[[[
$ git clone https://github.com/zsh-users/zsh-syntax-highlighting /tmp/zsh-syntax-highlighting
$ Src/zsh -f 
% source /tmp/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh 
% echo <TAB>
2: …/Src/exec.c:3961: BUG: q = 13 != queue_in = 14
               
]]]

Bisected to:
[[[
commit e51c9c17af51e4055efb5a2cc36739d1d7ae457f
Author: Barton E. Schaefer <schaefer@zsh.org>
Date:   Sun Jan 29 08:30:14 2017 -0800

    40453: signal handler safety for callers of patcompile(PAT_STATIC), which is not re-entrant.
]]]

Using these versions:
% echo $ZSH_PATCHLEVEL
zsh-5.3.1-229-gd78761c
% (cd /tmp/zsh-syntax-highlighting && git rev-parse HEAD)
ad522a091429ba180c930f84b2a023b40de4dbcc
% 

A similar issue was reported in 40463 (sic).

Cheers,

Daniel


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

* Re: Src/exec.c:3961: BUG: q = 13 != queue_in = 14
  2017-06-27  8:38 ` Src/exec.c:3961: BUG: q = 13 != queue_in = 14 Daniel Shahaf
@ 2017-06-27  9:07   ` Peter Stephenson
  2017-06-27  9:14     ` Daniel Shahaf
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Stephenson @ 2017-06-27  9:07 UTC (permalink / raw)
  To: zsh-workers

On Tue, 27 Jun 2017 08:38:42 +0000
Daniel Shahaf <d.s@daniel.shahaf.name> wrote:
> The error is reproducible, but my current recipe requires z-sy-h.
> Haven't tried yet to minimize that, sorry.
> 
> Recipe:
> [[[
> $ git clone https://github.com/zsh-users/zsh-syntax-highlighting /tmp/zsh-syntax-highlighting
> $ Src/zsh -f 
> % source /tmp/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh 
> % echo <TAB>
> 2: …/Src/exec.c:3961: BUG: q = 13 != queue_in = 14

I can see a couple in compctl.c... assuming zsh-syntax-highlighting
doesn't itself load compsys, they might be possibilities.

pws

diff --git a/Src/Zle/compctl.c b/Src/Zle/compctl.c
index 9e6ccb4..5414b8f 100644
--- a/Src/Zle/compctl.c
+++ b/Src/Zle/compctl.c
@@ -1867,6 +1867,7 @@ ccmakehookfn(UNUSED(Hookdef dummy), struct ccmakedat *dat)
 	    redup(osi, 0);
 
 	    dat->lst = 0;
+	    unqueue_signals();
 	    return 0;
 	}
 	if (lastmatches) {
@@ -1890,6 +1891,7 @@ ccmakehookfn(UNUSED(Hookdef dummy), struct ccmakedat *dat)
 	    redup(osi, 0);
 
 	    dat->lst = 0;
+	    unqueue_signals();
 	    return 0;
 	}
 	if (!m || !(m = m->next))


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

* Re: Src/exec.c:3961: BUG: q = 13 != queue_in = 14
  2017-06-27  9:07   ` Peter Stephenson
@ 2017-06-27  9:14     ` Daniel Shahaf
  0 siblings, 0 replies; 3+ messages in thread
From: Daniel Shahaf @ 2017-06-27  9:14 UTC (permalink / raw)
  To: Peter Stephenson, zsh-workers

Peter Stephenson wrote on Tue, 27 Jun 2017 10:07 +0100:
> On Tue, 27 Jun 2017 08:38:42 +0000
> Daniel Shahaf <d.s@daniel.shahaf.name> wrote:
> > % echo <TAB>
> > 2: …/Src/exec.c:3961: BUG: q = 13 != queue_in = 14
> 
> I can see a couple in compctl.c... assuming zsh-syntax-highlighting
> doesn't itself load compsys, they might be possibilities.

Thanks Peter, that fixed it.


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

end of thread, other threads:[~2017-06-27  9:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CGME20170627083923epcas3p32d15a148fe8bae48a74f97377d20a018@epcas3p3.samsung.com>
2017-06-27  8:38 ` Src/exec.c:3961: BUG: q = 13 != queue_in = 14 Daniel Shahaf
2017-06-27  9:07   ` Peter Stephenson
2017-06-27  9:14     ` Daniel Shahaf

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