[PATCH 1/5] winepulse: Add initial stub for pulseaudio support

Andrew Eikum aeikum at codeweavers.com
Mon Nov 2 08:43:54 CST 2015


On Mon, Nov 02, 2015 at 11:17:40PM +0900, Alexandre Julliard wrote:
> Andrew Eikum <aeikum at codeweavers.com> writes:
> 
> > From: Maarten Lankhorst <m.b.lankhorst at gmail.com>
> >
> > Includes API compatibility patch by Juergen Tretthahn <orson at orson.at>.
> >
> > Synchronous static data initialization by Andrew Eikum <aeikum at codeweavers.com>
> >
> > Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>
> 
> I know that this code is not supposed to be used yet, but if I try to
> use it, it's crashing all over the place. Is that really how it's
> supposed to be?
> 

I didn't really test it in-progress like that because lots of critical
functionality is missing in these early patches. I'm not sure how well
our winmm, dsound, and tests will handle missing IAudioRenderClient,
for example. It's split up for review purposes, because the full
driver is over 3000 lines.

I could try reproducing those issues and fixing them if you want, but
I'm not sure it's worth the time if the client DLLs and applications
won't handle it well anyway.

Andrew



More information about the wine-devel mailing list