wine-commits March 2010

wine-commits@winehq.org

3 participants
919 discussions

Roderick Colenbrander : wined3d: Use RECT instead of WINED3DRECT in more parts of BltOverride.
by Alexandre Julliard 30 Mar '10

30 Mar '10

Module: wine Branch: master Commit: a91f844a3d1c8e4f37b9d31f0025f3d20d1bbcb2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a91f844a3d1c8e4f37b9d31f0… Author: Roderick Colenbrander <thunderbird2k(a)gmail.com> Date: Thu Mar 25 23:23:12 2010 +0100 wined3d: Use RECT instead of WINED3DRECT in more parts of BltOverride. --- dlls/wined3d/surface.c | 14 ++++++++------ 1 files changed, 8 insertions(+), 6 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index b67672e..9d1e804 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -3769,6 +3769,7 @@ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *This, const WINED3DRECT rect; IWineD3DSwapChainImpl *srcSwapchain = NULL, *dstSwapchain = NULL; IWineD3DSurfaceImpl *Src = (IWineD3DSurfaceImpl *) SrcSurface; + RECT dst_rect; TRACE("(%p)->(%p,%p,%p,%08x,%p)\n", This, DestRect, SrcSurface, SrcRect, Flags, DDBltFx); @@ -3813,6 +3814,7 @@ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *This, const rect.x2 = This->currentDesc.Width; rect.y2 = This->currentDesc.Height; } + surface_get_rect(This, DestRect, &dst_rect); /* The only case where both surfaces on a swapchain are supported is a back buffer -> front buffer blit on the same swapchain */ if(dstSwapchain && dstSwapchain == srcSwapchain && dstSwapchain->backBuffer && @@ -4108,9 +4110,9 @@ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *This, const ClientToScreen(context->win_handle, &offset); GetClientRect(context->win_handle, &windowsize); h = windowsize.bottom - windowsize.top; - rect.x1 -= offset.x; rect.x2 -=offset.x; - rect.y1 -= offset.y; rect.y2 -=offset.y; - rect.y1 += This->currentDesc.Height - h; rect.y2 += This->currentDesc.Height - h; + dst_rect.left -= offset.x; dst_rect.right -=offset.x; + dst_rect.top -= offset.y; dst_rect.bottom -=offset.y; + dst_rect.top += This->currentDesc.Height - h; dst_rect.bottom += This->currentDesc.Height - h; } if (!is_identity_fixup(This->resource.format_desc->color_fixup)) @@ -4152,7 +4154,7 @@ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *This, const /* Draw a textured quad */ - draw_textured_quad(Src, &SourceRectangle, (RECT*)&rect, Filter); + draw_textured_quad(Src, &SourceRectangle, &dst_rect, Filter); if(Flags & (WINEDDBLT_KEYSRC | WINEDDBLT_KEYSRCOVERRIDE)) { glDisable(GL_ALPHA_TEST); @@ -4205,10 +4207,10 @@ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *This, const !(This == (IWineD3DSurfaceImpl*) dstSwapchain->frontBuffer || (dstSwapchain->backBuffer && This == (IWineD3DSurfaceImpl*) dstSwapchain->backBuffer[0]))) { TRACE("Surface is higher back buffer, falling back to software\n"); - return cpu_blit.color_fill(myDevice, This, (RECT*)&rect, color); + return cpu_blit.color_fill(myDevice, This, &dst_rect, color); } - return ffp_blit.color_fill(myDevice, This, (RECT*)&rect, color); + return ffp_blit.color_fill(myDevice, This, &dst_rect, color); } }

1 0

Roderick Colenbrander : wined3d: Add a helper function for getting a valid rectangle.
by Alexandre Julliard 30 Mar '10

30 Mar '10

Module: wine Branch: master Commit: 682c969e9afb33edfb4e215737cef213a036ea79 URL: http://source.winehq.org/git/wine.git/?a=commit;h=682c969e9afb33edfb4e21573… Author: Roderick Colenbrander <thunderbird2k(a)gmail.com> Date: Thu Mar 25 23:00:42 2010 +0100 wined3d: Add a helper function for getting a valid rectangle. --- dlls/wined3d/surface.c | 46 ++++++++++++++++------------------------------ 1 files changed, 16 insertions(+), 30 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 7bb983f..b67672e 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -273,6 +273,19 @@ static void surface_get_blt_info(GLenum target, const RECT *rect_in, GLsizei w, } } +static inline void surface_get_rect(IWineD3DSurfaceImpl *This, const RECT *rect_in, RECT *rect_out) +{ + if (rect_in) + *rect_out = *rect_in; + else + { + rect_out->left = 0; + rect_out->top = 0; + rect_out->right = This->currentDesc.Width; + rect_out->bottom = This->currentDesc.Height; + } +} + /* GL locking and context activation is done by the caller */ static void draw_textured_quad(IWineD3DSurfaceImpl *src_surface, const RECT *src_rect, const RECT *dst_rect, WINED3DTEXTUREFILTERTYPE Filter) { @@ -4027,17 +4040,7 @@ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *This, const TRACE("Blt from surface %p to rendertarget %p\n", Src, This); - if(SrcRect) { - SourceRectangle.left = SrcRect->left; - SourceRectangle.right = SrcRect->right; - SourceRectangle.top = SrcRect->top; - SourceRectangle.bottom = SrcRect->bottom; - } else { - SourceRectangle.left = 0; - SourceRectangle.right = Src->currentDesc.Width; - SourceRectangle.top = 0; - SourceRectangle.bottom = Src->currentDesc.Height; - } + surface_get_rect(Src, SrcRect, &SourceRectangle); /* When blitting from an offscreen surface to a rendertarget, the source * surface is not required to have a palette. Our rendering / conversion @@ -4324,17 +4327,7 @@ static HRESULT WINAPI IWineD3DSurfaceImpl_BltFast(IWineD3DSurface *iface, DWORD RECT SrcRect, DstRect; DWORD Flags=0; - if(rsrc) { - SrcRect.left = rsrc->left; - SrcRect.top= rsrc->top; - SrcRect.bottom = rsrc->bottom; - SrcRect.right = rsrc->right; - } else { - SrcRect.left = 0; - SrcRect.top = 0; - SrcRect.right = srcImpl->currentDesc.Width; - SrcRect.bottom = srcImpl->currentDesc.Height; - } + surface_get_rect(srcImpl, rsrc, &SrcRect); DstRect.left = dstx; DstRect.top=dsty; @@ -4755,14 +4748,7 @@ static inline void surface_blt_to_drawable(IWineD3DSurfaceImpl *This, const RECT struct wined3d_context *context; RECT src_rect, dst_rect; - if(rect_in) { - src_rect = *rect_in; - } else { - src_rect.left = 0; - src_rect.top = 0; - src_rect.right = This->currentDesc.Width; - src_rect.bottom = This->currentDesc.Height; - } + surface_get_rect(This, rect_in, &src_rect); context = context_acquire(device, (IWineD3DSurface*)This, CTXUSAGE_BLIT); if (context->render_offscreen)

