<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/1655/">https://reviewboard.asterisk.org/r/1655/</a>
     </td>
    </tr>
   </table>
   <br />








<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On January 9th, 2012, 6:13 a.m., <b>schmidts</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  



<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/1655/diff/2/?file=22855#file22855line773" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/include/asterisk/channel.h</a>
    <span style="font-weight: normal;">

     (Diff revision 2)

    </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; ">struct ast_channel {</pre></td>

  </tr>
 </tbody>



 
 




 <tbody>

  <tr>
    <th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">773</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                <span class="n">AST_STRING_FIELD</span><span class="p">(</span><span class="n">name</span><span class="p">);</span>                        <span class="cm">/*!&lt; ASCII unique channel name */</span></pre></td>
    <th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">773</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                <span class="n">AST_STRING_FIELD</span><span class="p">(</span><span class="n"><span class="hl">__do_not_use_</span>name</span><span class="p">);</span> <span class="cm">/*!&lt; ASCII unique channel name */</span></pre></td>
  </tr>

 </tbody>

</table>

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">just an idea. you could let name stay here and initialise it with a warning value like &quot;please dont use chan-&gt;name directly anymore use ast_channel_name(chan) instead&quot;. so this could help devs to fix their own apps.</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The main reason I am changing the name completely is that it allows the compiler to tell us when we&#39;ve messed up. I&#39;d never catch all of the uses without it. I also want to penalize people at compile-time for using the field. Just initializing it wouldn&#39;t do much good since it would just get overwritten almost immediately anyway.</pre>
<br />

<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On January 9th, 2012, 6:13 a.m., <b>schmidts</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  



<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/1655/diff/2/?file=22864#file22864line1629" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/main/channel.c</a>
    <span style="font-weight: normal;">

     (Diff revision 2)

    </span>
   </th>
  </tr>
 </thead>

 <tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
  <tr>

   <td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
   <td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">static int ast_channel_by_uniqueid_cb(void *obj, void *arg, void *data, int flags)</pre></td>

  </tr>
 </tbody>




 
 



 <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">1423</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">const</span> <span class="kt">char</span> <span class="o">*</span><span class="n">name</span> <span class="o">=</span> <span class="p">(</span><span class="n">flags</span> <span class="o">&amp;</span> <span class="n">OBJ_KEY</span><span class="p">)</span> <span class="o">?</span> <span class="n">arg</span> <span class="o">:</span> <span class="n">ast_channel_name</span><span class="p">((</span><span class="k">struct</span> <span class="n">ast_channel</span> <span class="o">*</span><span class="p">)</span><span class="n">arg</span><span class="p">);</span></pre></td>
  </tr>

 </tbody>

</table>

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">just a very short question about locking here. Imho it is possible that the name of the channel, given as argument, could change after this, shouldnt this channel (arg) be locked too?</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">arg is never a *real* channel and can&#39;t be locked. It is only passed as the address of a channel created on the stack with some fields just filled in like:

struct ast_channel tmp = {
   .name = something,
};

and I may have even gotten all of those removed in favor of searching in other ways, but I left support for the old-style just in case.</pre>
<br />

<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On January 9th, 2012, 6:13 a.m., <b>schmidts</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  



<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/1655/diff/2/?file=22864#file22864line1631" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/main/channel.c</a>
    <span style="font-weight: normal;">

     (Diff revision 2)

    </span>
   </th>
  </tr>
 </thead>

 <tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
  <tr>

   <td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
   <td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">static int ast_channel_by_uniqueid_cb(void *obj, void *arg, void *data, int flags)</pre></td>

  </tr>
 </tbody>




 
 



 <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">1425</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="kt">int</span> <span class="n">ret</span> <span class="o">=</span> <span class="n">CMP_MATCH</span><span class="p">;</span></pre></td>
  </tr>

 </tbody>

</table>

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">you could use CMP_MATCH &amp; CMP_STOP here cause channel name should be unique imho.

