patch 8.2.1568: prop_find() skips properties in the same line
Problem: prop_find() skips properties in the same line if "skipstart" is
used.
Solution: Use "continue" instead of "break". (closes #6840)
diff --git a/src/testdir/test_textprop.vim b/src/testdir/test_textprop.vim
index 9db5275..5990fbf 100644
--- a/src/testdir/test_textprop.vim
+++ b/src/testdir/test_textprop.vim
@@ -211,6 +211,22 @@
call prop_clear(1,6)
call prop_type_delete('prop_name')
+
+ " Multiple props per line, start on the first, should find the second.
+ let expected = {'lnum': 1, 'id': 0, 'col': 14, 'end': 1, 'type': 'misspell', 'length': 2, 'start': 1}
+ eval ['the quikc bronw fox jumsp over the layz dog']->repeat(2)->setline(1)
+ call prop_type_add('misspell', #{highlight: 'ErrorMsg'})
+ for lnum in [1, 2]
+ for col in [8, 14, 24, 38]
+ call prop_add(lnum, col, #{type: 'misspell', length: 2})
+ endfor
+ endfor
+ call cursor(1, 8)
+ let result = prop_find(#{type: 'misspell', skipstart: 1}, 'f')
+ call assert_equal(expected, result)
+
+ call prop_type_delete('misspell')
+ bwipe!
endfunc
func Test_prop_find_smaller_len_than_match_col()
diff --git a/src/textprop.c b/src/textprop.c
index 0645e1f..9dff6b8 100644
--- a/src/textprop.c
+++ b/src/textprop.c
@@ -718,7 +718,7 @@
// on a prop and we're not skipping.
if (start_pos_has_prop && !skipstart)
dir = -1;
- break;
+ continue;
}
// If skipstart is true, skip the prop at start pos (even if
@@ -726,7 +726,7 @@
if (start_pos_has_prop && skipstart && !seen_end)
{
start_pos_has_prop = 0;
- break;
+ continue;
}
prop_fill_dict(rettv->vval.v_dict, &prop, buf);
diff --git a/src/version.c b/src/version.c
index e1d3b12..840c325 100644
--- a/src/version.c
+++ b/src/version.c
@@ -755,6 +755,8 @@
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 1568,
+/**/
1567,
/**/
1566,