Patch 8.2.5163
Problem: Crash when deleting buffers in diff mode.
Solution: Recompute diffs later. Skip window without a valid buffer.
Files: src/diff.c, src/testdir/test_diffmode.vim
*** ../vim-8.2.5162/src/diff.c 2022-04-15 13:53:30.044708701 +0100
--- src/diff.c 2022-06-26 14:03:47.388884703 +0100
***************
*** 119,125 ****
tp->tp_diffbuf[i] = NULL;
tp->tp_diff_invalid = TRUE;
if (tp == curtab)
! diff_redraw(TRUE);
}
}
}
--- 119,130 ----
tp->tp_diffbuf[i] = NULL;
tp->tp_diff_invalid = TRUE;
if (tp == curtab)
! {
! // don't redraw right away, more might change or buffer state
! // is invalid right now
! need_diff_redraw = TRUE;
! redraw_later(VALID);
! }
}
}
}
***************
*** 670,676 ****
need_diff_redraw = FALSE;
FOR_ALL_WINDOWS(wp)
! if (wp->w_p_diff)
{
redraw_win_later(wp, SOME_VALID);
if (wp != curwin)
--- 675,682 ----
need_diff_redraw = FALSE;
FOR_ALL_WINDOWS(wp)
! // when closing windows or wiping buffers skip invalid window
! if (wp->w_p_diff && buf_valid(wp->w_buffer))
{
redraw_win_later(wp, SOME_VALID);
if (wp != curwin)
*** ../vim-8.2.5162/src/testdir/test_diffmode.vim 2022-06-24
12:38:33.986247430 +0100
--- src/testdir/test_diffmode.vim 2022-06-26 14:02:00.421049498 +0100
***************
*** 1603,1607 ****
--- 1603,1619 ----
call delete('Xright')
endfunc
+ " This was trying to update diffs for a buffer being closed
+ func Test_diff_only()
+ silent! lfile
+ set diff
+ lopen
+ norm o
+ silent! norm o
+
+ set nodiff
+ %bwipe!
+ endfunc
+
" vim: shiftwidth=2 sts=2 expandtab
*** ../vim-8.2.5162/src/version.c 2022-06-26 12:58:56.146042644 +0100
--- src/version.c 2022-06-26 13:31:46.389800150 +0100
***************
*** 737,738 ****
--- 737,740 ----
{ /* Add new patch number below this line */
+ /**/
+ 5163,
/**/
--
Everybody wants to go to heaven, but nobody wants to die.
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/vim_dev/20220626130453.7EDC61C4808%40moolenaar.net.