reverse-menu-complete re-starting completion on 5.2?

reverse-menu-complete re-starting completion on 5.2?

[Danek Duvall]

If, from zsh -f, I run

    bindkey -v $'\t' menu-complete
    bindkey -v $'\e\t' reverse-menu-complete

And my pwd contains the directories a<1-5>, then if I type "cd a<TAB>" and
keep on hitting tab, I'll cycle forward through the directories, and then
if I hit alt-tab, I'll start cycling backwards.  So far, so good; that's
the behavior I want.

If I then run

    autoload -Uz compinit
    compinit -i

and try the same experiment, I can cycle forward through the directories
just fine, but when I switch into reverse, it takes whatever directory is
on the command line and starts completing its subdirectories backwards.
Absolutely not what I want.

This worked just fine in 5.0.7, and doesn't in 5.2.  Is there some
configuration I missed between the two?  Or is there something broken in
5.2?

Thanks,
Danek

Re: reverse-menu-complete re-starting completion on 5.2?

[Bart Schaefer]

On Feb 26,  9:59am, Danek Duvall wrote:
} Subject: reverse-menu-complete re-starting completion on 5.2?
}
} I can cycle forward through the directories
} just fine, but when I switch into reverse, it takes whatever directory is
} on the command line and starts completing its subdirectories backwards.
} Absolutely not what I want.

This appears to have resulted from this commit:

commit 44c56f3af57d93d5d92ce18893cb020f8f36e2f3
Author: Oliver Kiddle <opk@zsh.org>
Date:   Mon Jun 29 01:53:59 2015 +0200

    35627: make reverse-menu-complete start with the last match in
    menu selection

:100644 100644 2ea6492... 9b5d7e2... M  ChangeLog
:100644 100644 d4051bd... ba538ca... M  Src/Zle/compcore.c
:100644 100644 189582d... bac533e... M  Src/Zle/compctl.c
:100644 100644 ea5e41f... 63e5b91... M  Src/Zle/complete.c
:100644 100644 18551d0... f37a432... M  Src/Zle/complist.c
:100644 100644 9f383f4... 7fec7c8... M  Src/Zle/compresult.c
:100644 100644 7217abc... e8dc93e... M  Src/Zle/zle_tricky.c

Specifically the changes to do_menucmp() in compresult.c.

Follow-ups to zsh-workers, please.