[FFmpeg-devel] [PATCH 2/3] hevcdsp: replace the SAOParams struct parameter from sao_band_filter

[James Almer]

Pass instead the two variables from the struct needed in the function.
This simplifies writing asm optimized versions of the function

Signed-off-by: James Almer <jamr...@gmail.com>
---
 libavcodec/hevc_filter.c      | 3 ++-
 libavcodec/hevcdsp.h          | 2 +-
 libavcodec/hevcdsp_template.c | 7 +++----
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/libavcodec/hevc_filter.c b/libavcodec/hevc_filter.c
index 30ade74..fb1aa37 100644
--- a/libavcodec/hevc_filter.c
+++ b/libavcodec/hevc_filter.c
@@ -259,7 +259,8 @@ static void sao_filter_CTB(HEVCContext *s, int x, int y)
         case SAO_BAND:
             copy_CTB(dst, src, width << s->sps->pixel_shift, height, 
stride_dst, stride_src);
             s->hevcdsp.sao_band_filter(src, dst, stride_src, stride_dst,
-                                       sao, width, height, c_idx);
+                                       sao->offset_val[c_idx], 
sao->band_position[c_idx],
+                                       width, height);
             restore_tqb_pixels(s, x, y, width, height, c_idx);
             sao->type_idx[c_idx] = SAO_APPLIED;
             break;
diff --git a/libavcodec/hevcdsp.h b/libavcodec/hevcdsp.h
index 353677a..a798fb1 100644
--- a/libavcodec/hevcdsp.h
+++ b/libavcodec/hevcdsp.h
@@ -59,7 +59,7 @@ typedef struct HEVCDSPContext {
     void (*idct_dc[4])(int16_t *coeffs);
 
     void (*sao_band_filter)(uint8_t *_dst, uint8_t *_src, ptrdiff_t 
_stride_dst, ptrdiff_t _stride_src,
-                            struct SAOParams *sao, int width, int height, int 
c_idx);
+                            int16_t *sao_offset_val, int sao_left_class, int 
width, int height);
 
     void (*sao_edge_filter[2])(uint8_t *_dst, uint8_t *_src, ptrdiff_t 
_stride_dst, ptrdiff_t _stride_src,
                                struct SAOParams *sao, int *borders, int _width,
diff --git a/libavcodec/hevcdsp_template.c b/libavcodec/hevcdsp_template.c
index a8fff45..23fdb10 100644
--- a/libavcodec/hevcdsp_template.c
+++ b/libavcodec/hevcdsp_template.c
@@ -302,16 +302,15 @@ IDCT_DC(32)
 #undef ADD_AND_SCALE
 
 static void FUNC(sao_band_filter_0)(uint8_t *_dst, uint8_t *_src,
-                                  ptrdiff_t stride_dst, ptrdiff_t stride_src, 
SAOParams *sao,
-                                  int width, int height, int c_idx)
+                                    ptrdiff_t stride_dst, ptrdiff_t stride_src,
+                                    int16_t *sao_offset_val, int 
sao_left_class,
+                                    int width, int height)
 {
     pixel *dst = (pixel *)_dst;
     pixel *src = (pixel *)_src;
     int offset_table[32] = { 0 };
     int k, y, x;
     int shift  = BIT_DEPTH - 5;
-    int16_t *sao_offset_val = sao->offset_val[c_idx];
-    int sao_left_class  = sao->band_position[c_idx];
 
     stride_dst /= sizeof(pixel);
     stride_src /= sizeof(pixel);
-- 
2.2.2

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel