On Tue, Apr 21, 2015 at 2:23 PM Carl Eugen Hoyos <ceho...@ag.or.at> wrote:

> Christopher Watkins <watk <at> chromium.org> writes:
>
> > Hi, this patch just removes a compiler warning.
>
> Please mention ticket #4467 is the commit message.
>
>
Oh, I didn't see that. Done.
From 041b6899c10a082c5d019678e78588dc26474a6c Mon Sep 17 00:00:00 2001
From: Chris Watkins <w...@chromium.org>
Date: Tue, 21 Apr 2015 14:14:19 -0700
Subject: [PATCH] avformat/mov.c: Use %d to print an enum, not PRIu8

Using the PRIu8 format specifier to print an enum value causes a
compiler warning, so use %d instead.

Fixes ticket #4467.

Signed-off-by: Chris Watkins <w...@chromium.org>
---
 libavformat/mov.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index 1af8fb6..0601830 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -1230,7 +1230,7 @@ static int mov_read_aclr(MOVContext *c, AVIOContext *pb, MOVAtom atom)
                         av_log(c, AV_LOG_WARNING, "ignored unknown aclr value (%d)\n", range_value);
                         break;
                     }
-                    av_dlog(c, "color_range: %"PRIu8"\n", codec->color_range);
+                    av_dlog(c, "color_range: %d\n", codec->color_range);
                 } else {
                   /* For some reason the whole atom was not added to the extradata */
                   av_log(c, AV_LOG_ERROR, "aclr not decoded - incomplete atom\n");
-- 
2.2.0.rc0.207.ga3a616c

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

Reply via email to