[Asterisk-code-review] modules.conf: Align the comments for more conclusiveness. (asterisk[master])

[Alexander Traud]

Alexander Traud has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/15128 )


Change subject: modules.conf: Align the comments for more conclusiveness.
......................................................................

modules.conf: Align the comments for more conclusiveness.

Change-Id: I79cc693cd5a6e5dd7d403b7e91d970ff1ddf8306
---
M configs/basic-pbx/modules.conf
M configs/samples/modules.conf.sample
2 files changed, 17 insertions(+), 12 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/28/15128/1

diff --git a/configs/basic-pbx/modules.conf b/configs/basic-pbx/modules.conf
index be35e28..cb0a92c 100644
--- a/configs/basic-pbx/modules.conf
+++ b/configs/basic-pbx/modules.conf
@@ -1,8 +1,9 @@
 [modules]
 autoload = no
 
-; This is a minimal module load. We are loading only the modules required for
-; the Asterisk features used in the Super Awesome Company configuration.
+; This is a minimal module load. We are loading only the modules
+; required for the Asterisk features used in the "Super Awesome
+; Company" configuration.
 
 ; Applications
 
@@ -110,7 +111,8 @@
 load = res_sorcery_realtime.so
 load = res_timing_timerfd.so
 
-; Don't load res_hep.so and kin unless you are using hep monitoring in your network
+; Do not load res_hep and kin unless you are using HEP monitoring
+; <http://sipcapture.org> in your network.
 
 noload = res_hep.so
 noload = res_hep_pjsip.so
diff --git a/configs/samples/modules.conf.sample b/configs/samples/modules.conf.sample
index 134ab89..d2291ae 100644
--- a/configs/samples/modules.conf.sample
+++ b/configs/samples/modules.conf.sample
@@ -24,7 +24,7 @@
 ; the "require" keyword. Asterisk will exit with a status code of 2
 ; if a required module does not load.
 ;
-; require = chan_pjsip.so
+;require = chan_pjsip.so
 ;
 ; If you want you can combine with preload
 ; preload-require = your_special_module.so
@@ -35,19 +35,22 @@
 ; By default, load chan_oss only (automatically).
 ;
 noload => chan_alsa.so
-;noload => chan_oss.so
 noload => chan_console.so
-
+;noload => chan_oss.so
+;
+; Do not load res_hep and kin unless you are using HEP monitoring
+; <http://sipcapture.org> in your network.
+;
 noload => res_hep.so
 noload => res_hep_pjsip.so
 noload => res_hep_rtcp.so
-
+;
 ; Do not load chan_sip by default, it may conflict with res_pjsip.
 noload => chan_sip.so
-
-; The default voicemail module is app_voicemal. All voicemail modules
-; are mutually exclusive. Therefore it is better to make sure they
-; are not loaded at startup
 ;
-noload => app_voicemail_odbc.so
+; Load one of the voicemail modules as they are mutually exclusive.
+; By default, load app_voicemail only (automatically).
+;
+;noload => app_voicemail.so
 noload => app_voicemail_imap.so
+noload => app_voicemail_odbc.so

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/15128
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I79cc693cd5a6e5dd7d403b7e91d970ff1ddf8306
Gerrit-Change-Number: 15128
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20201103/7ad1fc09/attachment-0001.html>