[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