[Asterisk-cvs] asterisk manager.c,1.62,1.63

[citats at lists.digium.com]

Update of /usr/cvsroot/asterisk
In directory mongoose.digium.com:/home/citats/cvs/asterisk

Modified Files:
	manager.c 
Log Message:
Cleanup recent manager changes.  We have a doc/CODING-GUIDELINES file for 
a reason.  Make sure all future patches follow the guidelines.

Some of the problems with this include:
	Unnecessary whitespace changes
	Use of spaces instead of tabs
	Code formatting that doesn't follow the published guidelines
	Nested statements without braces
	Comment problems


Index: manager.c
===================================================================
RCS file: /usr/cvsroot/asterisk/manager.c,v
retrieving revision 1.62
retrieving revision 1.63
diff -u -d -r1.62 -r1.63
--- manager.c	8 Jul 2004 15:39:35 -0000	1.62
+++ manager.c	8 Jul 2004 17:08:38 -0000	1.63
@@ -274,21 +274,22 @@
    ast_instring("this|that|more","this",',') == 1;
 
    feel free to move this to app.c -anthm */
-static int ast_instring(char *bigstr, char *smallstr, char delim) {
-    char *val = bigstr, *next;
+static int ast_instring(char *bigstr, char *smallstr, char delim) 
+{
+	char *val = bigstr, *next;
 
-    do {
-        if ( ( next = strchr ( val, delim ) ) )
-            if ( ! strncmp ( val , smallstr , ( next - val ) ) )
-                return 1;
-            else
-                continue;
-        else
-            return ! strcmp ( smallstr , val );
+	do {
+		if ((next = strchr(val, delim))) {
+			if (!strncmp(val, smallstr, (next - val)))
+				return 1;
+			else
+				continue;
+		} else
+			return !strcmp(smallstr, val);
 
-    } while ( * ( val = ( next + 1 ) ) ) ;
+	} while (*(val = (next + 1)));
 
-    return 0;
+	return 0;
 }
 
 static int get_perm(char *instr)
@@ -305,14 +306,15 @@
 	return ret;
 }
 
-static int ast_is_number(char *string) {
+static int ast_is_number(char *string) 
+{
 	int ret = 1, x = 0;
 
-	if(!string)
+	if (!string)
 		return 0;
 
-	for (x=0; x < strlen(string) ; x++ ) {
-		if (! (string[x] >= 48 && string[x] <= 57)) {
+	for (x=0; x < strlen(string); x++) {
+		if (!(string[x] >= 48 && string[x] <= 57)) {
 			ret = 0;
 			break;
 		}
@@ -321,15 +323,15 @@
 	return ret ? atoi(string) : 0;
 }
 
-
-static int ast_strings_to_mask(char *string) {
+static int ast_strings_to_mask(char *string) 
+{
 	int x = 0, ret = -1;
 	
 	x = ast_is_number(string);
 
-	if(x) 
+	if (x) 
 		ret = x;
-	else if (! string || ast_strlen_zero(string))
+	else if (!string || ast_strlen_zero(string))
 		ret = -1;
 	else if (!strcasecmp(string, "off") || ast_false(string))
 		ret = 0;
@@ -337,13 +339,13 @@
 		ret = -1;
 	else {
 		ret = 0;
-		for (x=0; x<sizeof(perms) / sizeof(perms[0]); x++) 
+		for (x=0; x<sizeof(perms) / sizeof(perms[0]); x++) {
 			if (ast_instring(string, perms[x].label, ',')) 
 				ret |= perms[x].num;		
+		}
 	}
 
 	return ret;
-
 }
 
 /* 
@@ -488,8 +490,8 @@
 "  client.\n"
 "Variables:\n"
 "	EventMask: 'on' if all events should be sent,\n"
-"   EventMask: 'system,call,log' if events should be sent matching these specific channels,\n" 
-"   EventMask: 'off' if no events should be sent.\n";
+"		'off' if no events should be sent,\n"
+"		'system,call,log' to select which flags events should have to be sent.\n";
 
 static int action_events(struct mansession *s, struct message *m)
 {