[asterisk-dev] [Code Review] Multicast RTP Paging Support

Russell Bryant russell at digium.com
Tue Jun 2 12:54:14 CDT 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/270/#review818
-----------------------------------------------------------



/trunk/channels/chan_multicast_rtp.c
<http://reviewboard.digium.com/r/270/#comment2009>

    Don't forget to set AF_INET.



/trunk/channels/chan_multicast_rtp.c
<http://reviewboard.digium.com/r/270/#comment2010>

    Probably LOAD_DECLINE ..



/trunk/res/res_rtp_multicast.c
<http://reviewboard.digium.com/r/270/#comment2011>

    You can just use structure assignment here.



/trunk/res/res_rtp_multicast.c
<http://reviewboard.digium.com/r/270/#comment2012>

    Space after casts.
    
    Also, if this fails, it would be good to log the strerror(errno).



/trunk/res/res_rtp_multicast.c
<http://reviewboard.digium.com/r/270/#comment2013>

    Isn't this equivalent to the STANDARD macro?


- Russell


On 2009-06-01 08:07:07, Joshua Colp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/270/
> -----------------------------------------------------------
> 
> (Updated 2009-06-01 08:07:07)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This patch implements two new modules: res_rtp_multicast and chan_multicast_rtp. The resource module is an RTP engine which can be used by any developer to send multicast RTP. The channel driver uses the RTP engine and configures it based on input given from the user in the Dial line. Any audio sent to the RTP engine is broadcast out.
> 
> 
> This addresses bug 11797.
>     https://issues.asterisk.org/view.php?id=11797
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/chan_multicast_rtp.c PRE-CREATION 
>   /trunk/res/res_rtp_multicast.c PRE-CREATION 
> 
> Diff: http://reviewboard.digium.com/r/270/diff
> 
> 
> Testing
> -------
> 
> Confirmed that audio is sent out as expected by dialing using the channel driver in the dialplan.
> 
> 
> Thanks,
> 
> Joshua
> 
>




More information about the asterisk-dev mailing list