diff options
author | Matthew Jordan <mjordan@digium.com> | 2014-05-28 22:54:12 +0000 |
---|---|---|
committer | Matthew Jordan <mjordan@digium.com> | 2014-05-28 22:54:12 +0000 |
commit | fb5690ce4b9ff896a56fe9b9eb9eeed1a2b5f028 (patch) | |
tree | 5fc5aa9c8099167cf81dc9cfdadb19ce3aa24743 /channels/chan_unistim.c | |
parent | 812f33d22218817f32ce856969e8196475f6d5fd (diff) |
Logger/CLI/etc.: Fix some aesthetic issues; reduce chatty verbose messages
This patch addresses some aesthetic issues in Asterisk. These are all just
minor tweaks to improve the look of the CLI when used in a variety of
settings. Specifically:
* A number of chatty verbose messages were removed or demoted to DEBUG
messages. Verbose messages with a verbosity level of 5 or higher were -
if kept as verbose messages - demoted to level 4. Several messages
that were emitted at verbose level 3 were demoted to 4, as announcement
of dialplan applications being executed occur at level 3 (and so the
effects of those applications should generally be less).
* Some verbose messages that only appear when their respective 'debug'
options are enabled were bumped up to always be displayed.
* Prefix/timestamping of verbose messages were moved to the verboser
handlers. This was done to prevent duplication of prefixes when the
timestamp option (-T) is used with the CLI.
* Verbose magic is removed from messages before being emitted to
non-verboser handlers. This prevents the magic in multi-line verbose
messages (such as SIP debug traces or the output of DumpChan) from
being written to files.
* _Slightly_ better support for the "light background" option (-W) was
added. This includes using ast_term_quit in the output of XML
documentation help, as well as changing the "Asterisk Ready" prompt to
bright green on the default background (which stands a better chance of
being displayed properly than bright white).
Review: https://reviewboard.asterisk.org/r/3547/
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@414798 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'channels/chan_unistim.c')
-rw-r--r-- | channels/chan_unistim.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/channels/chan_unistim.c b/channels/chan_unistim.c index 6762db391..7525f3921 100644 --- a/channels/chan_unistim.c +++ b/channels/chan_unistim.c @@ -973,7 +973,7 @@ static void send_client(int size, const unsigned char *data, struct unistimsessi /*#ifdef DUMP_PACKET */ if (unistimdebug) { - ast_verb(6, "Sending datas with seq #0x%.4x Using slot #%d :\n", (unsigned)pte->seq_server, buf_pos); + ast_verb(0, "Sending datas with seq #0x%.4x Using slot #%d :\n", (unsigned)pte->seq_server, buf_pos); } /*#endif */ send_raw_client(pte->wsabufsend[buf_pos].len, pte->wsabufsend[buf_pos].buf, &(pte->sin), @@ -986,7 +986,7 @@ static void send_ping(struct unistimsession *pte) { BUFFSEND; if (unistimdebug) { - ast_verb(6, "Sending ping\n"); + ast_verb(0, "Sending ping\n"); } pte->tick_next_ping = get_tick_count() + unistim_keepalive; memcpy(buffsend + SIZE_HEADER, packet_send_ping, sizeof(packet_send_ping)); @@ -1084,7 +1084,7 @@ static void check_send_queue(struct unistimsession *pte) /* Check if our send queue contained only one element */ if (pte->last_buf_available == 1) { if (unistimdebug) { - ast_verb(6, "Our single packet was ACKed.\n"); + ast_verb(0, "Our single packet was ACKed.\n"); } pte->last_buf_available--; set_ping_timer(pte); @@ -1093,14 +1093,14 @@ static void check_send_queue(struct unistimsession *pte) /* Check if this ACK catch up our latest packet */ else if (pte->last_seq_ack + 1 == pte->seq_server + 1) { if (unistimdebug) { - ast_verb(6, "Our send queue is completely ACKed.\n"); + ast_verb(0, "Our send queue is completely ACKed.\n"); } pte->last_buf_available = 0; /* Purge the send queue */ set_ping_timer(pte); return; } if (unistimdebug) { - ast_verb(6, "We still have packets in our send queue\n"); + ast_verb(0, "We still have packets in our send queue\n"); } return; } @@ -4659,7 +4659,7 @@ static void parsing(int size, unsigned char *buf, struct unistimsession *pte, if (buf[4] == 1) { ast_mutex_lock(&pte->lock); if (unistimdebug) { - ast_verb(6, "ACK received for packet #0x%.4x\n", (unsigned)seq); + ast_verb(0, "ACK received for packet #0x%.4x\n", (unsigned)seq); } pte->nb_retransmit = 0; |