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

[Måns Rullgård]

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

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

OK, now I did something silly.  I sent the old patch again.  New patch
attached.  Honest.

-- 
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: 90764 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060709/d5470a37/attachment.bin>