[asterisk-dev] [Code Review] 2633: Filter channels used as internal mechanisms

[Matt Jordan]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2633/#review8968
-----------------------------------------------------------



trunk/apps/confbridge/conf_chan_announce.c
<https://reviewboard.asterisk.org/r/2633/#comment17647>

    While we're here...
    
    We should probably rename this to be more reflective of its actual purpose. Yes, ConfBridge is what uses it currently, but other things will as well.
    
    How about just "Bridge Announcer Channel"?



trunk/apps/confbridge/conf_chan_record.c
<https://reviewboard.asterisk.org/r/2633/#comment17648>

    Same item here



trunk/main/manager_bridging.c
<https://reviewboard.asterisk.org/r/2633/#comment17650>

    For cleanliness, stick an extra CR between the variable declarations and the code.



trunk/main/manager_bridging.c
<https://reviewboard.asterisk.org/r/2633/#comment17649>

    For cleanliness, stick an extra CR between the variable declarations and the code.


- Matt Jordan


On June 21, 2013, 12:56 p.m., opticron wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2633/
> -----------------------------------------------------------
> 
> (Updated June 21, 2013, 12:56 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-21873
>     https://issues.asterisk.org/jira/browse/ASTERISK-21873
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This adds a new function pointer to the channel tech which determines whether a channel is fit for external consumption. Examples of channels that would not be fit for external consumption include the announcement and recording channels used by confbridge which are the only two marked as such by this patch.
> 
> A lot of this ended up being cleanup to ensure callers of various functions could handle a NULL return.
> 
> 
> Diffs
> -----
> 
>   trunk/apps/confbridge/conf_chan_announce.c 392407 
>   trunk/apps/confbridge/conf_chan_record.c 392407 
>   trunk/apps/confbridge/confbridge_manager.c 392407 
>   trunk/include/asterisk/channel.h 392407 
>   trunk/include/asterisk/stasis_channels.h 392407 
>   trunk/main/cdr.c 392407 
>   trunk/main/cel.c 392407 
>   trunk/main/manager_bridging.c 392407 
>   trunk/main/manager_channels.c 392407 
>   trunk/main/stasis_channels.c 392407 
>   trunk/res/parking/parking_manager.c 392407 
> 
> Diff: https://reviewboard.asterisk.org/r/2633/diff/
> 
> 
> Testing
> -------
> 
> Ran through CDR unit tests.
> 
> 
> Thanks,
> 
> opticron
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130625/7d17e9e4/attachment-0001.htm>