[MPlayer-dev-eng] [PATCH] add OpenAL backend device support

Reimar Döffinger Reimar.Doeffinger at gmx.de
Mon Sep 13 18:41:25 CEST 2010


On Mon, Sep 13, 2010 at 05:25:04PM +0200, Erik Hofman wrote:
> On Mon, 2010-09-13 at 17:02 +0200, Diego Biurrun wrote:
> 
> > Please respect the coding style of the rest of the file and format
> > your function headers accordingly.
> 
> I'm willing to changes the things you pointed out but only if you think
> it's a useful patch in the first place.

In principle yes, but having > 10% of code for this feature is quite
ridiculous.
I'm quite sure it's possible to get it to a reasonable size, but
if you want quick results just getting rid of the device list printing
is probably simplest.


More information about the MPlayer-dev-eng mailing list