diff options
author | Ron Yorston <rmy@pobox.com> | 2021-04-15 12:02:11 +0100 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2021-04-15 13:09:12 +0200 |
commit | 033fa3d5c65958a89fee155208ce8ac7f9049fcd (patch) | |
tree | daa0c68e8cf9ac9a56271e81ac7b970f999eea12 | |
parent | d9d19896a9be5b5cf35d00cae61c9d5621044ccf (diff) | |
download | busybox-w32-033fa3d5c65958a89fee155208ce8ac7f9049fcd.tar.gz busybox-w32-033fa3d5c65958a89fee155208ce8ac7f9049fcd.tar.bz2 busybox-w32-033fa3d5c65958a89fee155208ce8ac7f9049fcd.zip |
vi: code shrink motion by paragraph
Use a hand-coded loop to search for paragraph boundaries instead
of calling char_search(). We were using a loop anyway to skip
consecutive newlines.
function old new delta
do_cmd 4792 4752 -40
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-40) Total: -40 bytes
Signed-off-by: Ron Yorston <rmy@pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | editors/vi.c | 27 |
1 files changed, 15 insertions, 12 deletions
diff --git a/editors/vi.c b/editors/vi.c index f71897253..d37357edd 100644 --- a/editors/vi.c +++ b/editors/vi.c | |||
@@ -3636,21 +3636,24 @@ static void do_cmd(int c) | |||
3636 | break; | 3636 | break; |
3637 | case '{': // {- move backward paragraph | 3637 | case '{': // {- move backward paragraph |
3638 | case '}': // }- move forward paragraph | 3638 | case '}': // }- move forward paragraph |
3639 | dir = c == '}' ? FORWARD : BACK; | ||
3639 | do { | 3640 | do { |
3640 | dir = c == '}' ? FORWARD : BACK; | 3641 | int skip = TRUE; // initially skip consecutive empty lines |
3641 | // skip over consecutive empty lines | 3642 | while (dir == FORWARD ? dot < end - 1 : dot > text) { |
3642 | while ((dir == FORWARD ? dot < end - 1 : dot > text) && | 3643 | if (*dot == '\n' && dot[dir] == '\n') { |
3643 | *dot == '\n' && dot[dir] == '\n') { | 3644 | if (!skip) { |
3645 | if (dir == FORWARD) | ||
3646 | ++dot; // move to next blank line | ||
3647 | goto dc2; | ||
3648 | } | ||
3649 | } | ||
3650 | else { | ||
3651 | skip = FALSE; | ||
3652 | } | ||
3644 | dot += dir; | 3653 | dot += dir; |
3645 | } | 3654 | } |
3646 | q = char_search(dot, "\n\n", ((unsigned)dir << 1) | FULL); | 3655 | goto dc6; // end of file |
3647 | if (q != NULL) { // found blank line | 3656 | dc2: continue; |
3648 | dot = next_line(q); // move to next blank line | ||
3649 | } | ||
3650 | else { // blank line not found, move to end of file | ||
3651 | dot = dir == FORWARD ? end - 1 : text; | ||
3652 | break; | ||
3653 | } | ||
3654 | } while (--cmdcnt > 0); | 3657 | } while (--cmdcnt > 0); |
3655 | break; | 3658 | break; |
3656 | #endif /* FEATURE_VI_SEARCH */ | 3659 | #endif /* FEATURE_VI_SEARCH */ |