[MPlayer-dev-eng] MNG support for MPlayer

Stefan Schuermans stefan at blinkenarea.org
Wed Sep 17 09:50:46 CEST 2008


Uoti Urpala wrote:
> On Tue, 2008-09-16 at 23:55 +0200, Stefan Schuermans wrote:
>> I saw this somewhere in the source (e.g. in vidix/ivtv_vid.c), so I 
>> assumed this was okay to get rid of the warnings about unused 
>> parameters. I can simply remove the attribute stuff, or construct 
> 
> Do you get warnings about those parameters without the attributes? If
> so, with what compiler/flags? GCC 4.3 does not warn about unused
> function parameters unless you use -Wunused-parameter or -Wextra, which
> are not normally set by MPlayer.

I get those warnings due to the "-W" flag used after "configure 
--enable-debug". The compiler is "cc (GCC) 4.1.2 20061115 (prerelease) 
(Debian 4.1.1-21)" as compiler.

cc -Wdisabled-optimization -Wno-pointer-sign 
-Wdeclaration-after-statement -std=gnu99 -W -Wall -O2 -march=k8 
-mtune=k8 -pipe -g -D_REENTRANT -D_LARGEFILE_SOURCE 
-D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE -I. -I/usr/openwin/include 
  -I/usr/include/kde/artsc -pthread -I/usr/include/glib-2.0 
-I/usr/lib/glib-2.0/include   -I/usr/include/freetype2   -c -o 
libmpdemux/demux_mng.o libmpdemux/demux_mng.c
libmpdemux/demux_mng.c: In function 'demux_mng_free':
libmpdemux/demux_mng.c:77: warning: unused parameter 'size'
libmpdemux/demux_mng.c: In function 'demux_mng_closestream':
libmpdemux/demux_mng.c:95: warning: unused parameter 'h_mng'
libmpdemux/demux_mng.c: In function 'demux_mng_refresh':
libmpdemux/demux_mng.c:148: warning: unused parameter 'h_mng'
libmpdemux/demux_mng.c:149: warning: unused parameter 'x'
libmpdemux/demux_mng.c:150: warning: unused parameter 'y'
libmpdemux/demux_mng.c:151: warning: unused parameter 'width'
libmpdemux/demux_mng.c:152: warning: unused parameter 'height'
libmpdemux/demux_mng.c: In function 'demux_mng_fill_buffer':
libmpdemux/demux_mng.c:197: warning: unused parameter 'ds'
libmpdemux/demux_mng.c: In function 'demux_mng_seek':
libmpdemux/demux_mng.c:368: warning: unused parameter 'audio_delay'




More information about the MPlayer-dev-eng mailing list