1 0

Roderick Colenbrander : wined3d: Add color_fill to blit_shader.
by Alexandre Julliard 30 Mar '10

30 Mar '10

Module: wine Branch: master Commit: a7d3b616c43c66adcce9bc63ddc4f91d5c84bc82 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a7d3b616c43c66adcce9bc63d… Author: Roderick Colenbrander <thunderbird2k(a)gmail.com> Date: Thu Mar 25 17:01:33 2010 +0100 wined3d: Add color_fill to blit_shader. --- dlls/wined3d/arb_program_shader.c | 7 +++ dlls/wined3d/surface.c | 78 +++++++++++++++++++++++++++++------- dlls/wined3d/wined3d_private.h | 2 + 3 files changed, 72 insertions(+), 15 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index da8072c..8e613c9 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -6848,12 +6848,19 @@ static BOOL arbfp_blit_color_fixup_supported(const struct wined3d_gl_info *gl_in } } +static HRESULT arbfp_blit_color_fill(IWineD3DDeviceImpl *device, IWineD3DSurfaceImpl *dst_surface, const RECT *dst_rect, DWORD fill_color) +{ + FIXME("Color filling not implemented by arbfp_blit\n"); + return WINED3DERR_INVALIDCALL; +} + const struct blit_shader arbfp_blit = { arbfp_blit_alloc, arbfp_blit_free, arbfp_blit_set, arbfp_blit_unset, arbfp_blit_color_fixup_supported, + arbfp_blit_color_fill }; #undef GLINFO_LOCATION diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 2cfcb36..7bb983f 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -4183,20 +4183,10 @@ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *This, const } else { /* Source-Less Blit to render target */ if (Flags & WINEDDBLT_COLORFILL) { - /* This is easy to handle for the D3D Device... */ DWORD color; TRACE("Colorfill\n"); - /* This == (IWineD3DSurfaceImpl *) myDevice->render_targets[0] || dstSwapchain - must be true if we are here */ - if (This != (IWineD3DSurfaceImpl *) myDevice->render_targets[0] && - !(This == (IWineD3DSurfaceImpl*) dstSwapchain->frontBuffer || - (dstSwapchain->backBuffer && This == (IWineD3DSurfaceImpl*) dstSwapchain->backBuffer[0]))) { - TRACE("Surface is higher back buffer, falling back to software\n"); - return WINED3DERR_INVALIDCALL; - } - /* The color as given in the Blt function is in the format of the frame-buffer... * 'clear' expect it in ARGB format => we need to do some conversion :-) */ @@ -4206,10 +4196,16 @@ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *This, const return WINED3DERR_INVALIDCALL; } - TRACE("(%p) executing Render Target override, color = %x\n", This, color); - IWineD3DDeviceImpl_ClearSurface(myDevice, This, 1 /* Number of rectangles */, - &rect, WINED3DCLEAR_TARGET, color, 0.0f /* Z */, 0 /* Stencil */); - return WINED3D_OK; + /* This == (IWineD3DSurfaceImpl *) myDevice->render_targets[0] || dstSwapchain + must be true if we are here */ + if (This != (IWineD3DSurfaceImpl *) myDevice->render_targets[0] && + !(This == (IWineD3DSurfaceImpl*) dstSwapchain->frontBuffer || + (dstSwapchain->backBuffer && This == (IWineD3DSurfaceImpl*) dstSwapchain->backBuffer[0]))) { + TRACE("Surface is higher back buffer, falling back to software\n"); + return cpu_blit.color_fill(myDevice, This, (RECT*)&rect, color); + } + + return ffp_blit.color_fill(myDevice, This, (RECT*)&rect, color); } } @@ -5245,10 +5241,62 @@ static BOOL ffp_blit_color_fixup_supported(const struct wined3d_gl_info *gl_info return FALSE; } +static HRESULT ffp_blit_color_fill(IWineD3DDeviceImpl *device, IWineD3DSurfaceImpl *dst_surface, const RECT *dst_rect, DWORD fill_color) +{ + return IWineD3DDeviceImpl_ClearSurface(device, dst_surface, 1 /* Number of rectangles */, + (const WINED3DRECT*)dst_rect, WINED3DCLEAR_TARGET, fill_color, 0.0f /* Z */, 0 /* Stencil */); +} + const struct blit_shader ffp_blit = { ffp_blit_alloc, ffp_blit_free, ffp_blit_set, ffp_blit_unset, - ffp_blit_color_fixup_supported + ffp_blit_color_fixup_supported, + ffp_blit_color_fill +}; + +static HRESULT cpu_blit_alloc(IWineD3DDevice *iface) +{ + return WINED3D_OK; +} + +/* Context activation is done by the caller. */ +static void cpu_blit_free(IWineD3DDevice *iface) +{ +} + +/* Context activation is done by the caller. */ +static HRESULT cpu_blit_set(IWineD3DDevice *iface, const struct wined3d_format_desc *format_desc, + GLenum textype, UINT width, UINT height) +{ + return WINED3D_OK; +} + +/* Context activation is done by the caller. */ +static void cpu_blit_unset(IWineD3DDevice *iface) +{ +} + +static BOOL cpu_blit_color_fixup_supported(const struct wined3d_gl_info *gl_info, struct color_fixup_desc fixup) +{ + return FALSE; +} + +static HRESULT cpu_blit_color_fill(IWineD3DDeviceImpl *device, IWineD3DSurfaceImpl *dst_surface, const RECT *dst_rect, DWORD fill_color) +{ + WINEDDBLTFX BltFx; + memset(&BltFx, 0, sizeof(BltFx)); + BltFx.dwSize = sizeof(BltFx); + BltFx.u5.dwFillColor = color_convert_argb_to_fmt(fill_color, dst_surface->resource.format_desc->format); + return IWineD3DBaseSurfaceImpl_Blt((IWineD3DSurface*)dst_surface, dst_rect, NULL, NULL, WINEDDBLT_COLORFILL, &BltFx, WINED3DTEXF_POINT); +} + +const struct blit_shader cpu_blit = { + cpu_blit_alloc, + cpu_blit_free, + cpu_blit_set, + cpu_blit_unset, + cpu_blit_color_fixup_supported, + cpu_blit_color_fill }; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 0f134d7..77c7d83 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1177,10 +1177,12 @@ struct blit_shader GLenum textype, UINT width, UINT height); void (*unset_shader)(IWineD3DDevice *iface); BOOL (*color_fixup_supported)(const struct wined3d_gl_info *gl_info, struct color_fixup_desc fixup); + HRESULT (*color_fill)(IWineD3DDeviceImpl *device, IWineD3DSurfaceImpl *dst_surface, const RECT *dst_rect, DWORD fill_color); }; extern const struct blit_shader ffp_blit DECLSPEC_HIDDEN; extern const struct blit_shader arbfp_blit DECLSPEC_HIDDEN; +extern const struct blit_shader cpu_blit DECLSPEC_HIDDEN; typedef enum ContextUsage { CTXUSAGE_RESOURCELOAD = 1, /* Only loads textures: No State is applied */

