On Sun, Jan 20, 2019 at 6:40 PM Bram Moolenaar <[email protected]> wrote: > > > Tony wrote: > > > On Sun, Jan 20, 2019 at 3:31 PM Bram Moolenaar <[email protected]> wrote: > > > > > > > > > Patch 8.1.0785 > > > Problem: Depending on the configuration some functions are unused. > > > Solution: Add more #ifdefs, remove unused functions. (Dominique Pelle, > > > closes #3822) > > > Files: src/buffer.c, src/channel.c, src/ex_cmds2.c, src/ex_docmd.c, > > > src/fileio.c, src/getchar.c, src/gui_gtk_x11.c, src/hashtab.c, > > > src/json.c, src/mbyte.c, src/message.c, src/misc1.c, > > > src/misc2.c, > > > src/ops.c, src/option.c, src/os_unix.c, src/proto/os_unix.pro, > > > src/proto/regexp.pro, src/proto/terminal.pro, src/regexp.c, > > > src/screen.c, src/search.c, src/syntax.c, src/term.c, > > > src/terminal.c, src/ui.c, src/userfunc.c > > > > After applying this patch, I get three warnings, all in os.unix.c, and > > in Tiny and Small only, not in "bigger" builds, as follows: > > > > gcc -c -I. -Iproto -DHAVE_CONFIG_H -O2 -fno-strength-reduce -Wall > > -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -o objects/os_unix.o > > os_unix.c > > os_unix.c:1035:1: warning: ‘mch_didjmp’ defined but not used > > [-Wunused-function] > > mch_didjmp(void) > > ^~~~~~~~~~ > > os_unix.c:1029:1: warning: ‘mch_endjmp’ defined but not used > > [-Wunused-function] > > mch_endjmp(void) > > ^~~~~~~~~~ > > os_unix.c:1020:1: warning: ‘mch_startjmp’ defined but not used > > [-Wunused-function] > > mch_startjmp(void) > > ^~~~~~~~~~~~ > > The functions were made static, so now the compiler is checking whether > they are used. I'll tune the #ifdef.
I just created a PR to fix those warnings: https://github.com/vim/vim/pull/3832 It also fixes a few typos in warnings in the same file. Dominique -- -- 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.
