Patch 8.2.2029

4 views
Skip to first unread message

Bram Moolenaar

unread,
Nov 21, 2020, 8:17:28 AM11/21/20
to vim...@googlegroups.com

Patch 8.2.2029
Problem: Coverity warns for not checking return value.
Solution: Check that u_save_cursor() returns OK.
Files: src/ops.c


*** ../vim-8.2.2028/src/ops.c 2020-11-18 11:34:30.178191415 +0100
--- src/ops.c 2020-11-21 14:15:17.860940587 +0100
***************
*** 3871,3879 ****
else
{
(void)op_delete(oap);
! if (oap->motion_type == MLINE && has_format_option(FO_AUTO))
! u_save_cursor(); // cursor line wasn't saved yet
! auto_format(FALSE, TRUE);
}
break;

--- 3871,3880 ----
else
{
(void)op_delete(oap);
! // save cursor line for undo if it wasn't saved yet
! if (oap->motion_type == MLINE && has_format_option(FO_AUTO)
! && u_save_cursor() == OK)
! auto_format(FALSE, TRUE);
}
break;

*** ../vim-8.2.2028/src/version.c 2020-11-21 14:03:39.851158161 +0100
--- src/version.c 2020-11-21 14:16:00.984817194 +0100
***************
*** 752,753 ****
--- 752,755 ----
{ /* Add new patch number below this line */
+ /**/
+ 2029,
/**/

--
Facepalm statement #9: "Did you see, there is now even a hobbit book"

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
Reply all
Reply to author
Forward
0 new messages