<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/4159/">https://reviewboard.asterisk.org/r/4159/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#e0e0e0" width="100%" cellpadding="8" style="border: 1px gray solid;">
<tr>
<td>
<h1 style="margin-right: 0.2em; padding: 0; font-size: 10pt;">This change has been discarded.</h1>
</td>
</tr>
</table>
<br />
<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.</div>
<div>By Corey Farrell.</div>
<p style="color: grey;"><i>Updated Nov. 11, 2014, 2:17 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-24510">ASTERISK-24510</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;">Thank you in advance to anyone who takes the time to review this tedious patch. It is large but benign.
Scanned for / fixed instances of the regex search pattern (else|(if|while|for)\s*\(.*\))(\s*/\*.*\*/|)\s*$ in channels/chan_sip.c and channels/sip/*.c.
I've proposed this change for 13 and trunk. The set of braces for one 'if' statement was excluded from this change to allow the entire patch to apply cleanly to 13 and trunk with patch fuzz level 0. If you approve this change please note if you are approving it for 13+, or just trunk. I was late to AstriDevCon so I'm not sure if this is acceptable to 13, figured it doesn't hurt to ask. Asterisk 13 will be supported for a long time and chan_sip is in extended support status. I feel that adding the braces will make it easier/lower risk for me to fix bugs that are found in the next 4 years.
I have no idea what reviewboard is doing with the display for lines 6078 - 6254, no code is actually moved. Check the diff file for the actual changes in that section.</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;">Compiled, visually inspected. Ran all of tests/channel/SIP in testsuite.</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/13/channels/sip/reqresp_parser.c <span style="color: grey">(427641)</span></li>
<li>/branches/13/channels/sip/dialplan_functions.c <span style="color: grey">(427641)</span></li>
<li>/branches/13/channels/sip/config_parser.c <span style="color: grey">(427641)</span></li>
<li>/branches/13/channels/chan_sip.c <span style="color: grey">(427641)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/4159/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>