wine-commits December 2009

wine-commits@winehq.org

3 participants
1233 discussions

Nikolay Sivov : comctl32/updown: Fix formatting for hex mode.
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: f897a91247d9ed7a240f0db6fa902cd0148210e6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f897a91247d9ed7a240f0db6f… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Sat Dec 26 23:10:34 2009 +0300 comctl32/updown: Fix formatting for hex mode. --- dlls/comctl32/updown.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/comctl32/updown.c b/dlls/comctl32/updown.c index e87df87..076e315 100644 --- a/dlls/comctl32/updown.c +++ b/dlls/comctl32/updown.c @@ -309,7 +309,9 @@ static BOOL UPDOWN_GetBuddyInt (UPDOWN_INFO *infoPtr) */ static BOOL UPDOWN_SetBuddyInt (const UPDOWN_INFO *infoPtr) { - WCHAR fmt[3] = { '%', 'd', '\0' }; + static const WCHAR fmt_hex[] = { '0', 'x', '%', '0', '4', 'X', 0 }; + static const WCHAR fmt_dec_oct[] = { '%', 'd', '\0' }; + const WCHAR *fmt; WCHAR txt[20], txt_old[20] = { 0 }; int len; @@ -324,7 +326,7 @@ static BOOL UPDOWN_SetBuddyInt (const UPDOWN_INFO *infoPtr) } /* Regular window, so set caption to the number */ - if (infoPtr->Base == 16) fmt[1] = 'X'; + fmt = (infoPtr->Base == 16) ? fmt_hex : fmt_dec_oct; len = wsprintfW(txt, fmt, infoPtr->CurVal);

1 0

Nikolay Sivov : comctl32/rebar: Check for null handles before invalidation.
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: 46194d39f63317c6dbe4d55293c45765d01ad42b URL: http://source.winehq.org/git/wine.git/?a=commit;h=46194d39f63317c6dbe4d5529… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Sat Dec 26 19:17:06 2009 +0300 comctl32/rebar: Check for null handles before invalidation. This prevents all windows invalidation. --- dlls/comctl32/rebar.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/comctl32/rebar.c b/dlls/comctl32/rebar.c index 021bc16..2bb8b92 100644 --- a/dlls/comctl32/rebar.c +++ b/dlls/comctl32/rebar.c @@ -802,7 +802,7 @@ REBAR_CalcHorzBand (const REBAR_INFO *infoPtr, UINT rstart, UINT rend) work.right += SEP_WIDTH; work.bottom += SEP_WIDTH; InvalidateRect(infoPtr->hwndSelf, &work, TRUE); - InvalidateRect(lpBand->hwndChild, NULL, TRUE); + if (lpBand->hwndChild) InvalidateRect(lpBand->hwndChild, NULL, TRUE); } } @@ -923,7 +923,7 @@ REBAR_CalcVertBand (const REBAR_INFO *infoPtr, UINT rstart, UINT rend) work.bottom += SEP_WIDTH; work.right += SEP_WIDTH; InvalidateRect(infoPtr->hwndSelf, &work, TRUE); - InvalidateRect(lpBand->hwndChild, NULL, TRUE); + if (lpBand->hwndChild) InvalidateRect(lpBand->hwndChild, NULL, TRUE); } }

1 0

Henri Verbeet : wined3d: Let "SetupForBlit()" figure out the target' s width and height on its own.
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: 9c0bf89ed09ef529ebd544efe7274513583b5825 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9c0bf89ed09ef529ebd544efe… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Dec 27 19:57:23 2009 +0100 wined3d: Let "SetupForBlit()" figure out the target's width and height on its own. --- dlls/wined3d/context.c | 10 ++++------ 1 files changed, 4 insertions(+), 6 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index a0d16c9..c3c90ea 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -1624,16 +1624,16 @@ static inline void set_blit_dimension(UINT width, UINT height) { * Params: * This: Device to activate the context for * context: Context to setup - * width: render target width - * height: render target height * *****************************************************************************/ /* Context activation is done by the caller. */ -static inline void SetupForBlit(IWineD3DDeviceImpl *This, struct wined3d_context *context, UINT width, UINT height) +static void SetupForBlit(IWineD3DDeviceImpl *This, struct wined3d_context *context) { int i; const struct StateEntry *StateTable = This->StateTable; const struct wined3d_gl_info *gl_info = context->gl_info; + UINT width = ((IWineD3DSurfaceImpl *)context->current_rt)->currentDesc.Width; + UINT height = ((IWineD3DSurfaceImpl *)context->current_rt)->currentDesc.Height; DWORD sampler; TRACE("Setting up context %p for blitting\n", context); @@ -2205,9 +2205,7 @@ static void context_apply_state(struct wined3d_context *context, IWineD3DDeviceI break; case CTXUSAGE_BLIT: - SetupForBlit(device, context, - ((IWineD3DSurfaceImpl *)context->current_rt)->currentDesc.Width, - ((IWineD3DSurfaceImpl *)context->current_rt)->currentDesc.Height); + SetupForBlit(device, context); break; default:

1 0

Henri Verbeet : wined3d: Let "FindContext()" figure out the thread id on its own.
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: 57ff4b4dead687f7184305731da2dbee678f87a1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=57ff4b4dead687f7184305731… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Dec 27 19:57:22 2009 +0100 wined3d: Let "FindContext()" figure out the thread id on its own. It doesn't work with anything other than the current thread anyway. --- dlls/wined3d/context.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 797dd6b..a0d16c9 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -1857,11 +1857,12 @@ static struct wined3d_context *findThreadContextForSwapChain(IWineD3DSwapChain * * Returns: The needed context * *****************************************************************************/ -static inline struct wined3d_context *FindContext(IWineD3DDeviceImpl *This, IWineD3DSurface *target, DWORD tid) +static struct wined3d_context *FindContext(IWineD3DDeviceImpl *This, IWineD3DSurface *target) { IWineD3DSwapChain *swapchain = NULL; struct wined3d_context *current_context = context_get_current(); const struct StateEntry *StateTable = This->StateTable; + DWORD tid = GetCurrentThreadId(); struct wined3d_context *context; BOOL old_render_offscreen; @@ -2234,7 +2235,7 @@ struct wined3d_context *context_acquire(IWineD3DDeviceImpl *device, IWineD3DSurf TRACE("device %p, target %p, usage %#x.\n", device, target, usage); - context = FindContext(device, target, GetCurrentThreadId()); + context = FindContext(device, target); context_enter(context); if (!context->valid) return context;

1 0

