[MPlayer-dev-eng] [PATCH] new config parser related warning fixes (for GUI, too)

Dominik Mierzejewski dominik at rangers.eu.org
Sat Dec 28 20:09:27 CET 2002


These fix the following warnings:

Patch #1:
cfg.c:184: warning: passing arg 2 of `m_config_register_options' from incompatible pointer type
cfg.c:249: warning: passing arg 1 of `m_option_print' from incompatible pointer type

Patch #2:
mplayer.c:643: warning: passing arg 2 of `m_config_register_options' from incompatible pointer type
mencoder.c:397: warning: passing arg 2 of `m_config_register_options' from incompatible pointer type
input.c:1454: warning: passing arg 2 of `m_config_register_options' from incompatible pointer type

OK to commit?

-- 
MPlayer RPMs maintainer: http://www.piorunek.pl/~dominik/linux/pkgs/mplayer/
"The Universe doesn't give you any points for doing things that are easy."
        -- Sheridan to Garibaldi in Babylon 5:"The Geometry of Shadows"
-------------- next part --------------
--- MPlayer-20021226/Gui/cfg.c.gui	Tue Dec  3 20:58:38 2002
+++ MPlayer-20021226/Gui/cfg.c	Fri Dec 27 01:07:28 2002
@@ -7,6 +7,9 @@
 #include "../mp_msg.h"
 #include "../mplayer.h"
 #include "../cfgparser.h"
+#ifndef NEW_CONFIG
+#include "../playtree.h"
+#endif
 
 #ifdef USE_SETLOCALE
 #include <locale.h>
@@ -56,13 +59,21 @@
 
 // ---
 
+#ifdef NEW_CONFIG 
+extern int
+m_config_parse_config_file(m_config_t* config, char *conffile);
+#endif 
 extern char * get_path( char * filename );
 extern int    flip;
 extern int    frame_dropping;
 extern int    stop_xscreensaver;
 
 static m_config_t * gui_conf;
+#ifdef NEW_CONFIG
+static m_option_t gui_opts[] =
+#else
 static config_t gui_opts[] =
+#endif
 {
  { "enable_audio_equ",&gtkEnableAudioEqualizer,CONF_TYPE_FLAG,0,0,1,NULL },
  
-------------- next part --------------
--- MPlayer-20021228/input/input.c.warn	Thu Dec 26 23:46:19 2002
+++ MPlayer-20021228/input/input.c	Sat Dec 28 17:39:33 2002
@@ -367,7 +367,11 @@
   { NULL, NULL, 0, 0, 0, 0, NULL}
 };
 
+#ifdef NEW_CONFIG
+static m_option_t mp_input_opts[] = {
+#else
 static config_t mp_input_opts[] = {
+#endif
   { "input", &input_conf, CONF_TYPE_SUBCONFIG, 0, 0, 0, NULL},
   { "nojoystick", &use_joystick,  CONF_TYPE_FLAG, CONF_GLOBAL, 1, 0, NULL },
   { "joystick", &use_joystick,  CONF_TYPE_FLAG, CONF_GLOBAL, 0, 1, NULL },
--- MPlayer-20021228/cfg-mplayer.h.warn	Thu Dec 26 23:46:16 2002
+++ MPlayer-20021228/cfg-mplayer.h	Sat Dec 28 17:39:33 2002
@@ -174,7 +174,11 @@
  * by Folke
  */
 
+#ifdef NEW_CONFIG
+static m_option_t mplayer_opts[]={
+#else
 static config_t mplayer_opts[]={
+#endif
 	/* name, pointer, type, flags, min, max */
 	{"include", cfg_include, CONF_TYPE_FUNC_PARAM, CONF_NOSAVE, 0, 0, NULL}, /* this don't need anymore to be the first!!! */
 
--- MPlayer-20021228/cfg-mencoder.h.warn	Fri Dec 27 23:46:21 2002
+++ MPlayer-20021228/cfg-mencoder.h	Sat Dec 28 17:39:33 2002
@@ -139,7 +139,11 @@
 	{NULL, NULL, 0, 0, 0, 0, NULL}
 };
 
+#ifdef NEW_CONFIG
+static m_option_t mencoder_opts[]={
+#else
 static config_t mencoder_opts[]={
+#endif
 	/* name, pointer, type, flags, min, max */
 	{"include", cfg_include, CONF_TYPE_FUNC_PARAM, CONF_NOSAVE, 0, 0, NULL}, /* this must be the first!!! */
 


More information about the MPlayer-dev-eng mailing list