Tony Mechelynck wrote:

> I've been noticing some compiler warnings recently, so I did a "make 
> reconfig" (with logging of course) to catch any of them as there are in 
> what I compile.
> 
> My Huge build is made with the following settings (on 64-bit Linux):
> 
> export CONF_OPT_GUI='--enable-gnome-check'
> export CONF_OPT_LUA='--enable-luainterp'
> export CONF_OPT_PERL='--enable-perlinterp'
> export CONF_OPT_PYTHON='--enable-pythoninterp'
> export CONF_OPT_TCL='--enable-tclinterp'
> export CONF_OPT_RUBY='--enable-rubyinterp'
> export CONF_OPT_MZSCHEME='--disable-mzschemeinterp'
> export CONF_OPT_CSCOPE='--enable-cscope'
> export CONF_OPT_MULTIBYTE='--enable-multibyte'
> export CONF_OPT_FEAT='--with-features=huge'
> export CONF_OPT_COMPBY='"[email protected]"'
> export CONF_ARGS=''
> 
> I also define FEAT_XTERM_SAVE and undefine FEAT_TAG_OLDSTATIC in its 
> feature.h
> 
> As you can see, it includes almost all features but still leaves out the 
> Python3 and MzScheme modules. I also compile a Tiny build as a kind of 
> sanity test; there I exclude everything I can.
> 
> Results:
> 
> - No compiler warnings or errors in the Tiny build.
> 
> - The Huge build had the following warnings (one for a possibly 
> uninitialized variable in term.c and one for duplicate but in this case 
> not really conflicting defines in files included from if_ruby.c):
> 
> term.c: In function ‘check_termcode’:
> term.c:4217:42: warning: ‘row_char’ may be used uninitialized in this 
> function [-Wmaybe-uninitialized]
>     if (j == 1 && tp[i] == 'R' && row_char == '2' && col >= 2)
>                                            ^
> In file included from vim.h:275:0,
>                   from if_ruby.c:143:
> os_unix.h:568:0: warning: "HAVE_DUP" redefined [enabled by default]
>   #define HAVE_DUP  /* have dup() */
>   ^
> In file included from /usr/include/ruby-2.0.0/ruby/ruby.h:24:0,
>                   from /usr/include/ruby-2.0.0/ruby.h:33,
>                   from if_ruby.c:105:
> /usr/include/ruby-2.0.0/x86_64-linux/ruby/config.h:260:0: note: this is 
> the location of the previous definition
>   #define HAVE_DUP 1
>   ^
> 
> I suppose that the simplest way to avoid the latter message would be by 
> wrapping line 568 of os_unix.h (and possibly similar defines for other 
> platforms) as follows:
>       #ifndef HAVE_DUP        /* avoid duplication in if_ruby.h */
>       # define HAVE_DUP       /* have dup() */
>       #endif
> A better way, though, might be to give a different name to the define in 
> os_unix.h:568 but that means renaming it everywhere else that it is used.
> 
> - When I type "gcc --version" (without the quotes) at a shell prompt, I 
> get the following answer:
> gcc (SUSE Linux) 4.8.1 20130909 [gcc-4_8-branch revision 202388]
> Copyright (C) 2013 Free Software Foundation, Inc.
> This is free software; see the source for copying conditions.  There is NO
> warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
> 
> - I'm appending the :version message for this Huge build as an attachment.

Thanks, I'll make a patch.

-- 
            |

Ceci n'est pas une pipe.

 /// 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.

Raspunde prin e-mail lui