[Asterisk-cvs] asterisk/apps app_queue.c, 1.107, 1.108 app_test.c, 1.4, 1.5

markster at lists.digium.com markster at lists.digium.com
Sun Jan 2 19:41:03 CST 2005


Update of /usr/cvsroot/asterisk/apps
In directory mongoose.digium.com:/tmp/cvs-serv680/apps

Modified Files:
	app_queue.c app_test.c 
Log Message:
Fix various formatting errors (bug #3226)


Index: app_queue.c
===================================================================
RCS file: /usr/cvsroot/asterisk/apps/app_queue.c,v
retrieving revision 1.107
retrieving revision 1.108
diff -u -d -r1.107 -r1.108
--- app_queue.c	3 Jan 2005 01:42:37 -0000	1.107
+++ app_queue.c	3 Jan 2005 01:46:19 -0000	1.108
@@ -1921,8 +1921,8 @@
 	/* Our queue entry */
 	struct queue_ent qe;
 	
-	if (!data) {
-		ast_log(LOG_WARNING, "Queue requires an argument: queuename[|options[|URL][|announceoverride][|timeout]]\n"
+	if (!data || !ast_strlen_zero(data)) {
+		ast_log(LOG_WARNING, "Queue requires an argument: queuename[|options[|URL][|announceoverride][|timeout]]\n");
 		return -1;
 	}
 	

Index: app_test.c
===================================================================
RCS file: /usr/cvsroot/asterisk/apps/app_test.c,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -d -r1.4 -r1.5
--- app_test.c	22 Sep 2004 12:15:00 -0000	1.4
+++ app_test.c	3 Jan 2005 01:46:19 -0000	1.5
@@ -1,7 +1,7 @@
 /*
  * Asterisk -- A telephony toolkit for Linux.
  *
- * Applictions connected with CDR engine
+ * Applications to test connection and produce report in text file
  * 
  * Copyright (C) 2004, Digium, Inc.
  *
@@ -14,6 +14,7 @@
 
 #include <sys/types.h>
 #include <asterisk/channel.h>
+#include <asterisk/options.h>
 #include <asterisk/module.h>
 #include <asterisk/logger.h>
 #include <asterisk/lock.h>
@@ -29,13 +30,15 @@
 
 static char *tdesc = "Interface Test Application";
 
-static char *tests_descrip = "TestServer(): Perform test server function and write call report"
-							 "Results stored in /var/log/asterisk/testreports/<testid>.txt";
+static char *tests_descrip = 
+	 "TestServer(): Perform test server function and write call report.\n"
+	 "Results stored in /var/log/asterisk/testreports/<testid>-server.txt";
 static char *tests_app = "TestServer";
 static char *tests_synopsis = "Execute Interface Test Server";
 
-static char *testc_descrip = "TestClient(testid): Executes test client with given testid.\n"
-							 "Results stored in /var/log/asterisk/testreports/<testid>.txt";
+static char *testc_descrip = 
+	 "TestClient(testid): Executes test client with given testid.\n"
+	 "Results stored in /var/log/asterisk/testreports/<testid>-client.txt";
 
 static char *testc_app = "TestClient";
 static char *testc_synopsis = "Execute Interface Test Client";
@@ -135,15 +138,18 @@
 	/* Transmit client version */
 	if (!res)
 		res = ast_dtmf_stream(chan, NULL, "8378*1#", 0);
-	ast_log(LOG_DEBUG, "Transmit client version\n");
+	if (option_debug)
+		ast_log(LOG_DEBUG, "Transmit client version\n");
 	
 	/* Read server version */
-	ast_log(LOG_DEBUG, "Read server version\n");
+	if (option_debug)
+		ast_log(LOG_DEBUG, "Read server version\n");
 	if (!res) 
 		res = ast_app_getdata(chan, NULL, serverver, sizeof(serverver) - 1, 0);
 	if (res > 0)
 		res = 0;
-	ast_log(LOG_DEBUG, "server version: %s\n", serverver);
+	if (option_debug)
+		ast_log(LOG_DEBUG, "server version: %s\n", serverver);
 		
 	if (res > 0)
 		res = 0;
@@ -155,7 +161,8 @@
 		res = ast_dtmf_stream(chan, NULL, testid, 0);		
 	if (!res) 
 		res = ast_dtmf_stream(chan, NULL, "#", 0);		
-	ast_log(LOG_DEBUG, "send test identifier: %s\n", testid);
+	if (option_debug)
+		ast_log(LOG_DEBUG, "send test identifier: %s\n", testid);
 
 	if ((res >=0) && (!ast_strlen_zero(testid))) {
 		/* Make the directory to hold the test results in case it's not there */
@@ -171,7 +178,8 @@
 			
 			if (!res) {
 				/* Step 1: Wait for "1" */
-				ast_log(LOG_DEBUG, "TestClient: 2.  Wait DTMF 1\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 2.  Wait DTMF 1\n");
 				res = ast_waitfordigit(chan, 3000);
 				fprintf(f, "WAIT DTMF 1:   %s\n", (res != '1') ? "FAIL" : "PASS");
 				if (res == '1')
@@ -183,7 +191,8 @@
 				res = ast_safe_sleep(chan, 1000);
 			if (!res) {
 				/* Step 2: Send "2" */
-				ast_log(LOG_DEBUG, "TestClient: 2.  Send DTMF 2\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 2.  Send DTMF 2\n");
 				res = ast_dtmf_stream(chan, NULL, "2", 0);
 				fprintf(f, "SEND DTMF 2:   %s\n", (res < 0) ? "FAIL" : "PASS");
 				if (res > 0)
@@ -191,7 +200,8 @@
 			}
 			if (!res) {
 				/* Step 3: Wait one second */
-				ast_log(LOG_DEBUG, "TestClient: 3.  Wait one second\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 3.  Wait one second\n");
 				res = ast_safe_sleep(chan, 1000);
 				fprintf(f, "WAIT 1 SEC:    %s\n", (res < 0) ? "FAIL" : "PASS");
 				if (res > 0)
@@ -199,7 +209,8 @@
 			}			
 			if (!res) {
 				/* Step 4: Measure noise */
-				ast_log(LOG_DEBUG, "TestClient: 4.  Measure noise\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 4.  Measure noise\n");
 				res = measurenoise(chan, 5000, "TestClient");
 				fprintf(f, "MEASURENOISE:  %s (%d)\n", (res < 0) ? "FAIL" : "PASS", res);
 				if (res > 0)
@@ -207,7 +218,8 @@
 			}
 			if (!res) {
 				/* Step 5: Wait for "4" */
-				ast_log(LOG_DEBUG, "TestClient: 5.  Wait DTMF 4\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 5.  Wait DTMF 4\n");
 				res = ast_waitfordigit(chan, 3000);
 				fprintf(f, "WAIT DTMF 4:   %s\n", (res != '4') ? "FAIL" : "PASS");
 				if (res == '4')
@@ -217,13 +229,15 @@
 			}
 			if (!res) {
 				/* Step 6: Transmit tone noise */
-				ast_log(LOG_DEBUG, "TestClient: 6.  Transmit tone\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 6.  Transmit tone\n");
 				res = sendnoise(chan, 6000);
 				fprintf(f, "SENDTONE:      %s\n", (res < 0) ? "FAIL" : "PASS");
 			}
 			if (!res || (res == '5')) {
 				/* Step 7: Wait for "5" */
-				ast_log(LOG_DEBUG, "TestClient: 7.  Wait DTMF 5\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 7.  Wait DTMF 5\n");
 				if (!res)
 					res = ast_waitfordigit(chan, 3000);
 				fprintf(f, "WAIT DTMF 5:   %s\n", (res != '5') ? "FAIL" : "PASS");
@@ -234,7 +248,8 @@
 			}
 			if (!res) {
 				/* Step 8: Wait one second */
-				ast_log(LOG_DEBUG, "TestClient: 8.  Wait one second\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 8.  Wait one second\n");
 				res = ast_safe_sleep(chan, 1000);
 				fprintf(f, "WAIT 1 SEC:    %s\n", (res < 0) ? "FAIL" : "PASS");
 				if (res > 0)
@@ -242,7 +257,8 @@
 			}
 			if (!res) {
 				/* Step 9: Measure noise */
-				ast_log(LOG_DEBUG, "TestClient: 6.  Measure tone\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 6.  Measure tone\n");
 				res = measurenoise(chan, 4000, "TestClient");
 				fprintf(f, "MEASURETONE:   %s (%d)\n", (res < 0) ? "FAIL" : "PASS", res);
 				if (res > 0)
@@ -250,7 +266,8 @@
 			}
 			if (!res) {
 				/* Step 10: Send "7" */
-				ast_log(LOG_DEBUG, "TestClient: 7.  Send DTMF 7\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 7.  Send DTMF 7\n");
 				res = ast_dtmf_stream(chan, NULL, "7", 0);
 				fprintf(f, "SEND DTMF 7:   %s\n", (res < 0) ? "FAIL" : "PASS");
 				if (res > 0)
@@ -258,7 +275,8 @@
 			}
 			if (!res) {
 				/* Step 11: Wait for "8" */
-				ast_log(LOG_DEBUG, "TestClient: 11.  Wait DTMF 8\n");
+				if (option_debug)
+					ast_log(LOG_DEBUG, "TestClient: 11.  Wait DTMF 8\n");
 				res = ast_waitfordigit(chan, 3000);
 				fprintf(f, "WAIT DTMF 8:   %s\n", (res != '8') ? "FAIL" : "PASS");
 				if (res == '8')
@@ -266,12 +284,13 @@
 				else
 					res = -1;
 			}
-			if (!res) {
+			if (option_debug && !res ) {
 				/* Step 12: Hangup! */
 				ast_log(LOG_DEBUG, "TestClient: 12.  Hangup\n");
 			}
 
-			ast_log(LOG_DEBUG, "-- TEST COMPLETE--\n");
+			if (option_debug)
+				ast_log(LOG_DEBUG, "-- TEST COMPLETE--\n");
 			fprintf(f, "-- END TEST--\n");
 			fclose(f);
 			res = -1;
@@ -296,13 +315,16 @@
 	if (chan->_state != AST_STATE_UP)
 		res = ast_answer(chan);
 	/* Read version */
-	ast_log(LOG_DEBUG, "Read client version\n");
+	if (option_debug)
+		ast_log(LOG_DEBUG, "Read client version\n");
 	if (!res) 
 		res = ast_app_getdata(chan, NULL, testid, sizeof(testid) - 1, 0);
 	if (res > 0)
 		res = 0;
-	ast_log(LOG_DEBUG, "client version: %s\n", testid);
-	ast_log(LOG_DEBUG, "Transmit server version\n");
+	if (option_debug) {
+		ast_log(LOG_DEBUG, "client version: %s\n", testid);
+		ast_log(LOG_DEBUG, "Transmit server version\n");
+	}
 	res = ast_safe_sleep(chan, 1000);
 	if (!res)
 		res = ast_dtmf_stream(chan, NULL, "8378*1#", 0);
@@ -311,7 +333,8 @@
 
 	if (!res) 
 		res = ast_app_getdata(chan, NULL, testid, sizeof(testid) - 1, 0);		
-	ast_log(LOG_DEBUG, "read test identifier: %s\n", testid);
+	if (option_debug) 
+		ast_log(LOG_DEBUG, "read test identifier: %s\n", testid);
 	/* Check for sneakyness */
 	if (strchr(testid, '/'))
 		res = -1;
@@ -330,7 +353,8 @@
 			res = ast_safe_sleep(chan, 1000);
 			if (!res) {
 				/* Step 1: Send "1" */
-				ast_log(LOG_DEBUG, "TestServer: 1.  Send DTMF 1\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 1.  Send DTMF 1\n");
 				res = ast_dtmf_stream(chan, NULL, "1", 0);
 				fprintf(f, "SEND DTMF 1:   %s\n", (res < 0) ? "FAIL" : "PASS");
 				if (res > 0)
@@ -338,7 +362,8 @@
 			}
 			if (!res) {
 				/* Step 2: Wait for "2" */
-				ast_log(LOG_DEBUG, "TestServer: 2.  Wait DTMF 2\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 2.  Wait DTMF 2\n");
 				res = ast_waitfordigit(chan, 3000);
 				fprintf(f, "WAIT DTMF 2:   %s\n", (res != '2') ? "FAIL" : "PASS");
 				if (res == '2')
@@ -348,7 +373,8 @@
 			}
 			if (!res) {
 				/* Step 3: Measure noise */
-				ast_log(LOG_DEBUG, "TestServer: 3.  Measure noise\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 3.  Measure noise\n");
 				res = measurenoise(chan, 6000, "TestServer");
 				fprintf(f, "MEASURENOISE:  %s (%d)\n", (res < 0) ? "FAIL" : "PASS", res);
 				if (res > 0)
@@ -356,7 +382,8 @@
 			}
 			if (!res) {
 				/* Step 4: Send "4" */
-				ast_log(LOG_DEBUG, "TestServer: 4.  Send DTMF 4\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 4.  Send DTMF 4\n");
 				res = ast_dtmf_stream(chan, NULL, "4", 0);
 				fprintf(f, "SEND DTMF 4:   %s\n", (res < 0) ? "FAIL" : "PASS");
 				if (res > 0)
@@ -365,7 +392,8 @@
 		
 			if (!res) {
 				/* Step 5: Wait one second */
-				ast_log(LOG_DEBUG, "TestServer: 5.  Wait one second\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 5.  Wait one second\n");
 				res = ast_safe_sleep(chan, 1000);
 				fprintf(f, "WAIT 1 SEC:    %s\n", (res < 0) ? "FAIL" : "PASS");
 				if (res > 0)
@@ -374,7 +402,8 @@
 		
 			if (!res) {
 				/* Step 6: Measure noise */
-				ast_log(LOG_DEBUG, "TestServer: 6.  Measure tone\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 6.  Measure tone\n");
 				res = measurenoise(chan, 4000, "TestServer");
 				fprintf(f, "MEASURETONE:   %s (%d)\n", (res < 0) ? "FAIL" : "PASS", res);
 				if (res > 0)
@@ -383,7 +412,8 @@
 
 			if (!res) {
 				/* Step 7: Send "5" */
-				ast_log(LOG_DEBUG, "TestServer: 7.  Send DTMF 5\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 7.  Send DTMF 5\n");
 				res = ast_dtmf_stream(chan, NULL, "5", 0);
 				fprintf(f, "SEND DTMF 5:   %s\n", (res < 0) ? "FAIL" : "PASS");
 				if (res > 0)
@@ -392,14 +422,16 @@
 
 			if (!res) {
 				/* Step 8: Transmit tone noise */
-				ast_log(LOG_DEBUG, "TestServer: 8.  Transmit tone\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 8.  Transmit tone\n");
 				res = sendnoise(chan, 6000);
 				fprintf(f, "SENDTONE:      %s\n", (res < 0) ? "FAIL" : "PASS");
 			}
 		
 			if (!res || (res == '7')) {
 				/* Step 9: Wait for "7" */
-				ast_log(LOG_DEBUG, "TestServer: 9.  Wait DTMF 7\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 9.  Wait DTMF 7\n");
 				if (!res)
 					res = ast_waitfordigit(chan, 3000);
 				fprintf(f, "WAIT DTMF 7:   %s\n", (res != '7') ? "FAIL" : "PASS");
@@ -412,7 +444,8 @@
 				res = ast_safe_sleep(chan, 1000);
 			if (!res) {
 				/* Step 10: Send "8" */
-				ast_log(LOG_DEBUG, "TestServer: 10.  Send DTMF 8\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 10.  Send DTMF 8\n");
 				res = ast_dtmf_stream(chan, NULL, "8", 0);
 				fprintf(f, "SEND DTMF 8:   %s\n", (res < 0) ? "FAIL" : "PASS");
 				if (res > 0)
@@ -420,12 +453,13 @@
 			}
 			if (!res) {
 				/* Step 11: Wait for hangup to arrive! */
-				ast_log(LOG_DEBUG, "TestServer: 11.  Waiting for hangup\n");
+				if (option_debug) 
+					ast_log(LOG_DEBUG, "TestServer: 11.  Waiting for hangup\n");
 				res = ast_safe_sleep(chan, 10000);
 				fprintf(f, "WAIT HANGUP:   %s\n", (res < 0) ? "PASS" : "FAIL");
 			}
 
-			ast_log(LOG_DEBUG, "-- TEST COMPLETE--\n");
+			ast_log(LOG_NOTICE, "-- TEST COMPLETE--\n");
 			fprintf(f, "-- END TEST--\n");
 			fclose(f);
 			res = -1;




More information about the svn-commits mailing list