1 0

Roderick Colenbrander : wined3d: Introduce a helper function which converts a color from the surface format to ARGB .
by Alexandre Julliard 30 Mar '10

30 Mar '10

Module: wine Branch: master Commit: 9a1fd552d2030108311946edf04d3234a1517916 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9a1fd552d2030108311946edf… Author: Roderick Colenbrander <thunderbird2k(a)gmail.com> Date: Thu Mar 25 16:42:51 2010 +0100 wined3d: Introduce a helper function which converts a color from the surface format to ARGB. --- dlls/wined3d/surface.c | 94 +++++++++++++++++++++++++++++------------------- 1 files changed, 57 insertions(+), 37 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 9850731..2cfcb36 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -959,6 +959,61 @@ static inline BOOL surface_can_stretch_rect(IWineD3DSurfaceImpl *src, IWineD3DSu && is_identity_fixup(dst->resource.format_desc->color_fixup))); } +static BOOL surface_convert_color_to_argb(IWineD3DSurfaceImpl *This, DWORD color, DWORD *argb_color) +{ + IWineD3DDeviceImpl *device = This->resource.device; + + switch(This->resource.format_desc->format) + { + case WINED3DFMT_P8_UINT: + { + DWORD alpha; + + if (primary_render_target_is_p8(device)) + alpha = color << 24; + else + alpha = 0xFF000000; + + if (This->palette) { + *argb_color = (alpha | + (This->palette->palents[color].peRed << 16) | + (This->palette->palents[color].peGreen << 8) | + (This->palette->palents[color].peBlue)); + } else { + *argb_color = alpha; + } + } + break; + + case WINED3DFMT_B5G6R5_UNORM: + { + if (color == 0xFFFF) { + *argb_color = 0xFFFFFFFF; + } else { + *argb_color = ((0xFF000000) | + ((color & 0xF800) << 8) | + ((color & 0x07E0) << 5) | + ((color & 0x001F) << 3)); + } + } + break; + + case WINED3DFMT_B8G8R8_UNORM: + case WINED3DFMT_B8G8R8X8_UNORM: + *argb_color = 0xFF000000 | color; + break; + + case WINED3DFMT_B8G8R8A8_UNORM: + *argb_color = color; + break; + + default: + ERR("Unhandled conversion from %s to ARGB!\n", debug_d3dformat(This->resource.format_desc->format)); + return FALSE; + } + return TRUE; +} + static ULONG WINAPI IWineD3DSurfaceImpl_Release(IWineD3DSurface *iface) { IWineD3DSurfaceImpl *This = (IWineD3DSurfaceImpl *)iface; @@ -4145,44 +4200,9 @@ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *This, const /* The color as given in the Blt function is in the format of the frame-buffer... * 'clear' expect it in ARGB format => we need to do some conversion :-) */ - if (This->resource.format_desc->format == WINED3DFMT_P8_UINT) + if (!surface_convert_color_to_argb(This, DDBltFx->u5.dwFillColor, &color)) { - DWORD alpha; - - if (primary_render_target_is_p8(myDevice)) alpha = DDBltFx->u5.dwFillColor << 24; - else alpha = 0xFF000000; - - if (This->palette) { - color = (alpha | - (This->palette->palents[DDBltFx->u5.dwFillColor].peRed << 16) | - (This->palette->palents[DDBltFx->u5.dwFillColor].peGreen << 8) | - (This->palette->palents[DDBltFx->u5.dwFillColor].peBlue)); - } else { - color = alpha; - } - } - else if (This->resource.format_desc->format == WINED3DFMT_B5G6R5_UNORM) - { - if (DDBltFx->u5.dwFillColor == 0xFFFF) { - color = 0xFFFFFFFF; - } else { - color = ((0xFF000000) | - ((DDBltFx->u5.dwFillColor & 0xF800) << 8) | - ((DDBltFx->u5.dwFillColor & 0x07E0) << 5) | - ((DDBltFx->u5.dwFillColor & 0x001F) << 3)); - } - } - else if (This->resource.format_desc->format == WINED3DFMT_B8G8R8_UNORM - || This->resource.format_desc->format == WINED3DFMT_B8G8R8X8_UNORM) - { - color = 0xFF000000 | DDBltFx->u5.dwFillColor; - } - else if (This->resource.format_desc->format == WINED3DFMT_B8G8R8A8_UNORM) - { - color = DDBltFx->u5.dwFillColor; - } - else { - ERR("Wrong surface type for BLT override(Format doesn't match) !\n"); + /* The color conversion function already prints an error, so need to do it here */ return WINED3DERR_INVALIDCALL; }

1 0

Roderick Colenbrander : wined3d: Move argb_to_fmt to utils. c as a helper function to be used in more places.
by Alexandre Julliard 30 Mar '10

30 Mar '10

Module: wine Branch: master Commit: f724843f14f1c2fa094103760be382c4350c7d7d URL: http://source.winehq.org/git/wine.git/?a=commit;h=f724843f14f1c2fa094103760… Author: Roderick Colenbrander <thunderbird2k(a)gmail.com> Date: Thu Mar 25 16:30:13 2010 +0100 wined3d: Move argb_to_fmt to utils.c as a helper function to be used in more places. --- dlls/wined3d/device.c | 112 +--------------------------------------- dlls/wined3d/utils.c | 109 ++++++++++++++++++++++++++++++++++++++ dlls/wined3d/wined3d_private.h | 3 + 3 files changed, 114 insertions(+), 110 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 9b63dd0..cba1447 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -5525,114 +5525,6 @@ static void color_fill_fbo(IWineD3DDevice *iface, IWineD3DSurface *surface, context_release(context); } -static inline DWORD argb_to_fmt(DWORD color, WINED3DFORMAT destfmt) { - unsigned int r, g, b, a; - DWORD ret; - - if (destfmt == WINED3DFMT_B8G8R8A8_UNORM - || destfmt == WINED3DFMT_B8G8R8X8_UNORM - || destfmt == WINED3DFMT_B8G8R8_UNORM) - return color; - - TRACE("Converting color %08x to format %s\n", color, debug_d3dformat(destfmt)); - - a = (color & 0xff000000) >> 24; - r = (color & 0x00ff0000) >> 16; - g = (color & 0x0000ff00) >> 8; - b = (color & 0x000000ff) >> 0; - - switch(destfmt) - { - case WINED3DFMT_B5G6R5_UNORM: - if(r == 0xff && g == 0xff && b == 0xff) return 0xffff; - r = (r * 32) / 256; - g = (g * 64) / 256; - b = (b * 32) / 256; - ret = r << 11; - ret |= g << 5; - ret |= b; - TRACE("Returning %08x\n", ret); - return ret; - - case WINED3DFMT_B5G5R5X1_UNORM: - case WINED3DFMT_B5G5R5A1_UNORM: - a = (a * 2) / 256; - r = (r * 32) / 256; - g = (g * 32) / 256; - b = (b * 32) / 256; - ret = a << 15; - ret |= r << 10; - ret |= g << 5; - ret |= b << 0; - TRACE("Returning %08x\n", ret); - return ret; - - case WINED3DFMT_A8_UNORM: - TRACE("Returning %08x\n", a); - return a; - - case WINED3DFMT_B4G4R4X4_UNORM: - case WINED3DFMT_B4G4R4A4_UNORM: - a = (a * 16) / 256; - r = (r * 16) / 256; - g = (g * 16) / 256; - b = (b * 16) / 256; - ret = a << 12; - ret |= r << 8; - ret |= g << 4; - ret |= b << 0; - TRACE("Returning %08x\n", ret); - return ret; - - case WINED3DFMT_B2G3R3_UNORM: - r = (r * 8) / 256; - g = (g * 8) / 256; - b = (b * 4) / 256; - ret = r << 5; - ret |= g << 2; - ret |= b << 0; - TRACE("Returning %08x\n", ret); - return ret; - - case WINED3DFMT_R8G8B8X8_UNORM: - case WINED3DFMT_R8G8B8A8_UNORM: - ret = a << 24; - ret |= b << 16; - ret |= g << 8; - ret |= r << 0; - TRACE("Returning %08x\n", ret); - return ret; - - case WINED3DFMT_B10G10R10A2_UNORM: - a = (a * 4) / 256; - r = (r * 1024) / 256; - g = (g * 1024) / 256; - b = (b * 1024) / 256; - ret = a << 30; - ret |= r << 20; - ret |= g << 10; - ret |= b << 0; - TRACE("Returning %08x\n", ret); - return ret; - - case WINED3DFMT_R10G10B10A2_UNORM: - a = (a * 4) / 256; - r = (r * 1024) / 256; - g = (g * 1024) / 256; - b = (b * 1024) / 256; - ret = a << 30; - ret |= b << 20; - ret |= g << 10; - ret |= r << 0; - TRACE("Returning %08x\n", ret); - return ret; - - default: - FIXME("Add a COLORFILL conversion for format %s\n", debug_d3dformat(destfmt)); - return 0; - } -} - static HRESULT WINAPI IWineD3DDeviceImpl_ColorFill(IWineD3DDevice *iface, IWineD3DSurface *pSurface, const WINED3DRECT *pRect, WINED3DCOLOR color) { @@ -5654,7 +5546,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_ColorFill(IWineD3DDevice *iface, /* Just forward this to the DirectDraw blitting engine */ memset(&BltFx, 0, sizeof(BltFx)); BltFx.dwSize = sizeof(BltFx); - BltFx.u5.dwFillColor = argb_to_fmt(color, surface->resource.format_desc->format); + BltFx.u5.dwFillColor = color_convert_argb_to_fmt(color, surface->resource.format_desc->format); return IWineD3DSurface_Blt(pSurface, (const RECT *)pRect, NULL, NULL, WINEDDBLT_COLORFILL, &BltFx, WINED3DTEXF_POINT); } @@ -5702,7 +5594,7 @@ static void WINAPI IWineD3DDeviceImpl_ClearRendertargetView(IWineD3DDevice *ifac /* Just forward this to the DirectDraw blitting engine */ memset(&BltFx, 0, sizeof(BltFx)); BltFx.dwSize = sizeof(BltFx); - BltFx.u5.dwFillColor = argb_to_fmt(c, ((IWineD3DSurfaceImpl *)surface)->resource.format_desc->format); + BltFx.u5.dwFillColor = color_convert_argb_to_fmt(c, ((IWineD3DSurfaceImpl *)surface)->resource.format_desc->format); hr = IWineD3DSurface_Blt(surface, NULL, NULL, NULL, WINEDDBLT_COLORFILL, &BltFx, WINED3DTEXF_POINT); if (FAILED(hr)) { diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index e2e888d..46252eb 100644 --- a/dlls/wined3d/utils.c +++ b/dlls/wined3d/utils.c @@ -2202,6 +2202,115 @@ BOOL getDepthStencilBits(const struct wined3d_format_desc *format_desc, short *d return TRUE; } +DWORD color_convert_argb_to_fmt(DWORD color, WINED3DFORMAT destfmt) +{ + unsigned int r, g, b, a; + DWORD ret; + + if (destfmt == WINED3DFMT_B8G8R8A8_UNORM + || destfmt == WINED3DFMT_B8G8R8X8_UNORM + || destfmt == WINED3DFMT_B8G8R8_UNORM) + return color; + + TRACE("Converting color %08x to format %s\n", color, debug_d3dformat(destfmt)); + + a = (color & 0xff000000) >> 24; + r = (color & 0x00ff0000) >> 16; + g = (color & 0x0000ff00) >> 8; + b = (color & 0x000000ff) >> 0; + + switch(destfmt) + { + case WINED3DFMT_B5G6R5_UNORM: + if(r == 0xff && g == 0xff && b == 0xff) return 0xffff; + r = (r * 32) / 256; + g = (g * 64) / 256; + b = (b * 32) / 256; + ret = r << 11; + ret |= g << 5; + ret |= b; + TRACE("Returning %08x\n", ret); + return ret; + + case WINED3DFMT_B5G5R5X1_UNORM: + case WINED3DFMT_B5G5R5A1_UNORM: + a = (a * 2) / 256; + r = (r * 32) / 256; + g = (g * 32) / 256; + b = (b * 32) / 256; + ret = a << 15; + ret |= r << 10; + ret |= g << 5; + ret |= b << 0; + TRACE("Returning %08x\n", ret); + return ret; + + case WINED3DFMT_A8_UNORM: + TRACE("Returning %08x\n", a); + return a; + + case WINED3DFMT_B4G4R4X4_UNORM: + case WINED3DFMT_B4G4R4A4_UNORM: + a = (a * 16) / 256; + r = (r * 16) / 256; + g = (g * 16) / 256; + b = (b * 16) / 256; + ret = a << 12; + ret |= r << 8; + ret |= g << 4; + ret |= b << 0; + TRACE("Returning %08x\n", ret); + return ret; + + case WINED3DFMT_B2G3R3_UNORM: + r = (r * 8) / 256; + g = (g * 8) / 256; + b = (b * 4) / 256; + ret = r << 5; + ret |= g << 2; + ret |= b << 0; + TRACE("Returning %08x\n", ret); + return ret; + + case WINED3DFMT_R8G8B8X8_UNORM: + case WINED3DFMT_R8G8B8A8_UNORM: + ret = a << 24; + ret |= b << 16; + ret |= g << 8; + ret |= r << 0; + TRACE("Returning %08x\n", ret); + return ret; + + case WINED3DFMT_B10G10R10A2_UNORM: + a = (a * 4) / 256; + r = (r * 1024) / 256; + g = (g * 1024) / 256; + b = (b * 1024) / 256; + ret = a << 30; + ret |= r << 20; + ret |= g << 10; + ret |= b << 0; + TRACE("Returning %08x\n", ret); + return ret; + + case WINED3DFMT_R10G10B10A2_UNORM: + a = (a * 4) / 256; + r = (r * 1024) / 256; + g = (g * 1024) / 256; + b = (b * 1024) / 256; + ret = a << 30; + ret |= b << 20; + ret |= g << 10; + ret |= r << 0; + TRACE("Returning %08x\n", ret); + return ret; + + default: + FIXME("Add a COLORFILL conversion for format %s\n", debug_d3dformat(destfmt)); + return 0; + } +} + /* DirectDraw stuff */ WINED3DFORMAT pixelformat_for_depth(DWORD depth) { switch(depth) { diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 4237f04..0f134d7 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2626,6 +2626,9 @@ const char *debug_d3dtop(WINED3DTEXTUREOP d3dtop) DECLSPEC_HIDDEN; void dump_color_fixup_desc(struct color_fixup_desc fixup) DECLSPEC_HIDDEN; const char *debug_surflocation(DWORD flag) DECLSPEC_HIDDEN; +/* Color conversion routines */ +DWORD color_convert_argb_to_fmt(DWORD color, WINED3DFORMAT destfmt) DECLSPEC_HIDDEN; + /* Routines for GL <-> D3D values */ GLenum StencilOp(DWORD op) DECLSPEC_HIDDEN; GLenum CompareFunc(DWORD func) DECLSPEC_HIDDEN;

1 0

Alexandre Julliard : kernel32: Fix processing of spaces in command line for builtin exes.
by Alexandre Julliard 30 Mar '10

30 Mar '10

Module: wine Branch: master Commit: be0db98a232f208e8b011d86ad15f4b9b8eacbc7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=be0db98a232f208e8b011d86a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 29 19:37:35 2010 +0200 kernel32: Fix processing of spaces in command line for builtin exes. --- dlls/kernel32/process.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 0a404bc..541c794 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -1967,8 +1967,8 @@ static LPWSTR get_file_name( LPCWSTR appname, LPWSTR cmdline, LPWSTR buffer, ret = cmdline; break; } - if (!(*pos++ = *p++)) break; if (!first_space) first_space = pos; + if (!(*pos++ = *p++)) break; } if (!ret)

