Xavier de Gaye wrote:

> On Fri, May 17, 2013 at 11:56 PM, ZyX wrote:
> > These patches make all variants of invoking python use the same
> > DoPyCommand function (renamed from DoPy3Command and DoPythonCommand)
> > and moved actual implementation to if_py_both. Also made new py*do
> > commands use this and use other existing functions (like
> > GetBufferLine in place of pyline nonsense (it did not convert
> > newlines to nulls)).
> 
> 
> Would not it be better to have this code in an if_py_both.c source
> file instead of the if_py_both.h header file. It seems that
> if_py_both.h and dosinst.h are the only files in the vim source code
> that are actually source files instead of header files.

The new regexp code adds a regexp_nfa.c file that is included in
regexp.c.  It's a bit inconsistent.  Not sure if I should call it
regexp_nfa.h instead.

-- 
   [Another hideous roar.]
BEDEVERE: That's it!
ARTHUR:   What?
BEDEVERE: It's The Legendary Black Beast of Aaaaarrrrrrggghhh!
                 "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD

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