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

[Joshua Colp]

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



/branches/12/res/res_pjsip/config_transport.c
<https://reviewboard.asterisk.org/r/3324/#comment20800>

    Why not just use ast_asprintf for both of the to_str functions?


- Joshua Colp


On March 10, 2014, 7:04 p.m., Jonathan Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3324/
> -----------------------------------------------------------
> 
> (Updated March 10, 2014, 7:04 p.m.)
> 
> 
> Review request for Asterisk Developers, George Joseph and Matt Jordan.
> 
> 
> 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
> -----
> 
>   /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/07ef3755/attachment-0001.html>