commit 5eecb14c8e822a70dc333f986dd59e7a0d17ee7c
from: Mark Jamsek <mark@jamsek.dev>
via: Thomas Adam <thomas@xteddy.org>
date: Fri Jan 10 22:29:14 2025 UTC

tog: don't apply count modifier to the H, &, p keymaps

It's annoying to have the help (H) and limit (&) keymaps repeatedly fire
after closing their respective views if accidentally preceded by a count
modifier, and doesn't seem like a valid use case. Similarly, writing N
patch files to /tmp is wasteful.

ok stsp@

commit - fe3e505d00cfcda92541f40e41b973007f02450d
commit + 5eecb14c8e822a70dc333f986dd59e7a0d17ee7c
blob - 30793bea899b9efce485dce4feba761c81486309
blob + 3b225f2aa9fdc2ff9236830c2f59dc04580bcec0
--- tog/tog.c
+++ tog/tog.c
@@ -1871,6 +1871,7 @@ view_input(struct tog_view **new, int *done, struct to
 	case '?':
 	case 'H':
 	case KEY_F(1):
+		view->count = 0;
 		if (view->type == TOG_VIEW_HELP)
 			err = view->reset(view);
 		else
@@ -4902,6 +4903,7 @@ input_log_view(struct tog_view **new_view, struct tog_
 
 	switch (ch) {
 	case '&':
+		view->count = 0;
 		err = limit_log_view(view);
 		break;
 	case 'q':
@@ -7393,6 +7395,7 @@ input_diff_view(struct tog_view **new_view, struct tog
 		err = create_diff(s);
 		break;
 	case 'p':
+		view->count = 0;
 		err = diff_write_patch(view);
 		break;
 	default: