Re: bug 3514 fix (dsound primary buffer) needs fix?

[James Hawkins]

On 11/5/05, Peter Berg Larsen  wrote:
> Hi,
>
> The following patch
>
> http://www.winehq.org/pipermail/wine-patches/2005-October/021269.html
>
> fixes
>
> http://bugs.winehq.org/show_bug.cgi?id=3514
>
> but crashed Railroad tycoon, as it always sets device->primary->dsound to
> null when there are more than one user of the device.

The patch also breaks Neverwinter Nights.
>       if (This->device) {
>           if (DirectSoundDevice_Release(This->device) != 0) {
>               /* device not released so make sure primary reference to This removed */
> -
>            if (This->device->primary)
> -
>            if (This->device->primary && This->device->primary->dsound == This)
>                 This->device->primary->dsound = NULL;
>         }
>     }

My temporary workaround was to comment out the line that set dsound to
NULL, but this works.

--
James Hawkins