[MPlayer-dev-eng] [PATCH] make ao_oss use the subopt helper (was: Re: [MPlayer-DOCS] Re: CVS: main/DOCS/man/en mplayer.1, 1.1160, 1.1161)

Diego Biurrun diego at biurrun.de
Sun Dec 24 06:13:54 CET 2006


Digging through old mails ..

On Sun, Dec 18, 2005 at 12:25:38AM +0100, Tobias Diedrich wrote:
> Diego Biurrun wrote:
> > On Sun, Nov 27, 2005 at 07:22:35PM +0100, Tobias Diedrich wrote:
> > > 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. :-)
> > 
> > I understand what your problem is now.  I'd say it needs a more general
> > fix, though.
> > 
> > > I think it might be good to convert oss to use the suboptionparser
> > > (breaks compatibility)
> > 
> > Definitely.  Breaking compatibility should not be a problem since this
> > has only been in the CVS version so far.
> 
> How about the attached patch?

Here's a version that applies ..

Diego



More information about the MPlayer-dev-eng mailing list