[FFmpeg-devel] [PATCH] Add crop filter

Stefano Sabatini stefano.sabatini-lala
Wed Oct 21 00:51:42 CEST 2009


On date Tuesday 2009-10-20 01:14:20 +0200, Michael Niedermayer encoded:
> On Mon, Oct 19, 2009 at 11:50:08PM +0200, Stefano Sabatini wrote:
> > On date Friday 2009-10-16 12:49:33 +0200, Michael Niedermayer encoded:
> > > On Fri, Oct 16, 2009 at 12:34:07AM +0200, Stefano Sabatini wrote:
> [...]
> > > > +        round_hsub = crop->hsub;
> > > > +        round_vsub = crop->vsub;
> > > > +    }
> > > > +
> > > > +    crop->x &= ~((1 << round_hsub) - 1);
> > > > +    crop->y &= ~((1 << round_vsub) - 1);
> > > 
> > > > +    crop->w &= ~((1 << round_hsub) - 1);
> > > > +    crop->h &= ~((1 << round_vsub) - 1);
> > > 
> > > i dont think these are needed
> > 
> > How so?
> 
> ffmpeg should support all videos, even ones with odd with or height,
> they arent common but they do exist out there.

Patch updated with other minor fixes.

Regards.
-- 
FFmpeg = Fantastic and Free Monstrous Picky Extroverse Game
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-crop-filter.patch
Type: text/x-diff
Size: 9716 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091021/de0b46d7/attachment.patch>



More information about the ffmpeg-devel mailing list