[asterisk-dev] [Code Review] 2801: feature_config: Read and ignore parkinglots

[Matt Jordan]

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



/branches/12/main/features_config.c
<https://reviewboard.asterisk.org/r/2801/#comment18671>

    You really don't want any documentation for this.
    
    If you register the option as "_nodoc", I *think* it should handle the fact that neither the option nor the object should have documentation. If not, let me know and I'll help you fix it.



/branches/12/main/features_config.c
<https://reviewboard.asterisk.org/r/2801/#comment18669>

    I think a doxygen comment noting that this object type only exists to warn the user that it is no longer supported in features.conf is warranted.



/branches/12/main/features_config.c
<https://reviewboard.asterisk.org/r/2801/#comment18670>

    Register this with _nodoc (see the other unsupported options)


- Matt Jordan


On Aug. 28, 2013, 11:13 p.m., jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2801/
> -----------------------------------------------------------
> 
> (Updated Aug. 28, 2013, 11:13 p.m.)
> 
> 
> Review request for Asterisk Developers, kmoore, Matt Jordan, and Mark Michelson.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> Presence of a parkinglot in features.conf will currently cause Asterisk to fail to load due to the presence of an unrecognized configuration option. That's less than ideal and it makes having configurations across multiple versions of Asterisk difficult (which has impacted an automated parking test). This patch addresses that by reading in parkinglots and issuing a warning as they are created telling the user that parkinglots should no longer be configured in this module.
> 
> 
> Diffs
> -----
> 
>   /branches/12/main/config_options.c 397880 
>   /branches/12/main/features_config.c 397880 
> 
> Diff: https://reviewboard.asterisk.org/r/2801/diff/
> 
> 
> Testing
> -------
> 
> Tested loads and reloads with difference in the number and properties of parkinglots. Tested to make sure there was no impact on memory usage once the loads/reloads were finished.
> 
> 
> Thanks,
> 
> jrose
> 
>

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