On 10/03/14 11:37 AM, Diego Biurrun wrote:
> ---
>  libavformat/apetag.c         |    6 ++++--
>  libavformat/asfdec.c         |    8 +++++---
>  libavformat/avidec.c         |    4 ++--
>  libavformat/bink.c           |   10 +++++++---
>  libavformat/cafdec.c         |    5 ++++-
>  libavformat/crcenc.c         |    4 +++-
>  libavformat/dfa.c            |    7 +++++--
>  libavformat/dxa.c            |    5 ++++-
>  libavformat/electronicarts.c |    8 +++++---
>  libavformat/framecrcenc.c    |    4 +++-
>  libavformat/gxf.c            |    6 +++++-
>  libavformat/hnm.c            |   11 +++++++----
>  libavformat/iff.c            |    4 +++-
>  libavformat/lxfdec.c         |    9 ++++++---
>  libavformat/matroskadec.c    |    3 ++-
>  libavformat/mov.c            |    7 ++++---
>  libavformat/mvi.c            |    5 ++++-
>  libavformat/mxfdec.c         |   13 ++++++++-----
>  libavformat/omadec.c         |    8 +++++---
>  libavformat/rmdec.c          |    4 +++-
>  libavformat/rpl.c            |    4 ++--
>  libavformat/smacker.c        |    8 ++++++--
>  libavformat/smjpegdec.c      |    8 +++++---
>  libavformat/spdifenc.c       |    8 +++++---
>  libavformat/wtv.c            |    6 ++++--
>  libavformat/xmv.c            |    6 +++---
>  26 files changed, 114 insertions(+), 57 deletions(-)

[...]

> @@ -539,14 +539,15 @@ static int mxf_read_partition_pack(void *arg, 
> AVIOContext *pb, int tag, int size
>      }
>  
>      if (partition->kag_size <= 0 || partition->kag_size > (1 << 20)) {
> -        av_log(mxf->fc, AV_LOG_WARNING, "invalid KAGSize %i - guessing ", 
> partition->kag_size);
> +        av_log(mxf->fc, AV_LOG_WARNING, "invalid KAGSize %"PRId32" - 
> guessing ",
> +               partition->kag_size);

PRIi32? Ditto for any similar case.

[...]

> @@ -501,7 +503,7 @@ static void get_tag(AVFormatContext *s, AVIOContext *pb, 
> const char *key, int ty
>          return;
>  
>      if (type == 0 && length == 4) {
> -        snprintf(buf, buf_size, "%"PRIi32, avio_rl32(pb));
> +        snprintf(buf, buf_size, "%u", avio_rl32(pb));

Isn't this doing the opposite of what the patch was meant to do?
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to