February 2020 - Wine-devel - list.winehq.org

[PATCH 2/2 v2] dsound: Use an SRW lock for the buffer lock.
by Andrew Eikum 03 Feb '20

03 Feb '20

From: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> --- v2: Better match surrounding indent styles. dlls/dsound/buffer.c | 65 +++++++++++++----------------------- dlls/dsound/capture.c | 1 - dlls/dsound/dsound.c | 1 - dlls/dsound/dsound_convert.c | 1 - dlls/dsound/dsound_main.c | 1 - dlls/dsound/dsound_private.h | 2 +- dlls/dsound/duplex.c | 1 - dlls/dsound/mixer.c | 5 ++- dlls/dsound/primary.c | 5 ++- dlls/dsound/propset.c | 1 - dlls/dsound/sound3d.c | 1 - 11 files changed, 29 insertions(+), 55 deletions(-) diff --git a/dlls/dsound/buffer.c b/dlls/dsound/buffer.c index d585a7605ef..8d4a3b92fa6 100644 --- a/dlls/dsound/buffer.c +++ b/dlls/dsound/buffer.c @@ -27,7 +27,6 @@ #include "winbase.h" #include "winuser.h" #include "mmsystem.h" -#include "winternl.h" #include "vfwmsgs.h" #include "wine/debug.h" #include "dsound.h" @@ -200,8 +199,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_SetVolume(IDirectSoundBuffer8 *ifac return DSERR_INVALIDPARAM; } - /* **** */ - RtlAcquireResourceExclusive(&This->lock, TRUE); + AcquireSRWLockExclusive(&This->lock); if (This->dsbd.dwFlags & DSBCAPS_CTRL3D) { oldVol = This->ds3db_lVolume; @@ -216,8 +214,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_SetVolume(IDirectSoundBuffer8 *ifac DSOUND_RecalcVolPan(&(This->volpan)); } - RtlReleaseResource(&This->lock); - /* **** */ + ReleaseSRWLockExclusive(&This->lock); return hres; } @@ -268,8 +265,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_SetFrequency(IDirectSoundBuffer8 *i return DSERR_INVALIDPARAM; } - /* **** */ - RtlAcquireResourceExclusive(&This->lock, TRUE); + AcquireSRWLockExclusive(&This->lock); oldFreq = This->freq; This->freq = freq; @@ -280,8 +276,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_SetFrequency(IDirectSoundBuffer8 *i DSOUND_RecalcFormat(This); } - RtlReleaseResource(&This->lock); - /* **** */ + ReleaseSRWLockExclusive(&This->lock); return DS_OK; } @@ -295,8 +290,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Play(IDirectSoundBuffer8 *iface, DW TRACE("(%p,%08x,%08x,%08x)\n",This,reserved1,reserved2,flags); - /* **** */ - RtlAcquireResourceExclusive(&This->lock, TRUE); + AcquireSRWLockExclusive(&This->lock); This->playflags = flags; if (This->state == STATE_STOPPED) { @@ -309,8 +303,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Play(IDirectSoundBuffer8 *iface, DW IMediaObject_Discontinuity(This->filters[i].obj, 0); } - RtlReleaseResource(&This->lock); - /* **** */ + ReleaseSRWLockExclusive(&This->lock); return hres; } @@ -322,8 +315,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Stop(IDirectSoundBuffer8 *iface) TRACE("(%p)\n",This); - /* **** */ - RtlAcquireResourceExclusive(&This->lock, TRUE); + AcquireSRWLockExclusive(&This->lock); if (This->state == STATE_PLAYING) This->state = STATE_STOPPING; @@ -333,8 +325,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Stop(IDirectSoundBuffer8 *iface) DSOUND_CheckEvent(This, 0, 0); } - RtlReleaseResource(&This->lock); - /* **** */ + ReleaseSRWLockExclusive(&This->lock); return hres; } @@ -382,7 +373,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_GetCurrentPosition(IDirectSoundBuff TRACE("(%p,%p,%p)\n",This,playpos,writepos); - RtlAcquireResourceShared(&This->lock, TRUE); + AcquireSRWLockShared(&This->lock); pos = This->sec_mixpos; @@ -403,7 +394,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_GetCurrentPosition(IDirectSoundBuff *writepos %= This->buflen; } - RtlReleaseResource(&This->lock); + ReleaseSRWLockShared(&This->lock); TRACE("playpos = %d, writepos = %d, buflen=%d (%p, time=%d)\n", playpos?*playpos:-1, writepos?*writepos:-1, This->buflen, This, GetTickCount()); @@ -423,13 +414,13 @@ static HRESULT WINAPI IDirectSoundBufferImpl_GetStatus(IDirectSoundBuffer8 *ifac } *status = 0; - RtlAcquireResourceShared(&This->lock, TRUE); + AcquireSRWLockShared(&This->lock); if ((This->state == STATE_STARTING) || (This->state == STATE_PLAYING)) { *status |= DSBSTATUS_PLAYING; if (This->playflags & DSBPLAY_LOOPING) *status |= DSBSTATUS_LOOPING; } - RtlReleaseResource(&This->lock); + ReleaseSRWLockShared(&This->lock); TRACE("status=%x\n", *status); return DS_OK; @@ -509,8 +500,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Lock(IDirectSoundBuffer8 *iface, DW return DSERR_INVALIDPARAM; } - /* **** */ - RtlAcquireResourceShared(&This->lock, TRUE); + AcquireSRWLockShared(&This->lock); if (writecursor+writebytes <= This->buflen) { *(LPBYTE*)lplpaudioptr1 = This->buffer->memory+writecursor; @@ -543,8 +533,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Lock(IDirectSoundBuffer8 *iface, DW TRACE("Locked %p(%i bytes) and %p(%i bytes) writecursor=%d\n", *(LPBYTE*)lplpaudioptr1, *audiobytes1, lplpaudioptr2 ? *(LPBYTE*)lplpaudioptr2 : NULL, audiobytes2 ? *audiobytes2: 0, writecursor); } - RtlReleaseResource(&This->lock); - /* **** */ + ReleaseSRWLockShared(&This->lock); return DS_OK; } @@ -557,8 +546,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_SetCurrentPosition(IDirectSoundBuff TRACE("(%p,%d)\n",This,newpos); - /* **** */ - RtlAcquireResourceExclusive(&This->lock, TRUE); + AcquireSRWLockExclusive(&This->lock); /* start mixing from this new location instead */ newpos %= This->buflen; @@ -568,8 +556,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_SetCurrentPosition(IDirectSoundBuff /* at this point, do not attempt to reset buffers, mess with primary mix position, or anything like that to reduce latency. The data already prebuffered cannot be changed */ - RtlReleaseResource(&This->lock); - /* **** */ + ReleaseSRWLockExclusive(&This->lock); return hres; } @@ -591,16 +578,14 @@ static HRESULT WINAPI IDirectSoundBufferImpl_SetPan(IDirectSoundBuffer8 *iface, return DSERR_CONTROLUNAVAIL; } - /* **** */ - RtlAcquireResourceExclusive(&This->lock, TRUE); + AcquireSRWLockExclusive(&This->lock); if (This->volpan.lPan != pan) { This->volpan.lPan = pan; DSOUND_RecalcVolPan(&(This->volpan)); } - RtlReleaseResource(&This->lock); - /* **** */ + ReleaseSRWLockExclusive(&This->lock); return hres; } @@ -646,7 +631,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Unlock(IDirectSoundBuffer8 *iface, AcquireSRWLockShared(&This->device->buffer_list_lock); LIST_FOR_EACH_ENTRY(iter, &This->buffer->buffers, IDirectSoundBufferImpl, entry ) { - RtlAcquireResourceShared(&iter->lock, TRUE); + AcquireSRWLockShared(&iter->lock); if (x1) { if(x1 + (DWORD_PTR)p1 - (DWORD_PTR)iter->buffer->memory > iter->buflen) @@ -662,7 +647,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Unlock(IDirectSoundBuffer8 *iface, else iter->buffer->lockedbytes -= x2; } - RtlReleaseResource(&iter->lock); + ReleaseSRWLockShared(&iter->lock); } ReleaseSRWLockShared(&This->device->buffer_list_lock); } @@ -1125,7 +1110,7 @@ HRESULT secondarybuffer_create(DirectSoundDevice *device, const DSBUFFERDESC *ds } else DSOUND_RecalcVolPan(&(dsb->volpan)); - RtlInitializeResource(&dsb->lock); + InitializeSRWLock(&dsb->lock); /* register buffer */ err = DirectSoundDevice_AddBuffer(device, dsb); @@ -1148,7 +1133,6 @@ void secondarybuffer_destroy(IDirectSoundBufferImpl *This) This->device->drvcaps.dwFreeHwMixingAllBuffers++; DirectSoundDevice_RemoveBuffer(This->device, This); - RtlDeleteResource(&This->lock); This->buffer->ref--; list_remove(&This->entry); @@ -1190,13 +1174,13 @@ HRESULT IDirectSoundBufferImpl_Duplicate( return DSERR_OUTOFMEMORY; } - RtlAcquireResourceShared(&pdsb->lock, TRUE); + AcquireSRWLockShared(&pdsb->lock); CopyMemory(dsb, pdsb, sizeof(*dsb)); dsb->pwfx = DSOUND_CopyFormat(pdsb->pwfx); - RtlReleaseResource(&pdsb->lock); + ReleaseSRWLockShared(&pdsb->lock); if (dsb->pwfx == NULL) { HeapFree(GetProcessHeap(),0,dsb); @@ -1218,12 +1202,11 @@ HRESULT IDirectSoundBufferImpl_Duplicate( dsb->device = device; DSOUND_RecalcFormat(dsb); - RtlInitializeResource(&dsb->lock); + InitializeSRWLock(&dsb->lock); /* register buffer */ hres = DirectSoundDevice_AddBuffer(device, dsb); if (hres != DS_OK) { - RtlDeleteResource(&dsb->lock); list_remove(&dsb->entry); dsb->buffer->ref--; HeapFree(GetProcessHeap(),0,dsb->pwfx); diff --git a/dlls/dsound/capture.c b/dlls/dsound/capture.c index 8fc27499998..10690f7df0d 100644 --- a/dlls/dsound/capture.c +++ b/dlls/dsound/capture.c @@ -34,7 +34,6 @@ #include "winuser.h" #include "mmsystem.h" #include "mmddk.h" -#include "winternl.h" #include "winnls.h" #include "wine/debug.h" #include "dsound.h" diff --git a/dlls/dsound/dsound.c b/dlls/dsound/dsound.c index e7ab7922bb7..3085d86e5b5 100644 --- a/dlls/dsound/dsound.c +++ b/dlls/dsound/dsound.c @@ -30,7 +30,6 @@ #include "windef.h" #include "winbase.h" #include "winuser.h" -#include "winternl.h" #include "mmddk.h" #include "wingdi.h" #include "mmreg.h" diff --git a/dlls/dsound/dsound_convert.c b/dlls/dsound/dsound_convert.c index 6181f04be73..4735178a9a8 100644 --- a/dlls/dsound/dsound_convert.c +++ b/dlls/dsound/dsound_convert.c @@ -41,7 +41,6 @@ #include "windef.h" #include "winbase.h" #include "mmsystem.h" -#include "winternl.h" #include "wine/debug.h" #include "dsound.h" #include "dsound_private.h" diff --git a/dlls/dsound/dsound_main.c b/dlls/dsound/dsound_main.c index 4e752be6bae..3acee554ad9 100644 --- a/dlls/dsound/dsound_main.c +++ b/dlls/dsound/dsound_main.c @@ -43,7 +43,6 @@ #include "winnls.h" #include "winreg.h" #include "mmsystem.h" -#include "winternl.h" #include "mmddk.h" #include "wine/debug.h" #include "dsound.h" diff --git a/dlls/dsound/dsound_private.h b/dlls/dsound/dsound_private.h index 5605bded905..bdb9ebee544 100644 --- a/dlls/dsound/dsound_private.h +++ b/dlls/dsound/dsound_private.h @@ -136,7 +136,7 @@ struct IDirectSoundBufferImpl LONG ref, refn, ref3D, refiks; /* IDirectSoundBufferImpl fields */ DirectSoundDevice* device; - RTL_RWLOCK lock; + SRWLOCK lock; PWAVEFORMATEX pwfx; BufferMemory* buffer; DWORD playflags,state,leadin; diff --git a/dlls/dsound/duplex.c b/dlls/dsound/duplex.c index 4a88dd89b96..cccfeba0f06 100644 --- a/dlls/dsound/duplex.c +++ b/dlls/dsound/duplex.c @@ -28,7 +28,6 @@ #include "winuser.h" #include "mmsystem.h" #include "mmddk.h" -#include "winternl.h" #include "wine/debug.h" #include "dsound.h" #include "dsound_private.h" diff --git a/dlls/dsound/mixer.c b/dlls/dsound/mixer.c index c1bd3273f58..a6402b09eff 100644 --- a/dlls/dsound/mixer.c +++ b/dlls/dsound/mixer.c @@ -33,7 +33,6 @@ #include "mmsystem.h" #include "wingdi.h" #include "mmreg.h" -#include "winternl.h" #include "wine/debug.h" #include "dsound.h" #include "ks.h" @@ -593,7 +592,7 @@ static void DSOUND_MixToPrimary(const DirectSoundDevice *device, float *mix_buff if (dsb->buflen && dsb->state) { TRACE("Checking %p, frames=%d\n", dsb, frames); - RtlAcquireResourceShared(&dsb->lock, TRUE); + AcquireSRWLockShared(&dsb->lock); /* if buffer is stopping it is stopped now */ if (dsb->state == STATE_STOPPING) { dsb->state = STATE_STOPPED; @@ -609,7 +608,7 @@ static void DSOUND_MixToPrimary(const DirectSoundDevice *device, float *mix_buff *all_stopped = FALSE; } - RtlReleaseResource(&dsb->lock); + ReleaseSRWLockShared(&dsb->lock); } } } diff --git a/dlls/dsound/primary.c b/dlls/dsound/primary.c index 8d42fe0ed90..655f730b375 100644 --- a/dlls/dsound/primary.c +++ b/dlls/dsound/primary.c @@ -32,7 +32,6 @@ #include "winbase.h" #include "winuser.h" #include "mmsystem.h" -#include "winternl.h" #include "mmddk.h" #include "wine/debug.h" #include "dsound.h" @@ -274,9 +273,9 @@ static HRESULT DSOUND_PrimaryOpen(DirectSoundDevice *device, WAVEFORMATEX *wfx, device->playpos = 0; for (i = 0; i < device->nrofbuffers; i++) { - RtlAcquireResourceExclusive(&dsb[i]->lock, TRUE); + AcquireSRWLockExclusive(&dsb[i]->lock); DSOUND_RecalcFormat(dsb[i]); - RtlReleaseResource(&dsb[i]->lock); + ReleaseSRWLockExclusive(&dsb[i]->lock); } return DS_OK; diff --git a/dlls/dsound/propset.c b/dlls/dsound/propset.c index 92aae8fe0ad..dcb2b871783 100644 --- a/dlls/dsound/propset.c +++ b/dlls/dsound/propset.c @@ -28,7 +28,6 @@ #include "winbase.h" #include "winuser.h" #include "mmsystem.h" -#include "winternl.h" #include "winnls.h" #include "vfwmsgs.h" #include "mmddk.h" diff --git a/dlls/dsound/sound3d.c b/dlls/dsound/sound3d.c index 27fc416676f..06dbf58ca0b 100644 --- a/dlls/dsound/sound3d.c +++ b/dlls/dsound/sound3d.c @@ -44,7 +44,6 @@ #include "winbase.h" #include "winuser.h" #include "mmsystem.h" -#include "winternl.h" #include "mmddk.h" #include "wine/debug.h" #include "dsound.h" -- 2.25.0