same on the ast_channel_by_uniqueid_cp.</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">the ast_channel_by_name_cb is called specifically with OBJ_MULTIPLE and we are also comparing based on length, so the match won&#39;t necessarily be unique. For example, lots of code ends up doing a search for channel names like &quot;SIP/1234&quot; not &quot;SIP/1234-00000000&quot; so that we can return all channels like SIP/1234.</pre>
<br />

<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On January 9th, 2012, 6:13 a.m., <b>schmidts</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  



<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/1655/diff/2/?file=22864#file22864line1674" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/main/channel.c</a>
    <span style="font-weight: normal;">

     (Diff revision 2)

    </span>
   </th>
  </tr>
 </thead>

 <tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
  <tr>

   <td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
   <td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">static int ast_channel_by_uniqueid_cb(void *obj, void *arg, void *data, int flags)</pre></td>

  </tr>
 </tbody>




 
 



 <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">1468</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="kt">int</span> <span class="n">ret</span> <span class="o">=</span> <span class="n">CMP_MATCH</span><span class="p">;</span></pre></td>
  </tr>

 </tbody>

</table>

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">see comment above, CMP_MATCH &amp; CMP_STOP</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I prefer to let the caller choose via flags whether or not to continue searching for other matches. If they don&#39;t specifically pass OBJ_MULTIPLE, I think CMP_MATCH implies stopping. Maybe I should specifically pass 0 for flags when ast_channel_by_uniqueid_cb is called from ast_channel_cmp_cb?</pre>
<br />

<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On January 9th, 2012, 6:13 a.m., <b>schmidts</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  



