On Sun, 18 Sep 2011 09:43:27 -0700, Luca Barbato <[email protected]> wrote:
> On 9/18/11 7:02 AM, Anton Khirnov wrote:
> > From: Stefano Sabatini<[email protected]>
> >
> > The comment is meant to be about the align parameter.
> >
> > Signed-off-by: Anton Khirnov<[email protected]>
> > ---
> >   libavfilter/defaults.c |    2 +-
> >   1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/libavfilter/defaults.c b/libavfilter/defaults.c
> > index 5532953..89e401c 100644
> > --- a/libavfilter/defaults.c
> > +++ b/libavfilter/defaults.c
> > @@ -41,7 +41,7 @@ AVFilterBufferRef 
> > *avfilter_default_get_video_buffer(AVFilterLink *link, int per
> >       uint8_t *data[4];
> >       AVFilterBufferRef *picref = NULL;
> >
> > -    // +2 is needed for swscaler, +16 to be SIMD-friendly
> > +    // align: +2 is needed for swscaler, +16 to be SIMD-friendly
> >       if ((ret = av_image_alloc(data, linesize, w, h, link->format, 16))<  
> > 0)
> >           return NULL;
> >
> 
> I can't see the +2 and the +16 this comment is referring to...
> 

16 is the last argument of av_image_alloc(), +2 really isn't there,
i guess could be removed.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to