[FFmpeg-devel] [PATCH] man page grammar fixes

Stefano Sabatini stefasab at gmail.com
Sat Aug 10 10:08:32 CEST 2013


On date Friday 2013-08-09 19:03:04 -0800, Lou Logan encoded:
> On Fri, Aug 9, 2013, at 06:19 PM, Bryce W. Harrington wrote:
> > The first sentence of each of these man pages are worded a bit
> > awkwardly.  I think these minor copy-edits should make them
> > clearer.
> > 
> > Signed-off-by: Bryce Harrington <b.harrington at samsung.com>
> > ---
> >  doc/demuxers.texi  |    2 +-
> >  doc/ffmpeg.texi    |    2 +-
> >  doc/outdevs.texi   |    2 +-
> >  doc/protocols.texi |    4 ++--
> >  4 files changed, 5 insertions(+), 5 deletions(-)
> > 
> > diff --git a/doc/demuxers.texi b/doc/demuxers.texi
> > index e18448f..437d1ca 100644
> > --- a/doc/demuxers.texi
> > +++ b/doc/demuxers.texi
> > @@ -1,7 +1,7 @@
> >  @chapter Demuxers
> >  @c man begin DEMUXERS
> >  
> > -Demuxers are configured elements in FFmpeg which allow to read the
> > +Demuxers are configured elements in FFmpeg that can read the
> >  multimedia streams from a particular type of file.
> 
> Are the "are configured elements in FFmpeg" phrases really necessary? It
> does not contribute much to the description other than needless
> verbosity in my opinion.
> 
> Other than that your changes are an improvement and look good to me.

Will push it soon, thanks.
-- 
FFmpeg = Fundamentalist and Faithless Minimalistic Purposeless Elected Guru


More information about the ffmpeg-devel mailing list