1 0

Alexandre Julliard : kernel32: Add support for starting binaries in 32/ 64-bit mode from the other mode.
by Alexandre Julliard 30 Mar '10

30 Mar '10

Module: wine Branch: master Commit: 808c29b908db64723ff37147b0a5dea5c95988ac URL: http://source.winehq.org/git/wine.git/?a=commit;h=808c29b908db64723ff37147b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 29 17:57:02 2010 +0200 kernel32: Add support for starting binaries in 32/64-bit mode from the other mode. --- dlls/kernel32/process.c | 62 ++++++++++++++++++++++++++++++++++++++++++---- 1 files changed, 56 insertions(+), 6 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 30e7ca4..0a404bc 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -75,6 +75,7 @@ static UINT process_error_mode; static DWORD shutdown_flags = 0; static DWORD shutdown_priority = 0x280; static BOOL is_wow64; +static const int is_win64 = (sizeof(void *) > sizeof(int)); HMODULE kernel32_handle = 0; @@ -868,9 +869,7 @@ static void init_windows_dirs(void) ERR( "directory %s could not be created, error %u\n", debugstr_w(DIR_System), GetLastError() ); -#ifndef _WIN64 /* SysWow64 is always defined on 64-bit */ - if (is_wow64) -#endif + if (is_win64 || is_wow64) /* SysWow64 is always defined on 64-bit */ { len = strlenW( DIR_Windows ); buffer = HeapAlloc( GetProcessHeap(), 0, len * sizeof(WCHAR) + sizeof(default_syswow64W) ); @@ -1581,6 +1580,49 @@ done: return info; } +/*********************************************************************** + * get_alternate_loader + * + * Get the name of the alternate (32 or 64 bit) Wine loader. + */ +static const char *get_alternate_loader( char **ret_env ) +{ + char *env; + const char *loader = NULL; + const char *loader_env = getenv( "WINELOADER" ); + + *ret_env = NULL; + + if (wine_get_build_dir()) loader = is_win64 ? "loader/wine32" : "server/../loader/wine"; + + if (loader_env) + { + int len = strlen( loader_env ); + if (is_win64) + { + if (!(env = HeapAlloc( GetProcessHeap(), 0, sizeof("WINELOADER=") + len + 2 ))) return NULL; + strcpy( env, "WINELOADER=" ); + strcat( env, loader_env ); + strcat( env, "32" ); + } + else + { + if (!(env = HeapAlloc( GetProcessHeap(), 0, sizeof("WINELOADER=") + len ))) return NULL; + strcpy( env, "WINELOADER=" ); + strcat( env, loader_env ); + len += sizeof("WINELOADER=") - 1; + if (!strcmp( env + len - 2, "32" )) env[len - 2] = 0; + } + if (!loader) + { + if ((loader = strrchr( env, '/' ))) loader++; + else loader = env; + } + } + if (!loader) loader = is_win64 ? "wine32" : "wine"; + *ret_env = env; + return loader; +} /*********************************************************************** * create_process @@ -1598,6 +1640,8 @@ static BOOL create_process( HANDLE hFile, LPCWSTR filename, LPWSTR cmd_line, LPW HANDLE process_info; WCHAR *env_end; char *winedebug = NULL; + char *wineloader = NULL; + const char *loader = NULL; char **argv; startup_info_t *startup_info; DWORD startup_info_size; @@ -1605,7 +1649,7 @@ static BOOL create_process( HANDLE hFile, LPCWSTR filename, LPWSTR cmd_line, LPW pid_t pid; int err; - if (sizeof(void *) == sizeof(int) && !is_wow64 && (binary_info->flags & BINARY_FLAG_64BIT)) + if (!is_win64 && !is_wow64 && (binary_info->flags & BINARY_FLAG_64BIT)) { ERR( "starting 64-bit process %s not supported on this platform\n", debugstr_w(filename) ); SetLastError( ERROR_BAD_EXE_FORMAT ); @@ -1698,6 +1742,9 @@ static BOOL create_process( HANDLE hFile, LPCWSTR filename, LPWSTR cmd_line, LPW /* create the child process */ argv = build_argv( cmd_line, 1 ); + if (!is_win64 ^ !(binary_info->flags & BINARY_FLAG_64BIT)) + loader = get_alternate_loader( &wineloader ); + if (exec_only || !(pid = fork())) /* child */ { char preloader_reserve[64], socket_env[64]; @@ -1738,9 +1785,10 @@ static BOOL create_process( HANDLE hFile, LPCWSTR filename, LPWSTR cmd_line, LPW putenv( preloader_reserve ); putenv( socket_env ); if (winedebug) putenv( winedebug ); + if (wineloader) putenv( wineloader ); if (unixdir) chdir(unixdir); - if (argv) wine_exec_wine_binary( NULL, argv, getenv("WINELOADER") ); + if (argv) wine_exec_wine_binary( loader, argv, getenv("WINELOADER") ); _exit(1); } @@ -1751,6 +1799,7 @@ static BOOL create_process( HANDLE hFile, LPCWSTR filename, LPWSTR cmd_line, LPW close( socketfd[0] ); HeapFree( GetProcessHeap(), 0, argv ); HeapFree( GetProcessHeap(), 0, winedebug ); + HeapFree( GetProcessHeap(), 0, wineloader ); if (pid == -1) { FILE_SetDosError(); @@ -2072,7 +2121,8 @@ BOOL WINAPI DECLSPEC_HOTPATCH CreateProcessW( LPCWSTR app_name, LPWSTR cmd_line, inherit, flags, startup_info, info, unixdir, &binary_info, FALSE ); break; case BINARY_UNIX_LIB: - TRACE( "starting %s as Winelib app\n", debugstr_w(name) ); + TRACE( "starting %s as %d-bit Winelib app\n", + debugstr_w(name), (binary_info.flags & BINARY_FLAG_64BIT) ? 64 : 32 ); retv = create_process( hFile, name, tidy_cmdline, envW, cur_dir, process_attr, thread_attr, inherit, flags, startup_info, info, unixdir, &binary_info, FALSE ); break;

