FreeCalypso > hg > gsm-codec-lib
annotate libtwamr/dhf_subst2.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 | fa0652115168 |
| children |
| rev | line source |
|---|---|
|
464
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
1 /* |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
2 * The function implemented in this module is an aid for AMR-EFR interworking; |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
3 * it is meant to be invoked directly after amr_encode_frame(). It checks |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
4 * the encoder output for MR122 DHF and turns it into EFR DHF just like |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
5 * amr_dhf_subst_efr(), but only if the input PCM frame was an EHF. This |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
6 * slightly more complicated logic is needed in order to replicate the observed |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
7 * behavior of AMR-EFR speech encoder in the extant GSM network of T-Mobile USA. |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
8 */ |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
9 |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
10 #include "tw_amr.h" |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
11 #include "namespace.h" |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
12 #include "e_homing.h" |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
13 |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
14 void amr_dhf_subst_efr2(struct amr_param_frame *frame, const int16_t *pcm) |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
15 { |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
16 if (encoder_homing_frame_test(pcm)) |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
17 amr_dhf_subst_efr(frame); |
|
fa0652115168
libtwamr: add amr_dhf_subst_efr2()
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff
changeset
|
18 } |
