[Bug 10495] Wine should support PulseAudio

[wine-bugs at winehq.org]

http://bugs.winehq.org/show_bug.cgi?id=10495





--- Comment #202 from Art Taylor <theycallhimart at 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.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.