Re: [FFmpeg-devel] [PATCH] libavcodec/flacenc: Enable sample rates > 655350 Hz

2022-11-15 Thread Martijn van Beurden
Op ma 31 okt. 2022 om 19:33 schreef Martijn van Beurden : > > Op ma 31 okt. 2022 om 17:58 schreef Derek Buitenhuis > : > > > > It is interesting that the IETF RFC and the Xiph spec disagree on whether > > this is allowed. > > > > The Xiph spec also says the IETF spec is better, and it remains as >

Re: [FFmpeg-devel] [PATCH] libavcodec/flacenc: Enable sample rates > 655350 Hz

2022-10-31 Thread Derek Buitenhuis
On 10/31/2022 6:33 PM, Martijn van Beurden wrote: > The Xiph spec also says the IETF spec is better, and it remains as > historical reference and overview :) So it does. > The spec as it is on the FLAC website (which is being "preserved") is > wrong. I don't know how this came to be, I think it w

Re: [FFmpeg-devel] [PATCH] libavcodec/flacenc: Enable sample rates > 655350 Hz

2022-10-31 Thread Martijn van Beurden
Op ma 31 okt. 2022 om 17:58 schreef Derek Buitenhuis : > > It is interesting that the IETF RFC and the Xiph spec disagree on whether > this is allowed. > The Xiph spec also says the IETF spec is better, and it remains as historical reference and overview :) > The Xiph spec says: > > Sample r

Re: [FFmpeg-devel] [PATCH] libavcodec/flacenc: Enable sample rates > 655350 Hz

2022-10-31 Thread Derek Buitenhuis
On 10/31/2022 4:09 PM, Martijn van Beurden wrote: > Also, make use of the full sample rate code table > --- > libavcodec/flacenc.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) It is interesting that the IETF RFC and the Xiph spec disagree on whether this is allowed. The Xiph spec

[FFmpeg-devel] [PATCH] libavcodec/flacenc: Enable sample rates > 655350 Hz

2022-10-31 Thread Martijn van Beurden
Also, make use of the full sample rate code table --- libavcodec/flacenc.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/flacenc.c b/libavcodec/flacenc.c index 5d8c3f82be..bca71b3780 100644 --- a/libavcodec/flacenc.c +++ b/libavcodec/flacenc.c @@ -299,7 +299,7