Dominique wrote: > I see these compilation warnings when compiling > vim-7.4.2250 (and older) with -Wall -Wextra: > > gcc -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_GTK -pthread > -I/usr/include/gtk-2.0 -I/usr/lib/x86_64-linux-gnu/gtk-2.0/include > -I/usr/include/atk-1.0 -I/usr/include/cairo > -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/pango-1.0 > -I/usr/include/gio-unix-2.0/ -I/usr/include/freetype2 > -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include > -I/usr/include/pixman-1 -I/usr/include/libpng12 > -I/usr/include/harfbuzz -g -O0 -Wall -Wextra -Wshadow > -Wmissing-prototypes -Wunreachable-code -U_FORTIFY_SOURCE > -D_FORTIFY_SOURCE=1 -o objects/syntax.o syntax.c > In file included from syntax.c:14:0: > syntax.c: In function ‘set_hl_attr’: > vim.h:1635:6: warning: signed and unsigned type in conditional > expression [-Wsign-compare] > : (long_u)INVALCOLOR) \ > ^ > vim.h:1659:37: note: in expansion of macro ‘GUI_FUNCTION2’ > # define GUI_MCH_GET_RGB2(pixel) GUI_FUNCTION2(mch_get_rgb, (pixel)) > ^ > syntax.c:9264:28: note: in expansion of macro ‘GUI_MCH_GET_RGB2’
[...] > Attached patch fixes by removing a cast. Thanks. I actually had this but forgot to include the changes in the patch. -- hundred-and-one symptoms of being an internet addict: 77. The phone company asks you to test drive their new PBX system /// 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.
