[FFmpeg-cvslog] Merge commit '87c6c78604e4dd16f1f45862b27ca006da010527'

James Almer git at videolan.org
Sun Mar 19 20:15:45 EET 2017


ffmpeg | branch: master | James Almer <jamrial at gmail.com> | Sun Mar 19 15:10:28 2017 -0300| [e5623aafd8f6cba710f714e1d22c13d4a699792b] | committer: James Almer

Merge commit '87c6c78604e4dd16f1f45862b27ca006da010527'

* commit '87c6c78604e4dd16f1f45862b27ca006da010527':
  vp8: Change type of stride parameters to ptrdiff_t

Merged-by: James Almer <jamrial at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=e5623aafd8f6cba710f714e1d22c13d4a699792b
---

 libavcodec/vp8.c        | 19 ++++++++++---------
 libavcodec/vp8.h        |  4 ++--
 tests/checkasm/vp8dsp.c |  4 ++--
 3 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/libavcodec/vp8.c b/libavcodec/vp8.c
index 0682239..1e8808c 100644
--- a/libavcodec/vp8.c
+++ b/libavcodec/vp8.c
@@ -435,8 +435,8 @@ static void copy_chroma(AVFrame *dst, AVFrame *src, int width, int height)
     }
 }
 
-static void fade(uint8_t *dst, int dst_linesize,
-                 const uint8_t *src, int src_linesize,
+static void fade(uint8_t *dst, ptrdiff_t dst_linesize,
+                 const uint8_t *src, ptrdiff_t src_linesize,
                  int width, int height,
                  int alpha, int beta)
 {
@@ -1468,7 +1468,7 @@ void decode_mb_coeffs(VP8Context *s, VP8ThreadData *td, VP56RangeCoder *c,
 static av_always_inline
 void backup_mb_border(uint8_t *top_border, uint8_t *src_y,
                       uint8_t *src_cb, uint8_t *src_cr,
-                      int linesize, int uvlinesize, int simple)
+                      ptrdiff_t linesize, ptrdiff_t uvlinesize, int simple)
 {
     AV_COPY128(top_border, src_y + 15 * linesize);
     if (!simple) {
@@ -1479,7 +1479,7 @@ void backup_mb_border(uint8_t *top_border, uint8_t *src_y,
 
 static av_always_inline
 void xchg_mb_border(uint8_t *top_border, uint8_t *src_y, uint8_t *src_cb,
-                    uint8_t *src_cr, int linesize, int uvlinesize, int mb_x,
+                    uint8_t *src_cr, ptrdiff_t linesize, ptrdiff_t uvlinesize, int mb_x,
                     int mb_y, int mb_width, int simple, int xchg)
 {
     uint8_t *top_border_m1 = top_border - 32;     // for TL prediction
@@ -1632,7 +1632,8 @@ void intra_predict(VP8Context *s, VP8ThreadData *td, uint8_t *dst[3],
         for (y = 0; y < 4; y++) {
             uint8_t *topright = ptr + 4 - s->linesize;
             for (x = 0; x < 4; x++) {
-                int copy = 0, linesize = s->linesize;
+                int copy = 0;
+                ptrdiff_t linesize = s->linesize;
                 uint8_t *dst = ptr + 4 * x;
                 LOCAL_ALIGNED(4, uint8_t, copy_dst, [5 * 8]);
 
@@ -1738,7 +1739,7 @@ void vp8_mc_luma(VP8Context *s, VP8ThreadData *td, uint8_t *dst,
     uint8_t *src = ref->f->data[0];
 
     if (AV_RN32A(mv)) {
-        int src_linesize = linesize;
+        ptrdiff_t src_linesize = linesize;
 
         int mx = (mv->x * 2) & 7, mx_idx = subpel_idx[0][mx];
         int my = (mv->y * 2) & 7, my_idx = subpel_idx[0][my];
@@ -2084,8 +2085,8 @@ void filter_mb(VP8Context *s, uint8_t *dst[3], VP8FilterStrength *f,
     int filter_level = f->filter_level;
     int inner_limit = f->inner_limit;
     int inner_filter = f->inner_filter;
-    int linesize = s->linesize;
-    int uvlinesize = s->uvlinesize;
+    ptrdiff_t linesize   = s->linesize;
+    ptrdiff_t uvlinesize = s->uvlinesize;
     static const uint8_t hev_thresh_lut[2][64] = {
         { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1,
           2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2,
@@ -2171,7 +2172,7 @@ void filter_mb_simple(VP8Context *s, uint8_t *dst, VP8FilterStrength *f,
     int filter_level = f->filter_level;
     int inner_limit  = f->inner_limit;
     int inner_filter = f->inner_filter;
-    int linesize     = s->linesize;
+    ptrdiff_t linesize = s->linesize;
 
     if (!filter_level)
         return;
diff --git a/libavcodec/vp8.h b/libavcodec/vp8.h
index 6218fe0..3910b5c 100644
--- a/libavcodec/vp8.h
+++ b/libavcodec/vp8.h
@@ -143,8 +143,8 @@ typedef struct VP8Context {
 
     uint16_t mb_width;   /* number of horizontal MB */
     uint16_t mb_height;  /* number of vertical MB */
-    int linesize;
-    int uvlinesize;
+    ptrdiff_t linesize;
+    ptrdiff_t uvlinesize;
 
     uint8_t keyframe;
     uint8_t deblock_filter;
diff --git a/tests/checkasm/vp8dsp.c b/tests/checkasm/vp8dsp.c
index d3aff99..d93f730 100644
--- a/tests/checkasm/vp8dsp.c
+++ b/tests/checkasm/vp8dsp.c
@@ -171,7 +171,7 @@ static void check_idct_dc4(void)
     for (chroma = 0; chroma <= 1; chroma++) {
         void (*idct4dc)(uint8_t *, int16_t[4][16], ptrdiff_t) = chroma ? d.vp8_idct_dc_add4uv : d.vp8_idct_dc_add4y;
         if (check_func(idct4dc, "vp8_idct_dc_add4%s", chroma ? "uv" : "y")) {
-            int stride = chroma ? 8 : 16;
+            ptrdiff_t stride = chroma ? 8 : 16;
             int w      = chroma ? 2 : 4;
             for (i = 0; i < 4; i++) {
                 int blockx = 4 * (i % w);
@@ -365,7 +365,7 @@ static void randomize_loopfilter_buffers(int lineoff, int str,
 }
 
 // Fill the buffer with random pixels
-static void fill_loopfilter_buffers(uint8_t *buf, int stride, int w, int h)
+static void fill_loopfilter_buffers(uint8_t *buf, ptrdiff_t stride, int w, int h)
 {
     int x, y;
     for (y = 0; y < h; y++)


======================================================================

diff --cc libavcodec/vp8.c
index 0682239,546124c..1e8808c
--- a/libavcodec/vp8.c
+++ b/libavcodec/vp8.c
@@@ -435,9 -425,8 +435,9 @@@ static void copy_chroma(AVFrame *dst, A
      }
  }
  
- static void fade(uint8_t *dst, int dst_linesize,
-                  const uint8_t *src, int src_linesize,
 -static void fade(uint8_t *dst, uint8_t *src,
 -                 int width, int height, ptrdiff_t linesize,
++static void fade(uint8_t *dst, ptrdiff_t dst_linesize,
++                 const uint8_t *src, ptrdiff_t src_linesize,
 +                 int width, int height,
                   int alpha, int beta)
  {
      int i, j;
@@@ -1632,9 -1591,10 +1632,10 @@@ void intra_predict(VP8Context *s, VP8Th
          for (y = 0; y < 4; y++) {
              uint8_t *topright = ptr + 4 - s->linesize;
              for (x = 0; x < 4; x++) {
-                 int copy = 0, linesize = s->linesize;
+                 int copy = 0;
+                 ptrdiff_t linesize = s->linesize;
                  uint8_t *dst = ptr + 4 * x;
 -                DECLARE_ALIGNED(4, uint8_t, copy_dst)[5 * 8];
 +                LOCAL_ALIGNED(4, uint8_t, copy_dst, [5 * 8]);
  
                  if ((y == 0 || x == 3) && mb_y == 0) {
                      topright = tr_top;
@@@ -1738,10 -1698,10 +1739,10 @@@ void vp8_mc_luma(VP8Context *s, VP8Thre
      uint8_t *src = ref->f->data[0];
  
      if (AV_RN32A(mv)) {
-         int src_linesize = linesize;
+         ptrdiff_t src_linesize = linesize;
  
 -        int mx = (mv->x << 1) & 7, mx_idx = subpel_idx[0][mx];
 -        int my = (mv->y << 1) & 7, my_idx = subpel_idx[0][my];
 +        int mx = (mv->x * 2) & 7, mx_idx = subpel_idx[0][mx];
 +        int my = (mv->y * 2) & 7, my_idx = subpel_idx[0][my];
  
          x_off += mv->x >> 2;
          y_off += mv->y >> 2;



More information about the ffmpeg-cvslog mailing list