[MPlayer-dev-eng] [PATCH] more make warnings

[Diego Biurrun]

On Thu, May 31, 2007 at 07:31:55PM -0700, timwoj at ieee.org wrote:
> Here's fixes for most of the make warnings from the -Wall build I did the 
> other day.  It's mostly fixes for printf-style formats that were 
> passed to mp_msg, but there's a few other things in there as well.
> 
> --- libmpcodecs/vd_qtvideo.c	(revision 23440)
> +++ libmpcodecs/vd_qtvideo.c	(working copy)
> @@ -204,7 +204,7 @@
>  
> -    mp_msg(MSGT_DECVIDEO,MSGL_DBG2,"ImageCodecInitialize->%#x  size=%d (%d)\n",cres,icap.recordSize,icap.decompressRecordSize);
> +    mp_msg(MSGT_DECVIDEO,MSGL_DBG2,"ImageCodecInitialize->%#x  size=%ld (%ld)\n",cres,icap.recordSize,icap.decompressRecordSize);

This seems to add rather than remove a warning.

> --- libfaad2/specrec.c	(revision 23440)
> +++ libfaad2/specrec.c	(working copy)

That's an external library we would like to deprecate rather than patch.

> --- playtree.c	(revision 23440)
> +++ playtree.c	(working copy)

This file compiles without warnings for me.

Patch rejected.

Diego