[FFmpeg-devel] [PATCH] ffprobe: add "print_wrappers" option to default writer

[Stefano Sabatini]

On date Saturday 2012-05-12 17:28:29 +0200, Michael Niedermayer encoded:
> On Sat, May 12, 2012 at 02:44:14PM +0200, Stefano Sabatini wrote:
> > On date Thursday 2012-05-10 01:47:34 +0200, Stefano Sabatini encoded:
> > > On date Wednesday 2012-05-09 01:36:05 +0200, Stefano Sabatini encoded:
> > > > The option is useful for simplifying parsing.
> > [...]
> > > > + at item print_wrappers, w
> > > > +If set to 1 specify to print the section header and footer, if set to
> > > > +0 it will not. Default value is 1.
> > > > +
> > > > + at end table
> > > 
> > > Alternatively I could invert the logic and have noprint_wrappers, but
> > > I prefer to avoid negative logic.
> > 
> > Updated patch with fate fix.
> > -- 
> > FFmpeg = Fundamentalist Furious Meaningless Portable Extended Guide
> 
> >  doc/ffprobe.texi  |   13 +++++++++++
> >  ffprobe.c         |   60 ++++++++++++++++++++++++++++++++++++++++++++++++++----
> >  tests/fate-run.sh |    2 -
> 
> LGTM

Applied, after renaming it "noprint_wrappers".
-- 
FFmpeg = Fierce Frightening Monstrous Programmable Elastic Game