Fixed clang signed/unsigned warning in chunkcopy_safe.

nmoinvaz · Dead2 · commit 41faa0843d77 · 2022-03-29T13:14:18.000+02:00
inflate_p.h:159:18: warning: comparison of integers of different signs: 'int32_t' (aka 'int') and 'size_t' (aka 'unsigned long') [-Wsign-compare]
        tocopy = MIN(non_olap_size, len);
                 ^   ~~~~~~~~~~~~~  ~~~
zbuild.h:74:24: note: expanded from macro 'MIN'
#define MIN(a, b) ((a) &gt; (b) ? (b) : (a))
                    ~  ^  ~
diff --git a/inflate_p.h b/inflate_p.h
@@ -146,7 +146,7 @@ static inline uint8_t* chunkcopy_safe(uint8_t *out, uint8_t *from, size_t len, u
      * initial bulk memcpy of the nonoverlapping region. Then, we can leverage the size of this to determine the safest
      * atomic memcpy size we can pick such that we have non-overlapping regions. This effectively becomes a safe look
      * behind or lookahead distance */
-    int32_t non_olap_size = (int32_t)((from > out) ? from - out : out - from);
+    size_t non_olap_size = ((from > out) ? from - out : out - from);
 
     memcpy(out, from, non_olap_size);
     out += non_olap_size;
