Warning in Tiny and Small option.c after applying patches 8.1.1537 to 1551

10 views
Skip to first unread message

Tony Mechelynck

unread,
Jun 15, 2019, 8:05:38 PM6/15/19
to Bram Moolenaar, vim_dev
* 4 warnings "variable set but not used" in Tiny and Small option.c
after applying patches 1537 to 1551. Maybe a missing #ifdef ?
* No warning in Normal or higher.

Example for Tiny:

gcc -c -I. -Iproto -DHAVE_CONFIG_H -O2 -fno-strength-reduce
-Wall -D_REENTRANT -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -o
objects/option.o option.c
option.c: In function ‘set_string_option’:
option.c:6139:13: warning: variable ‘oldval_g’ set but not used
[-Wunused-but-set-variable]
char_u *oldval_g = NULL;
^~~~~~~~
option.c:6138:13: warning: variable ‘oldval_l’ set but not used
[-Wunused-but-set-variable]
char_u *oldval_l = NULL;
^~~~~~~~
option.c: In function ‘set_bool_option’:
option.c:8528:10: warning: variable ‘old_global_value’ set but not
used [-Wunused-but-set-variable]
int old_global_value = 0;
^~~~~~~~~~~~~~~~
option.c: In function ‘set_num_option’:
option.c:9133:10: warning: variable ‘old_global_value’ set but not
used [-Wunused-but-set-variable]
long old_global_value = 0; // only used when setting a local and
^~~~~~~~~~~~~~~~


Best regards,
Tony.

Bram Moolenaar

unread,
Jun 16, 2019, 10:39:41 AM6/16/19
to vim...@googlegroups.com, Tony Mechelynck
Thanks for reporting, I'll fix it.

--
hundred-and-one symptoms of being an internet addict:
207. You're given one phone call in prison and you ask them for a laptop.

/// Bram Moolenaar -- Br...@Moolenaar.net -- 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 ///
Reply all
Reply to author
Forward
0 new messages