[asterisk-commits] file: trunk r103318 - /trunk/main/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Feb 11 12:27:49 CST 2008


Author: file
Date: Mon Feb 11 12:27:47 2008
New Revision: 103318

URL: http://svn.digium.com/view/asterisk?view=rev&rev=103318
Log:
Just some minor coding style cleanup...

Modified:
    trunk/main/app.c
    trunk/main/asterisk.c
    trunk/main/indications.c
    trunk/main/logger.c
    trunk/main/manager.c
    trunk/main/pbx.c
    trunk/main/rtp.c
    trunk/main/translate.c
    trunk/main/utils.c

Modified: trunk/main/app.c
URL: http://svn.digium.com/view/asterisk/trunk/main/app.c?view=diff&rev=103318&r1=103317&r2=103318
==============================================================================
--- trunk/main/app.c (original)
+++ trunk/main/app.c Mon Feb 11 12:27:47 2008
@@ -1458,7 +1458,7 @@
 
 static int read_newoption(struct ast_channel *chan, struct ast_ivr_menu *menu, char *exten, int maxexten)
 {
-	int res=0;
+	int res = 0;
 	int ms;
 	while (option_matchmore(menu, exten)) {
 		ms = chan->pbx ? chan->pbx->dtimeout : 5000;
@@ -1476,7 +1476,7 @@
 static int ast_ivr_menu_run_internal(struct ast_channel *chan, struct ast_ivr_menu *menu, void *cbdata)
 {
 	/* Execute an IVR menu structure */
-	int res=0;
+	int res = 0;
 	int pos = 0;
 	int retries = 0;
 	char exten[AST_MAX_EXTENSION] = "s";

Modified: trunk/main/asterisk.c
URL: http://svn.digium.com/view/asterisk/trunk/main/asterisk.c?view=diff&rev=103318&r1=103317&r2=103318
==============================================================================
--- trunk/main/asterisk.c (original)
+++ trunk/main/asterisk.c Mon Feb 11 12:27:47 2008
@@ -947,7 +947,7 @@
 static void ast_network_puts(const char *string)
 {
 	int x;
-	for (x=0; x < AST_MAX_CONNECTS; x++) {
+	for (x = 0; x < AST_MAX_CONNECTS; x++) {
 		if (consoles[x].fd > -1) 
 			fdprint(consoles[x].p[1], string);
 	}
@@ -1905,7 +1905,7 @@
 					int tries;
 					int reconnects_per_second = 20;
 					fprintf(stderr, "Attempting to reconnect for 30 seconds\n");
-					for (tries=0; tries < 30 * reconnects_per_second; tries++) {
+					for (tries = 0; tries < 30 * reconnects_per_second; tries++) {
 						if (ast_tryconnect()) {
 							fprintf(stderr, "Reconnect succeeded after %.3f seconds\n", 1.0 / reconnects_per_second * tries);
 							printf(term_quit());
@@ -2152,7 +2152,7 @@
 
 	for (; count > 0; count--) {
 		numoutputline = 0;
-		for (i=0; i < limit && matches[idx]; i++, idx++) {
+		for (i = 0; i < limit && matches[idx]; i++, idx++) {
 
 			/* Don't print dupes */
 			if ( (matches[idx+1] != NULL && strcmp(matches[idx], matches[idx+1]) == 0 ) ) {
@@ -2264,7 +2264,7 @@
 			retval = CC_REFRESH;
 		} else {
 			/* Must be more than one match */
-			for (i=1, maxlen=0; matches[i]; i++) {
+			for (i = 1, maxlen = 0; matches[i]; i++) {
 				match_len = strlen(matches[i]);
 				if (match_len > maxlen)
 					maxlen = match_len;
@@ -2792,7 +2792,7 @@
 		fprintf(stderr, "Truncating argument size to %d\n", (int)(sizeof(_argv) / sizeof(_argv[0])) - 1);
 		argc = sizeof(_argv) / sizeof(_argv[0]) - 1;
 	}
-	for (x=0; x<argc; x++)
+	for (x = 0; x < argc; x++)
 		_argv[x] = argv[x];
 	_argv[x] = NULL;
 

Modified: trunk/main/indications.c
URL: http://svn.digium.com/view/asterisk/trunk/main/indications.c?view=diff&rev=103318&r1=103317&r2=103318
==============================================================================
--- trunk/main/indications.c (original)
+++ trunk/main/indications.c Mon Feb 11 12:27:47 2008
@@ -165,7 +165,7 @@
 		ps->v3_2 = pi->init_v3_2;
 		ps->oldnpos = ps->npos;
 	}
-	for (x=0;x<len/2;x++) {
+	for (x = 0; x < len/2; x++) {
 		ps->v1_1 = ps->v2_1;
 		ps->v2_1 = ps->v3_1;
 		ps->v3_1 = (pi->fac1 * ps->v2_1 >> 15) - ps->v1_1;
@@ -233,7 +233,7 @@
 		separator = ",";
 	s = strsep(&stringp,separator);
 	while (s && *s) {
-		int freq1, freq2, time, modulate=0, midinote=0;
+		int freq1, freq2, time, modulate = 0, midinote = 0;
 
 		if (s[0]=='!')
 			s++;
@@ -533,7 +533,7 @@
 
 	AST_RWLIST_WRLOCK(&tone_zones);
 	for (ps=NULL,ts=zone->tones; ts; ps=ts,ts=ts->next) {
-		if (strcasecmp(indication,ts->name)==0) {
+		if (!strcasecmp(indication,ts->name)) {
 			/* indication already there, replace */
 			ast_free((void*)ts->name);
 			ast_free((void*)ts->data);
@@ -573,7 +573,7 @@
 	AST_RWLIST_WRLOCK(&tone_zones);
 	ts = zone->tones;
 	while (ts) {
-		if (strcasecmp(indication,ts->name)==0) {
+		if (!strcasecmp(indication,ts->name)) {
 			/* indication found */
 			tmp = ts->next;
 			if (ps)

Modified: trunk/main/logger.c
URL: http://svn.digium.com/view/asterisk/trunk/main/logger.c?view=diff&rev=103318&r1=103317&r2=103318
==============================================================================
--- trunk/main/logger.c (original)
+++ trunk/main/logger.c Mon Feb 11 12:27:47 2008
@@ -1081,7 +1081,7 @@
 void ast_backtrace(void)
 {
 #ifdef HAVE_BKTR
-	int count=0, i=0;
+	int count = 0, i = 0;
 	void **addresses;
 	char **strings;
 
@@ -1089,7 +1089,7 @@
 		count = backtrace(addresses, MAX_BACKTRACE_FRAMES);
 		if ((strings = backtrace_symbols(addresses, count))) {
 			ast_debug(1, "Got %d backtrace record%c\n", count, count != 1 ? 's' : ' ');
-			for (i=0; i < count ; i++) {
+			for (i = 0; i < count; i++) {
 #if __WORDSIZE == 32
 				ast_log(LOG_DEBUG, "#%d: [%08X] %s\n", i, (unsigned int)addresses[i], strings[i]);
 #elif __WORDSIZE == 64

Modified: trunk/main/manager.c
URL: http://svn.digium.com/view/asterisk/trunk/main/manager.c?view=diff&rev=103318&r1=103317&r2=103318
==============================================================================
--- trunk/main/manager.c (original)
+++ trunk/main/manager.c Mon Feb 11 12:27:47 2008
@@ -395,7 +395,7 @@
 		return 0;
 	if (ast_true(string)) {	/* all permissions */
 		int x, ret = 0;
-		for (x=0; x<sizeof(perms) / sizeof(perms[0]); x++)
+		for (x = 0; x<sizeof(perms) / sizeof(perms[0]); x++)
 			ret |= perms[x].num;
 		return ret;
 	}
@@ -1009,7 +1009,7 @@
 			MD5Update(&md5, (unsigned char *) s->challenge, strlen(s->challenge));
 			MD5Update(&md5, (unsigned char *) user->secret, strlen(user->secret));
 			MD5Final(digest, &md5);
-			for (x=0; x<16; x++)
+			for (x = 0; x < 16; x++)
 				len += sprintf(md5key + len, "%2.2x", digest[x]);
 			if (!strcmp(md5key, key))
 				error = 0;
@@ -1334,7 +1334,7 @@
 	s->waiting_thread = pthread_self();	/* let new events wake up this thread */
 	ast_debug(1, "Starting waiting for an event!\n");
 
-	for (x=0; x < timeout || timeout < 0; x++) {
+	for (x = 0; x < timeout || timeout < 0; x++) {
 		ast_mutex_lock(&s->__lock);
 		if (NEW_EVENT(s))
 			needexit = 1;

Modified: trunk/main/pbx.c
URL: http://svn.digium.com/view/asterisk/trunk/main/pbx.c?view=diff&rev=103318&r1=103317&r2=103318
==============================================================================
--- trunk/main/pbx.c (original)
+++ trunk/main/pbx.c Mon Feb 11 12:27:47 2008
@@ -1127,7 +1127,7 @@
 
 static struct match_char *add_exten_to_pattern_tree(struct ast_context *con, struct ast_exten *e1, int findonly)
 {
-	struct match_char *m1=0,*m2=0;
+	struct match_char *m1 = NULL, *m2 = NULL;
 	int specif;
 	int already;
 	int pattern = 0;
@@ -1453,7 +1453,7 @@
 {
 	mode &= E_MATCH_MASK;	/* only consider the relevant bits */
 
-	if ( (mode == E_MATCH) && (pattern[0] == '_') && (strcasecmp(pattern,data)==0) ) /* note: if this test is left out, then _x. will not match _x. !!! */
+	if ( (mode == E_MATCH) && (pattern[0] == '_') && (!strcasecmp(pattern,data)) ) /* note: if this test is left out, then _x. will not match _x. !!! */
 		return 1;
 
 	if (pattern[0] != '_') { /* not a pattern, try exact or partial match */
@@ -1628,12 +1628,12 @@
 	const char *label, const char *callerid, enum ext_match_t action)
 {
 	int x, res;
-	struct ast_context *tmp=0;
-	struct ast_exten *e=0, *eroot=0;
-	struct ast_include *i = 0;
-	struct ast_sw *sw = 0;
-	struct ast_exten pattern = {0};
-	struct scoreboard score = {0};
+	struct ast_context *tmp = NULL;
+	struct ast_exten *e = NULL, *eroot = NULL;
+	struct ast_include *i = NULL;
+	struct ast_sw *sw = NULL;
+	struct ast_exten pattern = {NULL, };
+	struct scoreboard score = {0, };
 
 	pattern.label = label;
 	pattern.priority = priority;
@@ -1870,7 +1870,7 @@
  */
 static int parse_variable_name(char *var, int *offset, int *length, int *isfunc)
 {
-	int parens=0;
+	int parens = 0;
 
 	*offset = 0;
 	*length = INT_MAX;
@@ -2034,7 +2034,7 @@
 		if (places[i] == &globals)
 			ast_rwlock_rdlock(&globalslock);
 		AST_LIST_TRAVERSE(places[i], variables, entries) {
-			if (strcasecmp(ast_var_name(variables), var)==0) {
+			if (!strcasecmp(ast_var_name(variables), var)) {
 				s = ast_var_value(variables);
 				break;
 			}
@@ -4568,9 +4568,9 @@
 				if (includecount >= AST_PBX_MAX_STACK) {
 					ast_log(LOG_NOTICE, "Maximum include depth exceeded!\n");
 				} else {
-					int dupe=0;
+					int dupe = 0;
 					int x;
-					for (x=0;x<includecount;x++) {
+					for (x = 0; x < includecount; x++) {
 						if (!strcasecmp(includes[x], ast_get_include_name(i))) {
 							dupe++;
 							break;
@@ -4719,7 +4719,7 @@
 					struct ast_include *rinclude)
 {
 	struct ast_context *c;
-	int res=0, old_total_exten = dpc->total_exten;
+	int res = 0, old_total_exten = dpc->total_exten;
 
 	if (ast_strlen_zero(exten))
 		exten = NULL;
@@ -5407,7 +5407,7 @@
 	/* 2-minutes per bit, since the mask has only 32 bits :( */
 	/* Star is all times */
 	if (ast_strlen_zero(times) || !strcmp(times, "*")) {
-		for (x=0; x<24; x++)
+		for (x = 0; x < 24; x++)
 			i->minmask[x] = 0x3fffffff; /* 30 bits */
 		return;
 	}
@@ -5452,10 +5452,10 @@
 	/* Do the last one */
 	i->minmask[x/30] |= (1 << (x % 30));
 #else
-	for (cth=0; cth<24; cth++) {
+	for (cth = 0; cth < 24; cth++) {
 		/* Initialize masks to blank */
 		i->minmask[cth] = 0;
-		for (ctm=0; ctm<30; ctm++) {
+		for (ctm = 0; ctm < 30; ctm++) {
 			if (
 			/* First hour with more than one hour */
 			      (((cth == s1) && (ctm >= s2)) &&
@@ -5931,7 +5931,7 @@
 /*! \brief copy a string skipping whitespace */
 static int ext_strncpy(char *dst, const char *src, int len)
 {
-	int count=0;
+	int count = 0;
 
 	while (*src && (count < len - 1)) {
 		switch (*src) {
@@ -6903,7 +6903,7 @@
  */
 static int pbx_builtin_gotoiftime(struct ast_channel *chan, void *data)
 {
-	int res=0;
+	int res = 0;
 	char *s, *ts;
 	struct ast_timing timing;
 
@@ -7527,7 +7527,7 @@
 	__ast_custom_function_register(&exception_function, NULL);
 
 	/* Register builtin applications */
-	for (x=0; x<sizeof(builtins) / sizeof(struct pbx_builtin); x++) {
+	for (x = 0; x < sizeof(builtins) / sizeof(struct pbx_builtin); x++) {
 		ast_verb(1, "[%s]\n", builtins[x].name);
 		if (ast_register_application2(builtins[x].name, builtins[x].execute, builtins[x].synopsis, builtins[x].description, NULL)) {
 			ast_log(LOG_ERROR, "Unable to register builtin application '%s'\n", builtins[x].name);

Modified: trunk/main/rtp.c
URL: http://svn.digium.com/view/asterisk/trunk/main/rtp.c?view=diff&rev=103318&r1=103317&r2=103318
==============================================================================
--- trunk/main/rtp.c (original)
+++ trunk/main/rtp.c Mon Feb 11 12:27:47 2008
@@ -1542,7 +1542,7 @@
 
 	rtp->lastrxseqno = seqno;
 	
-	if (rtp->themssrc==0)
+	if (!rtp->themssrc)
 		rtp->themssrc = ntohl(rtpheader[2]); /* Record their SSRC to put in future RR */
 	
 	if (rtp_debug_test_addr(&sin))
@@ -1767,7 +1767,7 @@
 	rtp_bridge_lock(dest);
 	rtp_bridge_lock(src);
 
-	for (i=0; i < MAX_RTP_PT; ++i) {
+	for (i = 0; i < MAX_RTP_PT; ++i) {
 		dest->current_RTP_PT[i].isAstFormat = 
 			src->current_RTP_PT[i].isAstFormat;
 		dest->current_RTP_PT[i].code = 

Modified: trunk/main/translate.c
URL: http://svn.digium.com/view/asterisk/trunk/main/translate.c?view=diff&rev=103318&r1=103317&r2=103318
==============================================================================
--- trunk/main/translate.c (original)
+++ trunk/main/translate.c Mon Feb 11 12:27:47 2008
@@ -482,11 +482,11 @@
 	for (;;) {
 		int changed = 0;
 		for (x = 0; x < MAX_FORMAT; x++) {      /* source format */
-			for (y=0; y < MAX_FORMAT; y++) {    /* intermediate format */
+			for (y = 0; y < MAX_FORMAT; y++) {    /* intermediate format */
 				if (x == y)                     /* skip ourselves */
 					continue;
 
-				for (z=0; z<MAX_FORMAT; z++) {  /* dst format */
+				for (z = 0; z<MAX_FORMAT; z++) {  /* dst format */
 					int newcost;
 
 					if (z == x || z == y)       /* skip null conversions */

Modified: trunk/main/utils.c
URL: http://svn.digium.com/view/asterisk/trunk/main/utils.c?view=diff&rev=103318&r1=103317&r2=103318
==============================================================================
--- trunk/main/utils.c (original)
+++ trunk/main/utils.c Mon Feb 11 12:27:47 2008
@@ -92,8 +92,8 @@
 	} else {
 		char **p, **q;
 		char *pbuf;
-		int nbytes=0;
-		int naddr=0, naliases=0;
+		int nbytes = 0;
+		int naddr = 0, naliases = 0;
 		/* determine if we have enough space in buf */
 
 		/* count how many addresses */
@@ -180,7 +180,7 @@
 {
 	int res;
 	int herrno;
-	int dots=0;
+	int dots = 0;
 	const char *s;
 	struct hostent *result = NULL;
 	/* Although it is perfectly legitimate to lookup a pure integer, for




More information about the asterisk-commits mailing list