[Ffmpeg-devel] Re: [PATCH] MXF tag parser

Michael Niedermayer michaelni
Wed Aug 2 22:32:57 CEST 2006


On Wed, Aug 02, 2006 at 06:46:59PM +0200, Reimar D?ffinger wrote:
> > > Maybe, but if the end result will be really better? And what if one day
> > > we wanted to add checks if maybe tags are used that the specs do not
> > > allow in that section?
> > 
> > That would mean non valid files. Specs are specs and SMPTE usually
> > really consider everything in the specs. Like I said, if we really want
> > to support broken files, that code needs to be really isolated. Your
> > file with '0' tag size is broken.
> 1) I admit coming from MPlayer my goal is in the end to support as many
> files as possible, broken or not.

ffmpeg has that goal too, if a file exists in the real world
(intentionally generated files dont count) then we should support it
if we would drop everything which isnt spec conformant then <10% of
the avi files you can find would be playable
so IMHO if your files are obscure test streams generated by you or
some smartass who lacks a brain then we shouldnt support them, if
OTOH they are some normal video downloaded/bought/captured/encoded
by a unmodified encoder then we should try to support them 


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is

More information about the ffmpeg-devel mailing list