[MPlayer-dev-eng] [PATCH] -vo ggi suboptions

[Reimar Döffinger]

Hi,
the attached patch replaces any '.' by ',' in the vo_ggi suboption, as
',' can't be specified otherwise and is needed for many subdevice
specification (I want it for a little demo for the LinuxTag I'm
preparing).
Is it okay to apply? Any objections? Should I use a different character
than '.'?

Greetings,
Reimar Döffinger
-------------- next part --------------
Index: libvo/vo_ggi.c
===================================================================
RCS file: /cvsroot/mplayer/main/libvo/vo_ggi.c,v
retrieving revision 1.33
diff -u -r1.33 vo_ggi.c
--- libvo/vo_ggi.c	17 Dec 2004 19:36:52 -0000	1.33
+++ libvo/vo_ggi.c	21 Apr 2005 21:19:25 -0000
@@ -381,7 +381,15 @@
     }
 
     if ((char *)arg)
+    {
+	int i = 0;
 	ggi_conf.driver = strdup(arg);
+	while (ggi_conf.driver[i]) {
+	  if (ggi_conf.driver[i] == '.')
+	    ggi_conf.driver[i] = ',';
+	  i++;
+	}
+    }
     else
 	ggi_conf.driver = NULL;