<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/1655/diff/2/?file=22864#file22864line1825" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/main/channel.c</a>
    <span style="font-weight: normal;">

     (Diff revision 2)

    </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; ">struct ast_channel_iterator *ast_channel_iterator_all_new(void)</pre></td>

  </tr>
 </tbody>



 
 




 <tbody>

  <tr>
    <th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">1759</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">if</span> <span class="p">(</span><span class="n">exten</span><span class="p">)</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">1583</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">if</span> <span class="p">(</span><span class="n">ast_strlen_zero</span><span class="p">(</span><span class="n">name</span><span class="p">)</span> <span class="o">&amp;&amp;</span> <span class="p">(</span><span class="n">chan</span> <span class="o">=</span> <span class="n">ast_channel_callback</span><span class="p">(</span><span class="n">ast_channel_by_exten_cb</span><span class="p">,</span> <span class="p">(</span><span class="kt">void</span> <span class="o">*</span><span class="p">)</span> <span class="n">context</span><span class="p">,</span> <span class="p">(</span><span class="kt">void</span> <span class="o">*</span><span class="p">)</span> <span class="n">exten</span><span class="p">,</span> <span class="mi">0</span><span class="p">)))</span> <span class="p">{</span></pre></td>
  </tr>

 </tbody>

</table>

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">you should add this before the callback:

if (ast_strlen_zero(name) &amp;&amp; exten &amp;&amp; context &amp;&amp; (chan = ast_channel_callback....

imho you would get an error message for each channel (see row 1448) if this is called with an empty name and no exten &amp; context set. I dont know if this even could happen but row 1590 looks like it could be.</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This would change the logic so that it would be possible to then call the by_name callback with a zero-length name if, for instance, name and exten were both null. So it would just push the error to a different function. I&#39;m pretty sure that it is an actual bug in the calling code if this ever happens and I would rather warn people about it than just hide it by throwing in checks for the values just to avoid getting an error message. In fact, I&#39;m even considering putting in ast_assert() calls here so that in dev mode it would die if we did this.

Can you think of a reason this would be a bad thing to do? I&#39;ve been staring at the code for a while and may have tunnel vision.</pre>
<br />




<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On January 9th, 2012, 6:13 a.m., <b>schmidts</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">fine work and a good way to go.</pre>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Thanks!</pre>
<br />


<p>- Terry</p>


<br />
<p>On January 6th, 2012, 3:51 p.m., Terry Wilson 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 Terry Wilson.</div>


<p style="color: grey;"><i>Updated Jan. 6, 2012, 3:51 p.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;">There would be many benefits to making the ast_channel an opaque handle, from increasing maintainability to presenting ways to kill masquerades. I propose that we kick things off by taking things a field at a time, renaming the field to &#39;__do_not_use_${fieldname}&#39; and then writing setters/getters and converting the existing code to using them. When all fields are done, we can move ast_channel to a C file from channel.h and lop off the &#39;__do_not_use_&#39;.

This patch sets up main/channel_interal_api.c to be the only file that actually accesses the ast_channel&#39;s fields directly. The intent would be for any API functions in channel.c to use the accessor functions. No more monkeying around with channel internals. We should use our own APIs.

The interesting changes in this patch are the addition of channel_internal_api.c, the moving of the AST_DATA stuff from channel.c to channel_internal_api.c (note: the AST_DATA stuff will have to be reworked to use accessor functions when ast_channel is really opaque), and some re-working of the way channel iterators/callbacks are handled so as to avoid creating fake ast_channels on the stack to pass in matching data by directly accessing fields (since &quot;name&quot; is a stringfield and the fake channel doesn&#39;t init the stringfields, you can&#39;t use the ast_channel_name_set() function). I went with ast_channel_name(chan) for a getter, and ast_channel_name_set(chan, name) for a setter.

Ideas welcome!

The majority of the grunt-work for this change was done by writing a semantic patch using Coccinelle ( http://coccinelle.lip6.fr/ ).

note: There are some extra changes to some of the h323 files to comment out variables that were set and not used so that I could get it to compile under dev-mode.
</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;">I ran it through the testsuite and seemed to get the same number of failures that I got without it. I also made some calls and tested out &#39;core show channel &lt;tab&gt;&#39; and &#39;core show channel Console/dsp&#39;, and core show channels to make sure that the changes to the iterators seemed to work.</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/addons/chan_mobile.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/addons/chan_ooh323.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/addons/ooh323c/src/memheap.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_adsiprog.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_alarmreceiver.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_amd.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_chanisavail.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_chanspy.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_confbridge.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_dahdibarge.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_dahdiras.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_dial.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_directed_pickup.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_disa.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_dumpchan.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_externalivr.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_fax.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_flash.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_followme.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_getcpeid.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_jack.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_macro.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_meetme.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_milliwatt.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_minivm.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_mixmonitor.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_page.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_parkandannounce.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_playback.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_queue.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_readexten.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_record.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_rpt.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_sms.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_softhangup.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_stack.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_talkdetect.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_test.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_voicemail.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/apps/app_waitforsilence.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/bridges/bridge_multiplexed.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_agent.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_alsa.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_console.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_dahdi.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_gtalk.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_h323.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_iax2.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_jingle.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_local.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_mgcp.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_misdn.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_nbs.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_oss.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_phone.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_sip.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_skinny.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_unistim.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_usbradio.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/chan_vpb.cc <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/console_video.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/sig_analog.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/sig_pri.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/channels/sig_ss7.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/funcs/func_audiohookinherit.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/funcs/func_channel.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/funcs/func_frame_trace.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/funcs/func_global.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/funcs/func_groupcount.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/funcs/func_lock.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/include/asterisk/channel.h <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/abstract_jb.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/aoc.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/app.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/autochan.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/bridging.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/ccss.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/cdr.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/cel.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/channel.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/channel_internal_api.c <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/main/cli.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/dial.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/dsp.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/features.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/file.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/indications.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/manager.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/pbx.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/rtp_engine.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/say.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/main/udptl.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/pbx/pbx_dundi.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/pbx/pbx_lua.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/pbx/pbx_realtime.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/res/res_adsi.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/res/res_agi.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/res/res_calendar.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/res/res_fax.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/res/res_jabber.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/res/res_monitor.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/res/res_musiconhold.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/res/res_mutestream.c <span style="color: grey">(349871)</span></li>

 <li>/trunk/res/snmp/agent.c <span style="color: grey">(349871)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/1655/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>