Christian Brabandt wrote:

> Bram,
> gcc complains when compiling latest Vim version (7.3.1090) with
> In file included from regexp.c:7864:0:
> regexp_nfa.c: In function 'nfa_regpiece':
> regexp_nfa.c:1390:19: warning: declaration of 'new_state' shadows a global 
> declaration [-Wshadow]
> regexp_nfa.c:270:21: warning: shadowed declaration is here [-Wshadow]
> regexp_nfa.c: In function 'nfa_regtry':
> regexp_nfa.c:4727:7: warning: declaration of 'i' shadows a previous local 
> [-Wshadow]
> regexp_nfa.c:4638:10: warning: shadowed declaration is here [-Wshadow]
> 
> 
> Attached patch fixes it.

Thanks.  I think I slightly prefer renaming new_state() to
alloc_state().  And remove the extra declaration.


-- 
"Oh, no!  NOT the Spanish Inquisition!"
"NOBODY expects the Spanish Inquisition!!!"
                                -- Monty Python sketch --
"Oh, no!  NOT another option!"
"EVERYBODY expects another option!!!"
                                -- Discussion in vim-dev mailing list --

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


Raspunde prin e-mail lui