forked from aniani/vim
patch 8.2.3556: filler lines are incorrect for other window in diff mode
Problem: Filler lines are incorrect for other window in diff mode after making a change. Solution: Copy filler lines from the current window. (closes #8809)
This commit is contained in:
22
src/diff.c
22
src/diff.c
@@ -656,7 +656,8 @@ diff_redraw(
|
|||||||
{
|
{
|
||||||
win_T *wp;
|
win_T *wp;
|
||||||
win_T *wp_other = NULL;
|
win_T *wp_other = NULL;
|
||||||
int used_max_fill = FALSE;
|
int used_max_fill_other = FALSE;
|
||||||
|
int used_max_fill_curwin = FALSE;
|
||||||
int n;
|
int n;
|
||||||
|
|
||||||
need_diff_redraw = FALSE;
|
need_diff_redraw = FALSE;
|
||||||
@@ -681,16 +682,25 @@ diff_redraw(
|
|||||||
{
|
{
|
||||||
wp->w_topfill = n;
|
wp->w_topfill = n;
|
||||||
if (wp == curwin)
|
if (wp == curwin)
|
||||||
used_max_fill = TRUE;
|
used_max_fill_curwin = TRUE;
|
||||||
|
else if (wp_other != NULL)
|
||||||
|
used_max_fill_other = TRUE;
|
||||||
}
|
}
|
||||||
check_topfill(wp, FALSE);
|
check_topfill(wp, FALSE);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (wp_other != NULL && used_max_fill && curwin->w_p_scb)
|
if (wp_other != NULL && curwin->w_p_scb)
|
||||||
// The current window was set to used the maximum number of filler
|
{
|
||||||
// lines, may need to reduce them.
|
if (used_max_fill_curwin)
|
||||||
diff_set_topline(wp_other, curwin);
|
// The current window was set to used the maximum number of filler
|
||||||
|
// lines, may need to reduce them.
|
||||||
|
diff_set_topline(wp_other, curwin);
|
||||||
|
else if (used_max_fill_other)
|
||||||
|
// The other window was set to used the maximum number of filler
|
||||||
|
// lines, may need to reduce them.
|
||||||
|
diff_set_topline(curwin, wp_other);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static void
|
static void
|
||||||
|
20
src/testdir/dumps/Test_diff_scroll_change_03.dump
Normal file
20
src/testdir/dumps/Test_diff_scroll_change_03.dump
Normal file
@@ -0,0 +1,20 @@
|
|||||||
|
| +0#0000e05#a8a8a8255@1|-+0#4040ff13#afffff255@34||+1#0000000#ffffff0| +0#0000e05#a8a8a8255@1|6+0#0000000#5fd7ff255| @33
|
||||||
|
| +0#0000e05#a8a8a8255@1|-+0#4040ff13#afffff255@34||+1#0000000#ffffff0| +0#0000e05#a8a8a8255@1|7+0#0000000#5fd7ff255| @33
|
||||||
|
| +0#0000e05#a8a8a8255@1|-+0#4040ff13#afffff255@34||+1#0000000#ffffff0| +0#0000e05#a8a8a8255@1|8+0#0000000#5fd7ff255| @33
|
||||||
|
| +0#0000e05#a8a8a8255@1|9+0#0000000#ffffff0| @33||+1&&| +0#0000e05#a8a8a8255@1|9+0#0000000#ffffff0| @33
|
||||||
|
| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0|0| @32||+1&&| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0|0| @32
|
||||||
|
| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0@1| @32||+1&&| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0@1| @32
|
||||||
|
| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0|2| @32||+1&&| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0|2| @32
|
||||||
|
| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0|3| @32||+1&&| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0|3| @32
|
||||||
|
| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0|4| @32||+1&&| +0#0000e05#a8a8a8255@1|1+0#0000000#ffffff0|4| @32
|
||||||
|
| +0#0000e05#a8a8a8255@1|1+0#0000000#ffd7ff255|5|x+2&#ff404010| +0&#ffd7ff255@31||+1&#ffffff0| +0#0000e05#a8a8a8255@1|1+0#0000000#ffd7ff255|5>y+2&#ff404010| +0&#ffd7ff255@31
|
||||||
|
|~+0#4040ff13#ffffff0| @35||+1#0000000&|~+0#4040ff13&| @35
|
||||||
|
|~| @35||+1#0000000&|~+0#4040ff13&| @35
|
||||||
|
|~| @35||+1#0000000&|~+0#4040ff13&| @35
|
||||||
|
|~| @35||+1#0000000&|~+0#4040ff13&| @35
|
||||||
|
|~| @35||+1#0000000&|~+0#4040ff13&| @35
|
||||||
|
|~| @35||+1#0000000&|~+0#4040ff13&| @35
|
||||||
|
|~| @35||+1#0000000&|~+0#4040ff13&| @35
|
||||||
|
|~| @35||+1#0000000&|~+0#4040ff13&| @35
|
||||||
|
|[+1#0000000&|N|o| |N|a|m|e|]| |[|+|]| @5|7|,|3| @11|A|l@1| |[+3&&|N|o| |N|a|m|e|]| |[|+|]| @5|1|5|,|3| @10|B|o|t
|
||||||
|
| +0&&@74
|
@@ -1013,6 +1013,9 @@ func Test_diff_with_scroll_and_change()
|
|||||||
call term_sendkeys(buf, "ax\<Esc>")
|
call term_sendkeys(buf, "ax\<Esc>")
|
||||||
call VerifyScreenDump(buf, 'Test_diff_scroll_change_02', {})
|
call VerifyScreenDump(buf, 'Test_diff_scroll_change_02', {})
|
||||||
|
|
||||||
|
call term_sendkeys(buf, "\<C-W>lay\<Esc>")
|
||||||
|
call VerifyScreenDump(buf, 'Test_diff_scroll_change_03', {})
|
||||||
|
|
||||||
" clean up
|
" clean up
|
||||||
call StopVimInTerminal(buf)
|
call StopVimInTerminal(buf)
|
||||||
call delete('Xtest_scroll_change')
|
call delete('Xtest_scroll_change')
|
||||||
|
@@ -757,6 +757,8 @@ static char *(features[]) =
|
|||||||
|
|
||||||
static int included_patches[] =
|
static int included_patches[] =
|
||||||
{ /* Add new patch number below this line */
|
{ /* Add new patch number below this line */
|
||||||
|
/**/
|
||||||
|
3556,
|
||||||
/**/
|
/**/
|
||||||
3555,
|
3555,
|
||||||
/**/
|
/**/
|
||||||
|
Reference in New Issue
Block a user