From: Bram Moolenaar <Bram@vim.org>
Date: Sat, 21 Nov 2020 13:16:22 +0000 (+0100)
Subject: patch 8.2.2029: Coverity warns for not checking return value
X-Git-Tag: v8.2.2029
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5546688fb6798d37a770f366c13dedfa7e4cb417;p=vim

patch 8.2.2029: Coverity warns for not checking return value

Problem:    Coverity warns for not checking return value.
Solution:   Check that u_save_cursor() returns OK.
---

diff --git a/src/ops.c b/src/ops.c
index 417bb1621..b83394e01 100644
--- a/src/ops.c
+++ b/src/ops.c
@@ -3871,9 +3871,10 @@ do_pending_operator(cmdarg_T *cap, int old_col, int gui_yank)
 	    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);
+		// 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;
 
diff --git a/src/version.c b/src/version.c
index 0583f64b9..8e9d8103c 100644
--- a/src/version.c
+++ b/src/version.c
@@ -750,6 +750,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    2029,
 /**/
     2028,
 /**/