April 2011 - Wine-commits - list.winehq.org

Michael Stefaniuc : d3d9: COM cleanup for the IDirect3D9Ex iface.
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: a159569115d4f6f90d9d037437cae776183638ef URL: http://source.winehq.org/git/wine.git/?a=commit;h=a159569115d4f6f90d9d03743… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Sun Apr 10 22:20:18 2011 +0200 d3d9: COM cleanup for the IDirect3D9Ex iface. --- dlls/d3d9/d3d9_main.c | 4 +- dlls/d3d9/d3d9_private.h | 2 +- dlls/d3d9/directx.c | 98 ++++++++++++++++++++++++++++----------------- 3 files changed, 64 insertions(+), 40 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a159569115d4f6f90d9d0…

1 0

Francois Gouget : progman: Fix the Alt+Enter accelerator so it shows the selected element attributes.
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: 25a31a49c1382235278116a157020532c78e86e1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=25a31a49c1382235278116a15… Author: Francois Gouget <fgouget(a)free.fr> Date: Sun Apr 10 18:45:06 2011 +0200 progman: Fix the Alt+Enter accelerator so it shows the selected element attributes. --- programs/progman/progman.rc | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/programs/progman/progman.rc b/programs/progman/progman.rc index 4562093..2281b33 100644 --- a/programs/progman/progman.rc +++ b/programs/progman/progman.rc @@ -86,5 +86,5 @@ IDS_SYMBOLS_ICO, "Icons (*.ico)" ACCEL ACCELERATORS { -VK_RETURN, PM_EXECUTE, VIRTKEY, ALT +VK_RETURN, PM_ATTRIBUTES, VIRTKEY, ALT }

1 0

Francois Gouget : d3dx9_36: Make get_parameter_{handle, struct}() static.
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: 18e0e8c4c7dcbcc0d152ca1ba5e2fb49d0962edb URL: http://source.winehq.org/git/wine.git/?a=commit;h=18e0e8c4c7dcbcc0d152ca1ba… Author: Francois Gouget <fgouget(a)free.fr> Date: Sun Apr 10 18:45:11 2011 +0200 d3dx9_36: Make get_parameter_{handle,struct}() static. --- dlls/d3dx9_36/effect.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c index 1e3de0c..a03e8a3 100644 --- a/dlls/d3dx9_36/effect.c +++ b/dlls/d3dx9_36/effect.c @@ -91,12 +91,12 @@ static void skip_dword_unknown(const char **ptr, unsigned int count) } } -struct d3dx_parameter *get_parameter_struct(D3DXHANDLE handle) +static struct d3dx_parameter *get_parameter_struct(D3DXHANDLE handle) { return (struct d3dx_parameter *) handle; } -D3DXHANDLE get_parameter_handle(struct d3dx_parameter *parameter) +static D3DXHANDLE get_parameter_handle(struct d3dx_parameter *parameter) { return (D3DXHANDLE) parameter; }

1 0

Francois Gouget : mshtml: Make HTMLFiltersCollection_Create() static.
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: aa3f823d193ef76d23502bf1d1094896d497bccf URL: http://source.winehq.org/git/wine.git/?a=commit;h=aa3f823d193ef76d23502bf1d… Author: Francois Gouget <fgouget(a)free.fr> Date: Sun Apr 10 18:46:08 2011 +0200 mshtml: Make HTMLFiltersCollection_Create() static. --- dlls/mshtml/htmlelem.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/htmlelem.c b/dlls/mshtml/htmlelem.c index bf54739..74a4394 100644 --- a/dlls/mshtml/htmlelem.c +++ b/dlls/mshtml/htmlelem.c @@ -109,7 +109,7 @@ static inline HTMLFiltersCollection *impl_from_IHTMLFiltersCollection(IHTMLFilte return CONTAINING_RECORD(iface, HTMLFiltersCollection, IHTMLFiltersCollection_iface); } -IHTMLFiltersCollection *HTMLFiltersCollection_Create(void); +static IHTMLFiltersCollection *HTMLFiltersCollection_Create(void); WINE_DEFAULT_DEBUG_CHANNEL(mshtml); @@ -1971,7 +1971,7 @@ static dispex_static_data_t HTMLFiltersCollection_dispex = { HTMLFiltersCollection_iface_tids }; -IHTMLFiltersCollection *HTMLFiltersCollection_Create() +static IHTMLFiltersCollection *HTMLFiltersCollection_Create(void) { HTMLFiltersCollection *ret = heap_alloc(sizeof(HTMLFiltersCollection));

1 0

