January 2012 - Wine-commits

Michael Stefaniuc : dsound: COM cleanup for the IDirectSoundCaptureBuffer8 iface.
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: f1d7414971529436ae8c700e8791f9fa9e8eff84 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f1d7414971529436ae8c700e8… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Jan 12 00:06:48 2012 +0100 dsound: COM cleanup for the IDirectSoundCaptureBuffer8 iface. --- dlls/dsound/capture.c | 151 +++++++++++++++++------------------------- dlls/dsound/dsound_private.h | 7 +-- 2 files changed, 62 insertions(+), 96 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=f1d7414971529436ae8c7…

1 0

Michael Stefaniuc : dsound: Drop "Capture" from the IDirectSoundNotify method names.
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: e8f7ddf774edd26fe7bf8d3c6f39dd0fce9c044c URL: http://source.winehq.org/git/wine.git/?a=commit;h=e8f7ddf774edd26fe7bf8d3c6… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Jan 12 00:09:01 2012 +0100 dsound: Drop "Capture" from the IDirectSoundNotify method names. --- dlls/dsound/capture.c | 26 +++++++++++--------------- 1 files changed, 11 insertions(+), 15 deletions(-) diff --git a/dlls/dsound/capture.c b/dlls/dsound/capture.c index 70d70fb..c233ebc 100644 --- a/dlls/dsound/capture.c +++ b/dlls/dsound/capture.c @@ -56,12 +56,10 @@ struct IDirectSoundCaptureNotifyImpl }; /******************************************************************************* - * IDirectSoundCaptureNotify + * IDirectSoundNotify */ -static HRESULT WINAPI IDirectSoundCaptureNotifyImpl_QueryInterface( - LPDIRECTSOUNDNOTIFY iface, - REFIID riid, - LPVOID *ppobj) +static HRESULT WINAPI IDirectSoundNotifyImpl_QueryInterface(IDirectSoundNotify *iface, REFIID riid, + void **ppobj) { IDirectSoundCaptureNotifyImpl *This = (IDirectSoundCaptureNotifyImpl *)iface; TRACE("(%p,%s,%p)\n",This,debugstr_guid(riid),ppobj); @@ -74,7 +72,7 @@ static HRESULT WINAPI IDirectSoundCaptureNotifyImpl_QueryInterface( return IDirectSoundCaptureBuffer_QueryInterface((LPDIRECTSOUNDCAPTUREBUFFER)This->dscb, riid, ppobj); } -static ULONG WINAPI IDirectSoundCaptureNotifyImpl_AddRef(LPDIRECTSOUNDNOTIFY iface) +static ULONG WINAPI IDirectSoundNotifyImpl_AddRef(IDirectSoundNotify *iface) { IDirectSoundCaptureNotifyImpl *This = (IDirectSoundCaptureNotifyImpl *)iface; ULONG ref = InterlockedIncrement(&(This->ref)); @@ -82,7 +80,7 @@ static ULONG WINAPI IDirectSoundCaptureNotifyImpl_AddRef(LPDIRECTSOUNDNOTIFY ifa return ref; } -static ULONG WINAPI IDirectSoundCaptureNotifyImpl_Release(LPDIRECTSOUNDNOTIFY iface) +static ULONG WINAPI IDirectSoundNotifyImpl_Release(IDirectSoundNotify *iface) { IDirectSoundCaptureNotifyImpl *This = (IDirectSoundCaptureNotifyImpl *)iface; ULONG ref = InterlockedDecrement(&(This->ref)); @@ -97,10 +95,8 @@ static ULONG WINAPI IDirectSoundCaptureNotifyImpl_Release(LPDIRECTSOUNDNOTIFY if return ref; } -static HRESULT WINAPI IDirectSoundCaptureNotifyImpl_SetNotificationPositions( - LPDIRECTSOUNDNOTIFY iface, - DWORD howmuch, - LPCDSBPOSITIONNOTIFY notify) +static HRESULT WINAPI IDirectSoundNotifyImpl_SetNotificationPositions(IDirectSoundNotify *iface, + DWORD howmuch, const DSBPOSITIONNOTIFY *notify) { IDirectSoundCaptureNotifyImpl *This = (IDirectSoundCaptureNotifyImpl *)iface; TRACE("(%p,0x%08x,%p)\n",This,howmuch,notify); @@ -144,10 +140,10 @@ static HRESULT WINAPI IDirectSoundCaptureNotifyImpl_SetNotificationPositions( static const IDirectSoundNotifyVtbl dscnvt = { - IDirectSoundCaptureNotifyImpl_QueryInterface, - IDirectSoundCaptureNotifyImpl_AddRef, - IDirectSoundCaptureNotifyImpl_Release, - IDirectSoundCaptureNotifyImpl_SetNotificationPositions, + IDirectSoundNotifyImpl_QueryInterface, + IDirectSoundNotifyImpl_AddRef, + IDirectSoundNotifyImpl_Release, + IDirectSoundNotifyImpl_SetNotificationPositions }; static HRESULT IDirectSoundCaptureNotifyImpl_Create(

