[MPlayer-DOCS] Re: CVS: main/DOCS/man/en mplayer.1,1.1160,1.1161

[The Wanderer]

On 11/27/2005 01:22 PM, Tobias Diedrich wrote:

> Diego Biurrun wrote:
> 
>>> IMHO this makes the documentation a bit unclear, based on this
>>> I'd expect "-ao oss:mixer-device=/dev/mixer:dsp-device=/dev/dsp"
>>> to work...
>> 
>> Before my change, after my change or either way?
>> 
>> Hmm, I'm not going to claim that the current description is
>> perfect, but I don't see the problem you describe.  Other output
>> drivers are documented in the same way and where option=<value>
>> syntax is expected, this is written explicitly.  I was also hoping
>> that the <> would make clear that it should be replaced by a
>> concrete value...
> 
> I think it was more clear before the change. Most drivers use either
> the suboption parser or support only one parameter.  The syntax
> explanation suggests that the suboption order is not important.
> Before it was made clear that oss is an exception in that the order
> _is_ important, but maybe thats just me. :-)

It's not just you; I get the same impressions from both forms.

> I think it might be good to convert oss to use the suboptionparser
> (breaks compatibility) or reinclude the
> "oss[:dsp-device[:mixer-device[:mixer-channel]]]" 'usage example'.

Agreed. The former is probably the better solution, but the latter is
much the simpler.

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