[asterisk-dev] [Code Review] 2942: Give parking timeout comebacktoorigin channel DTMF features.

[Mark Michelson]

> On Oct. 21, 2013, 3:41 p.m., Mark Michelson wrote:
> > /branches/12/res/parking/parking_bridge_features.c, line 467
> > <https://reviewboard.asterisk.org/r/2942/diff/1/?file=47305#file47305line467>
> >
> >     Err on the safe side and make this buffer larger.
> 
> rmudgett wrote:
>     Safe side of what?  Five is the maximum the array needs to be.  Larger is totally unnecessary.

I'm picturing a potential future modification where a well-intentioned but misguided developer adds a new option to be applied but doesn't bump the size of the array up any, leading to disastrous results. While it's nice to declare the array to be exactly the right size necessary to apply all the current options, it doesn't hurt to throw a few more bytes on the array declaration.


- Mark


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


On Oct. 18, 2013, 11:30 p.m., rmudgett wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2942/
> -----------------------------------------------------------
> 
> (Updated Oct. 18, 2013, 11:30 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-22630
>     https://issues.asterisk.org/jira/browse/ASTERISK-22630
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> Parking timeouts did not set any DTMF features for the channel calling the parker back.
> 
> * Added code to set the parkedcalltransfers, parkedcallreparking, parkedcallhangup, and parkedcallrecording options appropriately for the channels when a parking timeout occurs.  The recall channel DTMF options are set using the BRIDGE_FEATURES channel variable to allow the other timeout options to have the DTMF features available.
> 
> * Updated the configuration option XML documentation to indicate that the parkedcalltransfers, parkedcallreparking, parkedcallhangup, and parkedcallrecording options also apply to parking timeouts.
> 
> 
> Diffs
> -----
> 
>   /branches/12/res/parking/parking_bridge.c 401272 
>   /branches/12/res/parking/parking_bridge_features.c 401272 
>   /branches/12/res/res_parking.c 401272 
> 
> Diff: https://reviewboard.asterisk.org/r/2942/diff/
> 
> 
> Testing
> -------
> 
> Tested with comebacktoorigin=yes and parkedcallreparking=caller configured in the parking lot.
> Had A call B with the tTkK Dial option flags.
> Made A park B.
> When the parking timeout happens, A is called back and has the DTMF one-touch-parking feature enabled but does not have transfer ability.
> Without the patch A has no DTMF features.
> 
> 
> Thanks,
> 
> rmudgett
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20131021/7b481a49/attachment.html>