[MPlayer-DOCS] r29635 - trunk/DOCS/man/en/mplayer.1

The Wanderer wanderer at fastmail.fm
Thu Sep 3 14:31:14 CEST 2009


On 09/03/2009 02:59 AM, Reimar Döffinger wrote:

> On Wed, Sep 02, 2009 at 11:35:21PM -0400, The Wanderer wrote:
 >
>> On 09/02/2009 03:44 PM, reimar wrote:

>>> +one is loaded from ~/.mplayer.
>> 
>> "If a file-specific" - missing "a" here.
>> 
>>> +In addition the \-use\-filedir\-conf also enables directory-specific configuration files,
>> 
>> "In addition" and "also" might be argued to be redundant, though
>> I'm not sure that that would be correct. Even if both are retained,
>> there should probably be a comma after "addition".
>> 
>>> +MPlayer tries to first load a mplayer.conf from the same directory as the file played
>>> +and then tries to load any file-specific configuration.
>> 
>> The two parts of this sentence aren't parallel. Either "MPlayer
>> first tries to load ... and then tries to load" or "MPlayer tries
>> to first load ... and then load" or similar would be more
>> appropriate; I'd probably prefer the former, myself.
> 
> It was late when I wrote it (and now it probably is too early). I
> changed it a bit more, because I think it was badly written overall.
> See if it is better.

Looks good to me.

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


More information about the MPlayer-DOCS mailing list