diff mbox series

[FFmpeg-devel,v2,07/10] avformat/aptxdec: Don't set AVCodecParameters.frame_size

Message ID AM7PR03MB6660E8E225B24A4715983D0B8FCC9@AM7PR03MB6660.eurprd03.prod.outlook.com
State New
Headers show
Series [FFmpeg-devel,v2,01/10] 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 success Make fate finished

Commit Message

Andreas Rheinhardt Aug. 31, 2021, 12:42 p.m. UTC 
This field was misunderstood: It gives the number of samples
in a packet, not the number of bytes. Its usage was wrong for APTX HD.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavformat/aptxdec.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Paul B Mahol Sept. 1, 2021, 6:32 a.m. UTC | #1 
lgtm
diff mbox series

Patch

diff --git a/libavformat/aptxdec.c b/libavformat/aptxdec.c
index 7153e262a1..a5ed921e2c 100644
--- a/libavformat/aptxdec.c
+++ b/libavformat/aptxdec.c
@@ -56,7 +56,6 @@  static int aptx_read_header(AVFormatContext *s)
     st->codecpar->codec_id = AV_CODEC_ID_APTX;
     st->codecpar->bits_per_coded_sample = 4;
     st->codecpar->block_align = APTX_BLOCK_SIZE;
-    st->codecpar->frame_size = APTX_PACKET_SIZE;
     return 0;
 }
 
@@ -68,7 +67,6 @@  static int aptx_hd_read_header(AVFormatContext *s)
     st->codecpar->codec_id = AV_CODEC_ID_APTX_HD;
     st->codecpar->bits_per_coded_sample = 6;
     st->codecpar->block_align = APTX_HD_BLOCK_SIZE;
-    st->codecpar->frame_size = APTX_HD_PACKET_SIZE;
     return 0;
 }