[FFmpeg-devel] [PATCH v2 1/3] libsvtav1: Rename without a -

Mark Thompson sw at jkqxz.net
Sat Aug 1 00:56:55 EEST 2020


On 31/07/2020 00:59, James Almer wrote:
> On 7/30/2020 6:31 PM, Mark Thompson wrote:
>> The external library is called libsvtav1, so use this name everywhere.
>> ---
>>   configure                                | 2 +-
>>   libavcodec/Makefile                      | 2 +-
>>   libavcodec/allcodecs.c                   | 2 +-
>>   libavcodec/{libsvt_av1.c => libsvtav1.c} | 8 ++++----
>>   4 files changed, 7 insertions(+), 7 deletions(-)
>>   rename libavcodec/{libsvt_av1.c => libsvtav1.c} (99%)
>>
>> ...
>> @@ -543,7 +543,7 @@ static const AVOption options[] = {
>>   };
>>   
>>   static const AVClass class = {
>> -    .class_name = "libsvt_av1",
>> +    .class_name = "libsvtav1",
>>       .item_name  = av_default_item_name,
>>       .option     = options,
>>       .version    = LIBAVUTIL_VERSION_INT,
>> @@ -557,8 +557,8 @@ static const AVCodecDefault eb_enc_defaults[] = {
>>       { NULL },
>>   };
>>   
>> -AVCodec ff_libsvt_av1_encoder = {
>> -    .name           = "libsvt_av1",
> 
> Subject mentions - (dash) but you're removing underscores.

Oops, fixed.

>> +AVCodec ff_libsvtav1_encoder = {
>> +    .name           = "libsvtav1",
>>       .long_name      = NULL_IF_CONFIG_SMALL("SVT-AV1(Scalable Video Technology for AV1) encoder"),
>>       .priv_data_size = sizeof(SvtContext),
>>       .type           = AVMEDIA_TYPE_VIDEO,
>> @@ -573,5 +573,5 @@ AVCodec ff_libsvt_av1_encoder = {
>>       .priv_class     = &class,
>>       .defaults       = eb_enc_defaults,
>>       .caps_internal  = FF_CODEC_CAP_INIT_CLEANUP,
>> -    .wrapper_name   = "libsvt_av1",
>> +    .wrapper_name   = "libsvtav1",
>>   };
> 
> LGTM with the subject fixed.

Applied.

Thanks,

- Mark


More information about the ffmpeg-devel mailing list