[asterisk-dev] [Code Review] 3136: cli: pjsip show endpoint <endpoint> shows allow/disallow codecs the same

Scott Griepentrog reviewboard at asterisk.org
Mon Feb 3 14:18:57 CST 2014


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

(Updated Feb. 3, 2014, 2:18 p.m.)


Review request for Asterisk Developers.


Changes
-------

Missed the config options table when I added the enum.  Now fixed.


Bugs: ASTERISK-23092
    https://issues.asterisk.org/jira/browse/ASTERISK-23092


Repository: Asterisk


Description (updated)
-------

WAS:

Insert a ! prefix in the display of endpoint disallow value.  Result is:

 disallow                      : !(ulaw|alaw)

NOW:

Remove the disallow option from generated lists, while still accepting it from a configuration file.


Diffs (updated)
-----

  /branches/12/res/res_pjsip/pjsip_configuration.c 407196 
  /branches/12/main/sorcery.c 407196 
  /branches/12/main/config_options.c 407196 
  /branches/12/include/asterisk/config_options.h 407196 

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


Testing
-------

Ran command and checked output.


Thanks,

Scott Griepentrog

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140203/9c610459/attachment.html>


More information about the asterisk-dev mailing list