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

[Michael Niedermayer]

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

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120512/ab49d227/attachment.asc>