[asterisk-dev] [Code Review] 2785: Update config framework/sorcery to allow for options/types without documentation; update documentation for various modules
Matt Jordan
reviewboard at asterisk.org
Thu Aug 22 10:03:59 CDT 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2785/
-----------------------------------------------------------
Review request for Asterisk Developers.
Bugs: ASTERISK-22112 and ASTERISK-22359
https://issues.asterisk.org/jira/browse/ASTERISK-22112
https://issues.asterisk.org/jira/browse/ASTERISK-22359
Repository: Asterisk
Description
-------
There are times when a configuration option should not have documentation.
1. Some options are registered with a particular object merely as a warning to users. These options aren't even really 'deprecated', they are just provided by a different configuration file.
2. Some object types - most notably some used by modules that use sorcery - are completely internal and should never be shown to the user.
3. Sorcery itself has several 'hidden' fields that should never be shown to a user.
This patch updates the configuration framework and sorcery with additional API calls that allow a module to register types as internal and options as not requiring documentation. This bypasses the XML documentation checking.
This patch also re-enables the strict XML documentation checking in trunk, as well as updates some documentation that was missing.
Diffs
-----
/trunk/res/res_pjsip/pjsip_configuration.c 397363
/trunk/res/res_pjsip.c 397363
/trunk/main/sorcery.c 397363
/trunk/main/features_config.c 397363
/trunk/main/config_options.c 397363
/trunk/include/asterisk/sorcery.h 397363
/trunk/include/asterisk/config_options.h 397363
Diff: https://reviewboard.asterisk.org/r/2785/diff/
Testing
-------
Thanks,
Matt Jordan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130822/df075905/attachment-0001.htm>
More information about the asterisk-dev
mailing list