Mans Rullgard <[EMAIL PROTECTED]> added the comment:

Michael Niedermayer <[EMAIL PROTECTED]> writes:

> Michael Niedermayer <[EMAIL PROTECTED]> added the comment:
>
> On Fri, Oct 19, 2007 at 08:15:08AM -0000, Mans Rullgard wrote:
> [...]
>> > as far is i understand this bug is caused by the r7853 code causing a
>> > correct detection of an invalidly set flag and then trying to resync from
>> > the last startcode, while e7852 did ignore the invalid flag
>> 
>> Yes, that's what's happening.  I'm happy to leave it this way, and
>> fail to play broken files.  However, you are usually of the opinion
>> that everything that can be played at all should be supported, even if
>> the files are invalid.  It's your call.
>
> yes, i do want as many files supported as possible, these vdr files included
> i was just trying to defend r7562 :) i didnt mean to suggest that we should
> not add a workaround (like your patch) which allows such files to be played

OK, but any workaround must print a warning about the invalidity of
the file.  It's our only hope of ever eradicating them.

______________________________________________________
FFmpeg issue tracker <[EMAIL PROTECTED]>
<https://roundup.mplayerhq.hu/roundup/ffmpeg/issue198>
______________________________________________________

Reply via email to