[MPlayer-cvslog] r33369 - trunk/gui/interface.h

ib subversion at mplayerhq.hu
Thu May 5 10:51:33 CEST 2011


Author: ib
Date: Thu May  5 10:51:33 2011
New Revision: 33369

Log:
Use free() instead of gfree() in guiSetFilename and guiSetDF macros.

The gfree() is pointless because there is a new assignment to the
freed variable right afterwards.

This also settles compiling issues with the Win32 part of the GUI
reported by Stephen Sheldon, sfsheldo gmail com, who suggested the
patch.

Modified:
   trunk/gui/interface.h

Modified: trunk/gui/interface.h
==============================================================================
--- trunk/gui/interface.h	Thu May  5 09:40:56 2011	(r33368)
+++ trunk/gui/interface.h	Thu May  5 10:51:33 2011	(r33369)
@@ -93,13 +93,13 @@ extern int use_gui;             // this 
 
 #define guiSetFilename(s, n) \
     { \
-        gfree((void **)&s); \
+        free(s); \
         s = gstrdup(n); \
     }
 
 #define guiSetDF(s, d, n) \
     { \
-        gfree((void **)&s); \
+        free(s); \
         s = malloc(strlen(d) + strlen(n) + 5); \
         sprintf(s, "%s/%s", d, n); \
     }


More information about the MPlayer-cvslog mailing list