[MPlayer-dev-eng] [PATCH] stream layer support for .ifo files
Vladimir Voroshilov
voroshil at gmail.com
Mon Dec 17 08:38:25 CET 2007
On Dec 17, 2007 1:29 PM, Reimar Döffinger
<Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> Hello,
> On Mon, Dec 17, 2007 at 08:49:41AM +0800, Ulion wrote:
> [...]
> > http://svn.mplayerhq.hu/mplayer/trunk/stream/stream_dvd.c?view=diff&r1=25237&r2=25238
> >
> > Which breaks mingw compile because the libgen.h and basename/dirname stuff.
>
ah. it does.
> And in addition that usage of basename/dirname is broken anyway since it
> may modify stream->url, that is not acceptable.
> I also have some doubts if the way those functions behave in corner
> cases is really what is desired...
>
well. is implementation something like:
idx=-1;
for(i=0;stream->url[i]; i++)
if(stream->url[i]=='/')
idx=i;
if(idx>=0)
{
filename=strdup(stream->url+idx+1);
dvd_device=malloc(idx);
strncpy(dvd_device,stream->url, idx);
}
acceptable or hole patch should be reverted (lossing these feature)?
--
Regards,
Vladimir Voroshilov mailto:voroshil at gmail.com
JID: voroshil at gmail.com, voroshil at jabber.ru
ICQ: 95587719
More information about the MPlayer-dev-eng
mailing list