[FFmpeg-devel] [PATCH] Add WebM to the Matroska demuxer name

Måns Rullgård mans
Fri Jul 16 09:00:12 CEST 2010


Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:

> On Thu, Jul 15, 2010 at 02:46:28PM -0700, Baptiste Coudurier wrote:
>> On 07/15/2010 02:37 PM, Reimar D?ffinger wrote:
>> >>>Under the current ABI it is a comma separated list that is allowed to
>> >>>grow (though It grows quite infrequently). It seems like doing it the
>> >>>right way is possible under the current ABI (all you would have to do
>> >>>is compare the first name up until the first comma or NUL). Perhaps
>> >>>you can enlighten us on why mplayer feels it necessary to do it the
>> >>>way it does.
>> >>
>> >>Probably for the same reason that av_guess_format does that?
>> >
>> >Btw. even av_find_input_format is rather questionable in my opinion.
>> >av_find_input_format(format->name) will return nothing for all formats
>> >that contain a ',' in name.
>> 
>> No, it will return the format.
>
> Sorry, that case is indeed fine.
> I think it does have some strange behaviour like accepting
> m4a,3
> for mov but not
> mp4,mov
> for example.

It accepts the entire string as stored or any subsequence from the
list.  It does not accept arbitrary subsets.  All very confusing.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list