1 0

Michael Stefaniuc : dsound: Add an extra refcount for the capture buffer object.
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: 3dac0e5a4fd12cd89936d9cf1e87753845c996b4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3dac0e5a4fd12cd89936d9cf1… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Jan 12 00:11:20 2012 +0100 dsound: Add an extra refcount for the capture buffer object. --- dlls/dsound/capture.c | 68 ++++++++++++++++++++++++------------------ dlls/dsound/dsound_private.h | 1 + 2 files changed, 40 insertions(+), 29 deletions(-) diff --git a/dlls/dsound/capture.c b/dlls/dsound/capture.c index c233ebc..8e6c77d 100644 --- a/dlls/dsound/capture.c +++ b/dlls/dsound/capture.c @@ -44,6 +44,34 @@ WINE_DEFAULT_DEBUG_CHANNEL(dsound); +static void capturebuffer_destroy(IDirectSoundCaptureBufferImpl *This) +{ + if (This->device->state == STATE_CAPTURING) + This->device->state = STATE_STOPPING; + + HeapFree(GetProcessHeap(),0, This->pdscbd); + + if (This->device->client) { + IAudioClient_Release(This->device->client); + This->device->client = NULL; + } + + if (This->device->capture) { + IAudioCaptureClient_Release(This->device->capture); + This->device->capture = NULL; + } + + /* remove from DirectSoundCaptureDevice */ + This->device->capture_buffer = NULL; + + if (This->notify) + IDirectSoundNotify_Release((LPDIRECTSOUNDNOTIFY)This->notify); + + HeapFree(GetProcessHeap(), 0, This->notifies); + HeapFree(GetProcessHeap(), 0, This); + TRACE("(%p) released\n", This); +} + /***************************************************************************** * IDirectSoundCaptureNotify implementation structure */ @@ -228,46 +256,26 @@ static HRESULT WINAPI IDirectSoundCaptureBufferImpl_QueryInterface(IDirectSoundC static ULONG WINAPI IDirectSoundCaptureBufferImpl_AddRef(IDirectSoundCaptureBuffer8 *iface) { IDirectSoundCaptureBufferImpl *This = impl_from_IDirectSoundCaptureBuffer8(iface); - ULONG ref = InterlockedIncrement(&(This->ref)); + ULONG ref = InterlockedIncrement(&This->ref); TRACE("(%p) ref was %d\n", This, ref - 1); + + if(ref == 1) + InterlockedIncrement(&This->numIfaces); + return ref; } static ULONG WINAPI IDirectSoundCaptureBufferImpl_Release(IDirectSoundCaptureBuffer8 *iface) { IDirectSoundCaptureBufferImpl *This = impl_from_IDirectSoundCaptureBuffer8(iface); - ULONG ref = InterlockedDecrement(&(This->ref)); + ULONG ref = InterlockedDecrement(&This->ref); TRACE("(%p) ref was %d\n", This, ref + 1); - if (!ref) { - TRACE("deleting object\n"); - if (This->device->state == STATE_CAPTURING) - This->device->state = STATE_STOPPING; - - HeapFree(GetProcessHeap(),0, This->pdscbd); - - if (This->device->client) { - IAudioClient_Release(This->device->client); - This->device->client = NULL; - } - - if (This->device->capture) { - IAudioCaptureClient_Release(This->device->capture); - This->device->capture = NULL; - } - - /* remove from DirectSoundCaptureDevice */ - This->device->capture_buffer = NULL; - - if (This->notify) - IDirectSoundNotify_Release((LPDIRECTSOUNDNOTIFY)This->notify); + if (!ref && !InterlockedDecrement(&This->numIfaces)) + capturebuffer_destroy(This); - HeapFree(GetProcessHeap(), 0, This->notifies); - HeapFree( GetProcessHeap(), 0, This ); - TRACE("(%p) released\n", This); - } return ref; } @@ -697,7 +705,8 @@ static HRESULT IDirectSoundCaptureBufferImpl_Create( LPBYTE newbuf; DWORD buflen; - This->ref = 1; + This->numIfaces = 0; + This->ref = 0; This->device = device; This->device->capture_buffer = This; This->notify = NULL; @@ -771,6 +780,7 @@ static HRESULT IDirectSoundCaptureBufferImpl_Create( device->buflen = buflen; } + IDirectSoundCaptureBuffer_AddRef(&This->IDirectSoundCaptureBuffer8_iface); *ppobj = This; TRACE("returning DS_OK\n"); diff --git a/dlls/dsound/dsound_private.h b/dlls/dsound/dsound_private.h index 6f3dd02..34419d1 100644 --- a/dlls/dsound/dsound_private.h +++ b/dlls/dsound/dsound_private.h @@ -248,6 +248,7 @@ struct DirectSoundCaptureDevice struct IDirectSoundCaptureBufferImpl { IDirectSoundCaptureBuffer8 IDirectSoundCaptureBuffer8_iface; + LONG numIfaces; /* "in use interfaces" refcount */ LONG ref; /* IDirectSoundCaptureBuffer fields */ DirectSoundCaptureDevice* device;

1 0

Michael Stefaniuc : dsound: Merge IDirectSoundNotify into the capture buffer object.
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: 13a4b03b443521127a48b8de111f9f7aab993cc8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=13a4b03b443521127a48b8de1… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Jan 12 00:19:48 2012 +0100 dsound: Merge IDirectSoundNotify into the capture buffer object. --- dlls/dsound/capture.c | 122 ++++++++++++++---------------------------- dlls/dsound/dsound_private.h | 8 +-- 2 files changed, 44 insertions(+), 86 deletions(-) diff --git a/dlls/dsound/capture.c b/dlls/dsound/capture.c index 8e6c77d..29d34dc 100644 --- a/dlls/dsound/capture.c +++ b/dlls/dsound/capture.c @@ -64,69 +64,59 @@ static void capturebuffer_destroy(IDirectSoundCaptureBufferImpl *This) /* remove from DirectSoundCaptureDevice */ This->device->capture_buffer = NULL; - if (This->notify) - IDirectSoundNotify_Release((LPDIRECTSOUNDNOTIFY)This->notify); - HeapFree(GetProcessHeap(), 0, This->notifies); HeapFree(GetProcessHeap(), 0, This); TRACE("(%p) released\n", This); } -/***************************************************************************** - * IDirectSoundCaptureNotify implementation structure - */ -struct IDirectSoundCaptureNotifyImpl -{ - /* IUnknown fields */ - const IDirectSoundNotifyVtbl *lpVtbl; - LONG ref; - IDirectSoundCaptureBufferImpl* dscb; -}; - /******************************************************************************* * IDirectSoundNotify */ +static inline struct IDirectSoundCaptureBufferImpl *impl_from_IDirectSoundNotify(IDirectSoundNotify *iface) +{ + return CONTAINING_RECORD(iface, IDirectSoundCaptureBufferImpl, IDirectSoundNotify_iface); +} + static HRESULT WINAPI IDirectSoundNotifyImpl_QueryInterface(IDirectSoundNotify *iface, REFIID riid, void **ppobj) { - IDirectSoundCaptureNotifyImpl *This = (IDirectSoundCaptureNotifyImpl *)iface; - TRACE("(%p,%s,%p)\n",This,debugstr_guid(riid),ppobj); + IDirectSoundCaptureBufferImpl *This = impl_from_IDirectSoundNotify(iface); - if (This->dscb == NULL) { - WARN("invalid parameter\n"); - return E_INVALIDARG; - } + TRACE("(%p,%s,%p)\n", This, debugstr_guid(riid), ppobj); - return IDirectSoundCaptureBuffer_QueryInterface((LPDIRECTSOUNDCAPTUREBUFFER)This->dscb, riid, ppobj); + return IDirectSoundCaptureBuffer_QueryInterface(&This->IDirectSoundCaptureBuffer8_iface, riid, ppobj); } static ULONG WINAPI IDirectSoundNotifyImpl_AddRef(IDirectSoundNotify *iface) { - IDirectSoundCaptureNotifyImpl *This = (IDirectSoundCaptureNotifyImpl *)iface; - ULONG ref = InterlockedIncrement(&(This->ref)); + IDirectSoundCaptureBufferImpl *This = impl_from_IDirectSoundNotify(iface); + ULONG ref = InterlockedIncrement(&This->refn); + TRACE("(%p) ref was %d\n", This, ref - 1); + + if(ref == 1) + InterlockedIncrement(&This->numIfaces); + return ref; } static ULONG WINAPI IDirectSoundNotifyImpl_Release(IDirectSoundNotify *iface) { - IDirectSoundCaptureNotifyImpl *This = (IDirectSoundCaptureNotifyImpl *)iface; - ULONG ref = InterlockedDecrement(&(This->ref)); + IDirectSoundCaptureBufferImpl *This = impl_from_IDirectSoundNotify(iface); + ULONG ref = InterlockedDecrement(&This->refn); + TRACE("(%p) ref was %d\n", This, ref + 1); - if (!ref) { - This->dscb->notify=NULL; - IDirectSoundCaptureBuffer_Release((LPDIRECTSOUNDCAPTUREBUFFER)This->dscb); - HeapFree(GetProcessHeap(),0,This); - TRACE("(%p) released\n", This); - } + if (!ref && !InterlockedDecrement(&This->numIfaces)) + capturebuffer_destroy(This); + return ref; } static HRESULT WINAPI IDirectSoundNotifyImpl_SetNotificationPositions(IDirectSoundNotify *iface, DWORD howmuch, const DSBPOSITIONNOTIFY *notify) { - IDirectSoundCaptureNotifyImpl *This = (IDirectSoundCaptureNotifyImpl *)iface; + IDirectSoundCaptureBufferImpl *This = impl_from_IDirectSoundNotify(iface); TRACE("(%p,0x%08x,%p)\n",This,howmuch,notify); if (howmuch > 0 && notify == NULL) { @@ -144,23 +134,23 @@ static HRESULT WINAPI IDirectSoundNotifyImpl_SetNotificationPositions(IDirectSou if (howmuch > 0) { /* Make an internal copy of the caller-supplied array. * Replace the existing copy if one is already present. */ - if (This->dscb->notifies) - This->dscb->notifies = HeapReAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, - This->dscb->notifies, howmuch * sizeof(DSBPOSITIONNOTIFY)); + if (This->notifies) + This->notifies = HeapReAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, This->notifies, + howmuch * sizeof(DSBPOSITIONNOTIFY)); else - This->dscb->notifies = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, - howmuch * sizeof(DSBPOSITIONNOTIFY)); + This->notifies = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, + howmuch * sizeof(DSBPOSITIONNOTIFY)); - if (This->dscb->notifies == NULL) { + if (!This->notifies) { WARN("out of memory\n"); return DSERR_OUTOFMEMORY; } - CopyMemory(This->dscb->notifies, notify, howmuch * sizeof(DSBPOSITIONNOTIFY)); - This->dscb->nrofnotifies = howmuch; + CopyMemory(This->notifies, notify, howmuch * sizeof(DSBPOSITIONNOTIFY)); + This->nrofnotifies = howmuch; } else { - HeapFree(GetProcessHeap(), 0, This->dscb->notifies); - This->dscb->notifies = NULL; - This->dscb->nrofnotifies = 0; + HeapFree(GetProcessHeap(), 0, This->notifies); + This->notifies = NULL; + This->nrofnotifies = 0; } return S_OK; @@ -174,30 +164,6 @@ static const IDirectSoundNotifyVtbl dscnvt = IDirectSoundNotifyImpl_SetNotificationPositions }; -static HRESULT IDirectSoundCaptureNotifyImpl_Create( - IDirectSoundCaptureBufferImpl *dscb, - IDirectSoundCaptureNotifyImpl **pdscn) -{ - IDirectSoundCaptureNotifyImpl * dscn; - TRACE("(%p,%p)\n",dscb,pdscn); - - dscn = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*dscn)); - - if (dscn == NULL) { - WARN("out of memory\n"); - return DSERR_OUTOFMEMORY; - } - - dscn->ref = 0; - dscn->lpVtbl = &dscnvt; - dscn->dscb = dscb; - dscb->notify = dscn; - IDirectSoundCaptureBuffer_AddRef((LPDIRECTSOUNDCAPTUREBUFFER)dscb); - - *pdscn = dscn; - return DS_OK; -} - static const char * const captureStateString[] = { "STATE_STOPPED", @@ -219,7 +185,6 @@ static HRESULT WINAPI IDirectSoundCaptureBufferImpl_QueryInterface(IDirectSoundC REFIID riid, void **ppobj) { IDirectSoundCaptureBufferImpl *This = impl_from_IDirectSoundCaptureBuffer8(iface); - HRESULT hres; TRACE( "(%p,%s,%p)\n", This, debugstr_guid(riid), ppobj ); @@ -230,18 +195,6 @@ static HRESULT WINAPI IDirectSoundCaptureBufferImpl_QueryInterface(IDirectSoundC *ppobj = NULL; - if ( IsEqualGUID( &IID_IDirectSoundNotify, riid ) ) { - if (!This->notify){ - hres = IDirectSoundCaptureNotifyImpl_Create(This, &This->notify); - if(FAILED(hres)) - return hres; - } - - IDirectSoundNotify_AddRef((LPDIRECTSOUNDNOTIFY)This->notify); - *ppobj = This->notify; - return DS_OK; - } - if ( IsEqualGUID( &IID_IDirectSoundCaptureBuffer, riid ) || IsEqualGUID( &IID_IDirectSoundCaptureBuffer8, riid ) ) { IDirectSoundCaptureBuffer8_AddRef(iface); @@ -249,6 +202,12 @@ static HRESULT WINAPI IDirectSoundCaptureBufferImpl_QueryInterface(IDirectSoundC return S_OK; } + if ( IsEqualGUID( &IID_IDirectSoundNotify, riid ) ) { + IDirectSoundNotify_AddRef(&This->IDirectSoundNotify_iface); + *ppobj = &This->IDirectSoundNotify_iface; + return S_OK; + } + FIXME("(%p,%s,%p) unsupported GUID\n", This, debugstr_guid(riid), ppobj); return E_NOINTERFACE; } @@ -707,9 +666,9 @@ static HRESULT IDirectSoundCaptureBufferImpl_Create( This->numIfaces = 0; This->ref = 0; + This->refn = 0; This->device = device; This->device->capture_buffer = This; - This->notify = NULL; This->nrofnotifies = 0; This->pdscbd = HeapAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY, @@ -724,6 +683,7 @@ static HRESULT IDirectSoundCaptureBufferImpl_Create( } This->IDirectSoundCaptureBuffer8_iface.lpVtbl = &dscbvt; + This->IDirectSoundNotify_iface.lpVtbl = &dscnvt; err = IMMDevice_Activate(device->mmdevice, &IID_IAudioClient, CLSCTX_INPROC_SERVER, NULL, (void**)&device->client); diff --git a/dlls/dsound/dsound_private.h b/dlls/dsound/dsound_private.h index 34419d1..da309a1 100644 --- a/dlls/dsound/dsound_private.h +++ b/dlls/dsound/dsound_private.h @@ -49,7 +49,6 @@ typedef struct IDirectSoundBufferImpl IDirectSoundBufferImpl; typedef struct IDirectSoundCaptureImpl IDirectSoundCaptureImpl; typedef struct IDirectSoundCaptureBufferImpl IDirectSoundCaptureBufferImpl; typedef struct IDirectSoundNotifyImpl IDirectSoundNotifyImpl; -typedef struct IDirectSoundCaptureNotifyImpl IDirectSoundCaptureNotifyImpl; typedef struct IDirectSound3DListenerImpl IDirectSound3DListenerImpl; typedef struct IDirectSound3DBufferImpl IDirectSound3DBufferImpl; typedef struct IKsBufferPropertySetImpl IKsBufferPropertySetImpl; @@ -248,15 +247,14 @@ struct DirectSoundCaptureDevice struct IDirectSoundCaptureBufferImpl { IDirectSoundCaptureBuffer8 IDirectSoundCaptureBuffer8_iface; + IDirectSoundNotify IDirectSoundNotify_iface; LONG numIfaces; /* "in use interfaces" refcount */ - LONG ref; + LONG ref, refn; /* IDirectSoundCaptureBuffer fields */ DirectSoundCaptureDevice* device; LPDSCBUFFERDESC pdscbd; DWORD flags; - - /* IDirectSoundCaptureNotifyImpl fields */ - IDirectSoundCaptureNotifyImpl* notify; + /* IDirectSoundNotify fields */ LPDSBPOSITIONNOTIFY notifies; int nrofnotifies; };

1 0

Michael Stefaniuc : dsound: Make capture behave like native in regards to COM aggregation.
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: a3120ce80ecca76c2138bb86a5ed9238659917af URL: http://source.winehq.org/git/wine.git/?a=commit;h=a3120ce80ecca76c2138bb86a… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Jan 12 00:28:21 2012 +0100 dsound: Make capture behave like native in regards to COM aggregation. --- dlls/dsound/capture.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/dlls/dsound/capture.c b/dlls/dsound/capture.c index 29d34dc..90cddb7 100644 --- a/dlls/dsound/capture.c +++ b/dlls/dsound/capture.c @@ -1065,6 +1065,11 @@ static HRESULT WINAPI IDirectSoundCaptureImpl_CreateCaptureBuffer(IDirectSoundCa TRACE( "(%p,%p,%p,%p)\n",iface,lpcDSCBufferDesc,lplpDSCaptureBuffer,pUnk); + if (pUnk) { + WARN("invalid parameter: pUnk != NULL\n"); + return DSERR_NOAGGREGATION; + } + if (lpcDSCBufferDesc == NULL) { WARN("invalid parameter: lpcDSCBufferDesc == NULL)\n"); return DSERR_INVALIDPARAM; @@ -1270,7 +1275,6 @@ HRESULT WINAPI DirectSoundCaptureCreate(LPCGUID lpcGUID, IDirectSoundCapture **p if (pUnkOuter) { WARN("invalid parameter: pUnkOuter != NULL\n"); - *ppDSC = NULL; return DSERR_NOAGGREGATION; }

1 0

Michael Stefaniuc : dsound/tests: Remove some returns in the capture buffer tests.
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: 4ac20cbc0fed529b691ea47ace12e5f0a0e87b48 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4ac20cbc0fed529b691ea47ac… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Jan 12 00:31:18 2012 +0100 dsound/tests: Remove some returns in the capture buffer tests. --- dlls/dsound/tests/capture.c | 12 ------------ 1 files changed, 0 insertions(+), 12 deletions(-) diff --git a/dlls/dsound/tests/capture.c b/dlls/dsound/tests/capture.c index 87a5a59..760f0cc 100644 --- a/dlls/dsound/tests/capture.c +++ b/dlls/dsound/tests/capture.c @@ -379,8 +379,6 @@ static void test_capture_buffer(LPDIRECTSOUNDCAPTURE dsco, (void **)&(state.notify)); ok((rc==DS_OK)&&(state.notify!=NULL), "IDirectSoundCaptureBuffer_QueryInterface() failed: %08x\n", rc); - if (rc!=DS_OK) - return; for (i = 0; i < NOTIFICATIONS; i++) { state.posnotify[i].dwOffset = (i * state.size) + state.size - 1; @@ -390,27 +388,19 @@ static void test_capture_buffer(LPDIRECTSOUNDCAPTURE dsco, rc=IDirectSoundNotify_SetNotificationPositions(state.notify,NOTIFICATIONS, state.posnotify); ok(rc==DS_OK,"IDirectSoundNotify_SetNotificationPositions() failed: %08x\n", rc); - if (rc!=DS_OK) - return; ref=IDirectSoundNotify_Release(state.notify); ok(ref==0,"IDirectSoundNotify_Release(): has %d references, should have " "0\n",ref); - if (ref!=0) - return; if (record) { rc=IDirectSoundCaptureBuffer_Start(dscbo,DSCBSTART_LOOPING); ok(rc==DS_OK,"IDirectSoundCaptureBuffer_Start() failed: %08x\n", rc); - if (rc!=DS_OK) - return; rc=IDirectSoundCaptureBuffer_GetStatus(dscbo,&status); ok(rc==DS_OK,"IDirectSoundCaptureBuffer_GetStatus() failed: %08x\n", rc); ok(status==(DSCBSTATUS_CAPTURING|DSCBSTATUS_LOOPING), "GetStatus: bad status: %x\n",status); - if (rc!=DS_OK) - return; /* wait for the notifications */ for (i = 0; i < (NOTIFICATIONS * 2); i++) { @@ -428,8 +418,6 @@ static void test_capture_buffer(LPDIRECTSOUNDCAPTURE dsco, rc=IDirectSoundCaptureBuffer_Stop(dscbo); ok(rc==DS_OK,"IDirectSoundCaptureBuffer_Stop() failed: %08x\n", rc); - if (rc!=DS_OK) - return; } }

1 0

Michael Stefaniuc : dsound/tests: Sanitize START_TEST() in capture.c.
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: 93bfa2b5be8aea6da5ba82526c7e6f2762250001 URL: http://source.winehq.org/git/wine.git/?a=commit;h=93bfa2b5be8aea6da5ba82526… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Jan 12 00:36:08 2012 +0100 dsound/tests: Sanitize START_TEST() in capture.c. --- dlls/dsound/tests/capture.c | 34 +++++++++++++++------------------- 1 files changed, 15 insertions(+), 19 deletions(-) diff --git a/dlls/dsound/tests/capture.c b/dlls/dsound/tests/capture.c index 760f0cc..02ef46a 100644 --- a/dlls/dsound/tests/capture.c +++ b/dlls/dsound/tests/capture.c @@ -177,7 +177,7 @@ EXIT: "should have 0\n", ref); } -static void IDirectSoundCapture_tests(void) +static void test_capture(void) { HRESULT rc; LPDIRECTSOUNDCAPTURE dsco=NULL; @@ -659,7 +659,7 @@ EXIT: return TRUE; } -static void capture_tests(void) +static void test_enumerate(void) { HRESULT rc; rc=pDirectSoundCaptureEnumerateA(&dscenum_callback,NULL); @@ -673,25 +673,21 @@ START_TEST(capture) CoInitialize(NULL); hDsound = LoadLibrary("dsound.dll"); - if (hDsound) - { - - pDirectSoundCaptureCreate=(void*)GetProcAddress(hDsound, - "DirectSoundCaptureCreate"); - pDirectSoundCaptureEnumerateA=(void*)GetProcAddress(hDsound, - "DirectSoundCaptureEnumerateA"); - if (pDirectSoundCaptureCreate && pDirectSoundCaptureEnumerateA) - { - IDirectSoundCapture_tests(); - capture_tests(); - } - else - skip("capture test skipped\n"); + if (!hDsound) { + skip("dsound.dll not found!\n"); + return; + } - FreeLibrary(hDsound); + pDirectSoundCaptureCreate = (void*)GetProcAddress(hDsound, "DirectSoundCaptureCreate"); + pDirectSoundCaptureEnumerateA = (void*)GetProcAddress(hDsound, "DirectSoundCaptureEnumerateA"); + if (!pDirectSoundCaptureCreate || !pDirectSoundCaptureEnumerateA) { + skip("capture test skipped\n"); + return; } - else - skip("dsound.dll not found!\n"); + test_capture(); + test_enumerate(); + + FreeLibrary(hDsound); CoUninitialize(); }

1 0

Michael Stefaniuc : dsound/tests: Add some COM tests for capture.
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: 66361f1bd532b685cad5ddd82ddea469f0633eac URL: http://source.winehq.org/git/wine.git/?a=commit;h=66361f1bd532b685cad5ddd82… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Jan 12 00:40:31 2012 +0100 dsound/tests: Add some COM tests for capture. --- dlls/dsound/tests/capture.c | 63 +++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 63 insertions(+), 0 deletions(-) diff --git a/dlls/dsound/tests/capture.c b/dlls/dsound/tests/capture.c index 02ef46a..cb53a02 100644 --- a/dlls/dsound/tests/capture.c +++ b/dlls/dsound/tests/capture.c @@ -666,6 +666,68 @@ static void test_enumerate(void) ok(rc==DS_OK,"DirectSoundCaptureEnumerateA() failed: %08x\n", rc); } +static void test_COM(void) +{ + IDirectSoundCapture *dsc = (IDirectSoundCapture*)0xdeadbeef; + IDirectSoundCaptureBuffer *buffer = (IDirectSoundCaptureBuffer*)0xdeadbeef; + IDirectSoundNotify *notify; + DSCBUFFERDESC bufdesc; + WAVEFORMATEX wfx; + HRESULT hr; + ULONG refcount; + + hr = pDirectSoundCaptureCreate(NULL, &dsc, (IUnknown*)0xdeadbeef); + ok(hr == DSERR_NOAGGREGATION, + "DirectSoundCaptureCreate failed: %08x, expected DSERR_NOAGGREGATION\n", hr); + ok(dsc == (IDirectSoundCapture*)0xdeadbeef, "dsc = %p\n", dsc); + + hr = pDirectSoundCaptureCreate(NULL, &dsc, NULL); + if (hr == DSERR_NODRIVER) { + skip("No driver\n"); + return; + } + ok(hr == DS_OK, "DirectSoundCaptureCreate failed: %08x, expected DS_OK\n", hr); + + init_format(&wfx, WAVE_FORMAT_PCM, 44100, 16, 1); + ZeroMemory(&bufdesc, sizeof(bufdesc)); + bufdesc.dwSize = sizeof(bufdesc); + bufdesc.dwBufferBytes = wfx.nAvgBytesPerSec; + bufdesc.lpwfxFormat = &wfx; + + hr = IDirectSoundCapture_CreateCaptureBuffer(dsc, &bufdesc, &buffer, (IUnknown*)0xdeadbeef); + ok(hr == DSERR_NOAGGREGATION, + "IDirectSoundCapture_CreateCaptureBuffer failed: %08x, expected DSERR_NOAGGREGATION\n", hr); + ok(buffer == (IDirectSoundCaptureBuffer*)0xdeadbeef, "buffer = %p\n", buffer); + + hr = IDirectSoundCapture_CreateCaptureBuffer(dsc, &bufdesc, &buffer, NULL); + ok(hr == DS_OK, "IDirectSoundCapture_CreateCaptureBuffer failed: %08x, expected DS_OK\n", hr); + + /* IDirectSoundCaptureBuffer and IDirectSoundNotify have separate refcounts */ + IDirectSoundCaptureBuffer_AddRef(buffer); + refcount = IDirectSoundCaptureBuffer_AddRef(buffer); + ok(refcount == 3, "IDirectSoundCaptureBuffer refcount is %u, expected 3\n", refcount); + hr = IDirectSoundCaptureBuffer_QueryInterface(buffer, &IID_IDirectSoundNotify, (void**)&notify); + ok(hr == DS_OK, "IDirectSoundCapture_QueryInterface failed: %08x, expected DS_OK\n", hr); + refcount = IDirectSoundNotify_AddRef(notify); + ok(refcount == 2, "IDirectSoundNotify refcount is %u, expected 2\n", refcount); + IDirectSoundCaptureBuffer_AddRef(buffer); + refcount = IDirectSoundCaptureBuffer_Release(buffer); + ok(refcount == 3, "IDirectSoundCaptureBuffer refcount is %u, expected 3\n", refcount); + + /* Release IDirectSoundCaptureBuffer while keeping IDirectSoundNotify alive */ + while (IDirectSoundCaptureBuffer_Release(buffer) > 0); + refcount = IDirectSoundNotify_AddRef(notify); + ok(refcount == 3, "IDirectSoundNotify refcount is %u, expected 3\n", refcount); + refcount = IDirectSoundCaptureBuffer_AddRef(buffer); + ok(refcount == 1, "IDirectSoundCaptureBuffer refcount is %u, expected 1\n", refcount); + + while (IDirectSoundNotify_Release(notify) > 0); + refcount = IDirectSoundCaptureBuffer_Release(buffer); + ok(refcount == 0, "IDirectSoundCaptureBuffer refcount is %u, expected 0\n", refcount); + refcount = IDirectSoundCapture_Release(dsc); + ok(refcount == 0, "IDirectSoundCapture refcount is %u, expected 0\n", refcount); +} + START_TEST(capture) { HMODULE hDsound; @@ -685,6 +747,7 @@ START_TEST(capture) return; } + test_COM(); test_capture(); test_enumerate();

1 0

Henri Verbeet : winecfg: Remove the Direct3D settings.
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: e960aa783f78d65feb1d4a93e008d06b0a65fc99 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e960aa783f78d65feb1d4a93e… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Jan 11 21:27:29 2012 +0100 winecfg: Remove the Direct3D settings. There should really be no reason to set any of these. --- po/ar.po | 254 +++++++++++++++++++---------------------- po/bg.po | 255 +++++++++++++++++++---------------------- po/ca.po | 254 +++++++++++++++++++---------------------- po/cs.po | 259 +++++++++++++++++++----------------------- po/da.po | 255 +++++++++++++++++++---------------------- po/de.po | 253 +++++++++++++++++++---------------------- po/el.po | 255 +++++++++++++++++++---------------------- po/en.po | 253 +++++++++++++++++++---------------------- po/en_US.po | 253 +++++++++++++++++++---------------------- po/eo.po | 255 +++++++++++++++++++---------------------- po/es.po | 259 +++++++++++++++++++----------------------- po/fa.po | 254 +++++++++++++++++++---------------------- po/fi.po | 254 +++++++++++++++++++---------------------- po/fr.po | 253 +++++++++++++++++++---------------------- po/he.po | 256 +++++++++++++++++++----------------------- po/hi.po | 253 +++++++++++++++++++---------------------- po/hu.po | 259 +++++++++++++++++++----------------------- po/it.po | 253 +++++++++++++++++++---------------------- po/ja.po | 254 +++++++++++++++++++---------------------- po/ko.po | 253 +++++++++++++++++++---------------------- po/lt.po | 256 +++++++++++++++++++---------------------- po/ml.po | 253 +++++++++++++++++++---------------------- po/nb_NO.po | 255 +++++++++++++++++++---------------------- po/nl.po | 254 +++++++++++++++++++---------------------- po/or.po | 253 +++++++++++++++++++---------------------- po/pa.po | 253 +++++++++++++++++++---------------------- po/pl.po | 254 +++++++++++++++++++---------------------- po/pt_BR.po | 259 +++++++++++++++++++----------------------- po/pt_PT.po | 259 +++++++++++++++++++----------------------- po/rm.po | 254 +++++++++++++++++++---------------------- po/ro.po | 259 +++++++++++++++++++----------------------- po/ru.po | 253 +++++++++++++++++++---------------------- po/sk.po | 255 +++++++++++++++++++---------------------- po/sl.po | 255 +++++++++++++++++++---------------------- po/sr_RS(a)cyrillic.po | 255 +++++++++++++++++++---------------------- po/sr_RS(a)latin.po | 259 +++++++++++++++++++----------------------- po/sv.po | 255 +++++++++++++++++++---------------------- po/te.po | 253 +++++++++++++++++++---------------------- po/th.po | 255 +++++++++++++++++++---------------------- po/tr.po | 255 +++++++++++++++++++---------------------- po/uk.po | 254 +++++++++++++++++++---------------------- po/wa.po | 254 +++++++++++++++++++---------------------- po/wine.pot | 253 +++++++++++++++++++---------------------- po/zh_CN.po | 259 +++++++++++++++++++----------------------- po/zh_TW.po | 259 +++++++++++++++++++----------------------- programs/winecfg/resource.h | 4 - programs/winecfg/winecfg.rc | 21 +--- programs/winecfg/x11drvdlg.c | 53 --------- 48 files changed, 5226 insertions(+), 6329 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=e960aa783f78d65feb1d4…

1 0

Henri Verbeet : wined3d: Hold the wined3d mutex in device_process_message() .
by Alexandre Julliard 12 Jan '12

12 Jan '12

Module: wine Branch: master Commit: be3a9b5ecb7f15f908323cf4b7ad09f3035a62ca URL: http://source.winehq.org/git/wine.git/?a=commit;h=be3a9b5ecb7f15f908323cf4b… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Jan 11 21:27:30 2012 +0100 wined3d: Hold the wined3d mutex in device_process_message(). --- dlls/wined3d/device.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 3551a14..b5d5111 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -6020,8 +6020,12 @@ void get_drawable_size_backbuffer(const struct wined3d_context *context, UINT *w LRESULT device_process_message(struct wined3d_device *device, HWND window, BOOL unicode, UINT message, WPARAM wparam, LPARAM lparam, WNDPROC proc) { + wined3d_mutex_lock(); + if (device->filter_messages) { + wined3d_mutex_unlock(); + TRACE("Filtering message: window %p, message %#x, wparam %#lx, lparam %#lx.\n", window, message, wparam, lparam); if (unicode) @@ -6043,6 +6047,8 @@ LRESULT device_process_message(struct wined3d_device *device, HWND window, BOOL device->device_parent->ops->mode_changed(device->device_parent); } + wined3d_mutex_unlock(); + if (unicode) return CallWindowProcW(proc, window, message, wparam, lparam); else

1 0