February 2011 - Wine-commits

Dmitry Timoshkov : winex11.drv: Make sure that default clipping region exists before calling ExcludeUpdateRgn ().
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: 38167e71d1ccff264d269c3bee7e1a448871829a URL: http://source.winehq.org/git/wine.git/?a=commit;h=38167e71d1ccff264d269c3be… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Mon Jan 31 19:30:43 2011 +0800 winex11.drv: Make sure that default clipping region exists before calling ExcludeUpdateRgn(). --- dlls/winex11.drv/window.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c index 65c046b..082c928 100644 --- a/dlls/winex11.drv/window.c +++ b/dlls/winex11.drv/window.c @@ -1605,7 +1605,7 @@ static void move_window_bits( struct x11drv_win_data *data, const RECT *old_rect RECT dst_rect = *new_rect; HDC hdc_src, hdc_dst; INT code; - HRGN rgn = 0; + HRGN rgn; HWND parent = 0; if (!data->whole_window) @@ -1624,6 +1624,9 @@ static void move_window_bits( struct x11drv_win_data *data, const RECT *old_rect hdc_src = hdc_dst = GetDCEx( data->hwnd, 0, DCX_CACHE ); } + rgn = CreateRectRgnIndirect( &dst_rect ); + SelectClipRgn( hdc_dst, rgn ); + DeleteObject( rgn ); ExcludeUpdateRgn( hdc_dst, data->hwnd ); code = X11DRV_START_EXPOSURES; @@ -1636,6 +1639,7 @@ static void move_window_bits( struct x11drv_win_data *data, const RECT *old_rect dst_rect.right - dst_rect.left, dst_rect.bottom - dst_rect.top, hdc_src, src_rect.left, src_rect.top, SRCCOPY ); + rgn = 0; code = X11DRV_END_EXPOSURES; ExtEscape( hdc_dst, X11DRV_ESCAPE, sizeof(code), (LPSTR)&code, sizeof(rgn), (LPSTR)&rgn );

1 0

Andrew Nguyen : shell32: Keep at least one reference to the autocompletion object at initialization until the edit window is destroyed .
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: fff50be3e8018df5292de632c88771f7cf78563d URL: http://source.winehq.org/git/wine.git/?a=commit;h=fff50be3e8018df5292de632c… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Tue Feb 1 04:16:38 2011 -0600 shell32: Keep at least one reference to the autocompletion object at initialization until the edit window is destroyed. Spotted with Valgrind. --- dlls/shell32/autocomplete.c | 19 ++++++++++++------- 1 files changed, 12 insertions(+), 7 deletions(-) diff --git a/dlls/shell32/autocomplete.c b/dlls/shell32/autocomplete.c index b259f6f..8db542a 100644 --- a/dlls/shell32/autocomplete.c +++ b/dlls/shell32/autocomplete.c @@ -203,18 +203,16 @@ static ULONG WINAPI IAutoComplete2_fnRelease( { IAutoCompleteImpl *This = (IAutoCompleteImpl *)iface; ULONG refCount = InterlockedDecrement(&This->ref); - + TRACE("(%p)->(%u)\n", This, refCount + 1); if (!refCount) { - TRACE(" destroying IAutoComplete(%p)\n",This); + TRACE("destroying IAutoComplete(%p)\n", This); HeapFree(GetProcessHeap(), 0, This->quickComplete); HeapFree(GetProcessHeap(), 0, This->txtbackup); - if (This->hwndListBox) - DestroyWindow(This->hwndListBox); - if (This->enumstr) - IEnumString_Release(This->enumstr); - HeapFree(GetProcessHeap(), 0, This); + if (This->enumstr) + IEnumString_Release(This->enumstr); + HeapFree(GetProcessHeap(), 0, This); } return refCount; } @@ -275,6 +273,8 @@ static HRESULT WINAPI IAutoComplete2_fnInit( This->initialized = TRUE; This->hwndEdit = hwndEdit; This->wpOrigEditProc = (WNDPROC) SetWindowLongPtrW( hwndEdit, GWLP_WNDPROC, (LONG_PTR) ACEditSubclassProc); + /* Keep at least one reference to the object until the edit window is destroyed. */ + IAutoComplete2_AddRef((IAutoComplete2 *)This); SetPropW( hwndEdit, autocomplete_propertyW, This ); if (This->options & ACO_AUTOSUGGEST) @@ -631,7 +631,12 @@ static LRESULT APIENTRY ACEditSubclassProc(HWND hwnd, UINT uMsg, WPARAM wParam, { WNDPROC proc = This->wpOrigEditProc; + RemovePropW(hwnd, autocomplete_propertyW); + SetWindowLongPtrW(hwnd, GWLP_WNDPROC, (LONG_PTR)proc); This->hwndEdit = NULL; + if (This->hwndListBox) + DestroyWindow(This->hwndListBox); + IAutoComplete2_Release((IAutoComplete2 *)This); return CallWindowProcW(proc, hwnd, uMsg, wParam, lParam); } default:

