upipe-v210: change pixel argument types to uintptr_t

The reasoning for this is:
- We want it to be unsigned because we don't expect a negative value to
  be passed.  We perform unsigned shifts on the value in some cases.
- We want it to be register sized so that we do not have to extend it on
  x86-64.

Author: JDarnley

lib/upipe-v210/upipe_v210enc.c

@@ -57,12 +57,12 @@
 /** @This defines an 8-bit packing function. */
 typedef void (*upipe_v210enc_pack_line_8)(
         const uint8_t *y, const uint8_t *u, const uint8_t *v,
-        uint8_t *dst, ptrdiff_t width);
+        uint8_t *dst, uintptr_t width);
 
 /** @This defines a 10-bit packing function. */
 typedef void (*upipe_v210enc_pack_line_10)(
         const uint16_t *y, const uint16_t *u, const uint16_t *v,
-        uint8_t *dst, ptrdiff_t width);
+        uint8_t *dst, uintptr_t width);
 
 /** upipe_v210enc structure with v210enc parameters */
 struct upipe_v210enc {

lib/upipe-v210/v210enc.c

@@ -62,7 +62,7 @@ static inline void wl32(uint8_t *dst, uint32_t u)
     } while (0)
 
 void upipe_planar_to_v210_8_c(const uint8_t *y, const uint8_t *u,
-                                 const uint8_t *v, uint8_t *dst, ptrdiff_t pixels)
+        const uint8_t *v, uint8_t *dst, uintptr_t pixels)
 {
     uint32_t val;
     int i;
@@ -81,7 +81,7 @@ void upipe_planar_to_v210_8_c(const uint8_t *y, const uint8_t *u,
 }
 
 void upipe_planar_to_v210_10_c(const uint16_t *y, const uint16_t *u,
-                                  const uint16_t *v, uint8_t *dst, ptrdiff_t pixels)
+        const uint16_t *v, uint8_t *dst, uintptr_t pixels)
 {
     uint32_t val;
     int i;

lib/upipe-v210/v210enc.h

@@ -5,18 +5,13 @@
 #include <stddef.h>
 #include <stdint.h>
 
-void upipe_planar_to_v210_8_c(const uint8_t *y, const uint8_t *u, const uint8_t *v, uint8_t *dst, ptrdiff_t pixels);
-void upipe_planar_to_v210_10_c(const uint16_t *y, const uint16_t *u, const uint16_t *v, uint8_t *dst, ptrdiff_t pixels);
+void upipe_planar_to_v210_8_c(const uint8_t *y, const uint8_t *u, const uint8_t *v, uint8_t *dst, uintptr_t pixels);
+void upipe_planar_to_v210_10_c(const uint16_t *y, const uint16_t *u, const uint16_t *v, uint8_t *dst, uintptr_t pixels);
 
-void upipe_planar_to_v210_10_avx2(const uint16_t *y, const uint16_t *u,
-                                    const uint16_t *v, uint8_t *dst, ptrdiff_t pixels);
-void upipe_planar_to_v210_10_ssse3(const uint16_t *y, const uint16_t *u,
-                                     const uint16_t *v, uint8_t *dst, ptrdiff_t pixels);
-void upipe_planar_to_v210_8_ssse3(const uint8_t *y, const uint8_t *u,
-                                    const uint8_t *v, uint8_t *dst, ptrdiff_t pixels);
-void upipe_planar_to_v210_8_avx(const uint8_t *y, const uint8_t *u,
-                                  const uint8_t *v, uint8_t *dst, ptrdiff_t pixels);
-void upipe_planar_to_v210_8_avx2(const uint8_t *y, const uint8_t *u,
-                                   const uint8_t *v, uint8_t *dst, ptrdiff_t pixels);
+void upipe_planar_to_v210_10_avx2(const uint16_t *y, const uint16_t *u, const uint16_t *v, uint8_t *dst, uintptr_t pixels);
+void upipe_planar_to_v210_10_ssse3(const uint16_t *y, const uint16_t *u, const uint16_t *v, uint8_t *dst, uintptr_t pixels);
+void upipe_planar_to_v210_8_ssse3(const uint8_t *y, const uint8_t *u, const uint8_t *v, uint8_t *dst, uintptr_t pixels);
+void upipe_planar_to_v210_8_avx(const uint8_t *y, const uint8_t *u, const uint8_t *v, uint8_t *dst, uintptr_t pixels);
+void upipe_planar_to_v210_8_avx2(const uint8_t *y, const uint8_t *u, const uint8_t *v, uint8_t *dst, uintptr_t pixels);
 
 #endif

tests/checkasm/planar10_input.c

@@ -35,7 +35,7 @@ static void randomize_buffers(uint16_t *src0, uint16_t *src1, int len)
 void checkasm_check_planar10_input(void)
 {
     struct {
-        void (*v210)(const uint16_t *y, const uint16_t *u, const uint16_t *v, uint8_t *dst, ptrdiff_t pixels);
+        void (*v210)(const uint16_t *y, const uint16_t *u, const uint16_t *v, uint8_t *dst, uintptr_t pixels);
     } s = {
         .v210 = upipe_planar_to_v210_10_c,
     };
@@ -62,7 +62,7 @@ void checkasm_check_planar10_input(void)
         uint8_t dst1[NUM_SAMPLES * 4 / 3 + 32];
         const int pixels = NUM_SAMPLES / 2 / 6 * 6;
 
-        declare_func(void, const uint16_t *y, const uint16_t *u, const uint16_t *v, uint8_t *dst, ptrdiff_t pixels);
+        declare_func(void, const uint16_t *y, const uint16_t *u, const uint16_t *v, uint8_t *dst, uintptr_t pixels);
 
         randomize_buffers(y0, y1, NUM_SAMPLES/2);
         randomize_buffers(u0, u1, NUM_SAMPLES/4);

tests/checkasm/planar8_input.c

@@ -37,7 +37,7 @@ static void randomize_buffers(uint8_t *src0, uint8_t *src1, int len)
 void checkasm_check_planar8_input(void)
 {
     struct {
-        void (*v210)(const uint8_t *y, const uint8_t *u, const uint8_t *v, uint8_t *dst, ptrdiff_t pixels);
+        void (*v210)(const uint8_t *y, const uint8_t *u, const uint8_t *v, uint8_t *dst, uintptr_t pixels);
     } s = {
         .v210 = upipe_planar_to_v210_8_c,
     };
@@ -67,7 +67,7 @@ void checkasm_check_planar8_input(void)
         uint8_t dst1[NUM_SAMPLES * 4 / 3 + 32];
         const int pixels = NUM_SAMPLES / 2 / 6 * 6;
 
-        declare_func(void, const uint8_t *y, const uint8_t *u, const uint8_t *v, uint8_t *dst, ptrdiff_t pixels);
+        declare_func(void, const uint8_t *y, const uint8_t *u, const uint8_t *v, uint8_t *dst, uintptr_t pixels);
 
         randomize_buffers(y0, y1, NUM_SAMPLES/2);
         randomize_buffers(u0, u1, NUM_SAMPLES/4);