[FFmpeg-devel] [RFC] documenting the table generator infrastructure

Michael Niedermayer michaelni
Wed Mar 24 20:03:44 CET 2010


On Wed, Mar 24, 2010 at 07:44:03PM +0100, Reimar D?ffinger wrote:
> On Wed, Mar 24, 2010 at 03:22:46PM +0100, Michael Niedermayer wrote:
> > On Mon, Mar 22, 2010 at 12:14:33AM +0100, Reimar D?ffinger wrote:
> > > A table generator consists of two files, *_tablegen.c and *_tablegen.h.
> > > The .h file will provide the variable declarations and initialization
> > > code for the tables, the .c describes the tables so they can be printed
> > > as a header file.
> > 
> > shouldnt the c then be called *_tableprint.c ?
> 
> Well, in my idea of the whole thing, the tablegen files only have the code
> to generate the tables and describe them, what the purpose if that description
> is is none of their concern (tableprint.c takes care of that).
> 
> > > In addition it must also contain a "const struct tabledef tables[]"
> > > array describing the tables to be generated.
> > 
> > this too is ugly, a print_tables(tables) with no naming constraint on
> > tables seems cleaner
> 
> Maybe it should be changed. As said, my plan was a bit beyond what we have
> now, so you could just e.g. link against a tableanalyze.c and it would not
> print the tables but process them.

see readelf/objdump and analyze all tables not just the 2% that are generated
with the generators


> However that already broke when I used printing functions instead of trying
> to describe the table completely, so maybe it makes sense to just give up on
> that.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Thouse who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100324/1a8a2b38/attachment.pgp>



More information about the ffmpeg-devel mailing list