<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/3324/">https://reviewboard.asterisk.org/r/3324/</a>
</td>
</tr>
</table>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://reviewboard.asterisk.org/r/3324/diff/1/?file=55753#file55753line589" style="color: black; font-weight: bold; text-decoration: underline;">/branches/12/res/res_pjsip/config_transport.c</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">static int cli_print_body(void *obj, void *arg, int flags)</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">577</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="tb"> </span><span class="n">ast_str_append</span><span class="p">(</span><span class="o">&</span><span class="n">context</span><span class="o">-></span><span class="n">output_buffer</span><span class="p">,</span> <span class="mi">0</span><span class="p">,</span> <span class="s">"%*s: %-21s %6s %5<span class="hl">x</span> %5<span class="hl">x</span> %s</span><span class="se">\n</span><span class="s">"</span><span class="p">,</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">589</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="tb"> </span><span class="n">ast_str_append</span><span class="p">(</span><span class="o">&</span><span class="n">context</span><span class="o">-></span><span class="n">output_buffer</span><span class="p">,</span> <span class="mi">0</span><span class="p">,</span> <span class="s">"%*s: %-21s %6s %5<span class="hl">d</span> %5<span class="hl">d</span> %s</span><span class="se">\n</span><span class="s">"</span><span class="p">,</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">So a little explanation here... I didn't really see that using hex values for COS/TOS had any purpose. We use decimal values mostly for the input of both of these and COS in particular is (or should be) a 3 bit value anyway and will never break a single digit in decimal in the first place. If this doesn't seem appropriate I can switch it back, but with the decimal values appearing in the sorcery object anyway I think this will be less confusing.</pre>
</div>
<br />
<p>- Jonathan Rose</p>
<br />
<p>On March 10th, 2014, 2:04 p.m. CDT, Jonathan Rose wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers, George Joseph and Matt Jordan.</div>
<div>By Jonathan Rose.</div>
<p style="color: grey;"><i>Updated March 10, 2014, 2:04 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-23235">ASTERISK-23235</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/branches/12/res/res_pjsip/pjsip_configuration.c <span style="color: grey">(410367)</span></li>
<li>/branches/12/res/res_pjsip/config_transport.c <span style="color: grey">(410367)</span></li>
<li>/branches/12/main/acl.c <span style="color: grey">(410367)</span></li>
<li>/branches/12/include/asterisk/acl.h <span style="color: grey">(410367)</span></li>
<li>/branches/12/CHANGES <span style="color: grey">(410367)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3324/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>