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 -- [email protected] -- 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 ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
