[FFmpeg-cvslog] r20027 - in trunk: doc/ffplay-doc.texi ffplay.c

Diego Biurrun diego
Sun Sep 27 15:23:50 CEST 2009


On Sun, Sep 27, 2009 at 03:11:48PM +0200, Stefano Sabatini wrote:
> On date Sunday 2009-09-27 14:38:01 +0200, Diego Biurrun wrote:
> > On Sat, Sep 26, 2009 at 07:18:03PM +0200, stefano wrote:
> > > 
> > > Log:
> > > Add support to the -loglevel option.
> > 
> > This message is wrong.  You can only add something to an entity if that
> > entity already exists.  This sounds as if you had opened a call center
> > where people would offer help for people having trouble with the
> > -loglevel option.
> > 
> > You probably meant to say
> > 
> >   Add support for the -loglevel option.
> > 
> > But this is needlessly complicated and still sounds as if the option
> > already existed somewhere else.  Just say
> > 
> >   Add -loglevel option.
> > 
> > or the slightly more descriptive
> > 
> >   Add -loglevel command line option.
> > 
> > However, it would be nice if you could spend a few words on explaining
> > what the option is all about.
> 
> Fixed - hope I did it more correctly this time.

Yes, thanks.

Diego



More information about the ffmpeg-cvslog mailing list