On 3/17/2016 3:36 AM, James Zern wrote:
> On Wed, Mar 16, 2016 at 9:03 PM, James Almer <jamr...@gmail.com> wrote:
>> Signed-off-by: James Almer <jamr...@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@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 

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

Reply via email to