1 0

[PATCH 1/2 v2] dsound: Use an SRW lock for buffer_list_lock.
by Andrew Eikum 03 Feb '20

03 Feb '20

From: Zebediah Figura <z.figura12(a)gmail.com> Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=48408 Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> --- v2: No changes. dlls/dsound/buffer.c | 4 ++-- dlls/dsound/dsound.c | 15 +++++++-------- dlls/dsound/dsound_private.h | 2 +- dlls/dsound/mixer.c | 4 ++-- dlls/dsound/primary.c | 4 ++-- 5 files changed, 14 insertions(+), 15 deletions(-) diff --git a/dlls/dsound/buffer.c b/dlls/dsound/buffer.c index f6006529141..d585a7605ef 100644 --- a/dlls/dsound/buffer.c +++ b/dlls/dsound/buffer.c @@ -643,7 +643,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Unlock(IDirectSoundBuffer8 *iface, if (x1 || x2) { - RtlAcquireResourceShared(&This->device->buffer_list_lock, TRUE); + AcquireSRWLockShared(&This->device->buffer_list_lock); LIST_FOR_EACH_ENTRY(iter, &This->buffer->buffers, IDirectSoundBufferImpl, entry ) { RtlAcquireResourceShared(&iter->lock, TRUE); @@ -664,7 +664,7 @@ static HRESULT WINAPI IDirectSoundBufferImpl_Unlock(IDirectSoundBuffer8 *iface, } RtlReleaseResource(&iter->lock); } - RtlReleaseResource(&This->device->buffer_list_lock); + ReleaseSRWLockShared(&This->device->buffer_list_lock); } return hres; diff --git a/dlls/dsound/dsound.c b/dlls/dsound/dsound.c index 8e2b839fe89..e7ab7922bb7 100644 --- a/dlls/dsound/dsound.c +++ b/dlls/dsound/dsound.c @@ -181,7 +181,7 @@ static HRESULT DirectSoundDevice_Create(DirectSoundDevice ** ppDevice) InitializeCriticalSection(&(device->mixlock)); device->mixlock.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": DirectSoundDevice.mixlock"); - RtlInitializeResource(&(device->buffer_list_lock)); + InitializeSRWLock(&device->buffer_list_lock); *ppDevice = device; @@ -238,7 +238,6 @@ static ULONG DirectSoundDevice_Release(DirectSoundDevice * device) HeapFree(GetProcessHeap(), 0, device->tmp_buffer); HeapFree(GetProcessHeap(), 0, device->cp_buffer); HeapFree(GetProcessHeap(), 0, device->buffer); - RtlDeleteResource(&device->buffer_list_lock); device->mixlock.DebugInfo->Spare[0] = 0; DeleteCriticalSection(&device->mixlock); HeapFree(GetProcessHeap(),0,device); @@ -612,7 +611,7 @@ HRESULT DirectSoundDevice_AddBuffer( TRACE("(%p, %p)\n", device, pDSB); - RtlAcquireResourceExclusive(&(device->buffer_list_lock), TRUE); + AcquireSRWLockExclusive(&device->buffer_list_lock); if (device->buffers) newbuffers = HeapReAlloc(GetProcessHeap(),0,device->buffers,sizeof(IDirectSoundBufferImpl*)*(device->nrofbuffers+1)); @@ -629,7 +628,7 @@ HRESULT DirectSoundDevice_AddBuffer( hr = DSERR_OUTOFMEMORY; } - RtlReleaseResource(&(device->buffer_list_lock)); + ReleaseSRWLockExclusive(&device->buffer_list_lock); return hr; } @@ -644,7 +643,7 @@ void DirectSoundDevice_RemoveBuffer(DirectSoundDevice * device, IDirectSoundBuff TRACE("(%p, %p)\n", device, pDSB); - RtlAcquireResourceExclusive(&(device->buffer_list_lock), TRUE); + AcquireSRWLockExclusive(&device->buffer_list_lock); if (device->nrofbuffers == 1) { assert(device->buffers[0] == pDSB); @@ -662,7 +661,7 @@ void DirectSoundDevice_RemoveBuffer(DirectSoundDevice * device, IDirectSoundBuff device->nrofbuffers--; TRACE("buffer count is now %d\n", device->nrofbuffers); - RtlReleaseResource(&(device->buffer_list_lock)); + ReleaseSRWLockExclusive(&device->buffer_list_lock); } /******************************************************************************* @@ -869,14 +868,14 @@ static HRESULT WINAPI IDirectSound8Impl_SetCooperativeLevel(IDirectSound8 *iface level==DSSCL_PRIORITY ? "DSSCL_PRIORITY" : "DSSCL_EXCLUSIVE"); } - RtlAcquireResourceExclusive(&device->buffer_list_lock, TRUE); + AcquireSRWLockExclusive(&device->buffer_list_lock); EnterCriticalSection(&device->mixlock); if ((level == DSSCL_WRITEPRIMARY) != (device->priolevel == DSSCL_WRITEPRIMARY)) hr = DSOUND_ReopenDevice(device, level == DSSCL_WRITEPRIMARY); if (SUCCEEDED(hr)) device->priolevel = level; LeaveCriticalSection(&device->mixlock); - RtlReleaseResource(&device->buffer_list_lock); + ReleaseSRWLockExclusive(&device->buffer_list_lock); return hr; } diff --git a/dlls/dsound/dsound_private.h b/dlls/dsound/dsound_private.h index 930c3e92ec8..5605bded905 100644 --- a/dlls/dsound/dsound_private.h +++ b/dlls/dsound/dsound_private.h @@ -80,7 +80,7 @@ struct DirectSoundDevice DWORD writelead, buflen, ac_frames, frag_frames, playpos, pad, stopped; int nrofbuffers; IDirectSoundBufferImpl** buffers; - RTL_RWLOCK buffer_list_lock; + SRWLOCK buffer_list_lock; CRITICAL_SECTION mixlock; IDirectSoundBufferImpl *primary; DWORD speaker_config; diff --git a/dlls/dsound/mixer.c b/dlls/dsound/mixer.c index 05ca99b5df4..c1bd3273f58 100644 --- a/dlls/dsound/mixer.c +++ b/dlls/dsound/mixer.c @@ -767,9 +767,9 @@ DWORD CALLBACK DSOUND_mixthread(void *p) if (!dev->ref) break; - RtlAcquireResourceShared(&(dev->buffer_list_lock), TRUE); + AcquireSRWLockShared(&dev->buffer_list_lock); DSOUND_PerformMix(dev); - RtlReleaseResource(&(dev->buffer_list_lock)); + ReleaseSRWLockShared(&dev->buffer_list_lock); } return 0; } diff --git a/dlls/dsound/primary.c b/dlls/dsound/primary.c index 852ec51b7ff..8d42fe0ed90 100644 --- a/dlls/dsound/primary.c +++ b/dlls/dsound/primary.c @@ -472,7 +472,7 @@ HRESULT primarybuffer_SetFormat(DirectSoundDevice *device, LPCWAVEFORMATEX passe } /* **** */ - RtlAcquireResourceExclusive(&(device->buffer_list_lock), TRUE); + AcquireSRWLockExclusive(&device->buffer_list_lock); EnterCriticalSection(&(device->mixlock)); if (device->priolevel == DSSCL_WRITEPRIMARY) { @@ -508,7 +508,7 @@ HRESULT primarybuffer_SetFormat(DirectSoundDevice *device, LPCWAVEFORMATEX passe out: LeaveCriticalSection(&(device->mixlock)); - RtlReleaseResource(&(device->buffer_list_lock)); + ReleaseSRWLockExclusive(&device->buffer_list_lock); /* **** */ return err; -- 2.25.0

1 0

[PATCH] mshtml: use ARRAY_SIZE instead of sizeof in MultiByteToWideChar
by Marcus Meissner 03 Feb '20

03 Feb '20

Signed-off-by: Marcus Meissner <meissner(a)suse.de> --- dlls/mshtml/tests/dom.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index 1c4a54cf63..2b0a5cde95 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -6370,7 +6370,7 @@ static void test_navigator(IHTMLDocument2 *doc) hres = ObtainUserAgentString(0, bufa, &size); ok(hres == S_OK, "ObtainUserAgentString failed: %08x\n", hres); - MultiByteToWideChar(CP_ACP, 0, bufa, size, buf, sizeof(buf)); + MultiByteToWideChar(CP_ACP, 0, bufa, size, buf, ARRAY_SIZE(buf)); bstr = NULL; hres = IOmNavigator_get_userAgent(navigator, &bstr); @@ -6390,7 +6390,7 @@ static void test_navigator(IHTMLDocument2 *doc) hres = UrlMkSetSessionOption(URLMON_OPTION_USERAGENT, ua, sizeof(ua), 0); ok(hres == S_OK, "UrlMkSetSessionOption failed: %08x\n", hres); - MultiByteToWideChar(CP_ACP, 0, ua, -1, buf, sizeof(buf)); + MultiByteToWideChar(CP_ACP, 0, ua, -1, buf, ARRAY_SIZE(buf)); hres = IOmNavigator_get_appVersion(navigator, &bstr); ok(hres == S_OK, "get_appVersion failed: %08x\n", hres); -- 2.25.0

2 1

[PATCH 1/2] dwrite: Do not create intermediate object to return face reference from family.
by Nikolay Sivov 03 Feb '20

03 Feb '20

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/dwrite/font.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/dlls/dwrite/font.c b/dlls/dwrite/font.c index 507099b24e..a3d36a5206 100644 --- a/dlls/dwrite/font.c +++ b/dlls/dwrite/font.c @@ -2429,21 +2429,19 @@ static HRESULT WINAPI dwritefontfamily1_GetFont(IDWriteFontFamily2 *iface, UINT3 static HRESULT WINAPI dwritefontfamily1_GetFontFaceReference(IDWriteFontFamily2 *iface, UINT32 index, IDWriteFontFaceReference **reference) { - IDWriteFont3 *font; - HRESULT hr; + struct dwrite_fontfamily *family = impl_from_IDWriteFontFamily2(iface); + const struct dwrite_font_data *font; TRACE("%p, %u, %p.\n", iface, index, reference); *reference = NULL; - hr = IDWriteFontFamily2_GetFont(iface, index, &font); - if (FAILED(hr)) - return hr; - - hr = IDWriteFont3_GetFontFaceReference(font, reference); - IDWriteFont3_Release(font); + if (index >= family->data->count) + return E_FAIL; - return hr; + font = family->data->fonts[index]; + return IDWriteFactory5_CreateFontFaceReference_((IDWriteFactory5 *)family->collection->factory, + font->file, font->face_index, font->simulations, reference); } static HRESULT WINAPI dwritefontfamily2_GetMatchingFonts(IDWriteFontFamily2 *iface, -- 2.24.1

2 2

[PATCH 8/8] jscript: Add GetSourcePosition implementation.
by Jacek Caban 03 Feb '20

03 Feb '20

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/jscript/jscript.c | 25 +++++++++++++++++++++---- dlls/jscript/tests/run.c | 3 --- 2 files changed, 21 insertions(+), 7 deletions(-)

1 0

[PATCH 7/8] jscript: Return error source and description from GetExceptionInfo.
by Jacek Caban 03 Feb '20

03 Feb '20

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/jscript/jscript.c | 4 ++++ dlls/jscript/tests/run.c | 3 +-- 2 files changed, 5 insertions(+), 2 deletions(-)

1 0

[PATCH 6/8] jscript: Propagate compiler error location.
by Jacek Caban 03 Feb '20

03 Feb '20

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/jscript/compile.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)

1 0

[PATCH 5/8] jscript: Propagate parser error location.
by Jacek Caban 03 Feb '20

03 Feb '20

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/jscript/compile.c | 2 +- dlls/jscript/parser.h | 4 +++- dlls/jscript/parser.y | 29 ++++++++++++++++++----------- 3 files changed, 22 insertions(+), 13 deletions(-)

1 0

[PATCH 4/8] jscript: Delay error object creation for builtin errors with message string until it's needed.
by Jacek Caban 03 Feb '20

03 Feb '20

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/jscript/dispex.c | 12 ++++++------ dlls/jscript/engine.c | 10 ++++------ dlls/jscript/error.c | 41 ++++++++--------------------------------- dlls/jscript/jscript.h | 2 +- 4 files changed, 19 insertions(+), 46 deletions(-)

1 0

[PATCH 3/8] jscript: Support propagating error message.
by Jacek Caban 03 Feb '20

03 Feb '20

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/jscript/dispex.c | 2 ++ dlls/jscript/engine.c | 14 ++++++++++++-- dlls/jscript/engine.h | 1 + dlls/jscript/error.c | 4 +++- dlls/jscript/jscript.c | 4 ++++ 5 files changed, 22 insertions(+), 3 deletions(-)

1 0