[asterisk-dev] [Code Review] Bridging API for Conference Bridge purposes

Joshua Colp jcolp at digium.com
Fri Jan 30 06:45:24 CST 2009



> On 2009-01-29 16:05:49, Terry Wilson wrote:
> > /trunk/main/bridging.c, lines 256-257
> > <http://reviewboard.digium.com/r/93/diff/2/?file=2260#file2260line256>
> >
> >     Braces around if

Done.


> On 2009-01-29 16:05:49, Terry Wilson wrote:
> > /trunk/main/bridging.c, lines 270-274
> > <http://reviewboard.digium.com/r/93/diff/2/?file=2260#file2260line270>
> >
> >     Braces around if

Done.


> On 2009-01-29 16:05:49, Terry Wilson wrote:
> > /trunk/main/bridging.c, lines 277-278
> > <http://reviewboard.digium.com/r/93/diff/2/?file=2260#file2260line277>
> >
> >     More braces

Done.


> On 2009-01-29 16:05:49, Terry Wilson wrote:
> > /trunk/main/bridging.c, line 308
> > <http://reviewboard.digium.com/r/93/diff/2/?file=2260#file2260line308>
> >
> >     spaces around operator

Done.


> On 2009-01-29 16:05:49, Terry Wilson wrote:
> > /trunk/main/bridging.c, line 430
> > <http://reviewboard.digium.com/r/93/diff/2/?file=2260#file2260line430>
> >
> >     We can't possibly

Done.


> On 2009-01-29 16:05:49, Terry Wilson wrote:
> > /trunk/main/bridging.c, lines 1308-1310
> > <http://reviewboard.digium.com/r/93/diff/2/?file=2260#file2260line1308>
> >
> >     Are these hard coded feature codes?  Would they eventually be made configurable?

Yes, they will eventually be made configurable.


- Joshua


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


On 2008-12-31 15:45:07, Joshua Colp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/93/
> -----------------------------------------------------------
> 
> (Updated 2008-12-31 15:45:07)
> 
> 
> Review request for Asterisk Developers and Russell Bryant.
> 
> 
> Summary
> -------
> 
> This patch implements the new bridging API and brings with it a module for conference bridges. It does *not* replace existing internal bridging or features yet and will not cause any regressions when put in. It will essentially be introduced as a first test phase to work out any unforeseen critical issues. The bridging core itself is fully implemented besides the following: jitterbuffer support, native bridging, and interval hooks (hooks that are time based versus action based). If you would like an explanation of what the bridging API is made up of and how it works that can be found in the bridging.h header file.
> 
> 
> Diffs
> -----
> 
>   /trunk/Makefile 164597 
>   /trunk/apps/app_confbridge.c PRE-CREATION 
>   /trunk/bridges/Makefile PRE-CREATION 
>   /trunk/bridges/bridge_multiplexed.c PRE-CREATION 
>   /trunk/bridges/bridge_simple.c PRE-CREATION 
>   /trunk/bridges/bridge_softmix.c PRE-CREATION 
>   /trunk/channels/chan_bridge.c PRE-CREATION 
>   /trunk/include/asterisk/bridging.h PRE-CREATION 
>   /trunk/include/asterisk/channel.h 164597 
>   /trunk/main/Makefile 164597 
>   /trunk/main/bridging.c PRE-CREATION 
> 
> Diff: http://reviewboard.digium.com/r/93/diff
> 
> 
> Testing
> -------
> 
> Conference bridge testing using app_confbridge with features. Joining two channels with simple frame exchange and joining three channels to move it to a true conference bridge. IVR capability of app_confbridge was also tested.
> 
> 
> Thanks,
> 
> Joshua
> 
>




More information about the asterisk-dev mailing list