[FFmpeg-cvslog] r15563 - trunk/doc/ffmpeg-doc.texi

[Stefano Sabatini]

On date Sunday 2008-10-05 13:58:57 -0400, The Wanderer wrote:
> Stefano Sabatini wrote:
> > On date Sunday 2008-10-05 07:07:32 -0400, The Wanderer wrote:
> > 
> >> stefano wrote:
> >> 
> >>> Author: stefano
> >>> Date: Sun Oct  5 11:45:54 2008
> >>> New Revision: 15563
> >>> 
> >>> Log:
> >>> Document the preset files system.
> >> 
> >>> +Preset files are specified with the @code{vpre}, @code{apre} and
> >>> + at code{spre} options. The options specified in a preset file are
> >>> +applied to the currently selected codec of the same type as specified
> >>> +by the preset option.
> >> 
> >> This isn't quite what I suggested - my suggestion would have
> >> omitted the words "specified by". This probably works, though.
> > 
> > Indeed I wasn't sure about the interpretation of your mail, so the
> > alternative would be: "The options specified in a preset file are
> > applied to the currently selected codec of the same type as the
> > preset option."
> > 
> > I have a slightly preference for the committed form, as it isn't
> > immediately clear what is the type of an option, but I'll change it
> > if you think it's preferable the second one.
> 
> I do think it's preferable to the second one, that being why I suggested
> it rather than the other, but I'm not sure it's worth changing it right
> off the bat.
> 
> The meaning of referring to the "type" of an option does seem clear to
> me at first glance ('a' type, 'v' type, 's' type), and in any case if
> it's not clear then it seems to me that that would also be a problem for
> the current form; both forms refer to the type of the option, it's just
> that one of them does so more verbosely.

Applied the proposed change.

Regards.