[Asterisk-code-review] translate: Avoid a warning message when doing FEC within Opu... (asterisk[13])

Alexander Traud asteriskteam at digium.com
Tue Dec 8 03:51:18 CST 2015


Alexander Traud has uploaded a new change for review.

  https://gerrit.asterisk.org/1778

Change subject: translate: Avoid a warning message when doing FEC within Opus Codec.
......................................................................

translate: Avoid a warning message when doing FEC within Opus Codec.

ASTERISK-25616 #close

Change-Id: Ibe729aaf2e6e25506cff247cec5149ec1e589319
---
M main/translate.c
1 file changed, 1 insertion(+), 9 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/78/1778/1

diff --git a/main/translate.c b/main/translate.c
index 44cfa2b..6a39bac 100644
--- a/main/translate.c
+++ b/main/translate.c
@@ -397,9 +397,6 @@
 /*! \brief framein wrapper, deals with bound checks.  */
 static int framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 {
-	int ret;
-	int samples = pvt->samples;	/* initial value */
-
 	/* Copy the last in jb timing info to the pvt */
 	ast_copy_flags(&pvt->f, f, AST_FRFLAG_HAS_TIMING_INFO);
 	pvt->f.ts = f->ts;
@@ -423,12 +420,7 @@
 	/* we require a framein routine, wouldn't know how to do
 	 * it otherwise.
 	 */
-	ret = pvt->t->framein(pvt, f);
-	/* diagnostic ... */
-	if (pvt->samples == samples)
-		ast_log(LOG_WARNING, "%s did not update samples %d\n",
-			pvt->t->name, pvt->samples);
-	return ret;
+	return pvt->t->framein(pvt, f);
 }
 
 /*! \brief generic frameout routine.

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibe729aaf2e6e25506cff247cec5149ec1e589319
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>



More information about the asterisk-code-review mailing list