[git commit] vi: improve handling of anchored searches
Denys Vlasenko
vda.linux at googlemail.com
Tue Jul 13 11:50:59 UTC 2021
commit: https://git.busybox.net/busybox/commit/?id=c76c78740a19ed3b1f9c5910313460221096536a
branch: https://git.busybox.net/busybox/commit/?id=refs/heads/master
Suppose we search for a git conflict marker '<<<<<<< HEAD' using
the command '/^<<<'. Using 'n' to go to the next match finds
'<<<' on the current line, apparently ignoring the '^' anchor.
Set a flag in the compiled regular expression to indicate that the
start of the string should not be considered a beginning-of-line
anchor. An exception has to be made when the search starts from
the beginning of the file. Make a similar change for end-of-line
anchors.
This doesn't affect a default build with VI_REGEX_SEARCH disabled.
When it's enabled:
function old new delta
char_search 247 285 +38
Signed-off-by: Ron Yorston <rmy at pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
editors/vi.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/editors/vi.c b/editors/vi.c
index 34d577e90..2941b8ae4 100644
--- a/editors/vi.c
+++ b/editors/vi.c
@@ -2384,6 +2384,8 @@ static char *char_search(char *p, const char *pat, int dir_and_range)
memset(&preg, 0, sizeof(preg));
err = re_compile_pattern(pat, strlen(pat), &preg);
+ preg.not_bol = p != text;
+ preg.not_eol = p != end - 1;
if (err != NULL) {
status_line_bold("bad search pattern '%s': %s", pat, err);
return p;
More information about the busybox-cvs
mailing list