1 0

Andrew Nguyen : ntdll: Fix one-time warning message output.
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: c698fc86b9ee15fed718c95b850b8017ce80349d URL: http://source.winehq.org/git/wine.git/?a=commit;h=c698fc86b9ee15fed718c95b8… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Tue Feb 1 04:33:47 2011 -0600 ntdll: Fix one-time warning message output. --- dlls/ntdll/directory.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ntdll/directory.c b/dlls/ntdll/directory.c index 5a5d101..66572da 100644 --- a/dlls/ntdll/directory.c +++ b/dlls/ntdll/directory.c @@ -847,7 +847,7 @@ static void add_fs_cache( dev_t dev, fsid_t fsid, BOOLEAN case_sensitive ) } /* Cache is out of space, warn */ - if (once++) + if (!once++) WARN( "FS cache is out of space, expect performance problems\n" ); }

1 0

Andrew Nguyen : msvcr90/tests: Avoid casting away const in comparison functions.
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: 39eb51f978042e209d78f962e280018f876cb3b5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=39eb51f978042e209d78f962e… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Tue Feb 1 04:40:14 2011 -0600 msvcr90/tests: Avoid casting away const in comparison functions. --- dlls/msvcr90/tests/msvcr90.c | 22 ++++++++++++++++------ 1 files changed, 16 insertions(+), 6 deletions(-) diff --git a/dlls/msvcr90/tests/msvcr90.c b/dlls/msvcr90/tests/msvcr90.c index d4a7262..67b00fc 100644 --- a/dlls/msvcr90/tests/msvcr90.c +++ b/dlls/msvcr90/tests/msvcr90.c @@ -462,25 +462,35 @@ static void test_wcsncat_s(void) } /* Based on dlls/ntdll/tests/string.c */ -static __cdecl int intcomparefunc(void *context, const void *a, const void*b) +static __cdecl int intcomparefunc(void *context, const void *a, const void *b) { + const int *p = a, *q = b; + ok (a != b, "must never get the same pointer\n"); ++*(int *) context; - return (*(int*)a) - (*(int*)b); + + return *p - *q; } -static __cdecl int charcomparefunc(void *context, const void *a, const void*b) +static __cdecl int charcomparefunc(void *context, const void *a, const void *b) { + const char *p = a, *q = b; + ok (a != b, "must never get the same pointer\n"); ++*(int *) context; - return (*(char*)a) - (*(char*)b); + + return *p - *q; } -static __cdecl int strcomparefunc(void *context, const void *a, const void*b) +static __cdecl int strcomparefunc(void *context, const void *a, const void *b) { + const char * const *p = a; + const char * const *q = b; + ok (a != b, "must never get the same pointer\n"); ++*(int *) context; - return lstrcmpA(*(char**)a,*(char**)b); + + return lstrcmpA(*p, *q); } static void test_qsort_s(void)

1 0

