aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRon Yorston <rmy@pobox.com>2021-04-15 12:02:11 +0100
committerDenys Vlasenko <vda.linux@googlemail.com>2021-04-15 13:09:12 +0200
commit033fa3d5c65958a89fee155208ce8ac7f9049fcd (patch)
treedaa0c68e8cf9ac9a56271e81ac7b970f999eea12
parentd9d19896a9be5b5cf35d00cae61c9d5621044ccf (diff)
downloadbusybox-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.c27
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 */