[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