[MPlayer-dev-eng] [PATCH] [RFC] Demuxer cleanup

[Gianluigi Tiesi]

On Tue, Jul 26, 2005 at 12:37:10AM +0200, Roberto Togni wrote:
> Hi all.
> 
> This patch replaces the big switch statements in demuxer.c with a
> modularized structure, like the one already used for codecs and streams.
> 
> Now to add a demuxer you only have to add a new DEMUXER_TYPE_ define,
> add a pointer to a demuxer description structure to the list at the top
> of demuxer.c, and add the demuxer_desc_t structure in your code.
> 
> This is a big patch, but most of the changes are very simple:
> - demuxer.c: almost all changes are here
> - demuxer_*.c: some functions made static, added description structures,
> created open_ and check_ functions to host code moved from demuxer.c
> (where needed)
> - mplayer.c: added code for -demuxer help
> - cfg-common.h: -demuxer now takes a string as parameter
> 
> As a bonus you get:
> - -demuxer help
> - demuxer selection by name
> 
> 
> Please test. If you can please test also external audiofile and tv
> (this is untested, i don't have a tv card).
> 
> If someone is missing in author field of a demuxer please report.
> 
> I'd like to apply it soon if there are no objections (this weekend?),
> unless you think it's better to delay it after pre9 (when?)
> 
> Ciao,
>  Roberto

It seams fine, btw:

+  char *filename; ///< Ugly hack, needed by avs_check_file

;( - There is no way to pass file descriptor/handle instead of filename
to avisynth, <semi_ot>also I would ask if someone intrested to help me to add
audio support, and more difficult, add linux support, this means add
exception handler to win32 loader on linux, it cannot be borrowed from
wine since it has a more complex implementation :(</semi_ot>


Bye

-- 
Gianluigi Tiesi <sherpya at netfarm.it>
EDP Project Leader
Netfarm S.r.l. - http://www.netfarm.it/
Free Software: http://oss.netfarm.it/