Henri Verbeet : wined3d: Introduce "context_apply_state()" to setup a context for a specific usage.
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: 9295de9b841ad46a628f7e1d802f93b2ce4a73ee URL: http://source.winehq.org/git/wine.git/?a=commit;h=9295de9b841ad46a628f7e1d8… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Dec 27 19:57:21 2009 +0100 wined3d: Introduce "context_apply_state()" to setup a context for a specific usage. --- dlls/wined3d/context.c | 148 ++++++++++++++++++++++++------------------------ 1 files changed, 75 insertions(+), 73 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 30590af..797dd6b 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -2105,65 +2105,11 @@ void context_set_draw_buffer(struct wined3d_context *context, GLenum buffer) context->draw_buffer_dirty = TRUE; } -/***************************************************************************** - * context_acquire - * - * Finds a rendering context and drawable matching the device and render - * target for the current thread, activates them and puts them into the - * requested state. - * - * Params: - * This: Device to activate the context for - * target: Requested render target - * usage: Prepares the context for blitting, drawing or other actions - * - *****************************************************************************/ -struct wined3d_context *context_acquire(IWineD3DDeviceImpl *This, IWineD3DSurface *target, enum ContextUsage usage) +/* Context activation is done by the caller. */ +static void context_apply_state(struct wined3d_context *context, IWineD3DDeviceImpl *device, enum ContextUsage usage) { - struct wined3d_context *current_context = context_get_current(); - DWORD tid = GetCurrentThreadId(); - DWORD i, dirtyState, idx; - BYTE shift; - const struct StateEntry *StateTable = This->StateTable; - const struct wined3d_gl_info *gl_info; - struct wined3d_context *context; - - TRACE("(%p): Selecting context for render target %p, thread %d\n", This, target, tid); - - context = FindContext(This, target, tid); - context_enter(context); - if (!context->valid) return context; - - gl_info = context->gl_info; - - /* Activate the opengl context */ - if (context != current_context) - { - if (!context_set_current(context)) ERR("Failed to activate the new context.\n"); - else This->frag_pipe->enable_extension((IWineD3DDevice *)This, !context->last_was_blit); - - if (context->vshader_const_dirty) - { - memset(context->vshader_const_dirty, 1, - sizeof(*context->vshader_const_dirty) * This->d3d_vshader_constantF); - This->highest_dirty_vs_const = This->d3d_vshader_constantF; - } - if (context->pshader_const_dirty) - { - memset(context->pshader_const_dirty, 1, - sizeof(*context->pshader_const_dirty) * This->d3d_pshader_constantF); - This->highest_dirty_ps_const = This->d3d_pshader_constantF; - } - } - else if (context->restore_ctx) - { - if (!pwglMakeCurrent(context->hdc, context->glCtx)) - { - DWORD err = GetLastError(); - ERR("Failed to make GL context %p current on device context %p, last error %#x.\n", - context->hdc, context->glCtx, err); - } - } + const struct StateEntry *state_table = device->StateTable; + unsigned int i; switch (usage) { case CTXUSAGE_CLEAR: @@ -2186,7 +2132,7 @@ struct wined3d_context *context_acquire(IWineD3DDeviceImpl *This, IWineD3DSurfac FIXME("Activating for CTXUSAGE_BLIT for an offscreen target with ORM_FBO. This should be avoided.\n"); ENTER_GL(); context_bind_fbo(context, GL_FRAMEBUFFER, &context->dst_fbo); - context_attach_surface_fbo(context, GL_FRAMEBUFFER, 0, target); + context_attach_surface_fbo(context, GL_FRAMEBUFFER, 0, context->current_rt); context_attach_depth_stencil_fbo(context, GL_FRAMEBUFFER, NULL, FALSE); LEAVE_GL(); } else { @@ -2215,7 +2161,7 @@ struct wined3d_context *context_acquire(IWineD3DDeviceImpl *This, IWineD3DSurfac case CTXUSAGE_CLEAR: if(context->last_was_blit) { - This->frag_pipe->enable_extension((IWineD3DDevice *) This, TRUE); + device->frag_pipe->enable_extension((IWineD3DDevice *)device, TRUE); } /* Blending and clearing should be orthogonal, but tests on the nvidia driver show that disabling @@ -2224,32 +2170,33 @@ struct wined3d_context *context_acquire(IWineD3DDeviceImpl *This, IWineD3DSurfac ENTER_GL(); glDisable(GL_BLEND); LEAVE_GL(); - Context_MarkStateDirty(context, STATE_RENDER(WINED3DRS_ALPHABLENDENABLE), StateTable); + Context_MarkStateDirty(context, STATE_RENDER(WINED3DRS_ALPHABLENDENABLE), state_table); ENTER_GL(); glEnable(GL_SCISSOR_TEST); checkGLcall("glEnable GL_SCISSOR_TEST"); LEAVE_GL(); context->last_was_blit = FALSE; - Context_MarkStateDirty(context, STATE_RENDER(WINED3DRS_SCISSORTESTENABLE), StateTable); - Context_MarkStateDirty(context, STATE_SCISSORRECT, StateTable); + Context_MarkStateDirty(context, STATE_RENDER(WINED3DRS_SCISSORTESTENABLE), state_table); + Context_MarkStateDirty(context, STATE_SCISSORRECT, state_table); break; case CTXUSAGE_DRAWPRIM: /* This needs all dirty states applied */ if(context->last_was_blit) { - This->frag_pipe->enable_extension((IWineD3DDevice *) This, TRUE); + device->frag_pipe->enable_extension((IWineD3DDevice *)device, TRUE); } - IWineD3DDeviceImpl_FindTexUnitMap(This); + IWineD3DDeviceImpl_FindTexUnitMap(device); ENTER_GL(); - for(i=0; i < context->numDirtyEntries; i++) { - dirtyState = context->dirtyArray[i]; - idx = dirtyState >> 5; - shift = dirtyState & 0x1f; + for (i = 0; i < context->numDirtyEntries; ++i) + { + DWORD rep = context->dirtyArray[i]; + DWORD idx = rep >> 5; + BYTE shift = rep & 0x1f; context->isStateDirty[idx] &= ~(1 << shift); - StateTable[dirtyState].apply(dirtyState, This->stateBlock, context); + state_table[rep].apply(rep, device->stateBlock, context); } LEAVE_GL(); context->numDirtyEntries = 0; /* This makes the whole list clean */ @@ -2257,14 +2204,69 @@ struct wined3d_context *context_acquire(IWineD3DDeviceImpl *This, IWineD3DSurfac break; case CTXUSAGE_BLIT: - SetupForBlit(This, context, - ((IWineD3DSurfaceImpl *)target)->currentDesc.Width, - ((IWineD3DSurfaceImpl *)target)->currentDesc.Height); + SetupForBlit(device, context, + ((IWineD3DSurfaceImpl *)context->current_rt)->currentDesc.Width, + ((IWineD3DSurfaceImpl *)context->current_rt)->currentDesc.Height); break; default: FIXME("Unexpected context usage requested\n"); } +} + +/***************************************************************************** + * context_acquire + * + * Finds a rendering context and drawable matching the device and render + * target for the current thread, activates them and puts them into the + * requested state. + * + * Params: + * This: Device to activate the context for + * target: Requested render target + * usage: Prepares the context for blitting, drawing or other actions + * + *****************************************************************************/ +struct wined3d_context *context_acquire(IWineD3DDeviceImpl *device, IWineD3DSurface *target, enum ContextUsage usage) +{ + struct wined3d_context *current_context = context_get_current(); + struct wined3d_context *context; + + TRACE("device %p, target %p, usage %#x.\n", device, target, usage); + + context = FindContext(device, target, GetCurrentThreadId()); + context_enter(context); + if (!context->valid) return context; + + if (context != current_context) + { + if (!context_set_current(context)) ERR("Failed to activate the new context.\n"); + else device->frag_pipe->enable_extension((IWineD3DDevice *)device, !context->last_was_blit); + + if (context->vshader_const_dirty) + { + memset(context->vshader_const_dirty, 1, + sizeof(*context->vshader_const_dirty) * device->d3d_vshader_constantF); + device->highest_dirty_vs_const = device->d3d_vshader_constantF; + } + if (context->pshader_const_dirty) + { + memset(context->pshader_const_dirty, 1, + sizeof(*context->pshader_const_dirty) * device->d3d_pshader_constantF); + device->highest_dirty_ps_const = device->d3d_pshader_constantF; + } + } + else if (context->restore_ctx) + { + if (!pwglMakeCurrent(context->hdc, context->glCtx)) + { + DWORD err = GetLastError(); + ERR("Failed to make GL context %p current on device context %p, last error %#x.\n", + context->hdc, context->glCtx, err); + } + } + + context_apply_state(context, device, usage); return context; }

