On Wed, Aug 20, 2014 at 01:02:21PM +0200, Stefano Sabatini wrote:
> On date Tuesday 2014-08-19 14:47:02 +0200, Clément Bœsch encoded:
> > From: Clément Bœsch <clem...@stupeflix.com>
> > 
> > ---
> >  doc/filters.texi            |  3 +++
> >  libavfilter/avf_showwaves.c | 24 +++++++++++++++++++-----
> >  2 files changed, 22 insertions(+), 5 deletions(-)
> > 
> > diff --git a/doc/filters.texi b/doc/filters.texi
> > index 0ca1d6f..828f2b4 100644
> > --- a/doc/filters.texi
> > +++ b/doc/filters.texi
> > @@ -10813,6 +10813,9 @@ Draw a vertical line for each sample.
> >  
> >  @item p2p
> >  Draw a point for each sample and a line between them.
> > +
> > +@item center
> > +Draw a centered vertical line for each sample.
> >  @end table
> >  
> >  Default value is @code{point}.
> > diff --git a/libavfilter/avf_showwaves.c b/libavfilter/avf_showwaves.c
> > index e025663..e4911cc 100644
> > --- a/libavfilter/avf_showwaves.c
> > +++ b/libavfilter/avf_showwaves.c
> > @@ -36,6 +36,7 @@ enum ShowWavesMode {
> >      MODE_POINT,
> >      MODE_LINE,
> >      MODE_P2P,
> 
> > +    MODE_CENTER,
> 
> Nit: what about CENTERED_LINE, then you can use cl or cline as abbreviation.
> 

Sure whatever.

[...]
> LGTM otherwise, thanks.

Applied, thanks

-- 
Clément B.

Attachment: pgpQp9M1gQ5Xv.pgp
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to