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)
--
--
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.
diff --git a/src/ops.c b/src/ops.c
index d285893..867c43b 100644
--- a/src/ops.c
+++ b/src/ops.c
@@ -5396,7 +5396,7 @@ op_addsub(oap, Prenum1, g_cmd)
pos.col = 0;
length = (colnr_T)STRLEN(ml_get(pos.lnum));
}
- else if (oap->motion_type == MCHAR)
+ else
{
if (!oap->inclusive)
dec(&(oap->end));
