[MPlayer-cvslog] r25740 - trunk/gui/win32/wincfg.c

13 Jan 2008

Author: reimar
Date: Sun Jan 13 17:25:33 2008
New Revision: 25740

Log:
Win32 gui has the same m_option_print error handling bug


Modified:
   trunk/gui/win32/wincfg.c

Modified: trunk/gui/win32/wincfg.c
==============================================================================

--- trunk/gui/win32/wincfg.c	(original)
+++ trunk/gui/win32/wincfg.c	Sun Jan 13 17:25:33 2008
@@ -130,13 +130,15 @@ int cfg_write(void)
         for (i=0; gui_opts[i].name; i++)
         {
             char *v = m_option_print(&gui_opts[i], gui_opts[i].p);
+            if(v == (char *)-1) {
+                mp_msg(MSGT_GPLAYER, MSGL_WARN, MSGTR_UnableToSaveOption, gui_opts[i].name);
+                v = NULL;
+            }
             if(v)
             {
                 fprintf(f, "%s = \"%s\"\n", gui_opts[i].name, v);
                 free(v);
             }
-            else if((int) v == -1)
-                mp_msg(MSGT_GPLAYER, MSGL_WARN, MSGTR_UnableToSaveOption, gui_opts[i].name);
         }
         fclose(f);
     }

    