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.

diff --git a/src/undo.c b/src/undo.c
index b3a6276865aefd5f3481d0e79d993fd159cad7b1..040ec548093c69e2a2e052de6f41308aa06e2f1c 100644 (file)
--- 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));

diff --git a/src/version.c b/src/version.c
index 1f94b015f319bf6977a2f8904bbd9d3e6aa2a68e..1dae71fcc6f7fe778bd885cacfb5288b9e7e1261 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -789,6 +789,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    179,
 /**/
     178,
 /**/