Henri Verbeet : wined3d: Explicitly handle 0 rt_count right at the start of context_apply_draw_buffers ().
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: bf99c0248f04655dfb402d03bc29a52a20dc5466 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bf99c0248f04655dfb402d03b… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Apr 8 20:22:49 2011 +0200 wined3d: Explicitly handle 0 rt_count right at the start of context_apply_draw_buffers(). It's trivial, and allows the rest of the code to assume at least 1 render target is present. --- dlls/wined3d/context.c | 11 +++++++++-- 1 files changed, 9 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 5b5ae5a..64380c8 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -1915,7 +1915,14 @@ static struct wined3d_context *FindContext(IWineD3DDeviceImpl *This, IWineD3DSur /* Context activation is done by the caller. */ static void context_apply_draw_buffers(struct wined3d_context *context, UINT rt_count, IWineD3DSurfaceImpl **rts) { - if (rt_count && !surface_is_offscreen(rts[0])) + if (!rt_count) + { + ENTER_GL(); + glDrawBuffer(GL_NONE); + checkGLcall("glDrawBuffer()"); + LEAVE_GL(); + } + else if (!surface_is_offscreen(rts[0])) { ENTER_GL(); glDrawBuffer(surface_get_gl_buffer(rts[0])); @@ -1951,7 +1958,7 @@ static void context_apply_draw_buffers(struct wined3d_context *context, UINT rt_ } else { - glDrawBuffer(rt_count ? rts[0]->resource.device->offscreenBuffer : GL_NONE); + glDrawBuffer(rts[0]->resource.device->offscreenBuffer); checkGLcall("glDrawBuffer()"); } LEAVE_GL();

1 0

Henri Verbeet : wined3d: Only set draw buffers that are explicitly specified in context_apply_draw_buffers ().
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: 2f3c147ab282b6e1a782df17b8a223496bfc5508 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2f3c147ab282b6e1a782df17b… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Apr 8 20:22:50 2011 +0200 wined3d: Only set draw buffers that are explicitly specified in context_apply_draw_buffers(). Anything that isn't explicitly specified is set to GL_NONE by glDrawBuffersARB(). --- dlls/wined3d/context.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 64380c8..d1a3189 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -1937,9 +1937,9 @@ static void context_apply_draw_buffers(struct wined3d_context *context, UINT rt_ const struct wined3d_gl_info *gl_info = context->gl_info; unsigned int i; - for (i = 0; i < gl_info->limits.buffers; ++i) + for (i = 0; i < rt_count; ++i) { - if (i < rt_count && rts[i] && rts[i]->resource.format->id != WINED3DFMT_NULL) + if (rts[i] && rts[i]->resource.format->id != WINED3DFMT_NULL) context->draw_buffers[i] = GL_COLOR_ATTACHMENT0 + i; else context->draw_buffers[i] = GL_NONE; @@ -1947,7 +1947,7 @@ static void context_apply_draw_buffers(struct wined3d_context *context, UINT rt_ if (gl_info->supported[ARB_DRAW_BUFFERS]) { - GL_EXTCALL(glDrawBuffersARB(gl_info->limits.buffers, context->draw_buffers)); + GL_EXTCALL(glDrawBuffersARB(rt_count, context->draw_buffers)); checkGLcall("glDrawBuffers()"); } else

1 0

Henri Verbeet : wined3d: Set the read buffer to GL_NONE for draws and clears on an FBO.
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: 8e12b2e61e8c6441ed25c116142c262efdf88b20 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8e12b2e61e8c6441ed25c1161… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Apr 8 20:22:51 2011 +0200 wined3d: Set the read buffer to GL_NONE for draws and clears on an FBO. For the benefit of depth-only draws and clears, that would otherwise fail FBO completeness, unless the driver supports ARB_ES2_compatibility. --- dlls/wined3d/context.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index d1a3189..c2bdd3c 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -2105,6 +2105,8 @@ BOOL context_apply_clear_state(struct wined3d_context *context, IWineD3DDeviceIm ++i; } context_apply_fbo_state(context, GL_FRAMEBUFFER, context->blit_targets, depth_stencil, SFLAG_INTEXTURE); + glReadBuffer(GL_NONE); + checkGLcall("glReadBuffer"); } else { @@ -2172,6 +2174,8 @@ BOOL context_apply_draw_state(struct wined3d_context *context, IWineD3DDeviceImp ENTER_GL(); context_apply_fbo_state(context, GL_FRAMEBUFFER, device->render_targets, device->depth_stencil, SFLAG_INTEXTURE); + glReadBuffer(GL_NONE); + checkGLcall("glReadBuffer"); LEAVE_GL(); } }

1 0

