<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/4315/">https://reviewboard.asterisk.org/r/4315/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Minor finding, so going to go ahead a ship.</pre>
<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/4315/diff/2/?file=70251#file70251line64" style="color: black; font-weight: bold; text-decoration: underline;">/branches/13/CHANGES</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">64</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> event. The ARI version has been bumped to 2.7.0 as a result.</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;">I think you meant 1.7.0?</pre>
</div>
<br />
<p>- Kevin Harwell</p>
<br />
<p>On January 7th, 2015, 1:36 p.m. CST, rmudgett 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.</div>
<div>By rmudgett.</div>
<p style="color: grey;"><i>Updated Jan. 7, 2015, 1:36 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-24049">ASTERISK-24049</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;">* Made the following AMI actions use list API calls for consistency:
Agents
BridgeInfo
BridgeList
BridgeTechnologyList
ConfbridgeLIst
ConfbridgeLIstRooms
CoreShowChannels
DAHDIShowChannels
DBGet
DeviceStateList
ExtensionStateList
FAXSessions
Hangup
IAXpeerlist
IAXpeers
IAXregistry
MeetmeList
MeetmeListRooms
MWIGet
ParkedCalls
Parkinglots
PJSIPShowEndpoint
PJSIPShowEndpoints
PJSIPShowRegistrationsInbound
PJSIPShowRegistrationsOutbound
PJSIPShowResourceLists
PJSIPShowSubscriptionsInbound
PJSIPShowSubscriptionsOutbound
PresenceStateList
PRIShowSpans
QueueStatus
QueueSummary
ShowDialPlan
SIPpeers
SIPpeerstatus
SIPshowregistry
SKINNYdevices
SKINNYlines
Status
VoicemailUsersList
* Incremented the AMI version to 2.7.0.
* Changed astman_send_listack() to not use the listflag parameter and
always set the value to "Start" so the start capitalization is consistent.
i.e., The FAXSessions used "Start" while the rest of the system used
"start". The corresponding complete event always used "Complete".
* Fixed ami_show_resource_lists() "PJSIPShowResourceLists" to output the
AMI ActionID for all of its list events.
* Fixed off-nominal AMI protocol error in manager_bridge_info(),
manager_parking_status_single_lot(), and
manager_parking_status_all_lots(). Use of astman_send_error() after
responding to the original AMI action request violates the action response
pattern by sending two responses.
* Fixed minor protocol error in action_getconfig() when no requested
categories are found. Each line needs to be formatted as "Header: text".
* Fixed off-nominal memory leak in manager_build_parked_call_string().
* Eliminated unnecessary use of RAII_VAR() in ami_subscription_detail().
</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;">Issued all of the AMI actions listed above to verify that the output was consistent.</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/res/res_pjsip_registrar.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/res/res_pjsip_pubsub.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/res/res_pjsip_outbound_registration.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/res/res_pjsip/pjsip_configuration.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/res/res_mwi_external_ami.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/res/res_manager_presencestate.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/res/res_manager_devicestate.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/res/res_fax.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/res/parking/parking_manager.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/main/pbx.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/main/manager_bridges.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/main/manager.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/main/db.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/main/bridge.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/include/asterisk/manager.h <span style="color: grey">(430354)</span></li>
<li>/branches/13/channels/chan_skinny.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/channels/chan_sip.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/channels/chan_iax2.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/channels/chan_dahdi.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/apps/app_voicemail.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/apps/app_queue.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/apps/app_meetme.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/apps/app_confbridge.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/apps/app_agent_pool.c <span style="color: grey">(430354)</span></li>
<li>/branches/13/UPGRADE.txt <span style="color: grey">(430354)</span></li>
<li>/branches/13/CHANGES <span style="color: grey">(430354)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/4315/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>