minor code optimizations to reduce the number of times that the ast_frame

and ast_trans_pvt pointers have to be dereferenced (issue #7069, Mithraen)


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@25059 65c4cc65-6c06-0410-ace0-fbb531ad65f3

codecs/codec_a_mu.c

@@ -55,28 +55,30 @@ static unsigned char a2mu[256];
 /*! \brief convert frame data and store into the buffer */
 static int alawtoulaw_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 {
-	int x;
+	int i;
+	int in_samples = f->samples;
 	unsigned char *src = f->data;
 	unsigned char *dst = (unsigned char *)pvt->outbuf + pvt->samples;
 
-	for ( x = 0 ; x < f->samples; x++)
-		dst[x] = a2mu[src[x]];
-	pvt->samples += f->samples;
-	pvt->datalen += f->datalen;
+	for (i = 0; i < in_samples; i++)
+		*dst++ = a2mu[*src++];
+	pvt->samples += in_samples;
+	pvt->datalen += in_samples;
 	return 0;
 }
 
 /*! \brief convert frame data and store into the buffer */
 static int ulawtoalaw_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 {
-	int x;
+	int i;
+	int in_samples = f->samples;
 	unsigned char *src = f->data;
 	unsigned char *dst = (unsigned char *)pvt->outbuf + pvt->samples;
 
-	for ( x = 0 ; x < f->samples; x++)
-		dst[x] = mu2a[src[x]];
-	pvt->samples += f->samples;
-	pvt->datalen += f->datalen;
+	for (i = 0; i < in_samples; i++)
+		*dst++ = mu2a[*src++];
+	pvt->samples += in_samples;
+	pvt->datalen += in_samples;
 	return 0;
 }
 

codecs/codec_alaw.c

@@ -57,25 +57,29 @@ static int alawtolin_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 	int i;
 	unsigned char *src = f->data;
 	int16_t *dst = (int16_t *)pvt->outbuf;
-
-	for ( i = 0; i < f->samples; i++)
-		dst[pvt->samples + i] = AST_ALAW(src[i]);
-	pvt->samples += f->samples;
-	pvt->datalen += 2*f->samples;	/* 2 bytes/sample */
+	int in_samples = f->samples;
+	int out_samples = pvt->samples;
+	
+	for (i = 0; i < in_samples; i++)
+		dst[out_samples++] = AST_ALAW(src[i]);
+
+	pvt->samples = out_samples;
+	pvt->datalen += in_samples * 2;	/* 2 bytes/sample */
 	return 0;
 }
 
 /*! \brief convert and store input samples in output buffer */
 static int lintoalaw_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 {
-	int x;
+	int i;
 	char *dst = pvt->outbuf + pvt->samples;
 	int16_t *src = f->data;
+	int in_samples = f->samples;
 
-	for ( x = 0; x < f->samples; x++) 
-		*dst++ = AST_LIN2A(src[x]);
-	pvt->samples += f->samples;
-	pvt->datalen += f->samples;	/* 1 byte/sample */
+	for (i = 0; i < in_samples; i++) 
+		*dst++ = AST_LIN2A(src[i]);
+	pvt->samples += in_samples;
+	pvt->datalen += in_samples;	/* 1 byte/sample */
 	return 0;
 }
 

codecs/codec_ulaw.c

@@ -57,13 +57,15 @@ static int ulawtolin_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 	int i;
 	unsigned char *src = f->data;
 	int16_t *dst = (int16_t *)pvt->outbuf;
+	int in_samples = f->samples;
+	int out_samples = pvt->samples;
 
 	/* convert and copy in outbuf */
-	for (i=0;  i<f->samples; i++)
-		dst[pvt->samples + i] = AST_MULAW(src[i]);
+	for (i = 0; i < in_samples; i++)
+		dst[out_samples++] = AST_MULAW(src[i]);
 
-	pvt->samples += f->samples;
-	pvt->datalen += 2 * f->samples;
+	pvt->samples = out_samples;
+	pvt->datalen += in_samples * 2;	/* 2 bytes/sample */
 	return 0;
 }
 
@@ -71,12 +73,14 @@ static int ulawtolin_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 static int lintoulaw_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 {
 	int i;
+	char *dst = pvt->outbuf + pvt->samples;
 	int16_t *src = f->data;
+	int in_samples = f->samples;
 
-	for (i=0 ; i < f->samples; i++) 
-		pvt->outbuf[pvt->samples + i] = AST_LIN2MU(src[i]);
-	pvt->samples += f->samples;
-	pvt->datalen += f->samples;	/* 1 byte/sample */
+	for (i = 0; i < in_samples; i++) 
+		*dst++ = AST_LIN2MU(src[i]);
+	pvt->samples += in_samples;
+	pvt->datalen += in_samples;	/* 1 byte/sample */
 	return 0;
 }
 
@@ -147,7 +151,7 @@ static void parse_config(void)
 {
 	struct ast_variable *var;
 	struct ast_config *cfg = ast_config_load("codecs.conf");
-	if (cfg == NULL)
+	if (!cfg)
 		return;
 	for (var = ast_variable_browse(cfg, "plc"); var; var = var->next) {
 		if (!strcasecmp(var->name, "genericplc")) {