[asterisk-commits] [svn-commits] trunk r17827 - /trunk/pbx/pbx_config.c

asterisk-commits at lists.digium.com asterisk-commits at lists.digium.com
Thu Apr 6 07:40:49 MST 2006


Author: rizzo
Date: Thu Apr  6 09:40:47 2006
New Revision: 17827

URL: http://svn.digium.com/view/asterisk?rev=17827&view=rev
Log:
whitespace and formatting fix postponed from last commit:
fix indentation of a large function, localize some variables
and add/remove whitespace where needed.


Modified:
    trunk/pbx/pbx_config.c

Modified: trunk/pbx/pbx_config.c
URL: http://svn.digium.com/view/asterisk/trunk/pbx/pbx_config.c?rev=17827&r1=17826&r2=17827&view=diff
==============================================================================
--- trunk/pbx/pbx_config.c (original)
+++ trunk/pbx/pbx_config.c Thu Apr  6 09:40:47 2006
@@ -1349,166 +1349,155 @@
 	char realvalue[256];
 	int lastpri = -2;
 	struct ast_context *con;
+	struct ast_variable *v;
+	char *cxt;
 
 	cfg = ast_config_load(config_file);
 	if (!cfg)
 		return;
-	{
-		struct ast_variable *v;
-		char *cxt;
-
-		/* Use existing config to populate the PBX table */
-		static_config = ast_true(ast_variable_retrieve(cfg, "general", "static"));
-		write_protect_config = ast_true(ast_variable_retrieve(cfg, "general", "writeprotect"));
-		autofallthrough_config = ast_true(ast_variable_retrieve(cfg, "general", "autofallthrough"));
-		clearglobalvars_config = ast_true(ast_variable_retrieve(cfg, "general", "clearglobalvars"));
-		ast_set2_flag(&ast_options, !ast_false(ast_variable_retrieve(cfg, "general", "priorityjumping")), AST_OPT_FLAG_PRIORITY_JUMPING);
-									    
-		for (v = ast_variable_browse(cfg, "globals"); v; v = v->next) {
-			memset(realvalue, 0, sizeof(realvalue));
-			pbx_substitute_variables_helper(NULL, v->value, realvalue, sizeof(realvalue) - 1);
-			pbx_builtin_setvar_helper(NULL, v->name, realvalue);
-		}
-		for (cxt = NULL; (cxt = ast_category_browse(cfg, cxt)); ) {
-
-			/* All categories but "general" or "globals" are considered contexts */
-			if (!strcasecmp(cxt, "general") || !strcasecmp(cxt, "globals"))
-				continue;
-			con=ast_context_create(&local_contexts,cxt, registrar);
-			if (con == NULL)
-				continue;
-
-			for (v = ast_variable_browse(cfg, cxt); v; v = v->next) {
-				if (!strcasecmp(v->name, "exten")) {
-					char *ext, *pri, *appl, *data, *cidmatch;
-					char *stringp=NULL;
+
+	/* Use existing config to populate the PBX table */
+	static_config = ast_true(ast_variable_retrieve(cfg, "general", "static"));
+	write_protect_config = ast_true(ast_variable_retrieve(cfg, "general", "writeprotect"));
+	autofallthrough_config = ast_true(ast_variable_retrieve(cfg, "general", "autofallthrough"));
+	clearglobalvars_config = ast_true(ast_variable_retrieve(cfg, "general", "clearglobalvars"));
+	ast_set2_flag(&ast_options, !ast_false(ast_variable_retrieve(cfg, "general", "priorityjumping")), AST_OPT_FLAG_PRIORITY_JUMPING);
+								    
+	for (v = ast_variable_browse(cfg, "globals"); v; v = v->next) {
+		memset(realvalue, 0, sizeof(realvalue));
+		pbx_substitute_variables_helper(NULL, v->value, realvalue, sizeof(realvalue) - 1);
+		pbx_builtin_setvar_helper(NULL, v->name, realvalue);
+	}
+	for (cxt = NULL; (cxt = ast_category_browse(cfg, cxt)); ) {
+
+		/* All categories but "general" or "globals" are considered contexts */
+		if (!strcasecmp(cxt, "general") || !strcasecmp(cxt, "globals"))
+			continue;
+		con=ast_context_create(&local_contexts,cxt, registrar);
+		if (con == NULL)
+			continue;
+
+		for (v = ast_variable_browse(cfg, cxt); v; v = v->next) {
+			if (!strcasecmp(v->name, "exten")) {
+				char *tc = strdup(v->value);
+				if (tc == NULL)
+					fprintf(stderr,"Error strdup returned NULL in %s\n",__PRETTY_FUNCTION__);
+				else {
 					int ipri = -2;
 					char realext[256]="";
 					char *plus, *firstp, *firstc;
-					char *tc = strdup(v->value);
-					if (tc == NULL)
-						fprintf(stderr,"Error strdup returned NULL in %s\n",__PRETTY_FUNCTION__);
-					else {
-						stringp=tc;
-						ext = strsep(&stringp, ",");
-						if (!ext)
-							ext="";
-						pbx_substitute_variables_helper(NULL, ext, realext, sizeof(realext) - 1);
-						cidmatch = strchr(realext, '/');
-						if (cidmatch) {
-							*cidmatch++ = '\0';
-							ast_shrink_phone_number(cidmatch);
+					char *pri, *appl, *data, *cidmatch;
+					char *stringp = tc;
+					char *ext = strsep(&stringp, ",");
+					if (!ext)
+						ext="";
+					pbx_substitute_variables_helper(NULL, ext, realext, sizeof(realext) - 1);
+					cidmatch = strchr(realext, '/');
+					if (cidmatch) {
+						*cidmatch++ = '\0';
+						ast_shrink_phone_number(cidmatch);
+					}
+					pri = strsep(&stringp, ",");
+					if (!pri)
+						pri="";
+					label = strchr(pri, '(');
+					if (label) {
+						*label++ = '\0';
+						end = strchr(label, ')');
+						if (end)
+							*end = '\0';
+						else
+							ast_log(LOG_WARNING, "Label missing trailing ')' at line %d\n", v->lineno);
+					}
+					plus = strchr(pri, '+');
+					if (plus)
+						*plus++ = '\0';
+					if (!strcmp(pri,"hint"))
+						ipri=PRIORITY_HINT;
+					else if (!strcmp(pri, "next") || !strcmp(pri, "n")) {
+						if (lastpri > -2)
+							ipri = lastpri + 1;
+						else
+							ast_log(LOG_WARNING, "Can't use 'next' priority on the first entry!\n");
+					} else if (!strcmp(pri, "same") || !strcmp(pri, "s")) {
+						if (lastpri > -2)
+							ipri = lastpri;
+						else
+							ast_log(LOG_WARNING, "Can't use 'same' priority on the first entry!\n");
+					} else if (sscanf(pri, "%d", &ipri) != 1 &&
+					    (ipri = ast_findlabel_extension2(NULL, con, realext, pri, cidmatch)) < 1) {
+						ast_log(LOG_WARNING, "Invalid priority/label '%s' at line %d\n", pri, v->lineno);
+						ipri = 0;
+					}
+					appl = S_OR(stringp, "");
+					/* Find the first occurrence of either '(' or ',' */
+					firstc = strchr(appl, ',');
+					firstp = strchr(appl, '(');
+					if (firstc && (!firstp || firstc < firstp)) {
+						/* comma found, no parenthesis */
+						/* or both found, but comma found first */
+						appl = strsep(&stringp, ",");
+						data = stringp;
+					} else if (!firstc && !firstp) {
+						/* Neither found */
+						data = "";
+					} else {
+						/* Final remaining case is parenthesis found first */
+						appl = strsep(&stringp, "(");
+						data = stringp;
+						end = strrchr(data, ')');
+						if ((end = strrchr(data, ')'))) {
+							*end = '\0';
+						} else {
+							ast_log(LOG_WARNING, "No closing parenthesis found? '%s(%s'\n", appl, data);
 						}
-						pri = strsep(&stringp, ",");
-						if (!pri)
-							pri="";
-						label = strchr(pri, '(');
-						if (label) {
-							*label++ = '\0';
-							end = strchr(label, ')');
-							if (end)
-								*end = '\0';
-							else
-								ast_log(LOG_WARNING, "Label missing trailing ')' at line %d\n", v->lineno);
+						ast_process_quotes_and_slashes(data, ',', '|');
+					}
+
+					if (!data)
+						data="";
+					appl = ast_skip_blanks(appl);
+					if (ipri) {
+						if (plus)
+							ipri += atoi(plus);
+						lastpri = ipri;
+						if (!ast_opt_dont_warn && !strcmp(realext, "_."))
+							ast_log(LOG_WARNING, "The use of '_.' for an extension is strongly discouraged and can have unexpected behavior.  Please use '_X.' instead at line %d\n", v->lineno);
+						if (ast_add_extension2(con, 0, realext, ipri, label, cidmatch, appl, strdup(data), FREE, registrar)) {
+							ast_log(LOG_WARNING, "Unable to register extension at line %d\n", v->lineno);
 						}
-						plus = strchr(pri, '+');
-						if (plus)
-							*plus++ = '\0';
-						if (!strcmp(pri,"hint"))
-							ipri=PRIORITY_HINT;
-						else if (!strcmp(pri, "next") || !strcmp(pri, "n")) {
-							if (lastpri > -2)
-								ipri = lastpri + 1;
-							else
-								ast_log(LOG_WARNING, "Can't use 'next' priority on the first entry!\n");
-						} else if (!strcmp(pri, "same") || !strcmp(pri, "s")) {
-							if (lastpri > -2)
-								ipri = lastpri;
-							else
-								ast_log(LOG_WARNING, "Can't use 'same' priority on the first entry!\n");
-						} else  {
-							if (sscanf(pri, "%d", &ipri) != 1) {
-								if ((ipri = ast_findlabel_extension2(NULL, con, realext, pri, cidmatch)) < 1) {
-									ast_log(LOG_WARNING, "Invalid priority/label '%s' at line %d\n", pri, v->lineno);
-									ipri = 0;
-								}
-							}
-						}
-						appl = stringp;
-						if (!appl)
-							appl="";
-						/* Find the first occurrence of either '(' or ',' */
-						firstc = strchr(appl, ',');
-						firstp = strchr(appl, '(');
-						if (firstc && ((!firstp) || (firstc < firstp))) {
-							/* comma found, no parenthesis */
-							/* or both found, but comma found first */
-							appl = strsep(&stringp, ",");
-							data = stringp;
-						} else if ((!firstc) && (!firstp)) {
-							/* Neither found */
-							data = "";
-						} else {
-							/* Final remaining case is parenthesis found first */
-							appl = strsep(&stringp, "(");
-							data = stringp;
-							end = strrchr(data, ')');
-							if ((end = strrchr(data, ')'))) {
-								*end = '\0';
-							} else {
-								ast_log(LOG_WARNING, "No closing parenthesis found? '%s(%s'\n", appl, data);
-							}
-							ast_process_quotes_and_slashes(data, ',', '|');
-						}
-
-						if (!data)
-							data="";
-						appl = ast_skip_blanks(appl);
-						if (ipri) {
-							if (plus)
-								ipri += atoi(plus);
-							lastpri = ipri;
-							if (!ast_opt_dont_warn) {
-								if (!strcmp(realext, "_."))
-									ast_log(LOG_WARNING, "The use of '_.' for an extension is strongly discouraged and can have unexpected behavior.  Please use '_X.' instead at line %d\n", v->lineno);
-							}
-							if (ast_add_extension2(con, 0, realext, ipri, label, cidmatch, appl, strdup(data), FREE, registrar)) {
-								ast_log(LOG_WARNING, "Unable to register extension at line %d\n", v->lineno);
-							}
-						}
-						free(tc);
 					}
-				} else if(!strcasecmp(v->name, "include")) {
-					memset(realvalue, 0, sizeof(realvalue));
+					free(tc);
+				}
+			} else if (!strcasecmp(v->name, "include")) {
+				memset(realvalue, 0, sizeof(realvalue));
+				pbx_substitute_variables_helper(NULL, v->value, realvalue, sizeof(realvalue) - 1);
+				if (ast_context_add_include2(con, realvalue, registrar))
+					ast_log(LOG_WARNING, "Unable to include context '%s' in context '%s'\n", v->value, cxt);
+			} else if (!strcasecmp(v->name, "ignorepat")) {
+				memset(realvalue, 0, sizeof(realvalue));
+				pbx_substitute_variables_helper(NULL, v->value, realvalue, sizeof(realvalue) - 1);
+				if (ast_context_add_ignorepat2(con, realvalue, registrar))
+					ast_log(LOG_WARNING, "Unable to include ignorepat '%s' in context '%s'\n", v->value, cxt);
+			} else if (!strcasecmp(v->name, "switch") || !strcasecmp(v->name, "lswitch") || !strcasecmp(v->name, "eswitch")) {
+				char *stringp= realvalue;
+				char *appl, *data;
+
+				memset(realvalue, 0, sizeof(realvalue));
+				if (!strcasecmp(v->name, "switch"))
 					pbx_substitute_variables_helper(NULL, v->value, realvalue, sizeof(realvalue) - 1);
-					if (ast_context_add_include2(con, realvalue, registrar))
-						ast_log(LOG_WARNING, "Unable to include context '%s' in context '%s'\n", v->value, cxt);
-				} else if(!strcasecmp(v->name, "ignorepat")) {
-					memset(realvalue, 0, sizeof(realvalue));
-					pbx_substitute_variables_helper(NULL, v->value, realvalue, sizeof(realvalue) - 1);
-					if (ast_context_add_ignorepat2(con, realvalue, registrar))
-						ast_log(LOG_WARNING, "Unable to include ignorepat '%s' in context '%s'\n", v->value, cxt);
-				} else if (!strcasecmp(v->name, "switch") || !strcasecmp(v->name, "lswitch") || !strcasecmp(v->name, "eswitch")) {
-					char *stringp= realvalue;
-					char *appl, *data;
-
-					memset(realvalue, 0, sizeof(realvalue));
-					if (!strcasecmp(v->name, "switch"))
-						pbx_substitute_variables_helper(NULL, v->value, realvalue, sizeof(realvalue) - 1);
-					else
-						ast_copy_string(realvalue, v->value, sizeof(realvalue));
-					appl = strsep(&stringp, "/");
-					data = strsep(&stringp, ""); /* XXX what for ? */
-					if (!data)
-						data = "";
-					if (ast_context_add_switch2(con, appl, data, !strcasecmp(v->name, "eswitch"), registrar))
-						ast_log(LOG_WARNING, "Unable to include switch '%s' in context '%s'\n", v->value, cxt);
-				}
+				else
+					ast_copy_string(realvalue, v->value, sizeof(realvalue));
+				appl = strsep(&stringp, "/");
+				data = strsep(&stringp, ""); /* XXX what for ? */
+				if (!data)
+					data = "";
+				if (ast_context_add_switch2(con, appl, data, !strcasecmp(v->name, "eswitch"), registrar))
+					ast_log(LOG_WARNING, "Unable to include switch '%s' in context '%s'\n", v->value, cxt);
 			}
 		}
-		ast_config_destroy(cfg);
-	}
-
+	}
+	ast_config_destroy(cfg);
 }
 
 static int pbx_load_module(void)
@@ -1528,7 +1517,8 @@
 
 STATIC_MODULE int load_module(void)
 {
-	if (pbx_load_module()) return -1;
+	if (pbx_load_module())
+		return -1;
  
 	ast_cli_register(&context_remove_extension_cli);
 	ast_cli_register(&context_dont_include_cli);

_______________________________________________
--Bandwidth and Colocation provided by Easynews.com --

svn-commits mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/svn-commits


More information about the asterisk-commits mailing list