On Mon, 20 Jan 2014 11:25:35 +0100, Luca Barbato <[email protected]> wrote: > From: Guillaume Martres <[email protected]> > > The src buffer should only contain values in the interval > [0, (1 << BIT_DEPTH) - 1]. > > Since shift = (BIT_DEPTH - 5), src[x] >> shift must be in > the interval [0, 31], so no clip is needed. > > Signed-off-by: Luca Barbato <[email protected]> > --- > libavcodec/hevcdsp_template.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/hevcdsp_template.c b/libavcodec/hevcdsp_template.c > index 7ebe490..104c1eb 100644 > --- a/libavcodec/hevcdsp_template.c > +++ b/libavcodec/hevcdsp_template.c > @@ -393,7 +393,7 @@ static void FUNC(sao_band_filter)(uint8_t *_dst, uint8_t > *_src, > offset_table[(k + sao_left_class) & 31] = sao_offset_val[k + 1]; > for (y = 0; y < height; y++) { > for (x = 0; x < width; x++) > - dst[x] = av_clip_pixel(src[x] + > offset_table[av_clip_pixel(src[x] >> shift)]); > + dst[x] = av_clip_pixel(src[x] + offset_table[src[x] >> shift]); > dst += stride; > src += stride; > } > -- > 1.8.5.1 >
Ok -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
