FreeCalypso > hg > gsm-codec-lib
view libgsmfr2/pp_good.c @ 477:4c9222d95647
libtwamr encoder: always emit frame->mode = mode;
In the original implementation of amr_encode_frame(), the 'mode' member
of the output struct was set to 0xFF if the output frame type is TX_NO_DATA.
This design was made to mimic the mode field (16-bit word) being set to
0xFFFF (or -1) in 3GPP test sequence format - but nothing actually depends
on this struct member being set in any way, and amr_frame_to_tseq()
generates the needed 0xFFFF on its own, based on frame->type being equal
to TX_NO_DATA.
It is simpler and more efficient to always set frame->mode to the actual
encoding mode in amr_encode_frame(), and this new behavior has already
been documented in doc/AMR-library-API description in anticipation of
the present change.
author | Mychaela Falconia <falcon@freecalypso.org> |
---|---|
date | Sat, 18 May 2024 22:30:42 +0000 |
parents | 573afa985df6 |
children | 96c4ed5529bf |
line wrap: on
line source
/* * In this module we implement preprocessing of received good frames. */ #include <stdint.h> #include <string.h> #include "tw_gsmfr.h" #include "pp_state.h" #include "pp_internal.h" void gsmfr_preproc_good_frame(struct gsmfr_preproc_state *st, uint8_t *frame) { int sid; /* always set correct magic */ frame[0] = 0xD0 | frame[0] & 0x0F; /* now classify by SID */ sid = gsmfr_preproc_sid_classify(frame); switch (sid) { case 0: /* good speech frame */ st->rx_state = SPEECH; memcpy(&st->speech_frame, frame, GSMFR_RTP_FRAME_LEN); return; case 1: /* invalid SID frame */ /* state-dependent handling, similar to BFI */ gsmfr_preproc_invalid_sid(st, frame); return; case 2: /* valid SID frame */ st->rx_state = COMFORT_NOISE; gsmfr_preproc_sid2cn(st, frame); return; } }