[FFmpeg-devel] [PATCH 2/2] libwebpenc_animencoder: print library messages in verbose log levels

James Almer jamrial at gmail.com
Thu Mar 17 15:21:28 CET 2016


On 3/17/2016 3:36 AM, James Zern wrote:
> On Wed, Mar 16, 2016 at 9:03 PM, James Almer <jamrial at gmail.com> wrote:
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavcodec/libwebpenc_animencoder.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
> 
> lgtm. Not really sure if this logging in the lib was really necessary...
> 
>> diff --git a/libavcodec/libwebpenc_animencoder.c b/libavcodec/libwebpenc_animencoder.c
>> index 35c456a..9068922 100644
>> --- a/libavcodec/libwebpenc_animencoder.c
>> +++ b/libavcodec/libwebpenc_animencoder.c
>> @@ -43,6 +43,8 @@ static av_cold int libwebp_anim_encode_init(AVCodecContext *avctx)
>>          LibWebPAnimContext *s = avctx->priv_data;
>>          WebPAnimEncoderOptions enc_options = { 0 };
>>          WebPAnimEncoderOptionsInit(&enc_options);
>> +        if (av_log_get_level() >= AV_LOG_VERBOSE)
>> +            enc_options.verbose = 1;
>>
> 
> Could be a one line assignment, either way.

Changed and both patches pushed. Thanks.

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 



More information about the ffmpeg-devel mailing list