FreeCalypso > hg > gsm-codec-lib
diff miscutil/gsmrec-dump.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 | edd2e20e7090 |
| children | 3816ba89a5a0 |
line wrap: on
line diff
--- a/miscutil/gsmrec-dump.c Sun Dec 11 03:25:21 2022 +0000 +++ b/miscutil/gsmrec-dump.c Sun Dec 11 04:00:13 2022 +0000 @@ -52,12 +52,12 @@ EFR_frame2params(frame, params); n = 0; for (i = 0; i < 5; i++) - printf(" %u", params[n++]); + printf(" %d", params[n++]); putchar('\n'); for (i = 0; i < 4; i++) { putchar(' '); for (j = 0; j < 13; j++) - printf(" %u", params[n++]); + printf(" %d", params[n++]); putchar('\n'); } break; @@ -66,12 +66,12 @@ gsm_explode(dummy_state, frame, params); n = 0; for (i = 0; i < 8; i++) - printf(" %u", params[n++]); + printf(" %d", params[n++]); putchar('\n'); for (i = 0; i < 4; i++) { putchar(' '); for (j = 0; j < 17; j++) - printf(" %u", params[n++]); + printf(" %d", params[n++]); putchar('\n'); } break;