[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#23536: 25.1.50; viper-mode "/" search includes "(default ...) " pref
bug#23536: 25.1.50; viper-mode "/" search includes "(default ...) " prefix
Sat, 14 May 2016 08:51:24 -0700
On Sat, May 14, 2016 at 1:50 AM, Eli Zaretskii <address@hidden> wrote:
>> From: Jim Meyering <address@hidden>
>> Date: Fri, 13 May 2016 22:31:38 -0700
>> This is using emacs commit (latest master)
>> How to repro?
>> Invoke this: emacs -nw -Q -f viper-mode
>> Then answer "n" to the viperize prompt.
>> Type "/foo" and hit ENTER.
>> You'll notice that the minibuffer now contains this:
>> ‘(default ) foo’: Pattern not found
>> implying that you've searched for a string significantly different than
>> If you try again, you'll see that the search string grows even longer:
>> ‘(default (default ) foo) foo’: Pattern not found
>> It looks like this problem was introduced via commit
>> 088acab3831b45e0e0749705226b8680076df4b6, since
>> if I revert that and rebuild, the problem goes away.
> Please see if the patch below fixes the problem, and doesn't bring new
> ones. (I don't use viper-mode, and we don't have a test suite for it,
> so I couldn't easily test the result beyond your simple recipe.)
> diff --git a/lisp/emulation/viper-cmd.el b/lisp/emulation/viper-cmd.el
> index da72b97..6951dbb 100644
> --- a/lisp/emulation/viper-cmd.el
> +++ b/lisp/emulation/viper-cmd.el
> @@ -2127,6 +2127,7 @@ viper-read-string-with-history
> (setq keymap (or keymap minibuffer-local-map)
> initial (or initial "")
> + viper-initial initial
Thank you. That does indeed solve the problem.
I'm closing this for now and will reopen if I notice anything untoward.