[MPlayer-cvslog] r33755 - trunk/gui/interface.c

ib subversion at mplayerhq.hu
Tue Jun 28 17:00:06 CEST 2011


Author: ib
Date: Tue Jun 28 17:00:05 2011
New Revision: 33755

Log:
Fix bug introduced as a side effect in r33750.

It was no longer possible to start gmplayer from the command line with
a file argument.

Every time gmplayer is called with a file argument this file goes into
the GUI specific playlist. As a side effect, GUI's file(name) to play
and MPlayer's filename point to the same address.

guiInit() pointlessly tries to set GUI's filename (free and strdup) from
MPlayer's filename instead of getting it from its playlist. The previous
memset set GUI's filename to NULL rendering the free() effectless while
now such an operation on the same address to strdup() afterwards isn't a
good idea.

Remove the pointless code.

Modified:
   trunk/gui/interface.c

Modified: trunk/gui/interface.c
==============================================================================
--- trunk/gui/interface.c	Tue Jun 28 12:55:17 2011	(r33754)
+++ trunk/gui/interface.c	Tue Jun 28 17:00:05 2011	(r33755)
@@ -366,9 +366,6 @@ void guiInit(void)
 
     uiSubRender = 1;
 
-    if (filename)
-        uiSetFileName(NULL, filename, STREAMTYPE_FILE);
-
     if (plCurrent && !filename)
         uiSetFileName(plCurrent->path, plCurrent->name, STREAMTYPE_FILE);
 


More information about the MPlayer-cvslog mailing list