1 0

Alexandre Julliard : configure: Create a few symlinks needed for running in Wow64 mode inside the build tree .
by Alexandre Julliard 30 Mar '10

30 Mar '10

Module: wine Branch: master Commit: e5c585e4a4d2162f4c45a0e6f909bf8524e2610c URL: http://source.winehq.org/git/wine.git/?a=commit;h=e5c585e4a4d2162f4c45a0e6f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 29 17:54:55 2010 +0200 configure: Create a few symlinks needed for running in Wow64 mode inside the build tree. --- configure | 12 ++++++++++++ configure.ac | 13 +++++++++++++ 2 files changed, 25 insertions(+), 0 deletions(-) diff --git a/configure b/configure index b02fe65..f133705 100755 --- a/configure +++ b/configure @@ -14713,6 +14713,18 @@ ac_config_commands="$ac_config_commands Makefile" ac_config_files="$ac_config_files Make.tmp:Makefile.in" + +if test -n "$with_wine64" +then +wine_fn_append_rule ALL_MAKEFILE_DEPENDS "all: fonts server $with_wine64/loader/wine32 +fonts server: + \$(RM) \$@ && \$(LN_S) $with_wine64/\$@ \$@ +$with_wine64/loader/wine32: + \$(RM) \$@ && \$(LN_S) $ac_pwd/loader/wine32 \$@ +clean:: + \$(RM) fonts server $with_wine64/loader/wine32" +fi + wine_fn_append_rule ALL_MAKEFILE_DEPENDS "uninstall:: -rmdir \$(DESTDIR)\$(datadir)/wine \$(DESTDIR)\$(fakedlldir) \$(DESTDIR)\$(dlldir)" diff --git a/configure.ac b/configure.ac index 25a1313..349a25a 100644 --- a/configure.ac +++ b/configure.ac @@ -2713,6 +2713,19 @@ _WINE_EOF AC_CONFIG_FILES([Make.tmp:Makefile.in]) dnl Some final makefile rules + +if test -n "$with_wine64" +then +WINE_APPEND_RULE([ALL_MAKEFILE_DEPENDS], +[all: fonts server $with_wine64/loader/wine32 +fonts server: + \$(RM) \$[@] && \$(LN_S) $with_wine64/\$[@] \$[@] +$with_wine64/loader/wine32: + \$(RM) \$[@] && \$(LN_S) $ac_pwd/loader/wine32 \$[@] +clean:: + \$(RM) fonts server $with_wine64/loader/wine32]) +fi + WINE_APPEND_RULE([ALL_MAKEFILE_DEPENDS], [uninstall:: -rmdir \$(DESTDIR)\$(datadir)/wine \$(DESTDIR)\$(fakedlldir) \$(DESTDIR)\$(dlldir)])