Andrew Nguyen : ntdll/tests: Avoid casting away const in comparison functions.
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: f7385699a9aa26125eabc4143a50254a289cda49 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f7385699a9aa26125eabc4143… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Tue Feb 1 04:40:44 2011 -0600 ntdll/tests: Avoid casting away const in comparison functions. --- dlls/ntdll/tests/string.c | 28 +++++++++++++++++++--------- 1 files changed, 19 insertions(+), 9 deletions(-) diff --git a/dlls/ntdll/tests/string.c b/dlls/ntdll/tests/string.c index 03fc436..14464d9 100644 --- a/dlls/ntdll/tests/string.c +++ b/dlls/ntdll/tests/string.c @@ -1143,22 +1143,32 @@ static void test_wcsrchr(void) "wcsrchr should have returned NULL\n"); } -static __cdecl int intcomparefunc(const void *a, const void*b) +static __cdecl int intcomparefunc(const void *a, const void *b) { - ok (a != b, "must never get the same pointer\n"); - return (*(int*)a) - (*(int*)b); + const int *p = a, *q = b; + + ok (a != b, "must never get the same pointer\n"); + + return *p - *q; } -static __cdecl int charcomparefunc(const void *a, const void*b) +static __cdecl int charcomparefunc(const void *a, const void *b) { - ok (a != b, "must never get the same pointer\n"); - return (*(char*)a) - (*(char*)b); + const char *p = a, *q = b; + + ok (a != b, "must never get the same pointer\n"); + + return *p - *q; } -static __cdecl int strcomparefunc(const void *a, const void*b) +static __cdecl int strcomparefunc(const void *a, const void *b) { - ok (a != b, "must never get the same pointer\n"); - return lstrcmpA(*(char**)a,*(char**)b); + const char * const *p = a; + const char * const *q = b; + + ok (a != b, "must never get the same pointer\n"); + + return lstrcmpA(*p, *q); } static void test_qsort(void)

1 0

Alexandre Julliard : ntdll: Avoid copying and restoring the context in RtlCaptureContext for i386.
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: 43682ce114ff5499e9e89e396a0fb66eb97f3dbc URL: http://source.winehq.org/git/wine.git/?a=commit;h=43682ce114ff5499e9e89e396… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Feb 1 12:13:19 2011 +0100 ntdll: Avoid copying and restoring the context in RtlCaptureContext for i386. --- dlls/ntdll/ntdll.spec | 2 +- dlls/ntdll/signal_i386.c | 35 ++++++++++++++++++++++++++++------- 2 files changed, 29 insertions(+), 8 deletions(-) diff --git a/dlls/ntdll/ntdll.spec b/dlls/ntdll/ntdll.spec index da674cd..0976bc5 100644 --- a/dlls/ntdll/ntdll.spec +++ b/dlls/ntdll/ntdll.spec @@ -435,7 +435,7 @@ # @ stub RtlAssert2 @ stdcall RtlAssert(ptr ptr long long) # @ stub RtlCancelTimer -@ stdcall -register RtlCaptureContext(ptr) +@ stdcall -norelay RtlCaptureContext(ptr) @ stdcall RtlCaptureStackBackTrace(long long ptr ptr) # @ stub RtlCaptureStackContext @ stdcall RtlCharToInteger(ptr long ptr) diff --git a/dlls/ntdll/signal_i386.c b/dlls/ntdll/signal_i386.c index f86c810..69c9c3f 100644 --- a/dlls/ntdll/signal_i386.c +++ b/dlls/ntdll/signal_i386.c @@ -1151,13 +1151,34 @@ static inline void restore_context( const CONTEXT *context, SIGCONTEXT *sigconte /*********************************************************************** * RtlCaptureContext (NTDLL.@) */ -void WINAPI __regs_RtlCaptureContext( CONTEXT *context, CONTEXT *regs ) -{ - *context = *regs; - if (fpux_support) save_fpux( context ); - else save_fpu( context ); -} -DEFINE_REGS_ENTRYPOINT( RtlCaptureContext, 1 ) +__ASM_STDCALL_FUNC( RtlCaptureContext, 4, + "pushl %eax\n\t" + __ASM_CFI(".cfi_adjust_cfa_offset 4\n\t") + "movl 8(%esp),%eax\n\t" /* context */ + "movl $0x10007,(%eax)\n\t" /* context->ContextFlags */ + "movw %gs,0x8c(%eax)\n\t" /* context->SegGs */ + "movw %fs,0x90(%eax)\n\t" /* context->SegFs */ + "movw %es,0x94(%eax)\n\t" /* context->SegEs */ + "movw %ds,0x98(%eax)\n\t" /* context->SegDs */ + "movl %edi,0x9c(%eax)\n\t" /* context->Edi */ + "movl %esi,0xa0(%eax)\n\t" /* context->Esi */ + "movl %ebx,0xa4(%eax)\n\t" /* context->Ebx */ + "movl %edx,0xa8(%eax)\n\t" /* context->Edx */ + "movl %ecx,0xac(%eax)\n\t" /* context->Ecx */ + "movl %ebp,0xb4(%eax)\n\t" /* context->Ebp */ + "movl 4(%esp),%edx\n\t" + "movl %edx,0xb8(%eax)\n\t" /* context->Eip */ + "movw %cs,0xbc(%eax)\n\t" /* context->SegCs */ + "pushfl\n\t" + __ASM_CFI(".cfi_adjust_cfa_offset 4\n\t") + "popl 0xc0(%eax)\n\t" /* context->EFlags */ + __ASM_CFI(".cfi_adjust_cfa_offset -4\n\t") + "leal 8(%esp),%edx\n\t" + "movl %edx,0xc4(%eax)\n\t" /* context->Esp */ + "movw %ss,0xc8(%eax)\n\t" /* context->SegSs */ + "popl 0xb0(%eax)\n\t" /* context->Eax */ + __ASM_CFI(".cfi_adjust_cfa_offset -4\n\t") + "ret $4" ) /***********************************************************************