1 0

Henri Verbeet : wined3d: Recognize "VMware, Inc." as VENDOR_MESA.
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: f30999f193e564b10b3826bd90e9096038d6239e URL: http://source.winehq.org/git/wine.git/?a=commit;h=f30999f193e564b10b3826bd9… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Dec 27 19:57:20 2009 +0100 wined3d: Recognize "VMware, Inc." as VENDOR_MESA. --- dlls/wined3d/directx.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index d0e6d3d..3ab1902 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -1123,7 +1123,8 @@ static enum wined3d_pci_vendor wined3d_guess_vendor(const char *gl_vendor, const if (strstr(gl_vendor, "Mesa") || strstr(gl_vendor, "DRI R300 Project") - || strstr(gl_vendor, "Tungsten Graphics, Inc")) + || strstr(gl_vendor, "Tungsten Graphics, Inc") + || strstr(gl_vendor, "VMware, Inc.")) return VENDOR_MESA; FIXME_(d3d_caps)("Received unrecognized GL_VENDOR %s. Returning VENDOR_WINE.\n", debugstr_a(gl_vendor));

1 0

Henri Verbeet : wined3d: Properly check for glBlitFramebuffer().
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: 7160e56ab100876c0039f29412aab45cbe7a34f2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7160e56ab100876c0039f2941… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Dec 27 19:57:19 2009 +0100 wined3d: Properly check for glBlitFramebuffer(). ARB_framebuffer_object can also provide this entry point. --- dlls/wined3d/swapchain.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/swapchain.c b/dlls/wined3d/swapchain.c index ad257f5..445fbbf 100644 --- a/dlls/wined3d/swapchain.c +++ b/dlls/wined3d/swapchain.c @@ -110,7 +110,7 @@ static inline void swapchain_blit(IWineD3DSwapChainImpl *This, struct wined3d_co if(w == window.right && h == window.bottom) gl_filter = GL_NEAREST; else gl_filter = GL_LINEAR; - if(gl_info->supported[EXT_FRAMEBUFFER_BLIT]) + if (gl_info->fbo_ops.glBlitFramebuffer) { ENTER_GL(); context_bind_fbo(context, GL_READ_FRAMEBUFFER, &context->src_fbo);

1 0

Michael Stefaniuc : user32: Remove some explicit LPARAM/WPARAM casts.
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: 6f3b49446352c4f06adba16fee7125a171525470 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6f3b49446352c4f06adba16fe… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Sun Dec 27 14:40:45 2009 +0100 user32: Remove some explicit LPARAM/WPARAM casts. --- dlls/user32/button.c | 2 +- dlls/user32/combo.c | 34 +++++++++++++++------------------- dlls/user32/edit.c | 8 ++++---- dlls/user32/mdi.c | 4 ++-- dlls/user32/message.c | 2 +- 5 files changed, 23 insertions(+), 27 deletions(-) diff --git a/dlls/user32/button.c b/dlls/user32/button.c index cec92e0..916d0c2 100644 --- a/dlls/user32/button.c +++ b/dlls/user32/button.c @@ -742,7 +742,7 @@ static void BUTTON_DrawLabel(HWND hwnd, HDC hdc, UINT dtFlags, const RECT *rc) lpOutputProc = BUTTON_DrawTextCallback; if (!(text = get_button_text( hwnd ))) return; lp = (LPARAM)text; - wp = (WPARAM)dtFlags; + wp = dtFlags; break; case BS_ICON: diff --git a/dlls/user32/combo.c b/dlls/user32/combo.c index 1dd4f37..86b5680 100644 --- a/dlls/user32/combo.c +++ b/dlls/user32/combo.c @@ -717,7 +717,7 @@ static void CBPaintText( if( (pText = HeapAlloc( GetProcessHeap(), 0, (size + 1) * sizeof(WCHAR))) ) { /* size from LB_GETTEXTLEN may be too large, from LB_GETTEXT is accurate */ - size=SendMessageW(lphc->hWndLBox, LB_GETTEXT, (WPARAM)id, (LPARAM)pText); + size=SendMessageW(lphc->hWndLBox, LB_GETTEXT, id, (LPARAM)pText); pText[size] = '\0'; /* just in case */ } else return; } @@ -730,7 +730,7 @@ static void CBPaintText( static const WCHAR empty_stringW[] = { 0 }; if( CB_HASSTRINGS(lphc) ) SetWindowTextW( lphc->hWndEdit, pText ? pText : empty_stringW ); if( lphc->wState & CBF_FOCUSED ) - SendMessageW(lphc->hWndEdit, EM_SETSEL, 0, (LPARAM)(-1)); + SendMessageW(lphc->hWndEdit, EM_SETSEL, 0, -1); } else /* paint text field ourselves */ { @@ -776,8 +776,7 @@ static void CBPaintText( dis.itemState = itemState; dis.hDC = hdc; dis.rcItem = rectEdit; - dis.itemData = SendMessageW(lphc->hWndLBox, LB_GETITEMDATA, - (WPARAM)id, 0 ); + dis.itemData = SendMessageW(lphc->hWndLBox, LB_GETITEMDATA, id, 0); /* * Clip the DC and have the parent draw the item. @@ -975,16 +974,15 @@ static INT CBUpdateLBox( LPHEADCOMBO lphc, BOOL bSelect ) if( pText ) { GetWindowTextW( lphc->hWndEdit, pText, length + 1); - idx = SendMessageW(lphc->hWndLBox, LB_FINDSTRING, - (WPARAM)(-1), (LPARAM)pText ); + idx = SendMessageW(lphc->hWndLBox, LB_FINDSTRING, -1, (LPARAM)pText); HeapFree( GetProcessHeap(), 0, pText ); } - SendMessageW(lphc->hWndLBox, LB_SETCURSEL, (WPARAM)(bSelect ? idx : -1), 0); + SendMessageW(lphc->hWndLBox, LB_SETCURSEL, bSelect ? idx : -1, 0); /* probably superfluous but Windows sends this too */ - SendMessageW(lphc->hWndLBox, LB_SETCARETINDEX, (WPARAM)(idx < 0 ? 0 : idx), 0); - SendMessageW(lphc->hWndLBox, LB_SETTOPINDEX, (WPARAM)(idx < 0 ? 0 : idx), 0); + SendMessageW(lphc->hWndLBox, LB_SETCARETINDEX, idx < 0 ? 0 : idx, 0); + SendMessageW(lphc->hWndLBox, LB_SETTOPINDEX, idx < 0 ? 0 : idx, 0); return idx; } @@ -1004,13 +1002,12 @@ static void CBUpdateEdit( LPHEADCOMBO lphc , INT index ) if( index >= 0 ) /* got an entry */ { - length = SendMessageW(lphc->hWndLBox, LB_GETTEXTLEN, (WPARAM)index, 0); + length = SendMessageW(lphc->hWndLBox, LB_GETTEXTLEN, index, 0); if( length != LB_ERR) { if( (pText = HeapAlloc( GetProcessHeap(), 0, (length + 1) * sizeof(WCHAR))) ) { - SendMessageW(lphc->hWndLBox, LB_GETTEXT, - (WPARAM)index, (LPARAM)pText ); + SendMessageW(lphc->hWndLBox, LB_GETTEXT, index, (LPARAM)pText); } } } @@ -1023,7 +1020,7 @@ static void CBUpdateEdit( LPHEADCOMBO lphc , INT index ) } if( lphc->wState & CBF_FOCUSED ) - SendMessageW(lphc->hWndEdit, EM_SETSEL, 0, (LPARAM)(-1)); + SendMessageW(lphc->hWndEdit, EM_SETSEL, 0, -1); HeapFree( GetProcessHeap(), 0, pText ); } @@ -1061,7 +1058,7 @@ static void CBDropDown( LPHEADCOMBO lphc ) lphc->droppedIndex = SendMessageW(lphc->hWndLBox, LB_GETCURSEL, 0, 0); SendMessageW(lphc->hWndLBox, LB_SETTOPINDEX, - (WPARAM)(lphc->droppedIndex == LB_ERR ? 0 : lphc->droppedIndex), 0 ); + lphc->droppedIndex == LB_ERR ? 0 : lphc->droppedIndex, 0); SendMessageW(lphc->hWndLBox, LB_CARETON, 0, 0); } @@ -1356,7 +1353,7 @@ static LRESULT COMBO_Command( LPHEADCOMBO lphc, WPARAM wParam, HWND hWnd ) lphc->wState |= CBF_NOLBSELECT; CBUpdateEdit( lphc, index ); /* select text in edit, as Windows does */ - SendMessageW(lphc->hWndEdit, EM_SETSEL, 0, (LPARAM)(-1)); + SendMessageW(lphc->hWndEdit, EM_SETSEL, 0, -1); } else { @@ -1655,8 +1652,7 @@ static LRESULT COMBO_SetItemHeight( LPHEADCOMBO lphc, INT index, INT height ) } } else if ( CB_OWNERDRAWN(lphc) ) /* set listbox item height */ - lRet = SendMessageW(lphc->hWndLBox, LB_SETITEMHEIGHT, - (WPARAM)index, (LPARAM)height ); + lRet = SendMessageW(lphc->hWndLBox, LB_SETITEMHEIGHT, index, height); return lRet; } @@ -1665,8 +1661,8 @@ static LRESULT COMBO_SetItemHeight( LPHEADCOMBO lphc, INT index, INT height ) */ static LRESULT COMBO_SelectString( LPHEADCOMBO lphc, INT start, LPARAM pText, BOOL unicode ) { - INT index = unicode ? SendMessageW(lphc->hWndLBox, LB_SELECTSTRING, (WPARAM)start, pText) : - SendMessageA(lphc->hWndLBox, LB_SELECTSTRING, (WPARAM)start, pText); + INT index = unicode ? SendMessageW(lphc->hWndLBox, LB_SELECTSTRING, start, pText) : + SendMessageA(lphc->hWndLBox, LB_SELECTSTRING, start, pText); if( index >= 0 ) { if( lphc->wState & CBF_EDIT ) diff --git a/dlls/user32/edit.c b/dlls/user32/edit.c index 01ca5df..d7ab5b4 100644 --- a/dlls/user32/edit.c +++ b/dlls/user32/edit.c @@ -3195,14 +3195,14 @@ static BOOL EDIT_CheckCombo(EDITSTATE *es, UINT msg, INT key) nEUI = 2; } - SendMessageW(hLBox, WM_KEYDOWN, (WPARAM)key, 0); + SendMessageW(hLBox, WM_KEYDOWN, key, 0); break; case WM_SYSKEYDOWN: /* Handle Alt+up/down arrows */ if (nEUI) SendMessageW(hCombo, CB_SHOWDROPDOWN, bDropped ? FALSE : TRUE, 0); else - SendMessageW(hLBox, WM_KEYDOWN, (WPARAM)VK_F4, 0); + SendMessageW(hLBox, WM_KEYDOWN, VK_F4, 0); break; } @@ -3326,7 +3326,7 @@ static LRESULT EDIT_WM_KeyDown(EDITSTATE *es, INT key) HWND hwDefCtrl = GetDlgItem(es->hwndParent, LOWORD(dw)); if (hwDefCtrl) { - SendMessageW(es->hwndParent, WM_NEXTDLGCTL, (WPARAM)hwDefCtrl, (LPARAM)TRUE); + SendMessageW(es->hwndParent, WM_NEXTDLGCTL, (WPARAM)hwDefCtrl, TRUE); PostMessageW(hwDefCtrl, WM_KEYDOWN, VK_RETURN, 0); } } @@ -3797,7 +3797,7 @@ static LRESULT EDIT_WM_SysKeyDown(EDITSTATE *es, INT key, DWORD key_data) if (EDIT_CheckCombo(es, WM_SYSKEYDOWN, key)) return 0; } - return DefWindowProcW(es->hwndSelf, WM_SYSKEYDOWN, (WPARAM)key, (LPARAM)key_data); + return DefWindowProcW(es->hwndSelf, WM_SYSKEYDOWN, key, key_data); } diff --git a/dlls/user32/mdi.c b/dlls/user32/mdi.c index 991ba37..b5e934a 100644 --- a/dlls/user32/mdi.c +++ b/dlls/user32/mdi.c @@ -1583,7 +1583,7 @@ LRESULT WINAPI DefMDIChildProcW( HWND hwnd, UINT message, case WM_SYSCHAR: if (wParam == '-') { - SendMessageW( hwnd, WM_SYSCOMMAND, (WPARAM)SC_KEYMENU, (DWORD)VK_SPACE); + SendMessageW( hwnd, WM_SYSCOMMAND, SC_KEYMENU, VK_SPACE); return 0; } break; @@ -1686,7 +1686,7 @@ BOOL WINAPI TranslateMDISysAccel( HWND hwndClient, LPMSG msg ) return 0; } TRACE("wParam = %04lx\n", wParam); - SendMessageW(ci->hwndActiveChild, WM_SYSCOMMAND, wParam, (LPARAM)msg->wParam); + SendMessageW(ci->hwndActiveChild, WM_SYSCOMMAND, wParam, msg->wParam); return 1; } } diff --git a/dlls/user32/message.c b/dlls/user32/message.c index d85eabb..022202a 100644 --- a/dlls/user32/message.c +++ b/dlls/user32/message.c @@ -1750,7 +1750,7 @@ static BOOL process_keyboard_message( MSG *msg, UINT hw_id, HWND hwnd_filter, { /* Handle VK_APPS key by posting a WM_CONTEXTMENU message */ if (msg->wParam == VK_APPS && !MENU_IsMenuActive()) - PostMessageW(msg->hwnd, WM_CONTEXTMENU, (WPARAM)msg->hwnd, (LPARAM)-1); + PostMessageW(msg->hwnd, WM_CONTEXTMENU, (WPARAM)msg->hwnd, -1); } }

