[asterisk-commits] tilghman: trunk r346655 - in /trunk: channels/ main/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Wed Nov 30 17:38:39 CST 2011
Author: tilghman
Date: Wed Nov 30 17:38:34 2011
New Revision: 346655
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=346655
Log:
Remove the few places where we try to ast_verbose() without a newline.
Modified:
trunk/channels/chan_sip.c
trunk/channels/chan_unistim.c
trunk/main/config.c
trunk/main/loader.c
trunk/main/tcptls.c
Modified: trunk/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_sip.c?view=diff&rev=346655&r1=346654&r2=346655
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Wed Nov 30 17:38:34 2011
@@ -7372,16 +7372,23 @@
case 4:
f = ast_rtp_instance_read(p->trtp, 0); /* RTP Text */
if (sipdebug_text) {
+ struct ast_str *out = ast_str_create(f->datalen * 4 + 6);
int i;
unsigned char* arr = f->data.ptr;
- for (i=0; i < f->datalen; i++) {
- ast_verbose("%c", (arr[i] > ' ' && arr[i] < '}') ? arr[i] : '.');
- }
- ast_verbose(" -> ");
- for (i=0; i < f->datalen; i++) {
- ast_verbose("%02X ", arr[i]);
- }
- ast_verbose("\n");
+ do {
+ if (!out) {
+ break;
+ }
+ for (i = 0; i < f->datalen; i++) {
+ ast_str_append(&out, 0, "%c", (arr[i] > ' ' && arr[i] < '}') ? arr[i] : '.');
+ }
+ ast_str_append(&out, 0, " -> ");
+ for (i = 0; i < f->datalen; i++) {
+ ast_str_append(&out, 0, "%02X ", arr[i]);
+ }
+ ast_verb(0, "%s\n", ast_str_buffer(out));
+ ast_free(out);
+ } while (0);
}
break;
case 5:
Modified: trunk/channels/chan_unistim.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_unistim.c?view=diff&rev=346655&r1=346654&r2=346655
==============================================================================
--- trunk/channels/chan_unistim.c (original)
+++ trunk/channels/chan_unistim.c Wed Nov 30 17:38:34 2011
@@ -1558,14 +1558,14 @@
int tmp, i = 0;
char addrmac[19];
int res = 0;
- if (unistimdebug)
- ast_verb(0, "Mac Address received : ");
for (tmp = 15; tmp < 15 + SIZE_HEADER; tmp++) {
sprintf(&addrmac[i], "%.2x", (unsigned char) buf[tmp]);
i += 2;
}
- if (unistimdebug)
- ast_verb(0, "%s\n", addrmac);
+ if (unistimdebug) {
+ ast_verb(0, "Mac Address received : %s\n", addrmac);
+ }
+
strcpy(pte->macaddr, addrmac);
res = unistim_register(pte);
if (!res) {
Modified: trunk/main/config.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/config.c?view=diff&rev=346655&r1=346654&r2=346655
==============================================================================
--- trunk/main/config.c (original)
+++ trunk/main/config.c Wed Nov 30 17:38:34 2011
@@ -1503,18 +1503,16 @@
if (cfmtime)
cfmtime->mtime = statbuf.st_mtime;
- ast_verb(2, "Parsing '%s': ", fn);
- fflush(stdout);
if (!(f = fopen(fn, "r"))) {
ast_debug(1, "No file to parse: %s\n", fn);
- ast_verb(2, "Not found (%s)\n", strerror(errno));
+ ast_verb(2, "Parsing '%s': Not found (%s)\n", fn, strerror(errno));
continue;
}
count++;
/* If we get to this point, then we're loading regardless */
ast_clear_flag(&flags, CONFIG_FLAG_FILEUNCHANGED);
ast_debug(1, "Parsing %s\n", fn);
- ast_verb(2, "Found\n");
+ ast_verb(2, "Parsing '%s': Found\n", fn);
while (!feof(f)) {
lineno++;
if (fgets(buf, sizeof(buf), f)) {
@@ -1839,7 +1837,7 @@
fclose(f); /* this should zero out the file */
} else {
ast_debug(1, "Unable to open for writing: %s\n", fn);
- ast_verb(2, "Unable to write %s (%s)", fn, strerror(errno));
+ ast_verb(2, "Unable to write %s (%s)\n", fn, strerror(errno));
}
if (fi) {
ao2_ref(fi, -1);
@@ -1856,7 +1854,7 @@
(f = fopen(fn, "w"))
#endif
) {
- ast_verb(2, "Saving '%s': ", fn);
+ ast_verb(2, "Saving '%s'\n", fn);
gen_header(f, configfile, fn, generator);
cat = cfg->root;
fclose(f);
@@ -1873,7 +1871,7 @@
f = fopen(fn, "a");
if (!f) {
ast_debug(1, "Unable to open for writing: %s\n", fn);
- ast_verb(2, "Unable to write %s (%s)", fn, strerror(errno));
+ ast_verb(2, "Unable to write %s (%s)\n", fn, strerror(errno));
if (fi) {
ao2_ref(fi, -1);
}
@@ -1964,7 +1962,7 @@
f = fopen(fn, "a");
if (!f) {
ast_debug(1, "Unable to open for writing: %s\n", fn);
- ast_verb(2, "Unable to write %s (%s)", fn, strerror(errno));
+ ast_verb(2, "Unable to write %s (%s)\n", fn, strerror(errno));
if (fi) {
ao2_ref(fi, -1);
}
@@ -2013,11 +2011,12 @@
}
cat = cat->next;
}
- if (!option_debug)
- ast_verb(2, "Saved\n");
+ if (!option_debug) {
+ ast_verb(2, "Saving '%s': saved\n", fn);
+ }
} else {
ast_debug(1, "Unable to open for writing: %s\n", fn);
- ast_verb(2, "Unable to write (%s)", strerror(errno));
+ ast_verb(2, "Unable to write '%s' (%s)\n", fn, strerror(errno));
if (fi) {
ao2_ref(fi, -1);
}
@@ -2034,7 +2033,7 @@
f = fopen(fn, "a");
if (!f) {
ast_debug(1, "Unable to open for writing: %s\n", fn);
- ast_verb(2, "Unable to write %s (%s)", fn, strerror(errno));
+ ast_verb(2, "Unable to write %s (%s)\n", fn, strerror(errno));
if (fi) {
ao2_ref(fi, -1);
}
Modified: trunk/main/loader.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/loader.c?view=diff&rev=346655&r1=346654&r2=346655
==============================================================================
--- trunk/main/loader.c (original)
+++ trunk/main/loader.c Wed Nov 30 17:38:34 2011
@@ -797,8 +797,11 @@
case AST_MODULE_LOAD_SUCCESS:
if (!ast_fully_booted) {
ast_verb(1, "%s => (%s)\n", mod->resource, term_color(tmp, mod->info->description, COLOR_BROWN, COLOR_BLACK, sizeof(tmp)));
- if (ast_opt_console && !option_verbose)
- ast_verbose( ".");
+ if (ast_opt_console && !option_verbose) {
+ /* This never looks good on anything but the root console, so
+ * it's best not to try to funnel it through the logger. */
+ fprintf(stdout, ".");
+ }
} else {
ast_verb(1, "Loaded %s => (%s)\n", mod->resource, mod->info->description);
}
Modified: trunk/main/tcptls.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/tcptls.c?view=diff&rev=346655&r1=346654&r2=346655
==============================================================================
--- trunk/main/tcptls.c (original)
+++ trunk/main/tcptls.c Wed Nov 30 17:38:34 2011
@@ -359,7 +359,7 @@
if (SSL_CTX_use_certificate_file(cfg->ssl_ctx, cfg->certfile, SSL_FILETYPE_PEM) == 0) {
if (!client) {
/* Clients don't need a certificate, but if its setup we can use it */
- ast_verb(0, "SSL error loading cert file. <%s>", cfg->certfile);
+ ast_verb(0, "SSL error loading cert file. <%s>\n", cfg->certfile);
sleep(2);
cfg->enabled = 0;
return 0;
@@ -368,7 +368,7 @@
if ((SSL_CTX_use_PrivateKey_file(cfg->ssl_ctx, tmpprivate, SSL_FILETYPE_PEM) == 0) || (SSL_CTX_check_private_key(cfg->ssl_ctx) == 0 )) {
if (!client) {
/* Clients don't need a private key, but if its setup we can use it */
- ast_verb(0, "SSL error loading private key file. <%s>", tmpprivate);
+ ast_verb(0, "SSL error loading private key file. <%s>\n", tmpprivate);
sleep(2);
cfg->enabled = 0;
return 0;
@@ -378,7 +378,7 @@
if (!ast_strlen_zero(cfg->cipher)) {
if (SSL_CTX_set_cipher_list(cfg->ssl_ctx, cfg->cipher) == 0 ) {
if (!client) {
- ast_verb(0, "SSL cipher error <%s>", cfg->cipher);
+ ast_verb(0, "SSL cipher error <%s>\n", cfg->cipher);
sleep(2);
cfg->enabled = 0;
return 0;
More information about the asterisk-commits
mailing list