Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_common: fix values for 24k sample rate
@ 2022-10-17  8:46 Paul B Mahol
  2022-10-18  7:10 ` Paul B Mahol
  2022-10-18  7:56 ` Andreas Rheinhardt
  0 siblings, 2 replies; 3+ messages in thread
From: Paul B Mahol @ 2022-10-17  8:46 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

[-- Attachment #1: Type: text/plain, Size: 16 bytes --]

Patch attached.

[-- Attachment #2: 0001-avcodec-mpegaudiodec_common-fix-values-for-24k-sampl.patch --]
[-- Type: text/x-patch, Size: 1043 bytes --]

From 5a068e66a9666a0a1c22c68315ec116c04e9c129 Mon Sep 17 00:00:00 2001
From: Paul B Mahol <onemda@gmail.com>
Date: Mon, 17 Oct 2022 10:46:34 +0200
Subject: [PATCH] avcodec/mpegaudiodec_common: fix values for 24k sample rate

Signed-off-by: Paul B Mahol <onemda@gmail.com>
---
 libavcodec/mpegaudiodec_common.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/mpegaudiodec_common.c b/libavcodec/mpegaudiodec_common.c
index 71fc2e7c81..5fcb39b325 100644
--- a/libavcodec/mpegaudiodec_common.c
+++ b/libavcodec/mpegaudiodec_common.c
@@ -369,7 +369,7 @@ const uint8_t ff_band_size_long[9][22] = {
 { 6, 6, 6, 6, 6, 6, 8, 10, 12, 14, 16,
   20, 24, 28, 32, 38, 46, 52, 60, 68, 58, 54, }, /* 22050 */
 { 6, 6, 6, 6, 6, 6, 8, 10, 12, 14, 16,
-  18, 22, 26, 32, 38, 46, 52, 64, 70, 76, 36, }, /* 24000 */
+  18, 22, 26, 32, 38, 46, 54, 62, 70, 76, 36, }, /* 24000 */
 { 6, 6, 6, 6, 6, 6, 8, 10, 12, 14, 16,
   20, 24, 28, 32, 38, 46, 52, 60, 68, 58, 54, }, /* 16000 */
 { 6, 6, 6, 6, 6, 6, 8, 10, 12, 14, 16,
-- 
2.37.2


[-- Attachment #3: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_common: fix values for 24k sample rate
  2022-10-17  8:46 [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_common: fix values for 24k sample rate Paul B Mahol
@ 2022-10-18  7:10 ` Paul B Mahol
  2022-10-18  7:56 ` Andreas Rheinhardt
  1 sibling, 0 replies; 3+ messages in thread
From: Paul B Mahol @ 2022-10-18  7:10 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

On 10/17/22, Paul B Mahol <onemda@gmail.com> wrote:
> Patch attached.
>

will apply soon
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_common: fix values for 24k sample rate
  2022-10-17  8:46 [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_common: fix values for 24k sample rate Paul B Mahol
  2022-10-18  7:10 ` Paul B Mahol
@ 2022-10-18  7:56 ` Andreas Rheinhardt
  1 sibling, 0 replies; 3+ messages in thread
From: Andreas Rheinhardt @ 2022-10-18  7:56 UTC (permalink / raw)
  To: ffmpeg-devel

Paul B Mahol:
> diff --git a/libavcodec/mpegaudiodec_common.c b/libavcodec/mpegaudiodec_common.c
> index 71fc2e7c81..5fcb39b325 100644
> --- a/libavcodec/mpegaudiodec_common.c
> +++ b/libavcodec/mpegaudiodec_common.c
> @@ -369,7 +369,7 @@ const uint8_t ff_band_size_long[9][22] = {
>  { 6, 6, 6, 6, 6, 6, 8, 10, 12, 14, 16,
>    20, 24, 28, 32, 38, 46, 52, 60, 68, 58, 54, }, /* 22050 */
>  { 6, 6, 6, 6, 6, 6, 8, 10, 12, 14, 16,
> -  18, 22, 26, 32, 38, 46, 52, 64, 70, 76, 36, }, /* 24000 */
> +  18, 22, 26, 32, 38, 46, 54, 62, 70, 76, 36, }, /* 24000 */
>  { 6, 6, 6, 6, 6, 6, 8, 10, 12, 14, 16,
>    20, 24, 28, 32, 38, 46, 52, 60, 68, 58, 54, }, /* 16000 */
>  { 6, 6, 6, 6, 6, 6, 8, 10, 12, 14, 16,

Can you add a pointer to the exact table in the spec on which this table
is based?

- Andreas

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-10-18  7:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-17  8:46 [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_common: fix values for 24k sample rate Paul B Mahol
2022-10-18  7:10 ` Paul B Mahol
2022-10-18  7:56 ` Andreas Rheinhardt

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git