[FFmpeg-devel] [PATCH 3/3] avformat/dvdvideodec: assign mono channel layout explicitly

Stefano Sabatini stefasab at gmail.com
Wed Mar 6 17:10:42 EET 2024


On date Wednesday 2024-03-06 01:19:13 -0600, Marth64 wrote:
> Signed-off-by: Marth64 <marth64 at proxyid.net>
> ---
>  libavformat/dvdvideodec.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/libavformat/dvdvideodec.c b/libavformat/dvdvideodec.c
> index 2c7ffdd148..6136060288 100644
> --- a/libavformat/dvdvideodec.c
> +++ b/libavformat/dvdvideodec.c
> @@ -1142,7 +1142,9 @@ static int dvdvideo_audio_stream_analyze(AVFormatContext *s, audio_attr_t audio_
>          return AVERROR_INVALIDDATA;
>      }
>  
> -    if (nb_channels == 2)
> +    if (nb_channels == 1)
> +        ch_layout = (AVChannelLayout) AV_CHANNEL_LAYOUT_MONO;
> +    else if (nb_channels == 2)
>          ch_layout = (AVChannelLayout) AV_CHANNEL_LAYOUT_STEREO;
>      else if (nb_channels == 6)
>          ch_layout = (AVChannelLayout) AV_CHANNEL_LAYOUT_5POINT1;

LGTM, but when this is needed?


More information about the ffmpeg-devel mailing list