[MPlayer-dev-eng] [PATCH] gmplayer fsFileSelect related fixes

Vadim Pisarev plv at farpost.com
Tue Apr 10 04:29:25 CEST 2007


Sergey Pinaev wrote:
> On Mon, 9 Apr 2007 06:51:39 -0400
> Rich Felker <dalias at aerifal.cx> wrote:
> 
>>>>> (btw, afaiu get_current_dir_name() returns malloc()ed string, that should
>>>>> be free()d too)
>>>> If so then that should be fixed..
>>>        get_current_dir_name,  which  is  only  prototyped  if  _GNU_SOURCE  is
>>>        defined, will malloc(3) an array big enough to hold the current  direc-
>>>        tory  name.
>> Uhg, so this is some nasty glibcism? Replace it with something
>> portable then, like getcwd().
> 
> gui uses gtk/glib, so for consistency it should use g_get_current_dir() =)
> 

So, should i replace get_current_dir_name() with g_get_current_dir()
everywhere in Gui/mplayer.gtk/fs.c ?

--
WBR,
 Vadim Pisarev



More information about the MPlayer-dev-eng mailing list