]> git.ipfire.org Git - thirdparty/vim.git/commit
patch 9.1.0310: Filler lines not checked properly in get_scroll_overlap() v9.1.0310
authorzeertzjq <zeertzjq@outlook.com>
Fri, 12 Apr 2024 16:38:38 +0000 (18:38 +0200)
committerChristian Brabandt <cb@256bit.org>
Fri, 12 Apr 2024 16:38:38 +0000 (18:38 +0200)
commit92325540876252eb7375e34ff8f92cd4265fa7ae
tree8bed886cc20ba6ba3945eedd2e2700d7ebbba9a7
parent0549c503ba20345097a14122f0a18dde69d470c5
patch 9.1.0310: Filler lines not checked properly in get_scroll_overlap()

Problem:  Filler lines not checked properly in get_scroll_overlap().
Solution: Add missing parentheses (zeertzjq).

The missing parentheses causes the second argument to diff_check_fill()
to always be 0 as it is the result of a comparison between a positive
integer and -1 (the value of BACKWARD), in which case diff_check_fill()
always returns 0 instead of the number of filler lines above a line.

It's very hard to add a test for this, because this mistake at most
leads to 2 screen lines of difference in scrolling behavior, and in
cases where it may indeed lead to a difference in behavior, neither
behavior achieves complete symmetry between CTRL-F and CTRL-B.

closes: #14527

Signed-off-by: zeertzjq <zeertzjq@outlook.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
src/move.c
src/version.c