On 24/02/2016 22:06, Bram Moolenaar wrote:
Because they actually cause an overflow problem (potentionally):

-            || top >= bot
+            || top - bot >= 0

The existing line doesn't do any subtraction thus can't overflow, it's a
simple comparison.  The second one could have an overflow.
So we are just avoiding a bogus warning, not fixing a problem.

The warning is likely to be from in-lining u_save() into u_save_cursor() when using -O3. The version of gcc being used sees that, assuming no signed integer overflow, top >= bot will always be false and the condition is redundant.

Mike
--
The face of a child can say it all, especially the mouth part of the face.

--
--
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.

Raspunde prin e-mail lui