[asterisk-dev] [Code Review] 3324: res_pjsip: Make ToS values show up as decimals in sorcery object output

Jonathan Rose reviewboard at asterisk.org
Thu Mar 13 11:11:54 CDT 2014


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

(Updated March 13, 2014, 11:11 a.m.)


Review request for Asterisk Developers, George Joseph and Matt Jordan.


Changes
-------

Use ast_asprintf


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


Repository: Asterisk


Description
-------

ToS values were showing up as strings in the output for the sorcery objects of res_pjsip endpoints and transports. This was causing test failures at one time and is also just not a very useful way of using TOS values that aren't from the predefined ToS names.  I also added a chart of named ToS values and their decimal equivalents to https://wiki.asterisk.org/wiki/display/AST/IP+Quality+of+Service This patch makes it so that they are output as decimal values only in the sorcery objects.


Diffs (updated)
-----

  /branches/12/res/res_pjsip/pjsip_configuration.c 410367 
  /branches/12/res/res_pjsip/config_transport.c 410367 
  /branches/12/main/acl.c 410367 
  /branches/12/include/asterisk/acl.h 410367 
  /branches/12/CHANGES 410367 

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


Testing
-------

Tested output for all the named ToS values and some values without equivalents.  Tested values above 255... since it uses the str2tos function, this has a sort of wrap around behavior for anything above 255. Not sure how appropriate that is, but it's been in place historically anyway. Updated tests that were evaluating PJSIP tos settings.  That's in another review.


Thanks,

Jonathan Rose

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


More information about the asterisk-dev mailing list