July 2008 - Wine-commits - list.winehq.org

Rob Shearman : rpcrt4: Clear the memory of embedded complex types in ComplexUnmarshall when fMustAlloc is TRUE to avoid passing uninitialised memory to the unmarshaller .
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: b75828fc90a28470694d1d7695bfed66f8f0b854 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b75828fc90a28470694d1d769… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Sun Jul 6 11:55:59 2008 +0100 rpcrt4: Clear the memory of embedded complex types in ComplexUnmarshall when fMustAlloc is TRUE to avoid passing uninitialised memory to the unmarshaller. Found by Valgrind. --- dlls/rpcrt4/ndr_marshall.c | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/dlls/rpcrt4/ndr_marshall.c b/dlls/rpcrt4/ndr_marshall.c index 4dcd500..3d06cf1 100644 --- a/dlls/rpcrt4/ndr_marshall.c +++ b/dlls/rpcrt4/ndr_marshall.c @@ -2426,6 +2426,13 @@ static unsigned char * ComplexUnmarshall(PMIDL_STUB_MESSAGE pStubMsg, desc = pFormat + *(const SHORT*)pFormat; size = EmbeddedComplexSize(pStubMsg, desc); TRACE("embedded complex (size=%ld) => %p\n", size, pMemory); + if (fMustAlloc) + /* we can't pass fMustAlloc=TRUE into the marshaller for this type + * since the type is part of the memory block that is encompassed by + * the whole complex type. Memory is forced to allocate when pointers + * are set to NULL, so we emulate that part of fMustAlloc=TRUE by + * clearing the memory we pass in to the unmarshaller */ + memset(pMemory, 0, size); m = NdrUnmarshaller[*desc & NDR_TABLE_MASK]; if (m) {

1 0

Alexandre Julliard : rpcrt4: Use interlocked functions to increment/ decrement ref counts.
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: 6f8749b7bcf60cf72d43742367d930229dd54890 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6f8749b7bcf60cf72d4374236… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Jul 7 12:09:45 2008 +0200 rpcrt4: Use interlocked functions to increment/decrement ref counts. --- dlls/rpcrt4/cpsf.c | 6 +++--- dlls/rpcrt4/ndr_ole.c | 11 ++++++----- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/dlls/rpcrt4/cpsf.c b/dlls/rpcrt4/cpsf.c index 23c4aeb..373c97a 100644 --- a/dlls/rpcrt4/cpsf.c +++ b/dlls/rpcrt4/cpsf.c @@ -65,7 +65,7 @@ static HRESULT WINAPI CStdPSFactory_QueryInterface(LPPSFACTORYBUFFER iface, if (IsEqualGUID(&IID_IUnknown,riid) || IsEqualGUID(&IID_IPSFactoryBuffer,riid)) { *obj = This; - This->RefCount++; + InterlockedIncrement( &This->RefCount ); return S_OK; } return E_NOINTERFACE; @@ -75,14 +75,14 @@ static ULONG WINAPI CStdPSFactory_AddRef(LPPSFACTORYBUFFER iface) { CStdPSFactoryBuffer *This = (CStdPSFactoryBuffer *)iface; TRACE("(%p)->AddRef()\n",iface); - return ++(This->RefCount); + return InterlockedIncrement( &This->RefCount ); } static ULONG WINAPI CStdPSFactory_Release(LPPSFACTORYBUFFER iface) { CStdPSFactoryBuffer *This = (CStdPSFactoryBuffer *)iface; TRACE("(%p)->Release()\n",iface); - return --(This->RefCount); + return InterlockedDecrement( &This->RefCount ); } static HRESULT WINAPI CStdPSFactory_CreateProxy(LPPSFACTORYBUFFER iface, diff --git a/dlls/rpcrt4/ndr_ole.c b/dlls/rpcrt4/ndr_ole.c index edfc78c..9bfab95 100644 --- a/dlls/rpcrt4/ndr_ole.c +++ b/dlls/rpcrt4/ndr_ole.c @@ -79,7 +79,7 @@ static HMODULE LoadCOM(void) typedef struct RpcStreamImpl { const IStreamVtbl *lpVtbl; - DWORD RefCount; + LONG RefCount; PMIDL_STUB_MESSAGE pMsg; LPDWORD size; unsigned char *data; @@ -95,7 +95,7 @@ static HRESULT WINAPI RpcStream_QueryInterface(LPSTREAM iface, IsEqualGUID(&IID_ISequentialStream, riid) || IsEqualGUID(&IID_IStream, riid)) { *obj = This; - This->RefCount++; + InterlockedIncrement( &This->RefCount ); return S_OK; } return E_NOINTERFACE; @@ -104,19 +104,20 @@ static HRESULT WINAPI RpcStream_QueryInterface(LPSTREAM iface, static ULONG WINAPI RpcStream_AddRef(LPSTREAM iface) { RpcStreamImpl *This = (RpcStreamImpl *)iface; - return ++(This->RefCount); + return InterlockedIncrement( &This->RefCount ); } static ULONG WINAPI RpcStream_Release(LPSTREAM iface) { RpcStreamImpl *This = (RpcStreamImpl *)iface; - if (!--(This->RefCount)) { + ULONG ref = InterlockedDecrement( &This->RefCount ); + if (!ref) { TRACE("size=%d\n", *This->size); This->pMsg->Buffer = This->data + *This->size; HeapFree(GetProcessHeap(),0,This); return 0; } - return This->RefCount; + return ref; } static HRESULT WINAPI RpcStream_Read(LPSTREAM iface,

1 0

Vitaliy Margolen : dinput: Add more traces. Prevent some traces from dereferencing NULL pointer.
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: 85e2aa1d8adb6ef9e160ed8c2f8e1a50bb29bba0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=85e2aa1d8adb6ef9e160ed8c2… Author: Vitaliy Margolen <wine-patches(a)kievinfo.com> Date: Sat Jul 5 22:24:37 2008 -0600 dinput: Add more traces. Prevent some traces from dereferencing NULL pointer. --- dlls/dinput/device.c | 2 +- dlls/dinput/effect_linuxinput.c | 6 +++--- dlls/dinput/joystick_linuxinput.c | 2 ++ 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/dlls/dinput/device.c b/dlls/dinput/device.c index 2c4cc6d..54fefbe 100644 --- a/dlls/dinput/device.c +++ b/dlls/dinput/device.c @@ -179,7 +179,7 @@ const char *_dump_dinput_GUID(const GUID *guid) { return guids[i].name; } } - return "Unknown GUID"; + return debugstr_guid(guid); } void _dump_DIDATAFORMAT(const DIDATAFORMAT *df) { diff --git a/dlls/dinput/effect_linuxinput.c b/dlls/dinput/effect_linuxinput.c index 7d603bb..4a31ec1 100644 --- a/dlls/dinput/effect_linuxinput.c +++ b/dlls/dinput/effect_linuxinput.c @@ -191,7 +191,7 @@ static void _dump_DIEFFECT(LPCDIEFFECT eff, REFGUID guid) TRACE(" - dwTriggerRepeatInterval: %d\n", eff->dwTriggerRepeatInterval); TRACE(" - cAxes: %d\n", eff->cAxes); TRACE(" - rgdwAxes: %p\n", eff->rgdwAxes); - if (TRACE_ON(dinput)) { + if (TRACE_ON(dinput) && eff->rgdwAxes) { TRACE(" "); for (i = 0; i < eff->cAxes; ++i) TRACE("%d ", eff->rgdwAxes[i]); @@ -261,7 +261,7 @@ static HRESULT WINAPI LinuxInputEffectImpl_Download( if (errno == ENOMEM) { return DIERR_DEVICEFULL; } else { - FIXME("Could not upload effect. Assuming a disconnected device.\n"); + FIXME("Could not upload effect. Assuming a disconnected device %d \"%s\".\n", *This->fd, strerror(errno)); return DIERR_INPUTLOST; } } @@ -824,7 +824,7 @@ HRESULT linuxinput_create_effect( HeapFree(GetProcessHeap(), 0, newEffect); return DIERR_INVALIDPARAM; default: - FIXME("Unknown force type.\n"); + FIXME("Unknown force type 0x%x.\n", type); HeapFree(GetProcessHeap(), 0, newEffect); return DIERR_INVALIDPARAM; } diff --git a/dlls/dinput/joystick_linuxinput.c b/dlls/dinput/joystick_linuxinput.c index 13bc1fb..24ed7d3 100644 --- a/dlls/dinput/joystick_linuxinput.c +++ b/dlls/dinput/joystick_linuxinput.c @@ -637,6 +637,8 @@ static HRESULT WINAPI JoystickAImpl_Acquire(LPDIRECTINPUTDEVICE8A iface) } } } + else + WARN("Failed to acquire: %x\n", res); return res; }

1 0

Vitaliy Margolen : dinput: Stop and unload effect when freeing it.
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: 91036a04a293caf2ac94a51954f23d390287bde3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=91036a04a293caf2ac94a5195… Author: Vitaliy Margolen <wine-patches(a)kievinfo.com> Date: Sat Jul 5 22:24:43 2008 -0600 dinput: Stop and unload effect when freeing it. --- dlls/dinput/effect_linuxinput.c | 25 ++++++++++++++----------- 1 files changed, 14 insertions(+), 11 deletions(-) diff --git a/dlls/dinput/effect_linuxinput.c b/dlls/dinput/effect_linuxinput.c index 4a31ec1..ab72d2c 100644 --- a/dlls/dinput/effect_linuxinput.c +++ b/dlls/dinput/effect_linuxinput.c @@ -724,17 +724,6 @@ static HRESULT WINAPI LinuxInputEffectImpl_SetParameters( return DI_OK; } -static ULONG WINAPI LinuxInputEffectImpl_Release( - LPDIRECTINPUTEFFECT iface) -{ - LinuxInputEffectImpl *This = (LinuxInputEffectImpl *)iface; - ULONG ref = InterlockedDecrement(&(This->ref)); - - if (ref == 0) - HeapFree(GetProcessHeap(), 0, This); - return ref; -} - static HRESULT WINAPI LinuxInputEffectImpl_Stop( LPDIRECTINPUTEFFECT iface) { @@ -768,6 +757,20 @@ static HRESULT WINAPI LinuxInputEffectImpl_Unload( return DI_OK; } +static ULONG WINAPI LinuxInputEffectImpl_Release(LPDIRECTINPUTEFFECT iface) +{ + LinuxInputEffectImpl *This = (LinuxInputEffectImpl *)iface; + ULONG ref = InterlockedDecrement(&(This->ref)); + + if (ref == 0) + { + LinuxInputEffectImpl_Stop(iface); + LinuxInputEffectImpl_Unload(iface); + HeapFree(GetProcessHeap(), 0, This); + } + return ref; +} + /****************************************************************************** * LinuxInputEffect */

1 0

Eric Pouech : winhelp: Remove the search button, hook the index button to the Finder, and add a new summary button to go back to main page.
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: fa25f187c37f38d48c1bfaa2a99d4f937b6571f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fa25f187c37f38d48c1bfaa2a… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Jul 5 21:33:37 2008 +0200 winhelp: Remove the search button, hook the index button to the Finder, and add a new summary button to go back to main page. --- programs/winhlp32/Bg.rc | 4 ++-- programs/winhlp32/Cs.rc | 4 ++-- programs/winhlp32/Da.rc | 4 ++-- programs/winhlp32/De.rc | 4 ++-- programs/winhlp32/En.rc | 4 ++-- programs/winhlp32/Eo.rc | 4 ++-- programs/winhlp32/Es.rc | 4 ++-- programs/winhlp32/Fi.rc | 4 ++-- programs/winhlp32/Fr.rc | 4 ++-- programs/winhlp32/Hu.rc | 4 ++-- programs/winhlp32/It.rc | 4 ++-- programs/winhlp32/Ko.rc | 4 ++-- programs/winhlp32/Nl.rc | 4 ++-- programs/winhlp32/No.rc | 4 ++-- programs/winhlp32/Pl.rc | 4 ++-- programs/winhlp32/Pt.rc | 8 ++++---- programs/winhlp32/Rm.rc | 4 ++-- programs/winhlp32/Ru.rc | 4 ++-- programs/winhlp32/Si.rc | 4 ++-- programs/winhlp32/Sk.rc | 4 ++-- programs/winhlp32/Sv.rc | 4 ++-- programs/winhlp32/Tr.rc | 4 ++-- programs/winhlp32/Wa.rc | 4 ++-- programs/winhlp32/Zh.rc | 4 ++-- programs/winhlp32/winhelp.c | 4 ++-- programs/winhlp32/winhelp_res.h | 2 +- 26 files changed, 53 insertions(+), 53 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=fa25f187c37f38d48c1bf…

1 0

Maarten Lankhorst : quartz: Fix parser/ pullpin to only care about the state transition stopped<->playing.
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: 53782ca47acfa8e5d7ae72b59602840e3b35cd81 URL: http://source.winehq.org/git/wine.git/?a=commit;h=53782ca47acfa8e5d7ae72b59… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Fri Jul 4 18:43:05 2008 -0700 quartz: Fix parser/pullpin to only care about the state transition stopped<->playing. --- dlls/quartz/parser.c | 36 +++++++++++++++----- dlls/quartz/pin.c | 88 ++++++++++++++++++++++++++++++++++++------------- dlls/quartz/pin.h | 3 +- 3 files changed, 93 insertions(+), 34 deletions(-) diff --git a/dlls/quartz/parser.c b/dlls/quartz/parser.c index 97be991..0848e84 100644 --- a/dlls/quartz/parser.c +++ b/dlls/quartz/parser.c @@ -219,15 +219,35 @@ HRESULT WINAPI Parser_Stop(IBaseFilter * iface) { ParserImpl *This = (ParserImpl *)iface; PullPin *pin = (PullPin *)This->ppPins[0]; + int i; TRACE("()\n"); EnterCriticalSection(&pin->thread_lock); + + IAsyncReader_BeginFlush(This->pInputPin->pReader); EnterCriticalSection(&This->csFilter); + + if (This->state == State_Stopped) { - This->state = State_Stopped; + LeaveCriticalSection(&This->csFilter); + LeaveCriticalSection(&pin->thread_lock); + return S_OK; } + + This->state = State_Stopped; + + for (i = 1; i < (This->cStreams + 1); i++) + { + OutputPin_DecommitAllocator((OutputPin *)This->ppPins[i]); + } + LeaveCriticalSection(&This->csFilter); + + PullPin_PauseProcessing(This->pInputPin); + PullPin_WaitForStateChange(This->pInputPin, INFINITE); + IAsyncReader_EndFlush(This->pInputPin->pReader); + LeaveCriticalSection(&pin->thread_lock); return S_OK; } @@ -260,8 +280,6 @@ HRESULT WINAPI Parser_Pause(IBaseFilter * iface) This->state = State_Paused; LeaveCriticalSection(&This->csFilter); - if (SUCCEEDED(hr)) - hr = PullPin_PauseProcessing(This->pInputPin); LeaveCriticalSection(&pin->thread_lock); return hr; @@ -280,8 +298,9 @@ HRESULT WINAPI Parser_Run(IBaseFilter * iface, REFERENCE_TIME tStart) EnterCriticalSection(&pin->thread_lock); EnterCriticalSection(&This->csFilter); { - if (This->state == State_Running) + if (This->state == State_Running || This->state == State_Paused) { + This->state = State_Running; LeaveCriticalSection(&This->csFilter); LeaveCriticalSection(&pin->thread_lock); return S_OK; @@ -289,12 +308,11 @@ HRESULT WINAPI Parser_Run(IBaseFilter * iface, REFERENCE_TIME tStart) This->rtStreamStart = tStart; - if (SUCCEEDED(hr) && (This->state == State_Stopped)) + for (i = 1; i < (This->cStreams + 1); i++) { - for (i = 1; i < (This->cStreams + 1); i++) - { - OutputPin_CommitAllocator((OutputPin *)This->ppPins[i]); - } + hr = OutputPin_CommitAllocator((OutputPin *)This->ppPins[i]); + if (FAILED(hr)) + break; } if (SUCCEEDED(hr)) diff --git a/dlls/quartz/pin.c b/dlls/quartz/pin.c index 5bce9bb..0ecf055 100644 --- a/dlls/quartz/pin.c +++ b/dlls/quartz/pin.c @@ -1175,6 +1175,35 @@ HRESULT OutputPin_CommitAllocator(OutputPin * This) return hr; } +HRESULT OutputPin_DecommitAllocator(OutputPin * This) +{ + HRESULT hr = S_OK; + + TRACE("(%p)->()\n", This); + + EnterCriticalSection(This->pin.pCritSec); + { + if (!This->pin.pConnectedTo || !This->pMemInputPin) + hr = VFW_E_NOT_CONNECTED; + else + { + IMemAllocator * pAlloc = NULL; + + hr = IMemInputPin_GetAllocator(This->pMemInputPin, &pAlloc); + + if (SUCCEEDED(hr)) + hr = IMemAllocator_Decommit(pAlloc); + + if (pAlloc) + IMemAllocator_Release(pAlloc); + } + } + LeaveCriticalSection(This->pin.pCritSec); + + TRACE("--> %08x\n", hr); + return hr; +} + HRESULT OutputPin_DeliverDisconnect(OutputPin * This) { HRESULT hr; @@ -1416,11 +1445,11 @@ static void CALLBACK PullPin_Flush(PullPin *This) IMediaSample *pSample; TRACE("Flushing!\n"); - EnterCriticalSection(This->pin.pCritSec); if (This->pReader) { /* Flush outstanding samples */ IAsyncReader_BeginFlush(This->pReader); + for (;;) { DWORD_PTR dwUser; @@ -1437,10 +1466,9 @@ static void CALLBACK PullPin_Flush(PullPin *This) IAsyncReader_EndFlush(This->pReader); } - LeaveCriticalSection(This->pin.pCritSec); } -static void CALLBACK PullPin_Thread_Process(PullPin *This, BOOL pause) +static void CALLBACK PullPin_Thread_Process(PullPin *This) { HRESULT hr; IMediaSample * pSample = NULL; @@ -1467,12 +1495,9 @@ static void CALLBACK PullPin_Thread_Process(PullPin *This, BOOL pause) if (FAILED(hr)) ERR("Request error: %x\n", hr); - if (!pause) - { - EnterCriticalSection(This->pin.pCritSec); - SetEvent(This->hEventStateChanged); - LeaveCriticalSection(This->pin.pCritSec); - } + EnterCriticalSection(This->pin.pCritSec); + SetEvent(This->hEventStateChanged); + LeaveCriticalSection(This->pin.pCritSec); if (SUCCEEDED(hr)) do @@ -1510,23 +1535,20 @@ static void CALLBACK PullPin_Thread_Process(PullPin *This, BOOL pause) /* Can't reset state to Sleepy here because that might race, instead PauseProcessing will do that for us * Flush remaining samples */ - TRACE("Almost done..\n"); - if (This->fnDone) This->fnDone(This->pin.pUserData); - if (pause) - { - EnterCriticalSection(This->pin.pCritSec); - This->state = Req_Sleepy; - SetEvent(This->hEventStateChanged); - LeaveCriticalSection(This->pin.pCritSec); - } - - TRACE("End: %08x, %d\n", hr, This->stop_playback); } +static void CALLBACK PullPin_Thread_Pause(PullPin *This) +{ + EnterCriticalSection(This->pin.pCritSec); + This->state = Req_Sleepy; + SetEvent(This->hEventStateChanged); + LeaveCriticalSection(This->pin.pCritSec); +} + static void CALLBACK PullPin_Thread_Stop(PullPin *This) { TRACE("(%p)->()\n", This); @@ -1545,6 +1567,16 @@ static void CALLBACK PullPin_Thread_Stop(PullPin *This) ExitThread(0); } +static void CALLBACK PullPin_Thread_Flush(PullPin *This) +{ + PullPin_Flush(This); + + EnterCriticalSection(This->pin.pCritSec); + This->state = Req_Sleepy; + SetEvent(This->hEventStateChanged); + LeaveCriticalSection(This->pin.pCritSec); +} + static DWORD WINAPI PullPin_Thread_Main(LPVOID pv) { PullPin *This = pv; @@ -1561,8 +1593,9 @@ static DWORD WINAPI PullPin_Thread_Main(LPVOID pv) switch (This->state) { case Req_Die: PullPin_Thread_Stop(This); break; - case Req_Run: PullPin_Thread_Process(This, FALSE); break; - case Req_Pause: PullPin_Thread_Process(This, TRUE); break; + case Req_Run: PullPin_Thread_Process(This); break; + case Req_Pause: PullPin_Thread_Pause(This); break; + case Req_Flush: PullPin_Thread_Flush(This); break; case Req_Sleepy: ERR("Should not be signalled with SLEEPY!\n"); break; default: ERR("Unknown state request: %d\n", This->state); break; } @@ -1710,13 +1743,20 @@ HRESULT WINAPI PullPin_BeginFlush(IPin * iface) EnterCriticalSection(&This->thread_lock); { + if (This->pReader) + IAsyncReader_BeginFlush(This->pReader); PullPin_WaitForStateChange(This, INFINITE); - if (This->hThread && !This->stop_playback) + if (This->hThread && This->state == Req_Run) { PullPin_PauseProcessing(This); PullPin_WaitForStateChange(This, INFINITE); } + + This->state = Req_Flush; + ResetEvent(This->hEventStateChanged); + SetEvent(This->thread_sleepy); + PullPin_WaitForStateChange(This, INFINITE); } LeaveCriticalSection(&This->thread_lock); @@ -1740,7 +1780,7 @@ HRESULT WINAPI PullPin_EndFlush(IPin * iface) FILTER_STATE state; IBaseFilter_GetState(This->pin.pinInfo.pFilter, INFINITE, &state); - if (This->stop_playback && state == State_Running) + if (This->stop_playback && state != State_Stopped) PullPin_StartProcessing(This); PullPin_WaitForStateChange(This, INFINITE); diff --git a/dlls/quartz/pin.h b/dlls/quartz/pin.h index b67ff1b..4170ca5 100644 --- a/dlls/quartz/pin.h +++ b/dlls/quartz/pin.h @@ -52,7 +52,6 @@ typedef HRESULT (* CLEANUPPROC) (LPVOID userdata); * If you implement it (it can be NULL for default behavior), you have to * call IMemAllocator_GetBuffer and IMemAllocator_RequestBuffer * This is useful if you want to request more than 1 buffer at simultaneously - * If PullPin->flushed is set, it means that all buffers queued previously are gone * * This will also cause the Sample Proc to be called with empty buffers to indicate * failure in retrieving the sample. @@ -141,6 +140,7 @@ typedef struct PullPin #define Req_Die 1 #define Req_Run 2 #define Req_Pause 3 +#define Req_Flush 4 /*** Constructors ***/ HRESULT InputPin_Construct(const IPinVtbl *InputPin_Vtbl, const PIN_INFO * pPinInfo, SAMPLEPROC_PUSH pSampleProc, LPVOID pUserData, QUERYACCEPTPROC pQueryAccept, CLEANUPPROC pCleanUp, LPCRITICAL_SECTION pCritSec, IMemAllocator *, IPin ** ppPin); @@ -184,6 +184,7 @@ HRESULT WINAPI OutputPin_EndFlush(IPin * iface); HRESULT WINAPI OutputPin_NewSegment(IPin * iface, REFERENCE_TIME tStart, REFERENCE_TIME tStop, double dRate); HRESULT OutputPin_CommitAllocator(OutputPin * This); +HRESULT OutputPin_DecommitAllocator(OutputPin * This); HRESULT OutputPin_GetDeliveryBuffer(OutputPin * This, IMediaSample ** ppSample, REFERENCE_TIME * tStart, REFERENCE_TIME * tStop, DWORD dwFlags); HRESULT OutputPin_SendSample(OutputPin * This, IMediaSample * pSample); HRESULT OutputPin_DeliverDisconnect(OutputPin * This);

1 0

Eric Pouech : winhelp: Removed the topics button, and forward search and topics to the same prop sheet.
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: 841720e491d4d82a83a3c83405e4ec59ef9f6376 URL: http://source.winehq.org/git/wine.git/?a=commit;h=841720e491d4d82a83a3c8340… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Jul 5 21:33:19 2008 +0200 winhelp: Removed the topics button, and forward search and topics to the same prop sheet. --- programs/winhlp32/Bg.rc | 1 - programs/winhlp32/Cs.rc | 1 - programs/winhlp32/Da.rc | 1 - programs/winhlp32/De.rc | 1 - programs/winhlp32/En.rc | 9 +++++++- programs/winhlp32/Eo.rc | 1 - programs/winhlp32/Es.rc | 1 - programs/winhlp32/Fi.rc | 1 - programs/winhlp32/Fr.rc | 1 - programs/winhlp32/Hu.rc | 1 - programs/winhlp32/It.rc | 1 - programs/winhlp32/Ko.rc | 1 - programs/winhlp32/Nl.rc | 1 - programs/winhlp32/No.rc | 1 - programs/winhlp32/Pl.rc | 1 - programs/winhlp32/Pt.rc | 2 - programs/winhlp32/Rm.rc | 1 - programs/winhlp32/Ru.rc | 1 - programs/winhlp32/Si.rc | 1 - programs/winhlp32/Sk.rc | 1 - programs/winhlp32/Sv.rc | 1 - programs/winhlp32/Tr.rc | 1 - programs/winhlp32/Wa.rc | 1 - programs/winhlp32/Zh.rc | 1 - programs/winhlp32/macro.c | 4 +- programs/winhlp32/winhelp.c | 41 +++++++++++++++++++++++++++++++++++--- programs/winhlp32/winhelp.h | 2 +- programs/winhlp32/winhelp_res.h | 2 +- 28 files changed, 49 insertions(+), 33 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=841720e491d4d82a83a3c…

1 0

Maarten Lankhorst : quartz: Make sure transform filter commits allocator.
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: 11e2b1ab2b0a867c8124a226475a12b199a1c0d5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=11e2b1ab2b0a867c8124a2264… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Fri Jul 4 16:59:01 2008 -0700 quartz: Make sure transform filter commits allocator. --- dlls/quartz/transform.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/quartz/transform.c b/dlls/quartz/transform.c index d602c69..f37e9e3 100644 --- a/dlls/quartz/transform.c +++ b/dlls/quartz/transform.c @@ -349,7 +349,7 @@ static HRESULT WINAPI TransformFilter_Pause(IBaseFilter * iface) EnterCriticalSection(&This->csFilter); { if (This->state == State_Stopped) - ((InputPin *)This->ppPins[0])->end_of_stream = 0; + IBaseFilter_Run(iface, -1); This->state = State_Paused; }

1 0

Maarten Lankhorst : quartz: Only call begin process functions in transform filter when stopped.
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: e58855e0e913adc55df56d31020f027d55afe522 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e58855e0e913adc55df56d310… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Fri Jul 4 18:42:09 2008 -0700 quartz: Only call begin process functions in transform filter when stopped. Fixes some graphical glitches that may occur otherwise. --- dlls/quartz/transform.c | 10 ++++++---- 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/dlls/quartz/transform.c b/dlls/quartz/transform.c index f37e9e3..d073970 100644 --- a/dlls/quartz/transform.c +++ b/dlls/quartz/transform.c @@ -349,7 +349,7 @@ static HRESULT WINAPI TransformFilter_Pause(IBaseFilter * iface) EnterCriticalSection(&This->csFilter); { if (This->state == State_Stopped) - IBaseFilter_Run(iface, -1); + ((InputPin *)This->ppPins[0])->end_of_stream = 0; This->state = State_Paused; } @@ -368,13 +368,15 @@ static HRESULT WINAPI TransformFilter_Run(IBaseFilter * iface, REFERENCE_TIME tS EnterCriticalSection(&This->csFilter); { if (This->state == State_Stopped) + { ((InputPin *)This->ppPins[0])->end_of_stream = 0; + if (This->pFuncsTable->pfnProcessBegin) + This->pFuncsTable->pfnProcessBegin(This); + OutputPin_CommitAllocator((OutputPin *)This->ppPins[1]); + } This->rtStreamStart = tStart; This->state = State_Running; - OutputPin_CommitAllocator((OutputPin *)This->ppPins[1]); - if (This->pFuncsTable->pfnProcessBegin) - This->pFuncsTable->pfnProcessBegin(This); } LeaveCriticalSection(&This->csFilter);

1 0

Dylan Smith : richedit: Made sure the caret position is stored when moving up/down.
by Alexandre Julliard 07 Jul '08

07 Jul '08

Module: wine Branch: master Commit: db95d69f992a1618e3f407a810b74fd6f8eaffaa URL: http://source.winehq.org/git/wine.git/?a=commit;h=db95d69f992a1618e3f407a81… Author: Dylan Smith <dylan.ah.smith(a)gmail.com> Date: Fri Jul 4 15:30:51 2008 -0400 richedit: Made sure the caret position is stored when moving up/down. The caret's x position is stored in ME_TextEditor.nUDArrowX so that when the caret is moved up or down, it will stay along the same horizontal position, or at the end of a line. Unfortunately, the value stored in nUDArrowX was being unconditionally discarded in ME_ArrowKey, preventing it from serving its purpose. --- dlls/riched20/caret.c | 1 - dlls/riched20/editor.c | 9 +++++++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/dlls/riched20/caret.c b/dlls/riched20/caret.c index ebf6c56..4a1a8a4 100644 --- a/dlls/riched20/caret.c +++ b/dlls/riched20/caret.c @@ -1349,7 +1349,6 @@ ME_ArrowKey(ME_TextEditor *editor, int nVKey, BOOL extend, BOOL ctrl) BOOL success = FALSE; ME_CheckCharOffsets(editor); - editor->nUDArrowX = -1; switch(nVKey) { case VK_LEFT: editor->bCaretAtEnd = 0; diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index 28850f9..dedc59e 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -1542,10 +1542,12 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) { case VK_LEFT: case VK_RIGHT: - case VK_UP: - case VK_DOWN: case VK_HOME: case VK_END: + editor->nUDArrowX = -1; + /* fall through */ + case VK_UP: + case VK_DOWN: case VK_PRIOR: case VK_NEXT: ME_CommitUndo(editor); /* End coalesced undos for typed characters */ @@ -1553,6 +1555,7 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) return TRUE; case VK_BACK: case VK_DELETE: + editor->nUDArrowX = -1; /* FIXME backspace and delete aren't the same, they act different wrt paragraph style of the merged paragraph */ if (GetWindowLongW(editor->hWnd, GWL_STYLE) & ES_READONLY) return FALSE; @@ -1583,6 +1586,8 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) return TRUE; default: + if (nKey != VK_SHIFT && nKey != VK_CONTROL && nKey && nKey != VK_MENU) + editor->nUDArrowX = -1; if (ctrl_is_down) { if (nKey == 'W')

1 0