[asterisk-dev] [Code Review] ParkedCall can now be 'exten at parkinglot' instead of just 'exten'

[David Vossel]

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

Ship it!


Great work!  I'm confused on why this behavior didn't already exist :)


/trunk/main/features.c
<https://reviewboard.asterisk.org/r/1209/#comment7163>

    make this
    
    const char * 



/trunk/main/features.c
<https://reviewboard.asterisk.org/r/1209/#comment7164>

    sscanf is a preferred method of converting strings to integers, longs, etc.  It allows you to specify the signedness of a variable if that is important and also detect a failure.
    
    This works though.


- David


On 2011-05-06 09:12:12, jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1209/
> -----------------------------------------------------------
> 
> (Updated 2011-05-06 09:12:12)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This feature addition allows the ParkedCall application to include the name of a parkinglot so that the caller can use this application on a specific parkinglot rather than just jumping to whichever parkinglot is already bound to the requesting channel.
> 
> 
> This addresses bug 18777.
>     https://issues.asterisk.org/view.php?id=18777
> 
> 
> Diffs
> -----
> 
>   /trunk/CHANGES 317381 
>   /trunk/main/features.c 317381 
> 
> Diff: https://reviewboard.asterisk.org/r/1209/diff
> 
> 
> Testing
> -------
> 
> Tested with multiple parkinglots of various names to make sure it searches the right parkinglots (parkinglot names have to be used, not their context names).  Also tested to make sure it didn't interfere with the normal behavior with no specified parkinglot (it was segfaulting, for what was an obvious reason, so testing was useful).
> 
> 
> Thanks,
> 
> jrose
> 
>

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