[vlc-devel] [vlc-commits] commit: Parser: generate the header file (not perfect but not too bad). ( Rémi Duraffort )

[Olivier Aubert]

On Wed, 2010-04-28 at 14:35 +0200, Rémi Duraffort wrote:
> Hello,
> > Great to autogenerate, but too bad to duplicate parsing efforts (hence
> > also duplicating maintenance of the parsers). Was some information
> > missing from the output of the parser from the python code generator?
> I created my own parser mainly because it took me more time to
> understand your parser than to actually write a simple one.
Well, I did not see much request for clarifications. And I would think that the documentation 
of the parse_include method:
"""Parse include file.

This generates a tuple for each function:
(return_type, method_name, parameter_list, comment)
with parameter_list being a list of tuples (parameter_type, parameter_name).
"""      
is rather clear, but I am obviously biased. Basically, its output could
be fed almost as-is to your generate_class (run generaty.py --debug
include/vlc/*.h to see its output), and would save multiple lines right
now, not counting the hassle it would save for later (when you write the
support for libvlc_media_list_player, with its multiline declarations,
and one line comments, etc).

Anyway, if you want to give it another try, I am willing to give a hand.
Else, too bad, let's go our own ways.

Regards,
Olivier