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

Andrew Eikum aeikum at codeweavers.com
Mon Nov 2 09:33:18 CST 2015


On Tue, Nov 03, 2015 at 12:22:09AM +0900, Alexandre Julliard wrote:
> Andrew Eikum <aeikum at codeweavers.com> writes:
> 
> > 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.
> 
> If it's just because things are missing, then that's OK. But you better
> hope that once the whole series is in, it no longer crashes here ;-)

After those five, there are three more patches to implement missing
critical interfaces (volume, session, and marshalling). All three
should be in the next wave. After those, I expect the audio tests to
pass. Then there are a couple of other patches for small improvements
and consistency with other drivers, split out to retain authorship.

Andrew



More information about the wine-devel mailing list