1 0

Alexandre Julliard : ntdll: Add a helper function for checking stack frame limits.
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: efbe69514759930257ccaf131563fd3af563677f URL: http://source.winehq.org/git/wine.git/?a=commit;h=efbe69514759930257ccaf131… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Feb 1 12:18:39 2011 +0100 ntdll: Add a helper function for checking stack frame limits. --- dlls/ntdll/signal_i386.c | 27 ++++++++++++++------------- 1 files changed, 14 insertions(+), 13 deletions(-) diff --git a/dlls/ntdll/signal_i386.c b/dlls/ntdll/signal_i386.c index 69c9c3f..13444ae 100644 --- a/dlls/ntdll/signal_i386.c +++ b/dlls/ntdll/signal_i386.c @@ -548,6 +548,16 @@ static inline TEB *get_current_teb(void) /******************************************************************* + * is_valid_frame + */ +static inline BOOL is_valid_frame( void *frame ) +{ + if ((ULONG_PTR)frame & 3) return FALSE; + return (frame >= NtCurrentTeb()->Tib.StackLimit && + (void **)frame < (void **)NtCurrentTeb()->Tib.StackBase - 1); +} + +/******************************************************************* * raise_handler * * Handler for exceptions happening inside a handler. @@ -594,9 +604,7 @@ static NTSTATUS call_stack_handlers( EXCEPTION_RECORD *rec, CONTEXT *context ) while (frame != (EXCEPTION_REGISTRATION_RECORD*)~0UL) { /* Check frame address */ - if (((void*)frame < NtCurrentTeb()->Tib.StackLimit) || - ((void*)(frame+1) > NtCurrentTeb()->Tib.StackBase) || - (ULONG_PTR)frame & 3) + if (!is_valid_frame( frame )) { rec->ExceptionFlags |= EH_STACK_INVALID; break; @@ -2381,10 +2389,7 @@ void WINAPI __regs_RtlUnwind( EXCEPTION_REGISTRATION_RECORD* pEndFrame, PVOID ta if (pEndFrame && (frame > pEndFrame)) raise_status( STATUS_INVALID_UNWIND_TARGET, pRecord ); - if (((void*)frame < NtCurrentTeb()->Tib.StackLimit) || - ((void*)(frame+1) > NtCurrentTeb()->Tib.StackBase) || - (UINT_PTR)frame & 3) - raise_status( STATUS_BAD_STACK, pRecord ); + if (!is_valid_frame( frame )) raise_status( STATUS_BAD_STACK, pRecord ); /* Call handler */ TRACE( "calling handler at %p code=%x flags=%x\n", @@ -2453,17 +2458,13 @@ USHORT WINAPI RtlCaptureStackBackTrace( ULONG skip, ULONG count, PVOID *buffer, while (skip--) { - if (((void *)frame < NtCurrentTeb()->Tib.StackLimit) || - ((void *)(frame + 1) >= NtCurrentTeb()->Tib.StackBase) || - ((ULONG_PTR)frame & 3)) return 0; + if (!is_valid_frame( frame )) return 0; frame = (ULONG *)*frame; } for (i = 0; i < count; i++) { - if (((void *)frame < NtCurrentTeb()->Tib.StackLimit) || - ((void *)(frame + 1) >= NtCurrentTeb()->Tib.StackBase) || - ((ULONG_PTR)frame & 3)) break; + if (!is_valid_frame( frame )) break; buffer[i] = (void *)frame[1]; if (hash) *hash += frame[1]; frame = (ULONG *)*frame;

