commit efc81331e7bceafa2d7bc452a98d6ec1943b4292
author Bram Moolenaar <Bram@vim.org> Fri Jul 13 16:31:19 2018 +0200
committer Bram Moolenaar <Bram@vim.org> Fri Jul 13 16:31:19 2018 +0200
tree 8942668ba3f82569214a27cd0acf5ccabd565a4d
parent e76c4b237d41d0e62ca5d4fc76d1dd163fe222c0

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.

src/undo.c

diff --git a/src/undo.c b/src/undo.c
index b3a6276..040ec54 100644
--- a/src/undo.c
+++ b/src/undo.c
@@ -266,10 +266,8 @@
     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));