[Ffmpeg-devel] [PATCH] enable/disable (de)muxers

Michael Niedermayer michaelni
Sun Jul 9 23:55:32 CEST 2006


Hi

On Sun, Jul 09, 2006 at 10:46:36PM +0100, M?ns Rullg?rd wrote:
> M?ns Rullg?rd <mru at inprovide.com> writes:
> 
> > Michael Niedermayer <michaelni at gmx.at> writes:
> >
> >> Hi
> >>
> >> On Sun, Jul 09, 2006 at 01:28:58PM +0100, M?ns Rullg?rd wrote:
> >>> Hi all,
> >>> 
> >>> This topic has been discussed previously, and I had the idea that I'd
> >>> finally do something about it.
> >>> 
> >>> To make individual (de)muxer selection possible, the configure script
> >>> needs to fetch a list of available formats somewhere.  Unfortunately,
> >>> there is no single lists that contains them all, as there is with the
> >>> codecs.  The allformats.c file instead calls bunch of functions, such
> >>> as mpegps_init(), which call av_register_input/output_format().  Thus
> >>> we have two options: 1) grep all the source files for declarations of
> >>> AVInputFormat and AVOutputFormat, or 2) move the av_register_*_format
> >>> calls to av_register_all().  In either case, the names of the structs
> >>> need to be made consistent.  This should not be a problem, as most of
> >>> them are declared static, so there can be no external references, not
> >>> that there should be any in the first place.
> >>> 
> >>> Please let me know your thoughts about this.
> >>
> >> i am in favor of 2. (move the av_register_*_format calls to
> >> av_register_all())
> >
> > That's what I prefer too.  I'll see what I can do.
> 
> OK, here's the initial patch for review.  I'd like to have someone
> else cast an eye on it before I commit, just in case I did something
> really silly.

ive quickly scrolled through it and cast 1/4 of an eye on it ...

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

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is




More information about the ffmpeg-devel mailing list