diff options
| author | Marc André Tanner <mat@brain-dump.org> | 2014-09-10 20:50:10 +0200 |
|---|---|---|
| committer | Marc André Tanner <mat@brain-dump.org> | 2014-09-10 20:52:18 +0200 |
| commit | 16bd3ede6241a4790246ad7fcbde21e8c1d626a0 (patch) | |
| tree | a05383c6320c9bdfbd19f43a1a99d183a713069c /text-motions.c | |
| parent | da3c41bca0f1b697f7a0eb364ca736ba8b30dbc8 (diff) | |
| download | vis-16bd3ede6241a4790246ad7fcbde21e8c1d626a0.tar.gz vis-16bd3ede6241a4790246ad7fcbde21e8c1d626a0.tar.xz | |
text-motions: fix no match case of text_find_char_{next,prev}
These functions should return the original position in case no
match was found.
Diffstat (limited to 'text-motions.c')
| -rw-r--r-- | text-motions.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/text-motions.c b/text-motions.c index f152c8a..25ff87c 100644 --- a/text-motions.c +++ b/text-motions.c @@ -51,7 +51,7 @@ size_t text_find_char_next(Text *txt, size_t pos, const char *s, size_t len) { matched = 0; text_iterator_byte_next(&it, NULL); } - return it.pos - (matched == len ? len : 0); + return matched == len ? it.pos - len : pos; } size_t text_find_char_prev(Text *txt, size_t pos, const char *s, size_t len) { @@ -62,14 +62,15 @@ size_t text_find_char_prev(Text *txt, size_t pos, const char *s, size_t len) { return pos; while (text_iterator_byte_get(&it, &c)) { if (c == s[matched]) { - if (matched-- == 0) + if (matched == 0) break; + matched--; } else { matched = len - 1; } text_iterator_byte_prev(&it, NULL); } - return it.pos; + return matched == 0 ? it.pos : pos; } size_t text_line_begin(Text *txt, size_t pos) { @@ -110,7 +111,11 @@ size_t text_line_finish(Text *txt, size_t pos) { } size_t text_line_end(Text *txt, size_t pos) { - return text_find_char_next(txt, pos, "\n", 1); + char c; + Iterator it = text_iterator_get(txt, pos); + while (text_iterator_byte_get(&it, &c) && c != '\n') + text_iterator_byte_next(&it, NULL); + return it.pos; } size_t text_line_next(Text *txt, size_t pos) { |
