diff options
author | Peter Stephenson <pws@users.sourceforge.net> | 2009-07-01 13:41:18 +0000 |
---|---|---|
committer | Peter Stephenson <pws@users.sourceforge.net> | 2009-07-01 13:41:18 +0000 |
commit | 88d07936a21c262ec5f86518cce3c85fd1951968 (patch) | |
tree | 9e3e9bdfbaefb6a74c01e24531598d2cd617c093 /Src/Zle/complist.c | |
parent | 10787467bc268079951b997b8605dd51cc9047a2 (diff) | |
download | zsh-88d07936a21c262ec5f86518cce3c85fd1951968.tar.gz zsh-88d07936a21c262ec5f86518cce3c85fd1951968.tar.xz zsh-88d07936a21c262ec5f86518cce3c85fd1951968.zip |
27080: it wasnt possible to exit menu selection cleanly
Diffstat (limited to 'Src/Zle/complist.c')
-rw-r--r-- | Src/Zle/complist.c | 20 |
1 files changed, 18 insertions, 2 deletions
diff --git a/Src/Zle/complist.c b/Src/Zle/complist.c index 1b6dd083a..41bc7622c 100644 --- a/Src/Zle/complist.c +++ b/Src/Zle/complist.c @@ -2572,7 +2572,7 @@ domenuselect(Hookdef dummy, Chdata dat) cmd != Th(z_selfinsertunmeta)))) { ungetkeycmd(); break; - } else if (cmd == Th(z_acceptline)) { + } else if (cmd == Th(z_acceptline) || cmd == Th(z_acceptsearch)) { if (mode == MM_FSEARCH || mode == MM_BSEARCH) { mode = 0; continue; @@ -3316,7 +3316,23 @@ domenuselect(Hookdef dummy, Chdata dat) mlbeg = -1; showinglist = ((validlist && !nolist) ? -2 : 0); onlyexpl = oe; - if (!smatches) + if (acc && listshown) { + /* + * Clear the list without spending sixteen weeks of + * redrawing it in slightly different states first. + * The following seems to work. I'm not sure what + * the difference is between listshown and showinglist, + * but listshown looks like the traditional thing to + * check for in this file at least. + * + * showinglist has a normally undocumented value of 1, + * and an extra-specially undocumented value of -2, which + * seems to be a force---it appears we need to kick it out + * of that state, though it worries me that in some places + * the code actually forces it back into that state. + */ + clearlist = listshown = showinglist = 1; + } else if (!smatches) clearlist = listshown = 1; zrefresh(); } |