[asterisk-dev] [Code Review] Prevent manager show commands cutting off the command name in favor of other, less important information.

jrose reviewboard at asterisk.org
Wed Oct 3 16:29:01 CDT 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2143/
-----------------------------------------------------------

Review request for Asterisk Developers.


Summary
-------

First, this patch cuts out information about the required authorization level.  There is really no need for it here and this information is freely avaialble with the 'manager show command [command]' information.

Second, the size field for the name is now picked based on the size of the largest command name. Any remaining space (out of 85, based on the sum of all previous fields, though I could add 2 since I removed some spaces as well and still be length neutral) will be given to the description.


This addresses bug ASTERISK-20396.
    https://issues.asterisk.org/jira/browse/ASTERISK-20396


Diffs
-----

  /trunk/main/manager.c 374334 

Diff: https://reviewboard.asterisk.org/r/2143/diff


Testing
-------

I used the command. It did what I thought it would. Seems pretty simplish.


Thanks,

jrose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20121003/4e9b698e/attachment.htm>


More information about the asterisk-dev mailing list