[FFmpeg-devel] [PATCH] flv: fix stereo flag when writing PCMA/PCMU
Marton Balint
cus at passwd.hu
Sun Feb 4 21:22:57 EET 2024
On Wed, 31 Jan 2024, aler9 wrote:
> Hello again, i'm bumping this patch since currently it's impossible to
> stream 16khz or stereo G711 tracks with RTMP, as these are always marked as
> 8khz, mono tracks.
> Please consider merging. Thanks.
Will apply, thanks.
Marton
>
>
> Il giorno dom 21 gen 2024 alle ore 16:16 Alessandro Ros <aler9.dev at gmail.com>
> ha scritto:
>
>> Currently, when writing PCMA or PCMU tracks with FLV or RTMP, the
>> stereo flag and sample rate flag inside RTMP audio messages are
>> overridden, making impossible to distinguish between mono and stereo
>> tracks. This patch fixes the issue by restoring the same flag mechanism
>> of all other codecs, that takes into consideration the right channel
>> count and sample rate.
>>
>> Signed-off-by: Alessandro Ros <aler9.dev at gmail.com>
>> ---
>> libavformat/flvenc.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c
>> index 874560fac1..772d891136 100644
>> --- a/libavformat/flvenc.c
>> +++ b/libavformat/flvenc.c
>> @@ -208,10 +208,10 @@ error:
>> flags |= FLV_CODECID_NELLYMOSER |
>> FLV_SAMPLESSIZE_16BIT;
>> break;
>> case AV_CODEC_ID_PCM_MULAW:
>> - flags = FLV_CODECID_PCM_MULAW | FLV_SAMPLERATE_SPECIAL |
>> FLV_SAMPLESSIZE_16BIT;
>> + flags |= FLV_CODECID_PCM_MULAW | FLV_SAMPLESSIZE_16BIT;
>> break;
>> case AV_CODEC_ID_PCM_ALAW:
>> - flags = FLV_CODECID_PCM_ALAW | FLV_SAMPLERATE_SPECIAL |
>> FLV_SAMPLESSIZE_16BIT;
>> + flags |= FLV_CODECID_PCM_ALAW | FLV_SAMPLESSIZE_16BIT;
>> break;
>> case 0:
>> flags |= par->codec_tag << 4;
>> --
>> 2.34.1
>>
>>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>
More information about the ffmpeg-devel
mailing list