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

Reimar Döffinger Reimar.Doeffinger
Fri Jul 16 07:39:31 CEST 2010


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.



More information about the ffmpeg-devel mailing list