[Ffmpeg-devel] Re: [PATCH] MXF unplayable file

Baptiste Coudurier baptiste.coudurier
Wed Aug 2 12:40:46 CEST 2006


Hi

Reimar D?ffinger wrote:
> Hello,
> On Wed, Aug 02, 2006 at 12:13:14AM +0200, Baptiste Coudurier wrote:
>> File is not valid. IMHO, no reason to play it. MXF is already a bloated
>> mess and I see no reason to polute the code to handle non valid files,
>> or if we really want to, isolate the code and isolate broken files as
>> soon as possible.
> 
> Would simply skipping tags with size == 0 maybe be acceptable? That
> should be very little code.
> And will you apply some of the fixes in this patch, like the missing
> CODEC_ID_NONE thing, or should I send a separate patch?
> 

Yes, it is acceptable. I already applied the CODEC_ID_NONE fix. Also Im
not against using container uls, it would just need to be complete and
accurate.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312




More information about the ffmpeg-devel mailing list