[Asterisk-code-review] app meetme: Set default value for audio buffers. (asterisk[11])

Joshua Colp asteriskteam at digium.com
Thu Dec 10 06:03:19 CST 2015


Joshua Colp has submitted this change and it was merged.

Change subject: app_meetme: Set default value for audio_buffers.
......................................................................


app_meetme: Set default value for audio_buffers.

The default value was never set for audio_buffers, causing bad
audio quality.  This ensures the default is always set.

ASTERISK-25569 #close

Change-Id: I2d2ee3e644120b0f9f6ea6ab9286d7d590942a44
---
M apps/app_meetme.c
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved



diff --git a/apps/app_meetme.c b/apps/app_meetme.c
index 2d8d556..ec2498c 100644
--- a/apps/app_meetme.c
+++ b/apps/app_meetme.c
@@ -5582,6 +5582,9 @@
 
 	/*  Logging of participants defaults to ON for compatibility reasons */
 	rt_log_members = 1;
+
+	/* Set default number of buffers to be allocated. */
+	audio_buffers = DEFAULT_AUDIO_BUFFERS;
 }
 
 static void load_config_meetme(int reload)

-- 
To view, visit https://gerrit.asterisk.org/1793
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2d2ee3e644120b0f9f6ea6ab9286d7d590942a44
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list