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

Måns Rullgård mru
Mon Jul 10 00:10:40 CEST 2006


M?ns Rullg?rd <mru at inprovide.com> writes:

> 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.

Hmm... already a flaw found.  Updated patch attached.

-- 
M?ns Rullg?rd
mru at inprovide.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: format-select.patch
Type: text/x-patch
Size: 90816 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060709/ef4d2ce8/attachment.bin>



More information about the ffmpeg-devel mailing list