[FFmpeg-devel] [PATCH] Document the preset files system

[The Wanderer]

Stefano Sabatini wrote:

> On date Sunday 2008-09-28 20:27:31 -0400, The Wanderer encoded:
> 
>> Stefano Sabatini wrote:
> 
>>> Hi, as in subject.
> 
> [...]
> 
>> Beyond that I can't really improve this, largely because I don't
>> understand it well enough for the degree of rewriting it would
>> require. (I think it's talking about automatically choosing a
>> preset based on the specified codec name and on something else, but
>> I have no real idea what the else is from a user perspective, and I
>> don't see where in the code this paragraph comes from in the first
>> place.)
>> 
>> Honestly, I would prefer to rewrite (and reorganize) these last
>> three paragraphs entirely, if I were certain that I understood them
>> correctly.
> 
> I rephrased almost entirely the whole text, trying to apply the
> proposed corrections, I think it should be more clear now, please
> check it again.

It's significantly better now, yes. I still see places to improve in
proofreading terms, but that's purely cosmetic, the text is now
understandable IMO. I'm in a bit of a hurry so I won't be reviewing it
immediately in detail, but I expect to get back to it as soon as I can.

-- 
       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.