[Ffmpeg-devel] [PATCH] mjpeg cleanup and again interlaced fix

[Michael Niedermayer]

Hi

On Wed, Apr 11, 2007 at 03:39:28PM +0200, Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
> > Hi
> > 
> > On Wed, Apr 11, 2007 at 02:17:14PM +0200, Baptiste Coudurier wrote:
> >> Hi
> >>
> >> 3 patches:
> >>
> >> - remove useless MpegEncContext.
> > 
> > ok if it works
> > 
> > 
> >> - fix odd field height decoding.
> >> - fix some interlaced content.
> >>
> >> justification:
> >> odd_heigth.mov, this sample is 720x405, and setting height *= 2 is then
> >> wrong, correct is to set it to container height.
> >> Since height change every field, do that for every field, not only for
> >> first picture.
> > 
> > as org_height is only set during coedec init i dont see how doing
> > height= org_height per field could change the height per field
> 
> It changes avctx height, not height per field ? Feel free to fix it
> in another way if you prefer.

fine ill accept the 2nd patch too

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

1. write clean, simple and working code
2. high level optimizations
3. low level optimizations
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070411/9ae55420/attachment.pgp>