diff frtest/max-out.c @ 128:a5ffec18e4cd

test programs: use printf %d format for codec parameters Even though all codec params (both FR and EFR) are small unsigned integers, we use signed int16_t data type for both, for interface reasons: in the case of FR it's the gsm_signal type of libgsm, and in the case of EFR it's the Word16 type of ETSI codec guts. Therefore, the correct printf format is %d, not %u, when the objective is to see what's in the variables (what the compiler sees) and catch any bugs.
author Mychaela Falconia <falcon@freecalypso.org>
date Sun, 11 Dec 2022 04:00:13 +0000
parents a8fd4ff6b013
children
line wrap: on
line diff
--- a/frtest/max-out.c	Sun Dec 11 03:25:21 2022 +0000
+++ b/frtest/max-out.c	Sun Dec 11 04:00:13 2022 +0000
@@ -75,12 +75,12 @@
 			gsm_explode(dec_state, frame, fr_params);
 			n = 0;
 			for (i = 0; i < 8; i++)
-				printf(" %u", fr_params[n++]);
+				printf(" %d", fr_params[n++]);
 			putchar('\n');
 			for (i = 0; i < 4; i++) {
 				putchar(' ');
 				for (j = 0; j < 17; j++)
-					printf(" %u", fr_params[n++]);
+					printf(" %d", fr_params[n++]);
 				putchar('\n');
 			}
 			break;
@@ -97,12 +97,12 @@
 		gsm_explode(dec_state, frame, fr_params);
 		n = 0;
 		for (i = 0; i < 8; i++)
-			printf(" %u", fr_params[n++]);
+			printf(" %d", fr_params[n++]);
 		putchar('\n');
 		for (i = 0; i < 4; i++) {
 			putchar(' ');
 			for (j = 0; j < 17; j++)
-				printf(" %u", fr_params[n++]);
+				printf(" %d", fr_params[n++]);
 			putchar('\n');
 		}
 		gsm_decode(dec_state, frame, pcm);