1 0

Alexandre Julliard : loader: Rename the 32-bit loader in the build tree too when doing a Wow64 build.
by Alexandre Julliard 30 Mar '10

30 Mar '10

Module: wine Branch: master Commit: 50c6ac83249a4de062f9e362a926dee2ce108046 URL: http://source.winehq.org/git/wine.git/?a=commit;h=50c6ac83249a4de062f9e362a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 29 17:53:30 2010 +0200 loader: Rename the 32-bit loader in the build tree too when doing a Wow64 build. --- .gitignore | 1 + loader/Makefile.in | 5 +++-- tools/winewrapper | 25 +++++++++---------------- 3 files changed, 13 insertions(+), 18 deletions(-) diff --git a/.gitignore b/.gitignore index 6ea657d..264e2de 100644 --- a/.gitignore +++ b/.gitignore @@ -229,6 +229,7 @@ loader/wine-preloader loader/wine.de.man loader/wine.fr.man loader/wine.man +loader/wine32 programs/Makeprog.rules programs/rpcss/epm.h programs/rpcss/epm_s.c diff --git a/loader/Makefile.in b/loader/Makefile.in index c314c9b..6e08066 100644 --- a/loader/Makefile.in +++ b/loader/Makefile.in @@ -13,6 +13,7 @@ EXTRA_BINARIES = @EXTRA_BINARIES@ PROGRAMS = \ wine \ + wine32 \ wine-installed \ wine-preloader @@ -27,7 +28,7 @@ INSTALLDIRS = \ $(DESTDIR)$(mandir)/de.UTF-8/man$(prog_manext) \ $(DESTDIR)$(mandir)/fr.UTF-8/man$(prog_manext) -all: wine wine-installed $(EXTRA_BINARIES) $(MANPAGES) +all: $(MAIN_BINARY) wine-installed $(EXTRA_BINARIES) $(MANPAGES) @MAKE_RULES@ @@ -37,7 +38,7 @@ LDEXECFLAGS = @LDEXECFLAGS@ wine-preloader: preloader.o Makefile.in $(CC) -o $@ -static -nostartfiles -nodefaultlibs -Wl,-Ttext=0x7c000000 preloader.o $(LIBPORT) $(LDFLAGS) -wine: main.o Makefile.in +$(MAIN_BINARY): main.o Makefile.in $(CC) -o $@ $(LDEXECFLAGS) main.o $(LIBWINE) $(LIBPORT) $(LIBPTHREAD) $(EXTRALIBS) $(LDFLAGS) $(LDRPATH_LOCAL) wine-installed: main.o Makefile.in diff --git a/tools/winewrapper b/tools/winewrapper index 13be936..c664142 100755 --- a/tools/winewrapper +++ b/tools/winewrapper @@ -81,7 +81,14 @@ else export LD_LIBRARY_PATH fi -WINELOADER="$topdir/loader/wine" +if [ -x "$topdir/loader/wine" ] +then WINELOADER="$topdir/loader/wine" +elif [ -x "$topdir/loader/wine32" ] +then WINELOADER="$topdir/loader/wine32" +else + echo "$0: could not find the wine loader in $topdir" + exit 1 +fi export WINELOADER # any local settings ? @@ -92,18 +99,4 @@ fi # and run the application -case "$0" in - wine|*/wine) - exec "$WINELOADER" "$@" - ;; - */*) - [ -f "$0.exe.so" ] && exec "$WINELOADER" "$0.exe.so" "$@" - echo "$0: cannot find corresponding application" - exit 1 - ;; - *) - [ -f "$appdir/$0.exe.so" ] && exec "$WINELOADER" "$appdir/$0.exe.so" "$@" - echo "$0: cannot find corresponding application" - exit 1 - ;; -esac +exec "$WINELOADER" "$@"

1 0

Henri Verbeet : wined3d: GL errors aren't FIXMEs.
by Alexandre Julliard 29 Mar '10

29 Mar '10

Module: wine Branch: master Commit: f69d0fa00de9d662eeaef1ab237c3a540920c574 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f69d0fa00de9d662eeaef1ab2… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Mar 29 13:31:11 2010 +0200 wined3d: GL errors aren't FIXMEs. --- dlls/wined3d/wined3d_private.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index a580a84..4237f04 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -789,7 +789,7 @@ do { \ TRACE("%s call ok %s / %d\n", A, __FILE__, __LINE__); \ \ } else do { \ - FIXME(">>>>>>>>>>>>>>>>> %s (%#x) from %s @ %s / %d\n", \ + ERR(">>>>>>>>>>>>>>>>> %s (%#x) from %s @ %s / %d\n", \ debug_glerror(err), err, A, __FILE__, __LINE__); \ err = glGetError(); \ } while (err != GL_NO_ERROR); \

1 0

← Newer
1
...
7
8
9
10
11
12
13
...
92
Older →

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 March 2010