[FFmpeg-devel] [PATCH 09/23] fftools/ffmpeg: stop handling AVMEDIA_TYPE_DATA in init_output_stream_encode()

James Almer jamrial at gmail.com
Sat Mar 25 21:43:46 EET 2023


On 3/25/2023 4:15 PM, Anton Khirnov wrote:
> We do not support data encoders, so this should never be reached.

nit: The abort() below could be an av_assert0(0) instead. It's the only 
abort() in the file, whereas av_assert0() is used a lot.

> ---
>   fftools/ffmpeg.c | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
> index f00b2d44e4..a424c69725 100644
> --- a/fftools/ffmpeg.c
> +++ b/fftools/ffmpeg.c
> @@ -3174,8 +3174,6 @@ static int init_output_stream_encode(OutputStream *ost, AVFrame *frame)
>               }
>           }
>   
> -        break;
> -    case AVMEDIA_TYPE_DATA:
>           break;
>       default:
>           abort();


More information about the ffmpeg-devel mailing list