<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/1754/">https://reviewboard.asterisk.org/r/1754/</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;">Looks good to me. I certainly would like to get someone to take a second look because I never feel comfortable just giving a "ship it!" with no suggestions...it just feels like I have to have missed something :)</pre>
<br />
<p>- Mark</p>
<br />
<p>On February 20th, 2012, 10:33 a.m., Joshua Colp wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Joshua Colp.</div>
<p style="color: grey;"><i>Updated Feb. 20, 2012, 10:33 a.m.</i></p>
<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;">The Page application was originally written to use MeetMe internally but as development has progressed ConfBridge has become the new favored conference bridge application. This task was originally to change Page to use the bridging API internally but that proved to be too low level and numerous features from ConfBridge would have been duplicated. Page has therefore been rewritten to use ConfBridge internally.</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 Paging multiple phones with different combinations of features.</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>/trunk/apps/app_confbridge.c <span style="color: grey">(355010)</span></li>
<li>/trunk/apps/app_page.c <span style="color: grey">(355010)</span></li>
<li>/trunk/apps/confbridge/conf_config_parser.c <span style="color: grey">(355010)</span></li>
<li>/trunk/apps/confbridge/include/confbridge.h <span style="color: grey">(355010)</span></li>
<li>/trunk/include/asterisk/dial.h <span style="color: grey">(355010)</span></li>
<li>/trunk/main/dial.c <span style="color: grey">(355010)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1754/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>