[Asterisk-code-review] loader: Convert reload classes to built-in modules. (asterisk[master])

Corey Farrell asteriskteam at digium.com
Wed Mar 14 04:21:03 CDT 2018


Hello Richard Mudgett, Jenkins2, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/8390

to look at the new patch set (#5).

Change subject: loader: Convert reload_classes to built-in modules.
......................................................................

loader: Convert reload_classes to built-in modules.

* acl (named_acl.c)
* cdr
* cel
* ccss
* dnsmgr
* dsp
* enum
* extconfig (config.c)
* features
* http
* indications
* logger
* manager
* plc
* sounds
* udptl

These modules are now loaded at appropriate time by the module loader.
Unlike loadable modules these use AST_MODULE_LOAD_FAILURE on error so
the module loader will abort startup on failure of these modules.

Some of these modules are still initialized or shutdown from outside the
module loader.  logger.c is initialized very early and shutdown very
late, manager.c is initialized by the module loader but is shutdown by
the Asterisk core (too much uses it without holding references).

Change-Id: I371a9a45064f20026c492623ea8062d02a1ab97f
---
M addons/cdr_mysql.c
M addons/chan_ooh323.c
M addons/res_config_mysql.c
M apps/app_celgenuserevent.c
M apps/app_dial.c
M apps/app_forkcdr.c
M cdr/cdr_adaptive_odbc.c
M cdr/cdr_beanstalkd.c
M cdr/cdr_csv.c
M cdr/cdr_custom.c
M cdr/cdr_manager.c
M cdr/cdr_odbc.c
M cdr/cdr_pgsql.c
M cdr/cdr_radius.c
M cdr/cdr_sqlite.c
M cdr/cdr_sqlite3_custom.c
M cdr/cdr_syslog.c
M cdr/cdr_tds.c
M cel/cel_beanstalkd.c
M cel/cel_custom.c
M cel/cel_manager.c
M cel/cel_odbc.c
M cel/cel_pgsql.c
M cel/cel_radius.c
M cel/cel_sqlite3_custom.c
M cel/cel_tds.c
M channels/chan_dahdi.c
M channels/chan_iax2.c
M channels/chan_sip.c
M funcs/func_callcompletion.c
M funcs/func_cdr.c
M funcs/func_enum.c
M include/asterisk/_private.h
M include/asterisk/acl.h
M include/asterisk/ccss.h
M include/asterisk/cdr.h
M include/asterisk/config.h
M include/asterisk/dsp.h
M include/asterisk/enum.h
M include/asterisk/features_config.h
M include/asterisk/logger.h
M include/asterisk/manager.h
M include/asterisk/module.h
M include/asterisk/sounds_index.h
M include/asterisk/udptl.h
M main/Makefile
M main/asterisk.c
M main/ccss.c
M main/cdr.c
M main/cel.c
M main/channel.c
M main/config.c
M main/dnsmgr.c
M main/dsp.c
M main/enum.c
M main/features.c
M main/features_config.c
A main/features_config.h
M main/http.c
M main/indications.c
M main/loader.c
M main/logger.c
M main/manager.c
M main/named_acl.c
M main/plc.c
M main/sounds.c
M main/udptl.c
M res/res_ari.c
M res/res_config_curl.c
M res/res_config_ldap.c
M res/res_config_odbc.c
M res/res_config_pgsql.c
M res/res_config_sqlite.c
M res/res_config_sqlite3.c
M res/res_http_post.c
M res/res_http_websocket.c
M res/res_phoneprov.c
M res/res_pjsip.c
M res/res_pjsip_t38.c
79 files changed, 526 insertions(+), 423 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/90/8390/5
-- 
To view, visit https://gerrit.asterisk.org/8390
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I371a9a45064f20026c492623ea8062d02a1ab97f
Gerrit-Change-Number: 8390
Gerrit-PatchSet: 5
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180314/5544f16e/attachment-0001.html>


More information about the asterisk-code-review mailing list