[asterisk-commits] russell: branch russell/ast_verbose_threadstorage r38400 - in /team/russell/a...

[asterisk-commits at lists.digium.com]

Author: russell
Date: Thu Jul 27 21:15:10 2006
New Revision: 38400

URL: http://svn.digium.com/view/asterisk?rev=38400&view=rev
Log:
minor cleanup of a loop and add some missing newlines in ast_verbose calls

Modified:
    team/russell/ast_verbose_threadstorage/asterisk.c
    team/russell/ast_verbose_threadstorage/logger.c
    team/russell/ast_verbose_threadstorage/res/res_agi.c

Modified: team/russell/ast_verbose_threadstorage/asterisk.c
URL: http://svn.digium.com/view/asterisk/team/russell/ast_verbose_threadstorage/asterisk.c?rev=38400&r1=38399&r2=38400&view=diff
==============================================================================
--- team/russell/ast_verbose_threadstorage/asterisk.c (original)
+++ team/russell/ast_verbose_threadstorage/asterisk.c Thu Jul 27 21:15:10 2006
@@ -2417,7 +2417,7 @@
 		WELCOME_MESSAGE;
 
 	if (ast_opt_console && !option_verbose) 
-		ast_verbose("[ Booting...");
+		ast_verbose("[ Booting...\n");
 
 	if (ast_opt_always_fork && (ast_opt_remote || ast_opt_console)) {
 		ast_log(LOG_WARNING, "'alwaysfork' is not compatible with console or remote console mode; ignored\n");
@@ -2435,7 +2435,7 @@
 	}
 
 	if (ast_opt_console && !option_verbose) 
-		ast_verbose("[ Reading Master Configuration ]");
+		ast_verbose("[ Reading Master Configuration ]\n");
 	ast_readconfig();
 
 	if (ast_opt_dump_core) {
@@ -2518,7 +2518,7 @@
 	fflush(stdout);
 
 	if (ast_opt_console && !option_verbose) 
-		ast_verbose("[ Initializing Custom Configuration Options ]");
+		ast_verbose("[ Initializing Custom Configuration Options ]\n");
 	/* custom config setup */
 	register_config_cli();
 	read_config_maps();

Modified: team/russell/ast_verbose_threadstorage/logger.c
URL: http://svn.digium.com/view/asterisk/team/russell/ast_verbose_threadstorage/logger.c?rev=38400&r1=38399&r2=38400&view=diff
==============================================================================
--- team/russell/ast_verbose_threadstorage/logger.c (original)
+++ team/russell/ast_verbose_threadstorage/logger.c Thu Jul 27 21:15:10 2006
@@ -342,14 +342,12 @@
 	}
 
 	var = ast_variable_browse(cfg, "logfiles");
-	while(var) {
-		chan = make_logchannel(var->name, var->value, var->lineno);
-		if (chan) {
-			chan->next = logchannels;
-			logchannels = chan;
-			global_logmask |= chan->logmask;
-		}
-		var = var->next;
+	for (; var; var = var->next) {
+		if (!(chan = make_logchannel(var->name, var->value, var->lineno)))
+			continue;
+		chan->next = logchannels;
+		logchannels = chan;
+		global_logmask |= chan->logmask;
 	}
 
 	ast_config_destroy(cfg);

Modified: team/russell/ast_verbose_threadstorage/res/res_agi.c
URL: http://svn.digium.com/view/asterisk/team/russell/ast_verbose_threadstorage/res/res_agi.c?rev=38400&r1=38399&r2=38400&view=diff
==============================================================================
--- team/russell/ast_verbose_threadstorage/res/res_agi.c (original)
+++ team/russell/ast_verbose_threadstorage/res/res_agi.c Thu Jul 27 21:15:10 2006
@@ -130,7 +130,7 @@
 		ast_log(LOG_ERROR, "Out of memory\n");
 	} else {
 		if (agidebug)
-			ast_verbose("AGI Tx >> %s", stuff);
+			ast_verbose("AGI Tx >> %s\n", stuff);
 		ast_carefulwrite(fd, stuff, strlen(stuff), 100);
 		free(stuff);
 	}