[FFmpeg-devel] [PATCH] mxfdec: Split essence container ul again by audio/video.
Reimar Döffinger
Reimar.Doeffinger at gmx.de
Fri Dec 9 21:41:53 CET 2011
On Fri, Dec 09, 2011 at 09:15:00PM +0100, Reimar Döffinger wrote:
> This fixes issue 721 where MP2 audio is detected and MPEG-2
> video, because only 14 bytes of the UL are compared and those
> are identical for both.
> In addition this is simply more robust: since we already check
> the codec_type anyway we should not needlessly risk assigning
> a codec_id that conflicts with it.
Read the reasoning, but forget the patch.
Carl's patch for it is actually correct, this one is not.
More information about the ffmpeg-devel
mailing list