[FFmpeg-devel] [PATCH] rotate filter

[Clément Bœsch]

On Wed, Jun 12, 2013 at 09:56:36AM +0200, Clément Bœsch wrote:
[...]
> > +AVFilter avfilter_vf_rotate = {
> > +    .name        = "rotate",
> > +    .description = NULL_IF_CONFIG_SMALL("Rotate the input image."),
> > +    .priv_size   = sizeof(RotContext),
> > +    .init        = init,
> > +    .uninit      = uninit,
> 
> > +    .query_formats = query_formats,
> 
> nit++: use this one for align basis.
> 
> > +    .inputs      = rotate_inputs,
> > +    .outputs     = rotate_outputs,
> > +    .priv_class  = &rotate_class,
> > +};
> 
> I didn't find anything obviously wrong, but I didn't look at the math
> involved. Patch LGTM.
> 

Ah I forgot; what about timeline support?

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130612/16f061ff/attachment.asc>