[FFmpeg-devel] PATCH: fourcc for VP6 Flash / FLV4 in .mkv

[Anssi Hannula]

Anssi Hannula kirjoitti keskiviikko, 28. huhtikuuta 2010 03:24:05:
> Christian P. Schmidt kirjoitti maanantai, 19. huhtikuuta 2010 13:24:43:
> > Hi,
> > 
> > I just encountered a file in the wild that contains FLV4 in .mkv. The
> 
> [...]
> 
> > +++ libavformat/riff.c  (working copy)
> > @@ -180,6 +180,7 @@
> > 
> >      { CODEC_ID_VP6,          MKTAG('V', 'P', '6', '0') },
> >      { CODEC_ID_VP6,          MKTAG('V', 'P', '6', '1') },
> >      { CODEC_ID_VP6,          MKTAG('V', 'P', '6', '2') },
> > 
> > +    { CODEC_ID_VP6F,         MKTAG('F', 'L', 'V', '4') },
> > 
> >      { CODEC_ID_ASV1,         MKTAG('A', 'S', 'V', '1') },
> >      { CODEC_ID_ASV2,         MKTAG('A', 'S', 'V', '2') },
> >      { CODEC_ID_VCR1,         MKTAG('V', 'C', 'R', '1') },
> 
> This was added before the previous VP6F entry:
>     { CODEC_ID_VP6F,         MKTAG('V', 'P', '6', 'F') },
> 
> so it is now used for encoding by default as well. Was that intended?

If not, attached is a patch to move the older VP6F entry above FLV4, closer to 
VP6/VP5/VP3 entries.

-- 
Anssi Hannula
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-move-vp6f-up.patch
Type: text/x-patch
Size: 1059 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100428/73c35e48/attachment.bin>