zsh-workers
 help / color / mirror / Atom feed
* [BUG]: zle-line-pre-redraw breaks vi-repeat-change
@ 2020-10-07 12:49 Roman Perepelitsa
  2020-10-07 13:25 ` Peter Stephenson
  0 siblings, 1 reply; 3+ messages in thread
From: Roman Perepelitsa @ 2020-10-07 12:49 UTC (permalink / raw)
  To: Zsh hackers list

If zle-line-pre-redraw is defined, vi-repeat-change behaves as if the
numeric argument was always 1.

To reproduce from `zsh -f`:

  % function zle-line-pre-redraw() {}
  % zle -N zle-line-pre-redraw
  % bindkey -v
  % 123456<ESC><2X><.>

The last line in more detail:

  - "123456" is self-insert
  - ESC is vi-cmd-mode
  - "2X" is vi-backward-delete-char with NUMERIC = 2
  - "." is vi-repeat-change

Expected: BUFFER contains "16".

Actual: BUFFER contains "126".

The actual behavior is unchanged if <.> is replaced with <3.>. The
expected BUFFER in this case is "6".

Roman.


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

* Re: [BUG]: zle-line-pre-redraw breaks vi-repeat-change
  2020-10-07 12:49 [BUG]: zle-line-pre-redraw breaks vi-repeat-change Roman Perepelitsa
@ 2020-10-07 13:25 ` Peter Stephenson
  2020-10-07 14:49   ` Roman Perepelitsa
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Stephenson @ 2020-10-07 13:25 UTC (permalink / raw)
  To: Roman Perepelitsa, Zsh hackers list

> On 07 October 2020 at 13:49 Roman Perepelitsa <roman.perepelitsa@gmail.com> wrote:
> 
> 
> If zle-line-pre-redraw is defined, vi-repeat-change behaves as if the
> numeric argument was always 1.

I wonder if we need to change zlecallhook().

pws

diff --git a/Src/Zle/zle_main.c b/Src/Zle/zle_main.c
index 8c0534708..1622d8a6b 100644
--- a/Src/Zle/zle_main.c
+++ b/Src/Zle/zle_main.c
@@ -1067,6 +1067,7 @@ redrawhook(void)
 	int old_incompfunc = incompfunc;
 	char *args[2];
 	Thingy lbindk_save = lbindk, bindk_save = bindk;
+	struct modifier zmod_save = zmod;
 
 	refthingy(lbindk_save);
 	refthingy(bindk_save);
@@ -1094,6 +1095,7 @@ redrawhook(void)
 	 * restore lastcmd manually so that we don't mess up the global state
 	 */
 	lastcmd = lastcmd_prev;
+	zmod = zmod_save;
     }
 }


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

* Re: [BUG]: zle-line-pre-redraw breaks vi-repeat-change
  2020-10-07 13:25 ` Peter Stephenson
@ 2020-10-07 14:49   ` Roman Perepelitsa
  0 siblings, 0 replies; 3+ messages in thread
From: Roman Perepelitsa @ 2020-10-07 14:49 UTC (permalink / raw)
  To: Peter Stephenson; +Cc: Zsh hackers list

On Wed, Oct 7, 2020 at 3:25 PM Peter Stephenson
<p.w.stephenson@ntlworld.com> wrote:
>
> I wonder if we need to change zlecallhook().

After applying this patch I can still reproduce the issue.

Roman.


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

end of thread, other threads:[~2020-10-07 14:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-07 12:49 [BUG]: zle-line-pre-redraw breaks vi-repeat-change Roman Perepelitsa
2020-10-07 13:25 ` Peter Stephenson
2020-10-07 14:49   ` Roman Perepelitsa

zsh-workers

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.vuxu.org/zsh-workers

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V1 zsh-workers zsh-workers/ http://inbox.vuxu.org/zsh-workers \
		zsh-workers@zsh.org
	public-inbox-index zsh-workers

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.vuxu.org/vuxu.archive.zsh.workers


code repositories for the project(s) associated with this inbox:

	https://git.vuxu.org/mirror/zsh/

AGPL code for this site: git clone https://public-inbox.org/public-inbox.git