May 2007 - Wine-Commits - list.winehq.org

Stefan Dösinger : wined3d: Support bump environment mapping using GL_NV_texture_shader.
by Alexandre Julliard May 10, 2007

May 10, 2007

Module: wine Branch: master Commit: 81504d6a2d2d5aa06e7f26e46b4afeb13a082eef URL: http://source.winehq.org/git/wine.git/?a=commit;h=81504d6a2d2d5aa06e7f26e46… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Mon May 7 21:08:14 2007 +0200 wined3d: Support bump environment mapping using GL_NV_texture_shader. --- dlls/wined3d/context.c | 24 +++++++ dlls/wined3d/directx.c | 31 ++++++++- dlls/wined3d/state.c | 132 +++++++++++++++++++++++++++++---------- dlls/wined3d/utils.c | 28 +++++++-- dlls/wined3d/wined3d_private.h | 1 + 5 files changed, 173 insertions(+), 43 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=81504d6a2d2d5aa06e7f2…

1 0

Stefan Dösinger : ddraw: Fix stupid bug when setting surface client memory.
by Alexandre Julliard May 10, 2007

May 10, 2007

Module: wine Branch: master Commit: 1b94a5eefde0871bc4eacd664a54e295c977f9a8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1b94a5eefde0871bc4eacd664… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Mon May 7 19:42:07 2007 +0200 ddraw: Fix stupid bug when setting surface client memory. --- dlls/ddraw/surface.c | 15 +++++++-------- 1 files changed, 7 insertions(+), 8 deletions(-) diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index 220444d..d0c93d7 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -2001,12 +2001,6 @@ IDirectDrawSurfaceImpl_SetSurfaceDesc(IDirectDrawSurface7 *iface, if(!DDSD) return DDERR_INVALIDPARAMS; - if (DDSD->dwFlags & DDSD_LPSURFACE && DDSD->lpSurface) - { - ERR("Setting the surface memory isn't supported yet\n"); - return DDERR_INVALIDPARAMS; - - } if (DDSD->dwFlags & DDSD_PIXELFORMAT) { newFormat = PixelFormat_DD2WineD3D(&DDSD->u4.ddpfPixelFormat); @@ -2047,14 +2041,19 @@ IDirectDrawSurfaceImpl_SetSurfaceDesc(IDirectDrawSurface7 *iface, DDCKEY_SRCBLT, (WINEDDCOLORKEY *) &DDSD->ddckCKSrcBlt); } - if (DDSD->dwFlags & DDSD_LPSURFACE) + if (DDSD->dwFlags & DDSD_LPSURFACE && DDSD->lpSurface) { hr = IWineD3DSurface_SetMem(This->WineD3DSurface, DDSD->lpSurface); if(hr != WINED3D_OK) { /* No need for a trace here, wined3d does that for us */ - return hr; + switch(hr) + { + case WINED3DERR_INVALIDCALL: return DDERR_INVALIDPARAMS; + default: break; /* Go on */ + } } + } This->surface_desc = *DDSD;

1 0

Stefan Dösinger : wined3d: Unload vertex weight data in unloadVertexData.
by Alexandre Julliard May 10, 2007

May 10, 2007

Module: wine Branch: master Commit: 4a8b55d456f4df544875baf0eabe765a7c96918b URL: http://source.winehq.org/git/wine.git/?a=commit;h=4a8b55d456f4df544875baf0e… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun May 6 23:47:20 2007 +0200 wined3d: Unload vertex weight data in unloadVertexData. --- dlls/wined3d/state.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/wined3d/state.c b/dlls/wined3d/state.c index 1b5115c..8d571a6 100644 --- a/dlls/wined3d/state.c +++ b/dlls/wined3d/state.c @@ -2472,6 +2472,9 @@ static inline void unloadVertexData(IWineD3DStateBlockImpl *stateblock) { if (GL_SUPPORT(EXT_SECONDARY_COLOR)) { glDisableClientState(GL_SECONDARY_COLOR_ARRAY_EXT); } + if (GL_SUPPORT(ARB_VERTEX_BLEND)) { + glDisableClientState(GL_WEIGHT_ARRAY_ARB); + } for (texture_idx = 0; texture_idx < GL_LIMITS(textures); ++texture_idx) { GL_EXTCALL(glClientActiveTextureARB(GL_TEXTURE0_ARB + texture_idx)); glDisableClientState(GL_TEXTURE_COORD_ARRAY);

1 0

Misha Koshelev : msi/tests: automation: Add more tests for Installer:: RegistryValue.
by Alexandre Julliard May 10, 2007

May 10, 2007

Module: wine Branch: master Commit: 2b7d8cb9d560848cf8e4ae7eb6181f84962ba495 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2b7d8cb9d560848cf8e4ae7eb… Author: Misha Koshelev <mk144210(a)bcm.edu> Date: Wed May 9 07:56:44 2007 -0500 msi/tests: automation: Add more tests for Installer::RegistryValue. Add test for a new error condition and tests for returning data from values of different registry value types. --- dlls/msi/tests/automation.c | 133 +++++++++++++++++++++++++++++++------------ 1 files changed, 96 insertions(+), 37 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=2b7d8cb9d560848cf8e4a…

1 0

Stefan Dösinger : ddraw: Always set fullscreen mode in wined3d.
by Alexandre Julliard May 10, 2007

May 10, 2007

Module: wine Branch: master Commit: df891b823ac6801d7359d7e398d5fa8832607328 URL: http://source.winehq.org/git/wine.git/?a=commit;h=df891b823ac6801d7359d7e39… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Mon May 7 18:32:19 2007 +0200 ddraw: Always set fullscreen mode in wined3d. DDSCL_NORMAL is not set at initialization. Instead of making this a & DDSCL_NORMAL || == 0 check just set fullscreen mode always. WineD3D will catch NOP changes anyway. --- dlls/ddraw/ddraw.c | 10 +++------- 1 files changed, 3 insertions(+), 7 deletions(-) diff --git a/dlls/ddraw/ddraw.c b/dlls/ddraw/ddraw.c index 16672f3..dd1b078 100644 --- a/dlls/ddraw/ddraw.c +++ b/dlls/ddraw/ddraw.c @@ -465,13 +465,9 @@ IDirectDrawImpl_SetCooperativeLevel(IDirectDraw7 *iface, } */ - /* Switch from normal to full screen mode? */ - if(This->cooperative_level & DDSCL_NORMAL) - { - This->cooperative_level &= ~DDSCL_NORMAL; - IWineD3DDevice_SetFullscreen(This->wineD3DDevice, - TRUE); - } + This->cooperative_level &= ~DDSCL_NORMAL; + IWineD3DDevice_SetFullscreen(This->wineD3DDevice, + TRUE); /* Don't override focus windows or private device windows */ if( hwnd &&

1 0

Stefan Dösinger : ddraw: Some caps fixes.
by Alexandre Julliard May 10, 2007

May 10, 2007

Module: wine Branch: master Commit: d9fef10b71b9add27cfaeffa6d7e3ab42342bba7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d9fef10b71b9add27cfaeffa6… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Wed May 9 11:49:27 2007 +0200 ddraw: Some caps fixes. Cards at DirectX7 times did not support non power of 2 textures, and some games check the capatiblity flags incorrectly. Rollcage expects the D3DPTEXTURECAPS_POW2 (limitation) flag set and fails with the reference rasterizer which support NP2 textures and doesn't have the cap set. --- dlls/ddraw/direct3d.c | 25 +++++++- dlls/ddraw/tests/d3d.c | 155 +++++++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 174 insertions(+), 6 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d9fef10b71b9add27cfae…

1 0

Eric Pouech : kernel32: Set the new definition for an activation context, and implement adding/releasing references to it.
by Alexandre Julliard May 9, 2007

May 9, 2007

Module: wine Branch: master Commit: fe3869388ab2e7dadd091ccc7878ba9bafb5dc3e URL: http://source.winehq.org/git/wine.git/?a=commit;h=fe3869388ab2e7dadd091ccc7… Author: Eric Pouech <eric.pouech(a)wanadoo.fr> Date: Mon May 7 21:49:47 2007 +0200 kernel32: Set the new definition for an activation context, and implement adding/releasing references to it. --- dlls/kernel32/actctx.c | 71 +++++++++++++++++++++++++++++++++++++++++------ 1 files changed, 62 insertions(+), 9 deletions(-) diff --git a/dlls/kernel32/actctx.c b/dlls/kernel32/actctx.c index fbf50d0..06bc8e5 100644 --- a/dlls/kernel32/actctx.c +++ b/dlls/kernel32/actctx.c @@ -46,6 +46,14 @@ WINE_DEFAULT_DEBUG_CHANNEL(actctx); #define ACTCTX_FAKE_HANDLE ((HANDLE) 0xf00baa) #define ACTCTX_FAKE_COOKIE ((ULONG_PTR) 0xf00bad) +#define ACTCTX_MAGIC 0xC07E3E11 + +struct actctx +{ + ULONG magic; + LONG ref_count; +}; + /*********************************************************************** * CreateActCtxA (KERNEL32.@) * @@ -130,15 +138,44 @@ done: */ HANDLE WINAPI CreateActCtxW(PCACTCTXW pActCtx) { - FIXME("%p %08x\n", pActCtx, pActCtx ? pActCtx->dwFlags : 0); + struct actctx* actctx; + DWORD ret = ERROR_SUCCESS; - if (!pActCtx) - return INVALID_HANDLE_VALUE; - if (pActCtx->cbSize != sizeof *pActCtx) - return INVALID_HANDLE_VALUE; - if (pActCtx->dwFlags & ~ACTCTX_FLAGS_ALL) + TRACE("%p %08x\n", pActCtx, pActCtx ? pActCtx->dwFlags : 0); + + if (!pActCtx || pActCtx->cbSize != sizeof(*pActCtx) || + (pActCtx->dwFlags & ~ACTCTX_FLAGS_ALL)) + { + SetLastError(ERROR_INVALID_PARAMETER); + return INVALID_HANDLE_VALUE; + } + actctx = HeapAlloc(GetProcessHeap(), 0, sizeof(*actctx)); + if (!actctx) return INVALID_HANDLE_VALUE; + + actctx->magic = ACTCTX_MAGIC; + actctx->ref_count = 1; + + if (ret == ERROR_SUCCESS) + { + return (HANDLE)actctx; + } + + ReleaseActCtx((HANDLE)actctx); + SetLastError(ret); return INVALID_HANDLE_VALUE; - return ACTCTX_FAKE_HANDLE; +} + +static struct actctx* check_actctx(HANDLE h) +{ + struct actctx* actctx = (struct actctx*)h; + + switch (actctx->magic) + { + case ACTCTX_MAGIC: return actctx; + default: + SetLastError(ERROR_INVALID_HANDLE); + return NULL; + } } /*********************************************************************** @@ -204,7 +241,12 @@ BOOL WINAPI GetCurrentActCtx(HANDLE* phActCtx) */ void WINAPI AddRefActCtx(HANDLE hActCtx) { - FIXME("%p\n", hActCtx); + struct actctx* actctx; + + TRACE("%p\n", hActCtx); + + if ((actctx = check_actctx(hActCtx))) + InterlockedIncrement( &actctx->ref_count ); } /*********************************************************************** @@ -214,7 +256,18 @@ void WINAPI AddRefActCtx(HANDLE hActCtx) */ void WINAPI ReleaseActCtx(HANDLE hActCtx) { - FIXME("%p\n", hActCtx); + struct actctx* actctx; + + TRACE("%p\n", hActCtx); + + if ((actctx = check_actctx(hActCtx))) + { + if (!InterlockedDecrement( &actctx->ref_count )) + { + actctx->magic = 0; + HeapFree(GetProcessHeap(), 0, actctx); + } + } } /***********************************************************************

1 0

Eric Pouech : kernel32: Implemented CreateActCtxA on top of CreateActCtxW.
by Alexandre Julliard May 9, 2007

May 9, 2007

Module: wine Branch: master Commit: 80f41b4937c77ac989c288919708992a6e93bfaf URL: http://source.winehq.org/git/wine.git/?a=commit;h=80f41b4937c77ac989c288919… Author: Eric Pouech <eric.pouech(a)wanadoo.fr> Date: Mon May 7 21:49:40 2007 +0200 kernel32: Implemented CreateActCtxA on top of CreateActCtxW. --- dlls/kernel32/actctx.c | 78 +++++++++++++++++++++++++++++++++++++++++++----- 1 files changed, 70 insertions(+), 8 deletions(-) diff --git a/dlls/kernel32/actctx.c b/dlls/kernel32/actctx.c index bdb6a1a..fbf50d0 100644 --- a/dlls/kernel32/actctx.c +++ b/dlls/kernel32/actctx.c @@ -2,6 +2,8 @@ * Activation contexts * * Copyright 2004 Jon Griffiths + * Copyright 2007 Eric Pouech + * Copyright 2007 Jacek Caban for CodeWeavers * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -25,6 +27,7 @@ #include "windef.h" #include "winbase.h" #include "winerror.h" +#include "winnls.h" #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(actctx); @@ -50,15 +53,74 @@ WINE_DEFAULT_DEBUG_CHANNEL(actctx); */ HANDLE WINAPI CreateActCtxA(PCACTCTXA pActCtx) { - FIXME("%p %08x\n", pActCtx, pActCtx ? pActCtx->dwFlags : 0); + ACTCTXW actw; + SIZE_T len; + HANDLE ret = INVALID_HANDLE_VALUE; + LPWSTR src = NULL, assdir = NULL, resname = NULL, appname = NULL; - if (!pActCtx) - return INVALID_HANDLE_VALUE; - if (pActCtx->cbSize != sizeof *pActCtx) - return INVALID_HANDLE_VALUE; - if (pActCtx->dwFlags & ~ACTCTX_FLAGS_ALL) - return INVALID_HANDLE_VALUE; - return ACTCTX_FAKE_HANDLE; + TRACE("%p %08x\n", pActCtx, pActCtx ? pActCtx->dwFlags : 0); + + if (!pActCtx || pActCtx->cbSize != sizeof(*pActCtx) || + (pActCtx->dwFlags & ~ACTCTX_FLAGS_ALL)) + { + SetLastError(ERROR_INVALID_PARAMETER); + return INVALID_HANDLE_VALUE; + } + + actw.cbSize = sizeof(actw); + actw.dwFlags = pActCtx->dwFlags; + if (pActCtx->lpSource) + { + len = MultiByteToWideChar(CP_ACP, 0, pActCtx->lpSource, -1, NULL, 0); + src = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (!src) return INVALID_HANDLE_VALUE; + MultiByteToWideChar(CP_ACP, 0, pActCtx->lpSource, -1, src, len); + } + actw.lpSource = src; + + if (actw.dwFlags & ACTCTX_FLAG_PROCESSOR_ARCHITECTURE_VALID) + actw.wProcessorArchitecture = pActCtx->wProcessorArchitecture; + if (actw.dwFlags & ACTCTX_FLAG_LANGID_VALID) + actw.wLangId = pActCtx->wLangId; + if (actw.dwFlags & ACTCTX_FLAG_ASSEMBLY_DIRECTORY_VALID) + { + len = MultiByteToWideChar(CP_ACP, 0, pActCtx->lpAssemblyDirectory, -1, NULL, 0); + assdir = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (!assdir) goto done; + MultiByteToWideChar(CP_ACP, 0, pActCtx->lpAssemblyDirectory, -1, assdir, len); + actw.lpAssemblyDirectory = assdir; + } + if (actw.dwFlags & ACTCTX_FLAG_RESOURCE_NAME_VALID) + { + if (!((ULONG_PTR)pActCtx->lpResourceName >> 16)) + { + len = MultiByteToWideChar(CP_ACP, 0, pActCtx->lpResourceName, -1, NULL, 0); + resname = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (!resname) goto done; + MultiByteToWideChar(CP_ACP, 0, pActCtx->lpResourceName, -1, resname, len); + actw.lpResourceName = resname; + } + else actw.lpResourceName = (LPWSTR)pActCtx->lpResourceName; + } + if (actw.dwFlags & ACTCTX_FLAG_APPLICATION_NAME_VALID) + { + len = MultiByteToWideChar(CP_ACP, 0, pActCtx->lpApplicationName, -1, NULL, 0); + appname = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (!appname) goto done; + MultiByteToWideChar(CP_ACP, 0, pActCtx->lpApplicationName, -1, appname, len); + actw.lpApplicationName = appname; + } + if (actw.dwFlags & ACTCTX_FLAG_HMODULE_VALID) + actw.hModule = pActCtx->hModule; + + ret = CreateActCtxW(&actw); + +done: + HeapFree(GetProcessHeap(), 0, src); + HeapFree(GetProcessHeap(), 0, assdir); + HeapFree(GetProcessHeap(), 0, resname); + HeapFree(GetProcessHeap(), 0, appname); + return ret; } /***********************************************************************

1 0

Stefan Dösinger : wined3d: Implement environment bump mapping with GL_ATI_envmap_bumpmap.
by Alexandre Julliard May 9, 2007

May 9, 2007

Module: wine Branch: master Commit: 28170f04747a396a4ca419c1f646212dab923627 URL: http://source.winehq.org/git/wine.git/?a=commit;h=28170f04747a396a4ca419c1f… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Fri May 4 14:28:09 2007 +0200 wined3d: Implement environment bump mapping with GL_ATI_envmap_bumpmap. --- dlls/ddraw/device.c | 37 +++++++++++++++++++++++++++++++++++++ dlls/wined3d/directx.c | 27 ++++++++++++++++++++++----- dlls/wined3d/state.c | 13 +++++++++++++ dlls/wined3d/utils.c | 40 ++++++++++++++++++++++++++++++++++++++++ include/wine/wined3d_gl.h | 8 ++++---- 5 files changed, 116 insertions(+), 9 deletions(-) diff --git a/dlls/ddraw/device.c b/dlls/ddraw/device.c index 63b06e8..de3c9db 100644 --- a/dlls/ddraw/device.c +++ b/dlls/ddraw/device.c @@ -1119,6 +1119,16 @@ IDirect3DDeviceImpl_7_EnumTextureFormats(IDirect3DDevice7 *iface, WINED3DFMT_DXT5, }; + WINED3DFORMAT BumpFormatList[] = { + WINED3DFMT_V8U8, + WINED3DFMT_L6V5U5, + WINED3DFMT_X8L8V8U8, + WINED3DFMT_Q8W8V8U8, + WINED3DFMT_V16U16, + WINED3DFMT_W11V11U10, + WINED3DFMT_A2W10V10U10 + }; + TRACE("(%p)->(%p,%p): Relay\n", This, Callback, Arg); if(!Callback) @@ -1150,6 +1160,33 @@ IDirect3DDeviceImpl_7_EnumTextureFormats(IDirect3DDevice7 *iface, } } } + + for(i = 0; i < sizeof(BumpFormatList) / sizeof(WINED3DFORMAT); i++) + { + hr = IWineD3D_CheckDeviceFormat(This->ddraw->wineD3D, + 0 /* Adapter */, + 0 /* DeviceType */, + 0 /* AdapterFormat */, + WINED3DUSAGE_QUERY_LEGACYBUMPMAP, + 0 /* ResourceType */, + BumpFormatList[i]); + if(hr == D3D_OK) + { + DDPIXELFORMAT pformat; + + memset(&pformat, 0, sizeof(pformat)); + pformat.dwSize = sizeof(pformat); + PixelFormat_WineD3DtoDD(&pformat, BumpFormatList[i]); + + TRACE("Enumerating WineD3DFormat %d\n", BumpFormatList[i]); + hr = Callback(&pformat, Arg); + if(hr != DDENUMRET_OK) + { + TRACE("Format enumeration cancelled by application\n"); + return D3D_OK; + } + } + } TRACE("End of enumeration\n"); return D3D_OK; } diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index 60e3694..ccca582 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -1679,6 +1679,20 @@ static HRESULT WINAPI IWineD3DImpl_CheckDeviceFormat(IWineD3D *iface, UINT Adapt TRACE_(d3d_caps)("[FAILED]\n"); return WINED3DERR_NOTAVAILABLE; } + } else if(Usage & WINED3DUSAGE_QUERY_LEGACYBUMPMAP) { + if(GL_SUPPORT(ATI_ENVMAP_BUMPMAP)) { + switch (CheckFormat) { + case WINED3DFMT_V8U8: + TRACE_(d3d_caps)("[OK]\n"); + return WINED3D_OK; + default: + TRACE_(d3d_caps)("[FAILED]\n"); + return WINED3DERR_NOTAVAILABLE; + } + /* TODO: GL_NV_TEXTURE_SHADER */ + } + TRACE_(d3d_caps)("[FAILED]\n"); + return WINED3DERR_NOTAVAILABLE; } if (GL_SUPPORT(EXT_TEXTURE_COMPRESSION_S3TC)) { @@ -2209,12 +2223,15 @@ static HRESULT WINAPI IWineD3DImpl_GetDeviceCaps(IWineD3D *iface, UINT Adapter, WINED3DTEXOPCAPS_MODULATEINVCOLOR_ADDALPHA; } - + if(GL_SUPPORT(ATI_ENVMAP_BUMPMAP)) { + *pCaps->TextureOpCaps |= WINED3DTEXOPCAPS_BUMPENVMAP; + } else if(GL_SUPPORT(NV_TEXTURE_SHADER)) { + /* TODO: environment bump mapping support with GL_NV_texture_shader */ + } #if 0 - *pCaps->TextureOpCaps |= WINED3DTEXOPCAPS_BUMPENVMAP; - /* FIXME: Add - WINED3DTEXOPCAPS_BUMPENVMAPLUMINANCE - WINED3DTEXOPCAPS_PREMODULATE */ + /* FIXME: Add + *pCaps->TextureOpCaps |= WINED3DTEXOPCAPS_BUMPENVMAPLUMINANCE + WINED3DTEXOPCAPS_PREMODULATE */ #endif *pCaps->MaxTextureBlendStages = GL_LIMITS(texture_stages); diff --git a/dlls/wined3d/state.c b/dlls/wined3d/state.c index e0c5fbf..1b5115c 100644 --- a/dlls/wined3d/state.c +++ b/dlls/wined3d/state.c @@ -2164,6 +2164,19 @@ static void tex_bumpenvmat(DWORD state, IWineD3DStateBlockImpl *stateblock, Wine shaderconstant(STATE_PIXELSHADERCONSTANT, stateblock, context); } } + + if(GL_SUPPORT(ATI_ENVMAP_BUMPMAP)) { + if(stage >= GL_LIMITS(texture_stages)) { + WARN("Bump env matrix of unsupported stage set\n"); + } else if(GL_SUPPORT(ARB_MULTITEXTURE)) { + GL_EXTCALL(glActiveTextureARB(GL_TEXTURE0_ARB + stage)); + checkGLcall("GL_EXTCALL(glActiveTextureARB(GL_TEXTURE0_ARB + stage))"); + } + GL_EXTCALL(glTexBumpParameterfvATI(GL_BUMP_ROT_MATRIX_ATI, + (float *) &(stateblock->textureState[stage][WINED3DTSS_BUMPENVMAT00]))); + checkGLcall("glTexBumpParameterfvATI"); + } + /* TODO: GL_NV_texture_shader */ } static void transform_world(DWORD state, IWineD3DStateBlockImpl *stateblock, WineD3DContext *context) { diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index 9ab4e76..e71b49b 100644 --- a/dlls/wined3d/utils.c +++ b/dlls/wined3d/utils.c @@ -2168,6 +2168,46 @@ void set_tex_op(IWineD3DDevice *iface, BOOL isAlpha, int Stage, WINED3DTEXTUREOP } else Handled = FALSE; break; + case WINED3DTOP_BUMPENVMAPLUMINANCE: + if(GL_SUPPORT(ATI_ENVMAP_BUMPMAP)) { + /* Some apps use BUMPENVMAPLUMINANCE instead of D3DTOP_BUMPENVMAP, although + * they check for the non-luminance cap flag. Well, give them what they asked + * for :-) + */ + WARN("Application uses WINED3DTOP_BUMPENVMAPLUMINANCE\n"); + } else { + Handled = FALSE; + break; + } + /* Fall through */ + case WINED3DTOP_BUMPENVMAP: + if(GL_SUPPORT(ATI_ENVMAP_BUMPMAP)) { + TRACE("Using ati bumpmap on stage %d, target %d\n", Stage, Stage + 1); + glTexEnvi(GL_TEXTURE_ENV, comb_target, GL_BUMP_ENVMAP_ATI); + checkGLcall("glTexEnvi(GL_TEXTURE_ENV, comb_target, GL_BUMP_ENVMAP_ATI)"); + glTexEnvi(GL_TEXTURE_ENV, GL_BUMP_TARGET_ATI, GL_TEXTURE0_ARB + Stage + 1); + checkGLcall("glTexEnvi(GL_TEXTURE_ENV, GL_BUMP_TARGET_ATI, GL_TEXTURE0_ARB + Stage + 1)"); + glTexEnvi(GL_TEXTURE_ENV, src0_target, src3); + checkGLcall("GL_TEXTURE_ENV, src0_target, src3"); + glTexEnvi(GL_TEXTURE_ENV, opr0_target, opr3); + checkGLcall("GL_TEXTURE_ENV, opr0_target, opr3"); + glTexEnvi(GL_TEXTURE_ENV, src1_target, src1); + checkGLcall("GL_TEXTURE_ENV, src0_target, src1"); + glTexEnvi(GL_TEXTURE_ENV, opr1_target, opr1); + checkGLcall("GL_TEXTURE_ENV, opr1_target, opr1"); + glTexEnvi(GL_TEXTURE_ENV, src2_target, src2); + checkGLcall("GL_TEXTURE_ENV, src0_target, src1"); + glTexEnvi(GL_TEXTURE_ENV, opr2_target, opr2); + checkGLcall("GL_TEXTURE_ENV, opr2_target, opr2"); + + Handled = TRUE; + } else { + /* If GL_NV_TEXTURE_SHADER is supported insted the GL_NV_register_combiner path + * will be taken instead + */ + Handled = FALSE; + } + break; default: Handled = FALSE; } diff --git a/include/wine/wined3d_gl.h b/include/wine/wined3d_gl.h index c325539..b8a0715 100644 --- a/include/wine/wined3d_gl.h +++ b/include/wine/wined3d_gl.h @@ -1192,10 +1192,10 @@ typedef void (APIENTRY * PGLFNFINISHOBJECTAPPLEPROC) (GLenum, GLuint); #define GL_BUMP_ENVMAP_ATI 0x877B #define GL_BUMP_TARGET_ATI 0x877C #endif -typedef void (APIENTRY * PGLFNTEXBUMPPARAMETERIVATIPROC) (GLenum, GLuint); -typedef void (APIENTRY * PGLFNTEXBUMPPARAMETERFVATIPROC) (GLenum, GLuint); -typedef void (APIENTRY * PGLFNGETTEXBUMPPARAMETERIVATIPROC) (GLenum, GLuint); -typedef void (APIENTRY * PGLFNGETTEXBUMPPARAMETERFVATIPROC) (GLenum, GLuint); +typedef void (APIENTRY * PGLFNTEXBUMPPARAMETERIVATIPROC) (GLenum, GLint *); +typedef void (APIENTRY * PGLFNTEXBUMPPARAMETERFVATIPROC) (GLenum, GLfloat *); +typedef void (APIENTRY * PGLFNGETTEXBUMPPARAMETERIVATIPROC) (GLenum, GLint *); +typedef void (APIENTRY * PGLFNGETTEXBUMPPARAMETERFVATIPROC) (GLenum, GLfloat *); /* GL_VERSION_2_0 */ #ifndef GL_VERSION_2_0

1 0

Stefan Dösinger : wined3d: Some more ProcessVertices fixes.
by Alexandre Julliard May 9, 2007

May 9, 2007

Module: wine Branch: master Commit: ca52d34189e06555bf966024b4492f8a8e914f3d URL: http://source.winehq.org/git/wine.git/?a=commit;h=ca52d34189e06555bf966024b… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun May 6 18:39:29 2007 +0200 wined3d: Some more ProcessVertices fixes. Remove vbos from the input strided data from the fixed function attributes explicitly and avoid writing beyond the end of a temporary buffer when cloning a windows bug. --- dlls/wined3d/device.c | 57 ++++++++++++++++++++++++++++++++---------------- 1 files changed, 38 insertions(+), 19 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 1bf959f..9a419f0 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -3537,7 +3537,13 @@ process_vertices_strided(IWineD3DDeviceImpl *This, DWORD dwDestIndex, DWORD dwCo } if(dest->vbo) { - dest_conv_addr = HeapAlloc(GetProcessHeap(), 0, dwCount * get_flexible_vertex_size(DestFVF)); + unsigned char extrabytes = 0; + /* If the destination vertex buffer has D3DFVF_XYZ position(non-rhw), native d3d writes RHW position, where the RHW + * gets written into the 4 bytes after the Z position. In the case of a dest buffer that only has D3DFVF_XYZ data, + * this may write 4 extra bytes beyond the area that should be written + */ + if(DestFVF == WINED3DFVF_XYZ) extrabytes = 4; + dest_conv_addr = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, dwCount * get_flexible_vertex_size(DestFVF) + extrabytes); if(!dest_conv_addr) { ERR("Out of memory\n"); /* Continue without storing converted vertices */ @@ -3871,25 +3877,38 @@ static HRESULT WINAPI IWineD3DDeviceImpl_ProcessVertices(IWineD3DDevice *iface, * * Also get the start index in, but only loop over all elements if there's something to add at all. */ - for(i=0; i < 16; i++) { - if(strided.u.input[i].VBO) { - IWineD3DVertexBufferImpl *vb = (IWineD3DVertexBufferImpl *) This->stateBlock->streamSource[strided.u.input[i].streamNo]; - - /* The vertex buffer is supposed to have a system memory copy */ - strided.u.input[i].VBO = 0; - strided.u.input[i].lpData = (BYTE *) ((unsigned long) strided.u.input[i].lpData + (unsigned long) vb->resource.allocatedMemory); - ENTER_GL(); - GL_EXTCALL(glDeleteBuffersARB(1, &vb->vbo)); - vb->vbo = 0; - LEAVE_GL(); - - /* To be safe. An app could technically draw, then call ProcessVertices, then draw again without ever changing the stream sources */ - IWineD3DDeviceImpl_MarkStateDirty(This, STATE_STREAMSRC); - } - if(strided.u.input[i].lpData) { - strided.u.input[i].lpData += strided.u.input[i].dwStride * SrcStartIndex; - } +#define FIXSRC(type) \ + if(strided.u.s.type.VBO) { \ + IWineD3DVertexBufferImpl *vb = (IWineD3DVertexBufferImpl *) This->stateBlock->streamSource[strided.u.s.type.streamNo]; \ + strided.u.s.type.VBO = 0; \ + strided.u.s.type.lpData = (BYTE *) ((unsigned long) strided.u.s.type.lpData + (unsigned long) vb->resource.allocatedMemory); \ + ENTER_GL(); \ + GL_EXTCALL(glDeleteBuffersARB(1, &vb->vbo)); \ + vb->vbo = 0; \ + LEAVE_GL(); \ + } \ + if(strided.u.s.type.lpData) { \ + strided.u.s.type.lpData += strided.u.s.type.dwStride * SrcStartIndex; \ + } + FIXSRC(position); + FIXSRC(blendWeights); + FIXSRC(blendMatrixIndices); + FIXSRC(normal); + FIXSRC(pSize); + FIXSRC(diffuse); + FIXSRC(specular); + for(i = 0; i < WINED3DDP_MAXTEXCOORD; i++) { + FIXSRC(texCoords[i]); } + FIXSRC(position2); + FIXSRC(normal2); + FIXSRC(tangent); + FIXSRC(binormal); + FIXSRC(tessFactor); + FIXSRC(fog); + FIXSRC(depth); + FIXSRC(sample); +#undef FIXSRC } return process_vertices_strided(This, DestIndex, VertexCount, &strided, (IWineD3DVertexBufferImpl *) pDestBuffer, Flags);

1 0