[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH] vi: fix undo when ':s' replacement string is empty
From:       Ron Yorston <rmy () pobox ! com>
Date:       2021-07-13 15:16:59
Message-ID: 60edae6b.JdEy2RyFgLGeg1Zv%rmy () pobox ! com
[Download RAW message or body]

Denys Vlasenko wrote:
>I missed this one, and then this part was rewritten in
>"allow regular expressions in ':s' commands".
>Sorry about the mess.

No worries.  Looks OK at first glance, though only time and continued
dogfooding of BusyBox vi will tell.

I think we may have ended up with a superfluous line.  Unless it was
intentional...

Ron
---
 editors/vi.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/editors/vi.c b/editors/vi.c
index c6bb74cfb..ce5740e49 100644
--- a/editors/vi.c
+++ b/editors/vi.c
@@ -3243,7 +3243,6 @@ static void colon(char *buf)
 								TEST_UNDO2 ? ALLOW_UNDO_CHAIN: ALLOW_UNDO);
 					found += bias;
 					ls += bias;
-					dot = ls;
 					//q += bias; - recalculated anyway
 				}
 #  if ENABLE_FEATURE_VI_REGEX_SEARCH
-- 
2.31.1

_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]