Dominique wrote:
> cppcheck static analyzer gives this warning: > > [/home/pel/sb/vim/src/memline.c:5032]: (style) Array index 'curix' is > used before limits check. > > It's hard to tell whether it can cause an > invalid memory access in practice, but it > is safer to swap the conditions in the 'if' as > in attached patch. I think the check was OK, but the fix helps to avoid false positives from cppcheck. -- Facepalm statement #8: "Drive faster, the petrol is running out" /// 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.
