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

[The Wanderer]

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.

-- 
       The Wanderer

Warning: Simply because I argue an issue does not mean I agree with any
side of it.

Secrecy is the beginning of tyranny.