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

[Michael Niedermayer]

On Sun, May 12, 2013 at 11:16:28PM +0200, Piotr Bandurski wrote:
> Hi,
> 
> > Please post patch again.
> 
> Not sure how to check the size of AVI1 + AVID. There is a value ($3d) before AVID
> tag, but is this really size of it (61 looks strange)? Also I would need to mess
> with len variable, so better someone else finish it.

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

I dont mind to finsh it if you prefer but i dont know why you check
that byte or if its even that byte that you wanted to check ...

Thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- 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/8953797e/attachment.asc>