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

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


Alexander Traud has uploaded a new change for review.

  https://gerrit.asterisk.org/1779

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/79/1779/1

diff --git a/main/translate.c b/main/translate.c
index e95e06b..c9932cb 100644
--- a/main/translate.c
+++ b/main/translate.c
@@ -328,9 +328,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;
@@ -354,12 +351,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/1779
To unsubscribe, visit https://gerrit.asterisk.org/settings

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



More information about the asterisk-code-review mailing list