[vlc-devel] commit: It's FOURCC update time! ( Felix Paul Kühne )

[Felix Paul Kühne]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


On 26.05.2008, at 18:09, Rémi Denis-Courmont wrote:
>
> Why duplicate MPEG-1 ???

Sorry for the late response. I've just compared ffmpeg.c from the  
bugfix branch with what we got in trunk. Somehow, certain MPEG1  
FOURCC's were missing in the branch, so I added them. If you read  
ffmpeg.c at branch now, you'll notice that this is all about MPEG1:

     /* MPEG-1 Video */
     { VLC_FOURCC('m','p','1','v'), CODEC_ID_MPEG1VIDEO,
       VIDEO_ES, "MPEG-1 Video" },
     { VLC_FOURCC('m','p','e','g'), CODEC_ID_MPEG1VIDEO,
       VIDEO_ES, "MPEG-1 Video" },
     { VLC_FOURCC('m','p','g','1'), CODEC_ID_MPEG1VIDEO,
       VIDEO_ES, "MPEG-1 Video" },
     { VLC_FOURCC('m','p','e','g'), CODEC_ID_MPEG1VIDEO,
       VIDEO_ES, "MPEG-1 Video" },
     { VLC_FOURCC('P','I','M','1'), CODEC_ID_MPEG1VIDEO,
       VIDEO_ES, "Pinnacle DC1000 (MPEG-1 Video)" },

(lines 523...533)

Thus, IMHO this commit is correct. Please tell me, if I am wrong.

Best,
Felix


-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.8 (Darwin)

iEYEARECAAYFAkg7H7YACgkQtUgezvXAcGOPIgCg5BBABp50XShOtsuz5tdKnnGe
C2QAnjEt04d0MXMUswkcKyq6byXhqdLw
=Xo1b
-----END PGP SIGNATURE-----