[Ffmpeg-cvslog] CVS: ffmpeg/doc ffserver-doc.texi,1.4,1.5

[Måns Rullgård]

The Wanderer <inverseparadox at comcast.net> writes:

> Diego Biurrun CVS wrote:
>
>> -LAME is important as it allows streaming of audio to Windows Media Player. Don't
>> -ask why the other audio types do not work.
>> +LAME is important as it allows streaming audio to Windows Media Player.
>> +Don't ask why the other audio types do not work.
>
> I think this is actually worse than the previous form.

Definitely.

> An alternate fix would have been to insert "the" before "streaming";
> I thought I had a possible fix for the current form as well, but
> somehow it's not coming to mind.

Perhaps "allows for streaming" would do.

>>  ffserver parameters must match the original parameters used to record the
>> -file. If not, then ffserver deletes the file before recording into it. (Now I write
>> -this, this seems broken).
>> +file. If not, then ffserver deletes the file before recording into it.
>> +(Now that I write this, it seems broken).
>
> "If they do not"
>
> (I *think* that's better.)

So do I.

>>  You can omit the YYYY-MM-DD, and then it refers to the current
>> day. However -note that @samp{?date=16:00:00}  refers to 4PM on the
>> current day -- this may be
>> -in the future and so unlikely to useful.
>> +note that @samp{?date=16:00:00} refers to 4pm on the current day -- this may be
>> +in the future and so is unlikely to be useful.
>
> I think that "PM" should in fact be capitalized; I certainly use it that
> way, although I would not capitalize the initial letters of the phrase
> it expands to.

Yes, PM should be capitalized.

> (On an FFmpeg-documentation blitz today, eh?)

He told someone he was active, and now he has to prove it.

-- 
M?ns Rullg?rd
mru at inprovide.com