[Ffmpeg-devel] [PATCH] VP6 to flv muxing

[Benjamin Larsson]

M?ns Rullg?rd wrote:

>Benjamin Larsson <banan at student.ltu.se> writes:
>
>  
>
>>Hi,
>>
>>Michael Niedermayer wrote:
>>
>>    
>>
>>>if you add a byte during flv muxing, shouldnt you remove i during flv
>>>demuxing?
>>>
>>>[...]
>>>
>>> 
>>>
>>>      
>>>
>>That could be done or define a new codec id and let that decoder handle
>>the extra byte.
>>    
>>
>
>I don't like that idea at all.  If one container requires some extra
>stuff, then it's the responsibility of the demuxer to deal with it.
>It's unfortunate when this happens, but there is no other sensible way
>to handle it.
>
>  
>
Ok, point taken. This patch adds the vp6 in flv demuxer part. But as
long as we don't have a native vp6 decoder and  mplayer refuses to use
the lavf demuxer to demux vp6 files with the binary decoder, the patch
is quite useless. And currently mencoder with the binary vp6 encoder
only encodes vp6 content with default settings so it's quite useless.
Based on that I'm holding of the muxer commit until the it's possible to
set quality properly. And the decoder commit will have to wait until
lavf flv demuxing is possible or a native decoder is presented. A native
decoder should be able to detect that extra byte so the patch might be
useless anyway.

MvH
Benjamin Larsson

-- 
"incorrect information" is an oxymoron. Information is, by definition, factual, correct.

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: vp6flv.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060611/3c4616ab/attachment.txt>