To: vim_dev@googlegroups.com
Subject: Patch 8.1.0179
Fcc: outbox
From: Bram Moolenaar <Bram@moolenaar.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
------------

Patch 8.1.0179
Problem:    Redundant condition for boundary check.
Solution:   Remove the condition. (Dominique Pelle). Change FALSE to FAIL.
Files:	    src/undo.c


*** ../vim-8.1.0178/src/undo.c	2018-07-08 16:50:33.111216814 +0200
--- src/undo.c	2018-07-13 16:27:37.721354663 +0200
***************
*** 266,275 ****
      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 + 2 == bot)
  	u_saveline((linenr_T)(top + 1));
--- 266,273 ----
      if (undo_off)
  	return OK;
  
!     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));
*** ../vim-8.1.0178/src/version.c	2018-07-11 22:57:47.947564638 +0200
--- src/version.c	2018-07-13 16:29:28.492772093 +0200
***************
*** 791,792 ****
--- 791,794 ----
  {   /* Add new patch number below this line */
+ /**/
+     179,
  /**/

-- 
"The amigos also appear to be guilty of not citing the work of others who had
gone before them.  Even worse, they have a chapter about modeling time and
space without making a single reference to Star Trek!"
			(Scott Ambler, reviewing the UML User Guide)

 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///