On 04/20/2011 03:45 PM, Anton Khirnov wrote: > From: Mark Himsley <m...@mdsh.com> > > draw_slice() checks that the fade factor is < 65536 and only > calculates the fade if so. But the fade factor is clipped in > end_frame() by av_clip_uint16() to 65535, so the fade is calculated > for every frame. This patch alters the check so that it compares with > < 65535 (UINT16_MAX). > > Signed-off-by: Anton Khirnov <an...@khirnov.net> > --- > libavfilter/vf_fade.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/libavfilter/vf_fade.c b/libavfilter/vf_fade.c > index 4c2cd4b..0c8668c 100644 > --- a/libavfilter/vf_fade.c > +++ b/libavfilter/vf_fade.c > @@ -101,7 +101,7 @@ static void draw_slice(AVFilterLink *inlink, int y, int > h, int slice_dir) > uint8_t *p; > int i, j, plane; > > - if (fade->factor < 65536) { > + if (fade->factor < UINT16_MAX) { > /* luma or rgb plane */ > for (i = 0; i < h; i++) { > p = outpic->data[0] + (y+i) * outpic->linesize[0];
looks correct to me. -Justin _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel