[asterisk-dev] [Code Review] 2546: res_parking: Default Parking Lot

[Paul Belanger]

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



/team/jrose/bridge_projects/res/res_parking.c
<https://reviewboard.asterisk.org/r/2546/#comment16829>

    Any way we cannot do this?  Always been a peeve of mine, that asterisk defaults to an extension number. Seems to be the only place in asterisk were we do this.
    
    I'd rather see the user has to explicitly define the extension number vs asterisk just assigning something.


- Paul Belanger


On May 15, 2013, 10:02 p.m., jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2546/
> -----------------------------------------------------------
> 
> (Updated May 15, 2013, 10:02 p.m.)
> 
> 
> Review request for Asterisk Developers, Matt Jordan and rmudgett.
> 
> 
> Bugs: ASTERISK-21643
>     https://issues.asterisk.org/jira/browse/ASTERISK-21643
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> Default Parking Lot. Pretty self explanatory. It's a parking lot.  It's named default.  It uses default options.
> 
> Actually, there is one difference between this parking lot and what you would get if you just stuck
> [default]
> 
> in the config file, and that's that if the default parking lot isn't specified in this way, it will set parkext to 700 so that dialplan generation
> will be used and an actual usable parking lot similar to the ones that used to be generated by features.conf will be set up.
> 
> 
> Diffs
> -----
> 
>   /team/jrose/bridge_projects/res/res_parking.c 388603 
> 
> Diff: https://reviewboard.asterisk.org/r/2546/diff/
> 
> 
> Testing
> -------
> 
> Tested with and without the presence of [default] in res_parking.conf
> Tested reloads removing and adding [default]
> Checked memory consumption before and after reloads for both adding and removing default.
> Checked options in the default parking lot created without the presence of [default]
> Checked options in the default parking lot created with the presence of [default]
> 
> 
> Thanks,
> 
> jrose
> 
>

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