[FFmpeg-cvslog] r20435 - trunk/ffmpeg.c

[Diego Biurrun]

On Sat, Dec 05, 2009 at 06:21:44PM -0200, Ramiro Polla wrote:
> On Sat, Dec 5, 2009 at 6:15 PM, Diego Biurrun <diego at biurrun.de> wrote:
> > On Fri, Dec 04, 2009 at 01:50:43PM -0200, Ramiro Polla wrote:
> >> On Thu, Dec 3, 2009 at 6:04 AM, Stefano Sabatini
> >> <stefano.sabatini-lala at poste.it> wrote:
> >> > On date Wednesday 2009-12-02 23:14:51 -0200, Ramiro Polla wrote:
> >> > [...]
> >> >> New patch attached with your suggestions, and a ':' on "according to
> >> >> the following rules:"
> >> >>
> >> >> --- doc/ffmpeg-doc.texi ? ? ? (revision 20711)
> >> >> +++ doc/ffmpeg-doc.texi ? ? ? (working copy)
> >> >> @@ -833,13 +833,17 @@
> >> >>
> >> >> +Preset files are specified with the @code{vpre}, @code{apre},
> >> >> + at code{spre}, and @code{fpre} options. The @code{fpre} option takes as
> >> >> +input the filename of the preset instead of a preset name and can be
> >
> > Fix Stefano's sentence reordering, "as input" should be at the end of
> > the sentence.
> 
> Hmm, it sounded fine for me too so I don't know what you want exactly.
> Do you mean:
> The @code{fpre} option takes the filename of the preset as input
> instead of a preset name and can be used for any kind of codec.
> or
> The @code{fpre} option takes the filename of the preset instead of a
> preset name as input and can be used for any kind of codec.

Either version is fine.  I have a slight preference for the latter.

Diego