Henri Verbeet : wined3d: Check FBO completeness after the read / draw buffers are specified.
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: 84103460bdab8a3f57f29817d7be2ceaf93ee2c3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=84103460bdab8a3f57f29817d… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Apr 8 20:22:52 2011 +0200 wined3d: Check FBO completeness after the read / draw buffers are specified. Unless the driver implements ARB_ES2_compatibility / GL4.1, FBO completeness depends on what read / draw buffers are set. --- dlls/wined3d/context.c | 25 ++++++++++++++++++++++--- dlls/wined3d/surface.c | 4 ++++ dlls/wined3d/swapchain.c | 1 + dlls/wined3d/wined3d_private.h | 1 + 4 files changed, 28 insertions(+), 3 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index c2bdd3c..be2ad3c 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -299,7 +299,7 @@ static void context_attach_surface_fbo(const struct wined3d_context *context, } /* GL locking is done by the caller */ -static void context_check_fbo_status(struct wined3d_context *context, GLenum target) +void context_check_fbo_status(struct wined3d_context *context, GLenum target) { const struct wined3d_gl_info *gl_info = context->gl_info; GLenum status; @@ -499,8 +499,6 @@ static void context_apply_fbo_state(struct wined3d_context *context, GLenum targ context->current_fbo = NULL; context_bind_fbo(context, target, NULL); } - - context_check_fbo_status(context, target); } /* GL locking is done by the caller */ @@ -2056,6 +2054,13 @@ void context_apply_blit_state(struct wined3d_context *context, IWineD3DDeviceImp context->draw_buffer_dirty = FALSE; } + if (wined3d_settings.offscreen_rendering_mode == ORM_FBO) + { + ENTER_GL(); + context_check_fbo_status(context, GL_FRAMEBUFFER); + LEAVE_GL(); + } + SetupForBlit(device, context); } @@ -2119,6 +2124,13 @@ BOOL context_apply_clear_state(struct wined3d_context *context, IWineD3DDeviceIm context_apply_draw_buffers(context, rt_count, rts); context->draw_buffer_dirty = TRUE; + if (wined3d_settings.offscreen_rendering_mode == ORM_FBO) + { + ENTER_GL(); + context_check_fbo_status(context, GL_FRAMEBUFFER); + LEAVE_GL(); + } + if (context->last_was_blit) { device->frag_pipe->enable_extension(TRUE); @@ -2186,6 +2198,13 @@ BOOL context_apply_draw_state(struct wined3d_context *context, IWineD3DDeviceImp context->draw_buffer_dirty = FALSE; } + if (wined3d_settings.offscreen_rendering_mode == ORM_FBO) + { + ENTER_GL(); + context_check_fbo_status(context, GL_FRAMEBUFFER); + LEAVE_GL(); + } + if (context->last_was_blit) { device->frag_pipe->enable_extension(TRUE); diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 801eb57..c2d92ed 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -3416,6 +3416,7 @@ static void surface_blt_fbo(IWineD3DDeviceImpl *device, const WINED3DTEXTUREFILT glReadBuffer(GL_COLOR_ATTACHMENT0); checkGLcall("glReadBuffer()"); } + context_check_fbo_status(context, GL_READ_FRAMEBUFFER); LEAVE_GL(); if (dst_location == SFLAG_INDRAWABLE) @@ -3438,6 +3439,7 @@ static void surface_blt_fbo(IWineD3DDeviceImpl *device, const WINED3DTEXTUREFILT context_apply_fbo_state_blit(context, GL_DRAW_FRAMEBUFFER, dst_surface, NULL, dst_location); context_set_draw_buffer(context, GL_COLOR_ATTACHMENT0); } + context_check_fbo_status(context, GL_DRAW_FRAMEBUFFER); glColorMask(GL_TRUE, GL_TRUE, GL_TRUE, GL_TRUE); IWineD3DDeviceImpl_MarkStateDirty(device, STATE_RENDER(WINED3DRS_COLORWRITEENABLE)); @@ -3516,9 +3518,11 @@ static void wined3d_surface_depth_blt_fbo(IWineD3DDeviceImpl *device, IWineD3DSu context_apply_fbo_state_blit(context, GL_READ_FRAMEBUFFER, NULL, src_surface, SFLAG_INTEXTURE); glReadBuffer(GL_NONE); checkGLcall("glReadBuffer()"); + context_check_fbo_status(context, GL_READ_FRAMEBUFFER); context_apply_fbo_state_blit(context, GL_DRAW_FRAMEBUFFER, NULL, dst_surface, SFLAG_INTEXTURE); context_set_draw_buffer(context, GL_NONE); + context_check_fbo_status(context, GL_DRAW_FRAMEBUFFER); if (gl_mask & GL_DEPTH_BUFFER_BIT) { diff --git a/dlls/wined3d/swapchain.c b/dlls/wined3d/swapchain.c index 5be6705..6db92d6 100644 --- a/dlls/wined3d/swapchain.c +++ b/dlls/wined3d/swapchain.c @@ -302,6 +302,7 @@ static void swapchain_blit(IWineD3DSwapChainImpl *This, struct wined3d_context * ENTER_GL(); context_apply_fbo_state_blit(context, GL_READ_FRAMEBUFFER, backbuffer, NULL, SFLAG_INTEXTURE); glReadBuffer(GL_COLOR_ATTACHMENT0); + context_check_fbo_status(context, GL_READ_FRAMEBUFFER); context_bind_fbo(context, GL_DRAW_FRAMEBUFFER, NULL); context_set_draw_buffer(context, GL_BACK); diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index d78d504..4ef1cb6 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1218,6 +1218,7 @@ void context_apply_fbo_state_blit(struct wined3d_context *context, GLenum target void context_attach_depth_stencil_fbo(struct wined3d_context *context, GLenum fbo_target, IWineD3DSurfaceImpl *depth_stencil, BOOL use_render_buffer) DECLSPEC_HIDDEN; void context_bind_fbo(struct wined3d_context *context, GLenum target, GLuint *fbo) DECLSPEC_HIDDEN; +void context_check_fbo_status(struct wined3d_context *context, GLenum target) DECLSPEC_HIDDEN; struct wined3d_context *context_create(IWineD3DSwapChainImpl *swapchain, IWineD3DSurfaceImpl *target, const struct wined3d_format *ds_format) DECLSPEC_HIDDEN; void context_destroy(IWineD3DDeviceImpl *This, struct wined3d_context *context) DECLSPEC_HIDDEN;

1 0

André Hentschel : winedbg: Implement get_addr on sparc .
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: 6c710ae1eddf7b8d5cfa214b57733fedfe346efd URL: http://source.winehq.org/git/wine.git/?a=commit;h=6c710ae1eddf7b8d5cfa214b5… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sat Apr 9 20:46:55 2011 +0200 winedbg: Implement get_addr on sparc. --- programs/winedbg/be_sparc.c | 10 +++++++++- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/programs/winedbg/be_sparc.c b/programs/winedbg/be_sparc.c index c9afc95..546518a 100644 --- a/programs/winedbg/be_sparc.c +++ b/programs/winedbg/be_sparc.c @@ -25,7 +25,15 @@ static unsigned be_sparc_get_addr(HANDLE hThread, const CONTEXT* ctx, enum be_cpu_addr bca, ADDRESS64* addr) { - dbg_printf("not done for Sparc\n"); + switch (bca) + { + case be_cpu_addr_pc: + return be_cpu_build_addr(hThread, ctx, addr, 0, ctx->pc); + case be_cpu_addr_stack: + return be_cpu_build_addr(hThread, ctx, addr, 0, ctx->o6); + case be_cpu_addr_frame: + return be_cpu_build_addr(hThread, ctx, addr, 0, ctx->i6); + } return FALSE; }

1 0

Nikolay Sivov : comctl32/tab: Fix invalid read of item data.
by Alexandre Julliard 11 Apr '11

11 Apr '11

Module: wine Branch: master Commit: 6867735e20e7d6615d77eabb10f79c296a778bd6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6867735e20e7d6615d77eabb1… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sat Apr 9 19:39:45 2011 +0400 comctl32/tab: Fix invalid read of item data. --- dlls/comctl32/tab.c | 2 +- dlls/comctl32/tests/tab.c | 3 +++ 2 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dlls/comctl32/tab.c b/dlls/comctl32/tab.c index fe82d59..f86fd0f 100644 --- a/dlls/comctl32/tab.c +++ b/dlls/comctl32/tab.c @@ -2500,7 +2500,7 @@ static void TAB_EnsureSelectionVisible( if (infoPtr->leftmostVisible >= iSelected) { - infoPtr->leftmostVisible = iSelected; + if (iSelected >= 0) infoPtr->leftmostVisible = iSelected; } else { diff --git a/dlls/comctl32/tests/tab.c b/dlls/comctl32/tests/tab.c index 4c464e6..5bd2ea0 100644 --- a/dlls/comctl32/tests/tab.c +++ b/dlls/comctl32/tests/tab.c @@ -1052,6 +1052,9 @@ static void test_insert_focus(HWND parent_wnd) hTab = createFilledTabControl(parent_wnd, TCS_FIXEDWIDTH, mask, 0); ok(hTab != NULL, "Failed to create tab control\n"); + r = SendMessage(hTab, TCM_GETCURSEL, 0, 0); + expect(-1, r); + flush_sequences(sequences, NUM_MSG_SEQUENCES); nTabsRetrieved = SendMessage(hTab, TCM_GETITEMCOUNT, 0, 0);

1 0