http://bugs.winehq.org/show_bug.cgi?id=10495
--- Comment #202 from Art Taylor theycallhimart@gmail.com 2010-02-12 18:39:11 --- All, it is not useful to debate the topic of "whether pulseaudio?" or "what is the best audio subsystem?" Those are not the topic of this bug report. Please refrain. Whether native or through alsa-pulse is on topic.
(In reply to comment #193)
(In reply to comment #191)
Is it because: *) patch code is of bad quality
The patch does not meet AJ's quality standards, and no one seems to want to fix this.
As the original author, I am willing to fix this. Last attempt at a code review was not by AJ and was on an old version.
*) patch doesn't conform coding standard
See above.
More useful feedback is needed, but it's quite clean. Naming and indentation is consistent. I should revisit pulse.c on a rainy weekend though.
*) patch is incomplete
Supports WaveOut, WaveIn and Dsound playback and capture. All testsuite tests pass.
*) current wine devs/mantainers don't want to mantain this piece of code even if it's well written and works?
If it's well written and works, then there would be little objection to this as required maintenance would be low. So far, no one has put their hand up to 1) fix the code standard, 2) prove that winepulse is 100% necessary and 3) maintain a fixed and proven version of the module long-term.
I am hesitant to put my hand up again because it's still scorched. I have been maintaining this patch out-of-tree for over a year now at http://art.ified.ca/?page_id=40 . I regularly receive email feedback on it and provide support.
(In reply to comment #192)
In the end it comes down to wine developers don't like having so many audio backends in the tree, and so don't want another committed to it.
No. What it comes down to is no one has proven programatically that winealsa can't be made to work with pulse's ALSA emulation.
Well, it can be coaxed into working by modifying buffer settings, and does work to a minimum extent, but it's not exactly ideal.