Re: [FFmpeg-devel] [PATCH] libavcodec/libxev: Update EVC codec naming

[Andreas Rheinhardt <andreas.rheinhardt@outlook.com>]

Dariusz Frankiewicz:
> Make consistency between encoder and decoder implementation.
> 
> Signed-off-by: Dariusz Frankiewicz <d.frankiewic@samsung.com>
> ---
>  libavcodec/libxevd.c | 2 +-
>  libavcodec/libxeve.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/libxevd.c b/libavcodec/libxevd.c
> index 520fdab7d8..51303ec71d 100644
> --- a/libavcodec/libxevd.c
> +++ b/libavcodec/libxevd.c
> @@ -469,7 +469,7 @@ static av_cold int libxevd_close(AVCodecContext *avctx)
>  }
>  
>  const FFCodec ff_libxevd_decoder = {
> -    .p.name             = "evc",
> +    .p.name             = "libxevd",

The codec name is used in avcodec_find_decoder_by_name() (and also in
the ffmpeg cli tool). Changing it causes breakage.

>      CODEC_LONG_NAME("EVC / MPEG-5 Essential Video Coding (EVC)"),
>      .p.type             = AVMEDIA_TYPE_VIDEO,
>      .p.id               = AV_CODEC_ID_EVC,
> diff --git a/libavcodec/libxeve.c b/libavcodec/libxeve.c
> index 1d9972619d..40eab08ef2 100644
> --- a/libavcodec/libxeve.c
> +++ b/libavcodec/libxeve.c
> @@ -600,7 +600,7 @@ static const FFCodecDefault libxeve_defaults[] = {
>  
>  const FFCodec ff_libxeve_encoder = {
>      .p.name             = "libxeve",
> -    .p.long_name        = NULL_IF_CONFIG_SMALL("libxeve MPEG-5 EVC"),
> +    CODEC_LONG_NAME("EVC / MPEG-5 Essential Video Coding (EVC)"),
>      .p.type             = AVMEDIA_TYPE_VIDEO,
>      .p.id               = AV_CODEC_ID_EVC,
>      .init               = libxeve_init,

_______________________________________________
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".