[FFmpeg-devel] [PATCH][RFC] Fix field order forinterlacedAVDJand AVRN in ntsc mode

[Michael Niedermayer]

On Sun, May 19, 2013 at 03:11:33AM +0200, Piotr Bandurski wrote:
> Hi,
> 
> > iam not sure what value to check for what, the app marker is 16bytes
> > in ntsc.avi the patch causes a byte to be read that is not part
> > of the app marker but outside
> > If you intend to read that byte thats probably part of the
> > AVID comment and would need to be read from mjpeg_decode_com()
> > the len (61) is probably also parsed corectly there and available for
> > checking
> 
> Ok, fixed. Please see attached patch.
> 
> The question is should I remove previous AVID string check and repleace it
> with new one (it will not break any samples?) or leave it as it is now?

i dont know, merging the 2 if it works is surely better than having
2 checks. 

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130519/2bd4bc91/attachment.asc>