On 2013-05-13 21:47, Luca Barbato wrote:
On 05/13/2013 09:42 PM, Diego Biurrun wrote:
jpeg2000dec.c:1200:20: warning: format ‘%lX’ expects argument of type ‘long 
unsigned int’, but argument 5 has type ‘long long unsigned int’
--- a/libavcodec/jpeg2000dec.c
+++ b/libavcodec/jpeg2000dec.c
@@ -1197,7 +1197,7 @@ static int 
jpeg2000_read_main_headers(Jpeg2000DecoderContext *s)
          default:
              av_log(s->avctx, AV_LOG_ERROR,
                     "unsupported marker 0x%.4X at pos 0x%lX\n",
-                   marker, (uint64_t)(s->buf - s->buf_start - 4));
+                   marker, s->buf - s->buf_start - 4);
              s->buf += len - 2;

probably ok, maybe "Do not cast to the wrong type" might describe better.

Huh, that's not the right version of the patch, the right printf modifier for pointer differences is 't'.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to