changeset 14327:b8f1167aa8ad v8.1.0179

patch 8.1.0179: redundant condition for boundary check commit https://github.com/vim/vim/commit/efc81331e7bceafa2d7bc452a98d6ec1943b4292 Author: Bram Moolenaar <Bram@vim.org> Date: Fri Jul 13 16:31:19 2018 +0200 patch 8.1.0179: redundant condition for boundary check Problem: Redundant condition for boundary check. Solution: Remove the condition. (Dominique Pelle). Change FALSE to FAIL.
author Christian Brabandt <cb@256bit.org>
date Fri, 13 Jul 2018 16:45:05 +0200
parents 19964d1e9f7c
children 05295e8ace8d
files src/undo.c src/version.c
diffstat 2 files changed, 4 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/src/undo.c
+++ b/src/undo.c
@@ -266,10 +266,8 @@ u_save(linenr_T top, linenr_T bot)
     if (undo_off)
 	return OK;
 
-    if (top > curbuf->b_ml.ml_line_count
-	    || top >= bot
-	    || bot > curbuf->b_ml.ml_line_count + 1)
-	return FALSE;	/* rely on caller to do error messages */
+    if (top >= bot || bot > curbuf->b_ml.ml_line_count + 1)
+	return FAIL;	// rely on caller to give an error message
 
     if (top + 2 == bot)
 	u_saveline((linenr_T)(top + 1));
--- a/src/version.c
+++ b/src/version.c
@@ -790,6 +790,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    179,
+/**/
     178,
 /**/
     177,