diff options
author | Ron Yorston | 2021-04-10 11:16:47 +0100 |
---|---|---|
committer | Denys Vlasenko | 2021-04-11 00:18:55 +0200 |
commit | fe76569daa88429276e72dc0c51c0b58d275aee7 (patch) | |
tree | c97b2ab3277f5bb228a49cb09c888488152546b6 | |
parent | b18c7bf702b2d7d967b2baf7d70fd8c408191eb4 (diff) | |
download | busybox-fe76569daa88429276e72dc0c51c0b58d275aee7.zip busybox-fe76569daa88429276e72dc0c51c0b58d275aee7.tar.gz |
vi: allow 'r' command to be aborted, repeated
Make the 'r' command behave more like vi:
- abort the command if ESC is entered after the 'r';
- allow a repeat count to be entered before the 'r';
- if the repeat count exceeds the space available on the line don't
change any characters and issue an alert.
function old new delta
do_cmd 4679 4746 +67
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 1/0 up/down: 67/0) Total: 67 bytes
v2: Don't break build when FEATURE_VI_UNDO is disabled.
Signed-off-by: Ron Yorston <rmy@pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | editors/vi.c | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/editors/vi.c b/editors/vi.c index e752685..fad5317 100644 --- a/editors/vi.c +++ b/editors/vi.c @@ -3920,9 +3920,18 @@ static void do_cmd(int c) break; case 'r': // r- replace the current char with user input c1 = get_one_char(); // get the replacement char - if (*dot != '\n') { - dot = text_hole_delete(dot, dot, ALLOW_UNDO); - dot = char_insert(dot, c1, ALLOW_UNDO_CHAIN); + if (c1 != 27) { + if (end_line(dot) - dot < (cmdcnt ?: 1)) { + indicate_error(); + goto dc6; + } + do { + dot = text_hole_delete(dot, dot, allow_undo); +#if ENABLE_FEATURE_VI_UNDO + allow_undo = ALLOW_UNDO_CHAIN; +#endif + dot = char_insert(dot, c1, allow_undo); + } while (--cmdcnt > 0); dot_left(); } end_cmd_q(); // stop adding to q |