[FFmpeg-devel] [PATCH v3 2/3] avutil/log: add av_log_set_opts function

wm4 nfxjfg at googlemail.com
Tue Apr 3 18:03:50 EEST 2018


On Tue, 3 Apr 2018 10:25:56 +0200
Paul B Mahol <onemda at gmail.com> wrote:

> On 4/3/18, Tobias Rapp <t.rapp at noa-archive.com> wrote:
> > On 29.03.2018 20:38, Michael Niedermayer wrote:  
> >> On Wed, Mar 28, 2018 at 05:03:39PM +0200, Tobias Rapp wrote:  
> >>> Allows to set log level and flag values from string.
> >>>
> >>> Signed-off-by: Tobias Rapp <t.rapp at noa-archive.com>
> >>> ---
> >>>   doc/APIchanges      |  3 +++
> >>>   libavutil/log.c     | 76
> >>> +++++++++++++++++++++++++++++++++++++++++++++++++++++
> >>>   libavutil/log.h     | 16 +++++++++++
> >>>   libavutil/version.h |  2 +-
> >>>   4 files changed, 96 insertions(+), 1 deletion(-)  
> >>
> >> iam not intending to override anyone blocking this but as i looked at the
> >> code
> >> and it LGTM  
> >
> > Thanks for review. Moved code into fftools/cmdutils.c and pushed to
> > master. Will send a separate patch to ML for the missing documentation
> > update.  
> 
> This was blocked, please revert ASAP!

Looks fine to me. None of what was pushed was blocked.


More information about the ffmpeg-devel mailing list