diff mbox series

[FFmpeg-devel,06/11] avcodec/utils: Support APTX (HD) in av_get_audio_frame_duration()

Message ID AM7PR03MB66601CC2DD5BA0506E8DBA618FCA9@AM7PR03MB6660.eurprd03.prod.outlook.com
State Superseded
Headers show
Series [FFmpeg-devel,01/11] tests/fate-run: Allow multiple inputs for transcode() | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc fail Make fate failed

Commit Message

Andreas Rheinhardt Aug. 29, 2021, 9:27 p.m. UTC 
APTX decodes four bytes of input to four stereo samples; APTX HD
does the same with six bytes of input. So it can be easily supported
in av_get_audio_frame_duration().

This fixes invalid durations and (derived) timestamps of demuxed
APTX HD packets and therefore fixed the timestamp in the aptx-hd
FATE test.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavcodec/utils.c     |  4 ++++
 tests/ref/fate/aptx-hd | 18 +++++++++++-------
 2 files changed, 15 insertions(+), 7 deletions(-)
diff mbox series

Patch

diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index cfc07cbcb8..9090638385 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -678,6 +678,10 @@  static int get_audio_frame_duration(enum AVCodecID id, int sr, int ch, int ba,
             return 256 * (frame_bytes / 64);
         if (id == AV_CODEC_ID_RA_144)
             return 160 * (frame_bytes / 20);
+        if (id == AV_CODEC_ID_APTX)
+            return 4 * (frame_bytes / 4);
+        if (id == AV_CODEC_ID_APTX_HD)
+            return 4 * (frame_bytes / 6);
 
         if (bps > 0) {
             /* calc from frame_bytes and bits_per_coded_sample */
diff --git a/tests/ref/fate/aptx-hd b/tests/ref/fate/aptx-hd
index 825d0439b1..c924186ef8 100644
--- a/tests/ref/fate/aptx-hd
+++ b/tests/ref/fate/aptx-hd
@@ -7,10 +7,14 @@ 
 #channel_layout 0: 3
 #channel_layout_name 0: stereo
 0,          0,          0,     1024,     8192, 0x85791bdb
-0,       1536,       1536,     1024,     8192, 0x8db6dfa9
-0,       3072,       3072,     1024,     8192, 0x617b2190
-0,       4608,       4608,     1024,     8192, 0x733c4d71
-0,       6144,       6144,     1024,     8192, 0xb254420c
-0,       7680,       7680,     1024,     8192, 0xc166b305
-0,       9216,       9216,     1024,     8192, 0x93e8a20e
-0,      10752,      10752,     1024,     8192, 0xd91c6790
+0,       1024,       1024,     1024,     8192, 0x8db6dfa9
+0,       2048,       2048,     1024,     8192, 0x617b2190
+0,       3072,       3072,     1024,     8192, 0x733c4d71
+0,       4096,       4096,     1024,     8192, 0xb254420c
+0,       5120,       5120,     1024,     8192, 0xc166b305
+0,       6144,       6144,     1024,     8192, 0x93e8a20e
+0,       7168,       7168,     1024,     8192, 0xd91c6790
+0,       8192,       8192,     1024,     8192, 0xee6584ae
+0,       9216,       9216,     1024,     8192, 0x74a6246d
+0,      10240,      10240,     1024,     8192, 0xa20e6fb6
+0,      11264,      11264,      736,     5888, 0x06a6f058