Deti Fliegl
2004-Aug-22  07:27 UTC
[Asterisk-Users] app_mp3 with bri-stuff.0.1.0RC4a does not work
Hi there,
app_mp3 still does not work with the latest bri-stuff patch and the 
zaphfc driver. Here in my place it only works with the patch attached. 
For me it seems the bri-stuff worsens the asterisk timing... has anybody 
else made experiences with it?
Deti
-------------- next part --------------
Index: app_mp3.c
==================================================================RCS file:
/usr/cvsroot/asterisk/apps/app_mp3.c,v
retrieving revision 1.19
diff -u -r1.19 app_mp3.c
--- app_mp3.c	22 Jun 2004 19:32:52 -0000	1.19
+++ app_mp3.c	22 Aug 2004 14:20:49 -0000
@@ -60,6 +60,7 @@
 			close(x);
 	}
 	/* Execute mpg123, but buffer if it's a net connection */
+#if 0
 	if (!strncmp(filename, "http://", 7)) {
 		/* Most commonly installed in /usr/local/bin */
 	    execl(LOCAL_MPG_123, "mpg123", "-q", "-s",
"-b", "1024", "-f", "8192",
"--mono", "-r", "8000", filename, (char *)NULL);
@@ -68,7 +69,9 @@
 		/* As a last-ditch effort, try to use PATH */
 	    execlp("mpg123", "mpg123", "-q",
"-s", "-b", "1024",  "-f",
"8192", "--mono", "-r", "8000",
filename, (char *)NULL);
 	}
-	else {
+	else
+#endif
+	{
 		/* Most commonly installed in /usr/local/bin */
 	    execl(MPG_123, "mpg123", "-q", "-s",
"-f", "8192", "--mono", "-r",
"8000", filename, (char *)NULL);
 		/* But many places has it in /usr/bin */
@@ -176,6 +179,7 @@
 					res = 0;
 					break;
 				}
+				gettimeofday(&next, NULL);
 				next.tv_usec += res / 2 * 125;
 				if (next.tv_usec >= 1000000) {
 					next.tv_usec -= 1000000;
