[Asterisk-code-review] Remove redundant module checks and references. (asterisk[master])

Corey Farrell asteriskteam at digium.com
Thu Jan 18 08:27:33 CST 2018


Hello Richard Mudgett, Jenkins2, 

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

    https://gerrit.asterisk.org/7989

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

Change subject: Remove redundant module checks and references.
......................................................................

Remove redundant module checks and references.

This removes references that are no longer needed due to automatic
references created by module dependencies.

In addition this removes most calls to ast_module_check as they were
checking modules which are listed as dependencies.

Change-Id: I332a6e8383d4c72c8e89d988a184ab8320c4872e
---
M apps/app_confbridge.c
M apps/app_meetme.c
M apps/app_stack.c
M apps/app_stasis.c
M channels/chan_pjsip.c
M funcs/func_curl.c
M funcs/func_periodic_hook.c
M include/asterisk/agi.h
M include/asterisk/ari.h
M include/asterisk/res_mwi_external.h
M include/asterisk/res_pjproject.h
M include/asterisk/res_pjsip.h
M include/asterisk/res_pjsip_pubsub.h
M include/asterisk/res_pjsip_session.h
M include/asterisk/stasis_app.h
M res/res_agi.c
M res/res_ari.c
M res/res_ari_applications.c
M res/res_ari_asterisk.c
M res/res_ari_bridges.c
M res/res_ari_channels.c
M res/res_ari_device_states.c
M res/res_ari_endpoints.c
M res/res_ari_events.c
M res/res_ari_mailboxes.c
M res/res_ari_playbacks.c
M res/res_ari_recordings.c
M res/res_ari_sounds.c
M res/res_config_curl.c
M res/res_crypto.c
M res/res_curl.c
M res/res_fax.c
M res/res_hep_pjsip.c
M res/res_hep_rtcp.c
M res/res_http_media_cache.c
M res/res_http_websocket.c
M res/res_monitor.c
M res/res_mwi_external.c
M res/res_mwi_external_ami.c
M res/res_pjproject.c
M res/res_pjsip.c
M res/res_pjsip/config_auth.c
M res/res_pjsip/config_transport.c
M res/res_pjsip/include/res_pjsip_private.h
M res/res_pjsip/location.c
M res/res_pjsip/pjsip_distributor.c
M res/res_pjsip/pjsip_global_headers.c
M res/res_pjsip/pjsip_message_filter.c
M res/res_pjsip/pjsip_options.c
M res/res_pjsip/pjsip_session.c
M res/res_pjsip_acl.c
M res/res_pjsip_authenticator_digest.c
M res/res_pjsip_caller_id.c
M res/res_pjsip_dialog_info_body_generator.c
M res/res_pjsip_diversion.c
M res/res_pjsip_dlg_options.c
M res/res_pjsip_dtmf_info.c
M res/res_pjsip_empty_info.c
M res/res_pjsip_endpoint_identifier_anonymous.c
M res/res_pjsip_endpoint_identifier_ip.c
M res/res_pjsip_endpoint_identifier_user.c
M res/res_pjsip_exten_state.c
M res/res_pjsip_header_funcs.c
M res/res_pjsip_history.c
M res/res_pjsip_logger.c
M res/res_pjsip_messaging.c
M res/res_pjsip_mwi.c
M res/res_pjsip_mwi_body_generator.c
M res/res_pjsip_nat.c
M res/res_pjsip_notify.c
M res/res_pjsip_one_touch_record_info.c
M res/res_pjsip_outbound_authenticator_digest.c
M res/res_pjsip_outbound_publish.c
M res/res_pjsip_outbound_registration.c
M res/res_pjsip_path.c
M res/res_pjsip_phoneprov_provider.c
M res/res_pjsip_pidf_body_generator.c
M res/res_pjsip_pidf_digium_body_supplement.c
M res/res_pjsip_pidf_eyebeam_body_supplement.c
M res/res_pjsip_publish_asterisk.c
M res/res_pjsip_pubsub.c
M res/res_pjsip_refer.c
M res/res_pjsip_registrar.c
M res/res_pjsip_registrar_expire.c
M res/res_pjsip_rfc3326.c
M res/res_pjsip_sdp_rtp.c
M res/res_pjsip_send_to_voicemail.c
M res/res_pjsip_session.c
M res/res_pjsip_sips_contact.c
M res/res_pjsip_t38.c
M res/res_pjsip_transport_management.c
M res/res_pjsip_transport_websocket.c
M res/res_pjsip_xpidf_body_generator.c
M res/res_pktccops.c
M res/res_smdi.c
M res/res_stasis.c
M res/res_stasis_mailbox.c
M res/res_statsd.c
M res/stasis/app.c
M rest-api-templates/res_ari_resource.c.mustache
M tests/test_res_pjsip_scheduler.c
M tests/test_res_stasis.c
M tests/test_utils.c
103 files changed, 135 insertions(+), 752 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/89/7989/4
-- 
To view, visit https://gerrit.asterisk.org/7989
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I332a6e8383d4c72c8e89d988a184ab8320c4872e
Gerrit-Change-Number: 7989
Gerrit-PatchSet: 4
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/20180118/b4cf4926/attachment-0001.html>


More information about the asterisk-code-review mailing list