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

Diego Biurrun diego
Sun Sep 27 14:38:01 CEST 2009


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.

Diego



More information about the ffmpeg-cvslog mailing list