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

Michael Niedermayer michaelni
Fri Apr 13 23:27:04 CEST 2007


Hi

On Wed, Apr 11, 2007 at 08:36:04PM +0200, Michael Niedermayer wrote:
> 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

and the 2nd patch should be split in 2 as it changes to seperate things

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- 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/20070413/72ad6421/attachment.pgp>



More information about the ffmpeg-devel mailing list