[FFmpeg-devel] [PATCH]New option -hide_banner

Michael Niedermayer michaelni at gmx.at
Sun Dec 29 11:12:08 CET 2013


On Sun, Dec 29, 2013 at 10:51:36AM +0100, Stefano Sabatini wrote:
> On date Saturday 2013-12-28 21:48:55 +0100, Carl Eugen Hoyos encoded:
> > Hi!
> > 
> > Attached patch contains Stefanos improvements for the documentation,
> > I prefer the current variable name.
> > 
> > Please comment, Carl Eugen
> 
> > From 5bcb96dc452432ef64128b2487bce2c58aff3d43 Mon Sep 17 00:00:00 2001
> > From: Carl Eugen Hoyos <cehoyos at minerva.hye>
> > Date: Sat, 28 Dec 2013 05:18:39 +0100
> 
> > Subject: [PATCH] Allow hiding the banner.
> 
> nit: cmdutils: allow hiding the banner
> 
> > 
> > Fixes ticket #3246.
> > ---
> >  cmdutils.c                   |    6 +++++-
> >  cmdutils.h                   |    1 +
> >  cmdutils_common_opts.h       |    1 +
> >  doc/fftools-common-opts.texi |    7 +++++++
> >  4 files changed, 14 insertions(+), 1 deletions(-)
> 
> LGTM but I'm not the cmdutils maintainer.

LGTM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- 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/20131229/370a0f67/attachment.asc>


More information about the ffmpeg-devel mailing list