[Asterisk-code-review] app queue: Show reason of pause on CLI (asterisk[master])

Richard Mudgett asteriskteam at digium.com
Mon Nov 30 15:07:38 CST 2015


Richard Mudgett has posted comments on this change.

Change subject: app_queue: Show reason of pause on CLI
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.asterisk.org/#/c/1666/1/apps/app_queue.c
File apps/app_queue.c:

Line 9262: 							" (paused", mem->reason_paused, ")" , ast_term_reset());
> Sorry, i don't understand. You can explain a little more, please.
He was asking what the output of the CLI "queue show" or AMI action Queues looks like with your change.  Is the new information squished together so you cannot read it?  It certainly looks like there is no delimiter between what was there before and the new information.
(pausedreason) vs (paused reason)


-- 
To view, visit https://gerrit.asterisk.org/1666
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I887028a40cd97b350da9a3bb2719616b7fec9864
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list