From e43805d40188f2797619cdf068b64d2c3ada8a8f Mon Sep 17 00:00:00 2001 From: Guillaume Martres Date: Sat, 11 Jan 2014 22:46:25 +0100 Subject: [PATCH] hevc: remove useless clip in FUNC(sao_band_filter)() 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. This removes the code that was changed in 5856bca360c5bc3e340a357d91b1f993c80a7bea as the clip that was repositioned in that commit is removed Signed-off-by: Michael Niedermayer (cherry picked from commit b00a8b4d194f1bf23343f3f42138affa1fe26641) Signed-off-by: Michael Niedermayer --- 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 8e9db63583..4f6b7119d2 100644 --- a/libavcodec/hevcdsp_template.c +++ b/libavcodec/hevcdsp_template.c @@ -391,7 +391,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; }