Help with 6120d7cc145, causing bugs

Jeff Cook jeff at deserettechnology.com
Sat Aug 28 23:13:41 CDT 2010


Wylda reported success with the attached patch on bug 23902. Does it
seem OK to everyone else? In this case, the extra channel needed for
capture is not added when a micelem is found, because micelem should
only be found when there is no other suitable playback or capture
controls (i.e. "Mic" element only). Though I'm not sure if that's
working, so the micelem detection could probably still stand for some
reworking, but this seems to fix things for now.

On Fri, Aug 27, 2010 at 7:51 AM, Henri Verbeet <hverbeet at gmail.com> wrote:
> On 27 August 2010 15:14,  <wylda at volny.cz> wrote:
>> All apps are working perfectly! Could i kindly ask you for sending
>> to patches today? It looks like i will have spare time for testing,
>> so it will help me to not to mess with revering.
>>
> It's probably not quite correct. In particular, before 6120d7cc145
> it's likely the Mic element would be counted in capcontrols as well,
> and skipped for the chan count in some cases. Also, the channel count
> for USB mics will probably end up as 2 now, which would make it hit
> the "No channels found, skipping device!\n" path again. I think I'd
> prefer the original author or Maarten to have a look at this, the code
> looks somewhat complicated to me, perhaps more so than really needed.
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: micelem_chans.diff
Type: application/octet-stream
Size: 877 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20100828/5dd7e8c0/attachment.obj>


More information about the wine-devel mailing list