From: Bram Moolenaar <Bram@vim.org>
Date: Mon, 3 May 2021 17:49:51 +0000 (+0200)
Subject: patch 8.2.2828: Coverity complains about not checking rename() return value
X-Git-Tag: v8.2.2828
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=97a6c6a1fb6043fd6520fbaaafc6970334186167;p=vim

patch 8.2.2828: Coverity complains about not checking rename() return value

Problem:    Coverity complains about not checking the rename() return value.
Solution:   Add "(void)", can't do anything in case of a failure.
---

diff --git a/src/fileio.c b/src/fileio.c
index 1af0b4c59..b59c01d26 100644
--- a/src/fileio.c
+++ b/src/fileio.c
@@ -3733,7 +3733,7 @@ vim_rename(char_u *from, char_u *to)
 			return 0;
 		    // Strange, the second step failed.  Try moving the
 		    // file back and return failure.
-		    mch_rename(tempname, (char *)from);
+		    (void)mch_rename(tempname, (char *)from);
 		    return -1;
 		}
 		// If it fails for one temp name it will most likely fail
diff --git a/src/version.c b/src/version.c
index f20c0099d..444f35fda 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 */
+/**/
+    2828,
 /**/
     2827,
 /**/