1 0

Andrew Nguyen : dxdiagn: Avoid zeroing memory in container functions.
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: 99e36ffabde8a5588c7ee9ae85abede4c7b33bd9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=99e36ffabde8a5588c7ee9ae8… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Tue Feb 1 04:31:47 2011 -0600 dxdiagn: Avoid zeroing memory in container functions. --- dlls/dxdiagn/container.c | 16 ++++++++++++---- 1 files changed, 12 insertions(+), 4 deletions(-) diff --git a/dlls/dxdiagn/container.c b/dlls/dxdiagn/container.c index 8585f7d..d3a3cc1 100644 --- a/dlls/dxdiagn/container.c +++ b/dlls/dxdiagn/container.c @@ -244,13 +244,17 @@ HRESULT WINAPI IDxDiagContainerImpl_AddProp(PDXDIAGCONTAINER iface, LPCWSTR pwsz return E_INVALIDARG; } - pNew = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(IDxDiagContainerImpl_Property)); + pNew = HeapAlloc(GetProcessHeap(), 0, sizeof(IDxDiagContainerImpl_Property)); if (NULL == pNew) { return E_OUTOFMEMORY; } VariantInit(&pNew->v); VariantCopy(&pNew->v, pVarProp); - pNew->vName = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, (lstrlenW(pwszPropName) + 1) * sizeof(WCHAR)); + pNew->vName = HeapAlloc(GetProcessHeap(), 0, (lstrlenW(pwszPropName) + 1) * sizeof(WCHAR)); + if (NULL == pNew->vName) { + HeapFree(GetProcessHeap(), 0, pNew); + return E_OUTOFMEMORY; + } lstrcpyW(pNew->vName, pwszPropName); pNew->next = NULL; @@ -278,12 +282,16 @@ HRESULT WINAPI IDxDiagContainerImpl_AddChildContainer(PDXDIAGCONTAINER iface, LP return E_INVALIDARG; } - pNew = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(IDxDiagContainerImpl_SubContainer)); + pNew = HeapAlloc(GetProcessHeap(), 0, sizeof(IDxDiagContainerImpl_SubContainer)); if (NULL == pNew) { return E_OUTOFMEMORY; } pNew->pCont = pSubCont; - pNew->contName = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, (lstrlenW(pszContName) + 1) * sizeof(WCHAR)); + pNew->contName = HeapAlloc(GetProcessHeap(), 0, (lstrlenW(pszContName) + 1) * sizeof(WCHAR)); + if (NULL == pNew->contName) { + HeapFree(GetProcessHeap(), 0, pNew) + return E_OUTOFMEMORY; + } lstrcpyW(pNew->contName, pszContName); pNew->next = NULL;

1 0

