On Mon, Jan 11, 2016 at 6:01 AM, h_east <[email protected]> wrote: > Hi Tony, > > 2016-1-11(Mon) 11:43:51 UTC+9 Tony Mechelynck: >> Here is what appears in the Tiny build (the Huge build has the same >> warning with a much longer gcc command-line): >> >> gcc -c -I. -Iproto -DHAVE_CONFIG_H -O2 -fno-strength-reduce -Wall >> -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -o objects/ops.o ops.c >> ops.c: In function ‘op_addsub’: >> ops.c:5419:17: warning: ‘length’ may be used uninitialized in this >> function [-Wmaybe-uninitialized] >> one_change = do_addsub(oap->op_type, &pos, length, amount); > > I couldn't reproduce it. > But attached patch would be fixed this warning. > Please confirm this. > Thanks for reporting. > -- > Best regards, > Hirohito Higashi (a.k.a h_east) > The warning has indeed disappeared.
I am not qualified to say if, in this case, the code does the same thing without the additional "if" Best regards, Tony. -- -- 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.
