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