Andrew Nguyen : dxdiagn: Convert the container linked list to a standard Wine list.
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: a421adf0a0334da70e71471f9fa2956acf2938a7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a421adf0a0334da70e71471f9… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Tue Feb 1 04:32:05 2011 -0600 dxdiagn: Convert the container linked list to a standard Wine list. --- dlls/dxdiagn/container.c | 35 +++++++++++++---------------------- dlls/dxdiagn/dxdiag_private.h | 11 ++++++----- 2 files changed, 19 insertions(+), 27 deletions(-) diff --git a/dlls/dxdiagn/container.c b/dlls/dxdiagn/container.c index d3a3cc1..317d098 100644 --- a/dlls/dxdiagn/container.c +++ b/dlls/dxdiagn/container.c @@ -86,7 +86,7 @@ static HRESULT WINAPI IDxDiagContainerImpl_GetNumberOfChildContainers(PDXDIAGCON static HRESULT WINAPI IDxDiagContainerImpl_EnumChildContainerNames(PDXDIAGCONTAINER iface, DWORD dwIndex, LPWSTR pwszContainer, DWORD cchContainer) { IDxDiagContainerImpl *This = (IDxDiagContainerImpl *)iface; - IDxDiagContainerImpl_SubContainer* p = NULL; + IDxDiagContainerImpl_SubContainer *p; DWORD i = 0; TRACE("(%p, %u, %p, %u)\n", iface, dwIndex, pwszContainer, cchContainer); @@ -95,15 +95,14 @@ static HRESULT WINAPI IDxDiagContainerImpl_EnumChildContainerNames(PDXDIAGCONTAI return E_INVALIDARG; } - p = This->subContainers; - while (NULL != p) { + LIST_FOR_EACH_ENTRY(p, &This->subContainers, IDxDiagContainerImpl_SubContainer, entry) + { if (dwIndex == i) { TRACE("Found container name %s, copying string\n", debugstr_w(p->contName)); lstrcpynW(pwszContainer, p->contName, cchContainer); return (cchContainer <= strlenW(p->contName)) ? DXDIAG_E_INSUFFICIENT_BUFFER : S_OK; } - p = p->next; ++i; } @@ -114,16 +113,16 @@ static HRESULT WINAPI IDxDiagContainerImpl_EnumChildContainerNames(PDXDIAGCONTAI static HRESULT IDxDiagContainerImpl_GetChildContainerInternal(PDXDIAGCONTAINER iface, LPCWSTR pwszContainer, IDxDiagContainer** ppInstance) { IDxDiagContainerImpl *This = (IDxDiagContainerImpl *)iface; - IDxDiagContainerImpl_SubContainer* p = NULL; + IDxDiagContainerImpl_SubContainer *p; - p = This->subContainers; - while (NULL != p) { + LIST_FOR_EACH_ENTRY(p, &This->subContainers, IDxDiagContainerImpl_SubContainer, entry) + { if (0 == lstrcmpW(p->contName, pwszContainer)) { *ppInstance = p->pCont; return S_OK; } - p = p->next; } + return E_INVALIDARG; } @@ -273,8 +272,7 @@ HRESULT WINAPI IDxDiagContainerImpl_AddProp(PDXDIAGCONTAINER iface, LPCWSTR pwsz HRESULT WINAPI IDxDiagContainerImpl_AddChildContainer(PDXDIAGCONTAINER iface, LPCWSTR pszContName, PDXDIAGCONTAINER pSubCont) { IDxDiagContainerImpl *This = (IDxDiagContainerImpl *)iface; - IDxDiagContainerImpl_SubContainer* p = NULL; - IDxDiagContainerImpl_SubContainer* pNew = NULL; + IDxDiagContainerImpl_SubContainer *pNew; TRACE("(%p, %s, %p)\n", iface, debugstr_w(pszContName), pSubCont); @@ -286,24 +284,16 @@ HRESULT WINAPI IDxDiagContainerImpl_AddChildContainer(PDXDIAGCONTAINER iface, LP if (NULL == pNew) { return E_OUTOFMEMORY; } - pNew->pCont = pSubCont; + pNew->contName = HeapAlloc(GetProcessHeap(), 0, (lstrlenW(pszContName) + 1) * sizeof(WCHAR)); if (NULL == pNew->contName) { - HeapFree(GetProcessHeap(), 0, pNew) + HeapFree(GetProcessHeap(), 0, pNew); return E_OUTOFMEMORY; } lstrcpyW(pNew->contName, pszContName); - pNew->next = NULL; + pNew->pCont = pSubCont; - p = This->subContainers; - if (NULL == p) { - This->subContainers = pNew; - } else { - while (NULL != p->next) { - p = p->next; - } - p->next = pNew; - } + list_add_tail(&This->subContainers, &pNew->entry); ++This->nSubContainers; return S_OK; } @@ -334,5 +324,6 @@ HRESULT DXDiag_CreateDXDiagContainer(REFIID riid, LPVOID *ppobj) { } container->lpVtbl = &DxDiagContainer_Vtbl; container->ref = 0; /* will be inited with QueryInterface */ + list_init(&container->subContainers); return IDxDiagContainerImpl_QueryInterface((PDXDIAGCONTAINER)container, riid, ppobj); } diff --git a/dlls/dxdiagn/dxdiag_private.h b/dlls/dxdiagn/dxdiag_private.h index 8a2ebad..5e6d2b8 100644 --- a/dlls/dxdiagn/dxdiag_private.h +++ b/dlls/dxdiagn/dxdiag_private.h @@ -27,6 +27,7 @@ #include "winbase.h" #include "wingdi.h" +#include "wine/list.h" #include "dxdiag.h" /* DXDiag Interfaces: */ @@ -55,9 +56,9 @@ struct IDxDiagProviderImpl { /* ---------------- */ typedef struct IDxDiagContainerImpl_SubContainer { - IDxDiagContainer* pCont; - WCHAR* contName; - struct IDxDiagContainerImpl_SubContainer* next; + struct list entry; + WCHAR *contName; + IDxDiagContainer *pCont; } IDxDiagContainerImpl_SubContainer; typedef struct IDxDiagContainerImpl_Property { @@ -75,8 +76,8 @@ struct IDxDiagContainerImpl { const IDxDiagContainerVtbl *lpVtbl; LONG ref; /* IDxDiagContainer fields */ - IDxDiagContainerImpl_Property* properties; - IDxDiagContainerImpl_SubContainer* subContainers; + IDxDiagContainerImpl_Property* properties; + struct list subContainers; DWORD nProperties; DWORD nSubContainers; };

1 0

Stefan Dösinger : d3d9: Work around an odd nvidia windows bug.
by Alexandre Julliard 01 Feb '11

01 Feb '11

Module: wine Branch: master Commit: 73c4fbcc5120b57dc7eb00622070f3d92220ed34 URL: http://source.winehq.org/git/wine.git/?a=commit;h=73c4fbcc5120b57dc7eb00622… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun Jan 30 21:10:50 2011 +0100 d3d9: Work around an odd nvidia windows bug. --- dlls/d3d9/tests/visual.c | 14 ++++++++++++++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index d73e603..6c93bb7 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -490,6 +490,16 @@ static void clear_test(IDirect3DDevice9 *device) IDirect3DDevice9_Present(device, NULL, NULL, NULL, NULL); + /* Hack to work around a nvidia windows driver bug. The clear below is supposed to + * clear the red quad in the top left part of the render target. For some reason it + * doesn't work if the clear color is 0xffffffff on some versions of the Nvidia Windows + * driver(tested on 8.17.12.5896, Win7). A clear with a different color works around + * this bug and fixes the clear with the white color. Even 0xfeffffff works, but let's + * pick some obvious value + */ + hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET, 0xdeadbabe, 0.0, 0); + ok(hr == D3D_OK, "IDirect3DDevice9_Clear failed with %08x\n", hr); + /* Test how the viewport affects clears */ hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET, 0xffffffff, 0.0, 0); ok(hr == D3D_OK, "IDirect3DDevice9_Clear failed with %08x\n", hr); @@ -626,6 +636,10 @@ static void clear_test(IDirect3DDevice9 *device) hr = IDirect3DDevice9_SetRenderState(device, D3DRS_COLORWRITEENABLE, D3DCOLORWRITEENABLE_RED); ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed with %08x\n", hr); + /* Same nvidia windows driver trouble with white clears as earlier in the same test */ + hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET, 0xdeadbeef, 0.0, 0); + ok(hr == D3D_OK, "IDirect3DDevice9_Clear failed with %08x\n", hr); + hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET, 0xffffffff, 0.0, 0); ok(hr == D3D_OK, "IDirect3DDevice9_Clear failed with %08x\n", hr);

1 0