1 0

Michael Stefaniuc : winex11.drv: Remove some explicit LPARAM/WPARAM casts.
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: 425cc7e033a7e82c701441c1f43513c639c791d8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=425cc7e033a7e82c701441c1f… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Sun Dec 27 14:40:57 2009 +0100 winex11.drv: Remove some explicit LPARAM/WPARAM casts. --- dlls/winex11.drv/clipboard.c | 2 +- dlls/winex11.drv/opengl.c | 2 +- dlls/winex11.drv/systray.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/winex11.drv/clipboard.c b/dlls/winex11.drv/clipboard.c index 7735b56..706d378 100644 --- a/dlls/winex11.drv/clipboard.c +++ b/dlls/winex11.drv/clipboard.c @@ -790,7 +790,7 @@ static BOOL X11DRV_CLIPBOARD_RenderFormat(Display *display, LPWINE_CLIPDATA lpDa * data requested into the clipboard. */ TRACE("Sending WM_RENDERFORMAT message to hwnd(%p)\n", cbInfo.hWndOwner); - SendMessageW(cbInfo.hWndOwner, WM_RENDERFORMAT, (WPARAM)lpData->wFormatID, 0); + SendMessageW(cbInfo.hWndOwner, WM_RENDERFORMAT, lpData->wFormatID, 0); if (!lpData->hData) bret = FALSE; } diff --git a/dlls/winex11.drv/opengl.c b/dlls/winex11.drv/opengl.c index 5bd0255..e784844 100644 --- a/dlls/winex11.drv/opengl.c +++ b/dlls/winex11.drv/opengl.c @@ -1601,7 +1601,7 @@ static BOOL internal_SetPixelFormat(X11DRV_PDEVICE *physDev, return FALSE; } - if(!SendMessageW(hwnd, WM_X11DRV_SET_WIN_FORMAT, (WPARAM)fmt->fmt_id, 0)) { + if(!SendMessageW(hwnd, WM_X11DRV_SET_WIN_FORMAT, fmt->fmt_id, 0)) { ERR("Couldn't set format of the window, returning failure\n"); return FALSE; } diff --git a/dlls/winex11.drv/systray.c b/dlls/winex11.drv/systray.c index ddeb223..bbe6b67 100644 --- a/dlls/winex11.drv/systray.c +++ b/dlls/winex11.drv/systray.c @@ -185,7 +185,7 @@ static LRESULT WINAPI tray_wndproc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lp case WM_MBUTTONDBLCLK: /* notify the owner hwnd of the message */ TRACE("relaying 0x%x\n", msg); - ret = PostMessageW(icon->owner, icon->callback_message, (WPARAM) icon->id, (LPARAM) msg); + ret = PostMessageW(icon->owner, icon->callback_message, icon->id, msg); if (!ret && (GetLastError() == ERROR_INVALID_WINDOW_HANDLE)) { WARN( "application window was destroyed, removing icon %u\n", icon->id );

1 0

Michael Stefaniuc : user32/tests: Remove some explicit LPARAM/WPARAM casts.
by Alexandre Julliard 28 Dec '09

28 Dec '09

Module: wine Branch: master Commit: 7e2b6248963ccb5cbaf4b6c7d5be70df975fbf12 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7e2b6248963ccb5cbaf4b6c7d… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Sun Dec 27 14:40:19 2009 +0100 user32/tests: Remove some explicit LPARAM/WPARAM casts. --- dlls/user32/tests/edit.c | 40 ++++++++++++++++++++-------------------- dlls/user32/tests/listbox.c | 16 ++++++++-------- dlls/user32/tests/msg.c | 4 ++-- 3 files changed, 30 insertions(+), 30 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=7e2b6248963ccb5cbaf4b…

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits December 2009