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

Rodrigo Ramirez Norambuena asteriskteam at digium.com
Wed Dec 2 19:31:29 CST 2015


Rodrigo Ramirez Norambuena 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 9261: 						ast_str_append(&out, 0, "%s%s:%s%s%s", 	ast_term_color(COLOR_BROWN, COLOR_BLACK),
         : 							" (paused", mem->reason_paused, ")" , ast_term_reset());
> There's no reason to have "(paused" and ")" as passed in parameters when th
Thanks for recommendations. I will do the changes.


-- 
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