[Mplayer-cvslog] CVS: main/libmpdemux open.c,1.89,1.90

[D Richard Felker III]

On Wed, Sep 03, 2003 at 09:27:06PM +0200, Alex Beregszaszi wrote:
> Update of /cvsroot/mplayer/main/libmpdemux
> In directory mail:/var/tmp.root/cvs-serv9896
> 
> Modified Files:
> 	open.c 
> Log Message:
> cdda && cddb aren't uris too (hopefully all the pseudo urls are catched before url_new())
> 
> Index: open.c
> ===================================================================
> RCS file: /cvsroot/mplayer/main/libmpdemux/open.c,v
> retrieving revision 1.89
> retrieving revision 1.90
> diff -u -r1.89 -r1.90
> --- open.c	31 Aug 2003 18:51:05 -0000	1.89
> +++ open.c	3 Sep 2003 19:26:33 -0000	1.90
> @@ -478,6 +478,7 @@
>    // FIXME: to avoid nonsense error messages...
>    if (strncmp("tv://", filename, 5) && strncmp("mf://", filename, 5) &&
>      strncmp("vcd://", filename, 6) && strncmp("dvb://", filename, 6) &&
> +    strncmp("cdda://", filename, 7) && strncmp("cddb://", filename, 7) &&
>      strstr(filename, "://"))

What about dvd?? IMO this is dumb -- url_read should just accept the
(very few) prefixes that are valid urls, and silently ignore the rest,
so we don't have to keep adding tests like this...

Rich