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

Stefano Sabatini stefano.sabatini-lala
Sun Sep 27 15:11:48 CEST 2009


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.

Regards.



More information about the ffmpeg-cvslog mailing list