wine-commits June 2012

wine-commits@winehq.org

3 participants
827 discussions

Hans Leidekker : msi: Ignore the value if a registry value name is special.
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: fd0c2dabdf1818efcd2e0fc05b70e628227145c1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fd0c2dabdf1818efcd2e0fc05… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Mon Jun 11 15:36:22 2012 +0200 msi: Ignore the value if a registry value name is special. --- dlls/msi/action.c | 43 +++++++++++++++++++------------------------ 1 files changed, 19 insertions(+), 24 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index 17e8696..5ba7ec3 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -2390,13 +2390,17 @@ static UINT ACTION_CostFinalize(MSIPACKAGE *package) return MSI_SetFeatureStates(package); } -/* OK this value is "interpreted" and then formatted based on the - first few characters */ -static LPSTR parse_value(MSIPACKAGE *package, LPCWSTR value, DWORD *type, - DWORD *size) +static LPSTR parse_value(MSIPACKAGE *package, LPCWSTR value, DWORD *type, DWORD *size) { LPSTR data = NULL; + if (!value) + { + data = (LPSTR)strdupW(szEmpty); + *size = sizeof(szEmpty); + *type = REG_SZ; + return data; + } if (value[0]=='#' && value[1]!='#' && value[1]!='%') { if (value[1]=='x') @@ -2578,19 +2582,19 @@ static WCHAR *get_keypath( MSICOMPONENT *comp, HKEY root, const WCHAR *path ) return strdupW( path ); } -static BOOL is_special_entry( const WCHAR *name, const WCHAR *value ) +static BOOL is_special_entry( const WCHAR *name ) { - return (name && (name[0] == '*' || name[0] == '+') && !name[1] && !value); + return (name && (name[0] == '*' || name[0] == '+') && !name[1]); } static UINT ITERATE_WriteRegistryValues(MSIRECORD *row, LPVOID param) { MSIPACKAGE *package = param; - LPSTR value_data = NULL; + LPSTR value; HKEY root_key, hkey; DWORD type,size; LPWSTR deformated, uikey, keypath; - LPCWSTR szRoot, component, name, key, value; + LPCWSTR szRoot, component, name, key; MSICOMPONENT *comp; MSIRECORD * uirow; INT root; @@ -2643,25 +2647,16 @@ static UINT ITERATE_WriteRegistryValues(MSIRECORD *row, LPVOID param) msi_free(keypath); return ERROR_SUCCESS; } - - value = MSI_RecordGetString(row,5); - if (value) - value_data = parse_value(package, value, &type, &size); - else - { - value_data = (LPSTR)strdupW(szEmpty); - size = sizeof(szEmpty); - type = REG_SZ; - } - + value = parse_value(package, MSI_RecordGetString(row, 5), &type, &size); deformat_string(package, name, &deformated); - if (!is_special_entry( name , value )) + + if (!is_special_entry( name )) { if (!check_first) { TRACE("Setting value %s of %s\n", debugstr_w(deformated), debugstr_w(uikey)); - RegSetValueExW(hkey, deformated, 0, type, (LPBYTE)value_data, size); + RegSetValueExW(hkey, deformated, 0, type, (LPBYTE)value, size); } else { @@ -2677,7 +2672,7 @@ static UINT ITERATE_WriteRegistryValues(MSIRECORD *row, LPVOID param) TRACE("Checked and setting value %s of %s\n", debugstr_w(deformated), debugstr_w(uikey)); if (deformated || size) - RegSetValueExW(hkey, deformated, 0, type, (LPBYTE)value_data, size); + RegSetValueExW(hkey, deformated, 0, type, (LPBYTE)value, size); } } } @@ -2687,11 +2682,11 @@ static UINT ITERATE_WriteRegistryValues(MSIRECORD *row, LPVOID param) MSI_RecordSetStringW(uirow,2,deformated); MSI_RecordSetStringW(uirow,1,uikey); if (type == REG_SZ || type == REG_EXPAND_SZ) - MSI_RecordSetStringW(uirow,3,(LPWSTR)value_data); + MSI_RecordSetStringW(uirow, 3, (LPWSTR)value); msi_ui_actiondata( package, szWriteRegistryValues, uirow ); msiobj_release( &uirow->hdr ); - msi_free(value_data); + msi_free(value); msi_free(deformated); msi_free(uikey); msi_free(keypath);

1 0

Hans Leidekker : msi: Add support for 64-bit registry components.
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: f0ea60049c4c0a484f7006a14ca3375cb0fd9260 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f0ea60049c4c0a484f7006a14… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Mon Jun 11 15:36:03 2012 +0200 msi: Add support for 64-bit registry components. --- dlls/msi/action.c | 30 +++++++--- dlls/msi/msi_main.c | 1 + dlls/msi/msipriv.h | 1 + dlls/msi/package.c | 2 - dlls/msi/tests/install.c | 140 ++++++++++++++++++++++++++++++++++++++++++++++ 5 files changed, 162 insertions(+), 12 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=f0ea60049c4c0a484f700…

1 0

Jacek Caban : mshtml: Wine Gecko 1.6 release.
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: 9fa443dcd9d16265a2d2b41441cea17c3c07e782 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9fa443dcd9d16265a2d2b4144… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Jun 11 15:32:18 2012 +0200 mshtml: Wine Gecko 1.6 release. --- dlls/appwiz.cpl/addons.c | 6 ++-- dlls/mshtml/htmlelem.c | 26 ++----------- dlls/mshtml/htmlnode.c | 2 +- dlls/mshtml/nsiface.idl | 90 ++++++++++++++++++++++----------------------- 4 files changed, 52 insertions(+), 72 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=9fa443dcd9d16265a2d2b…

1 0

Jacek Caban : urlmon: Use error dialog for ERROR_INTERNET_SEC_CERT_REV_FAILED error if IHttpSecurity returns S_FALSE .
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: 9af95b32433f2227c3d62b70dc7740a5fa0b81cf URL: http://source.winehq.org/git/wine.git/?a=commit;h=9af95b32433f2227c3d62b70d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Jun 11 16:14:45 2012 +0200 urlmon: Use error dialog for ERROR_INTERNET_SEC_CERT_REV_FAILED error if IHttpSecurity returns S_FALSE. --- dlls/urlmon/http.c | 12 +++++++----- 1 files changed, 7 insertions(+), 5 deletions(-) diff --git a/dlls/urlmon/http.c b/dlls/urlmon/http.c index d5f2b33..c67d36e 100644 --- a/dlls/urlmon/http.c +++ b/dlls/urlmon/http.c @@ -129,6 +129,8 @@ static HRESULT handle_http_error(HttpProtocol *This, DWORD error) DWORD res; HRESULT hres; + TRACE("(%p %u)\n", This, error); + switch(error) { case ERROR_INTERNET_SEC_CERT_DATE_INVALID: case ERROR_INTERNET_SEC_CERT_CN_INVALID: @@ -162,6 +164,8 @@ static HRESULT handle_http_error(HttpProtocol *This, DWORD error) hres = IHttpSecurity_OnSecurityProblem(http_security, error); IHttpSecurity_Release(http_security); + TRACE("OnSecurityProblem returned %08x\n", hres); + if(hres != S_FALSE) { BOOL res = FALSE; @@ -195,15 +199,13 @@ static HRESULT handle_http_error(HttpProtocol *This, DWORD error) switch(error) { case ERROR_INTERNET_SEC_CERT_REV_FAILED: - if(hres == S_FALSE) { - hres = internet_error_to_hres(error); - }else { + if(hres != S_FALSE) { /* Silently ignore the error. We will get more detailed error from wininet anyway. */ set_security_flag(This, SECURITY_FLAG_IGNORE_REVOCATION); hres = RPC_E_RETRY; + break; } - break; - + /* fallthrough */ default: hres = IServiceProvider_QueryService(serv_prov, &IID_IWindowForBindingUI, &IID_IWindowForBindingUI, (void**)&wfb_ui); if(SUCCEEDED(hres)) {

1 0

Aric Stewart : usp10: Corrections for ShapeCharGlyphProp_Thai.
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: f241014f46aedbae233409c4bfe0150d55c9e184 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f241014f46aedbae233409c4b… Author: Aric Stewart <aric(a)codeweavers.com> Date: Mon Jun 11 09:49:14 2012 -0500 usp10: Corrections for ShapeCharGlyphProp_Thai. --- dlls/usp10/shape.c | 32 ++++++++------------------------ 1 files changed, 8 insertions(+), 24 deletions(-) diff --git a/dlls/usp10/shape.c b/dlls/usp10/shape.c index f2b41b1..36f6854 100644 --- a/dlls/usp10/shape.c +++ b/dlls/usp10/shape.c @@ -411,7 +411,7 @@ static const ScriptShapeData ShapingData[] = {{ tibetan_features, 2}, NULL, 0, NULL, ShapeCharGlyphProp_Tibet}, {{ phags_features, 3}, NULL, 0, ContextualShape_Phags_pa, ShapeCharGlyphProp_Thai}, {{ thai_features, 1}, NULL, 0, ContextualShape_Thai, ShapeCharGlyphProp_Thai}, - {{ thai_features, 1}, NULL, 0, ContextualShape_Thai, ShapeCharGlyphProp_Thai}, + {{ thai_features, 1}, NULL, 0, ContextualShape_Thai, NULL}, {{ thai_features, 1}, required_lao_features, 0, ContextualShape_Lao, ShapeCharGlyphProp_Thai}, {{ thai_features, 1}, required_lao_features, 0, ContextualShape_Lao, ShapeCharGlyphProp_Thai}, {{ devanagari_features, 6}, required_devanagari_features, MS_MAKE_TAG('d','e','v','2'), ContextualShape_Devanagari, ShapeCharGlyphProp_Devanagari}, @@ -2838,13 +2838,9 @@ static void ShapeCharGlyphProp_Arabic( HDC hdc, ScriptCache *psc, SCRIPT_ANALYSI static void ShapeCharGlyphProp_Thai( HDC hdc, ScriptCache *psc, SCRIPT_ANALYSIS *psa, const WCHAR* pwcChars, const INT cChars, const WORD* pwGlyphs, const INT cGlyphs, WORD *pwLogClust, SCRIPT_CHARPROP *pCharProp, SCRIPT_GLYPHPROP *pGlyphProp ) { - int i,k; + int i; int finaGlyph; INT dirL; - BYTE *spaces; - - spaces = HeapAlloc(GetProcessHeap(),0,cGlyphs); - memset(spaces,0,cGlyphs); if (!psa->fLogicalOrder && psa->fRTL) { @@ -2857,20 +2853,11 @@ static void ShapeCharGlyphProp_Thai( HDC hdc, ScriptCache *psc, SCRIPT_ANALYSIS dirL = 1; } - for (i = 0; i < cGlyphs; i++) - { - for (k = 0; k < cChars; k++) - if (pwLogClust[k] == i) - { - if (pwcChars[k] == 0x0020) - spaces[i] = 1; - } - } - OpenType_GDEF_UpdateGlyphProps(hdc, psc, pwGlyphs, cGlyphs, pwLogClust, cChars, pGlyphProp); for (i = 0; i < cGlyphs; i++) { + int k; int char_index[20]; int char_count = 0; @@ -2881,25 +2868,22 @@ static void ShapeCharGlyphProp_Thai( HDC hdc, ScriptCache *psc, SCRIPT_ANALYSIS char_index[char_count++] = k; } + if (i == finaGlyph) + pGlyphProp[i].sva.uJustification = SCRIPT_JUSTIFY_NONE; + else + pGlyphProp[i].sva.uJustification = SCRIPT_JUSTIFY_CHARACTER; + if (char_count == 0) continue; if (char_count ==1 && pwcChars[char_index[0]] == 0x0020) /* space */ - { - pGlyphProp[i].sva.uJustification = SCRIPT_JUSTIFY_CHARACTER; pCharProp[char_index[0]].fCanGlyphAlone = 1; - } - else if (i == finaGlyph) - pGlyphProp[i].sva.uJustification = SCRIPT_JUSTIFY_NONE; - else - pGlyphProp[i].sva.uJustification = SCRIPT_JUSTIFY_CHARACTER; /* handle Thai SARA AM (U+0E33) differently than GDEF */ if (char_count == 1 && pwcChars[char_index[0]] == 0x0e33) pGlyphProp[i].sva.fClusterStart = 0; } - HeapFree(GetProcessHeap(),0,spaces); UpdateClustersFromGlyphProp(cGlyphs, cChars, pwLogClust, pGlyphProp); /* Do not allow justification between marks and their base */

1 0

Aric Stewart : usp10: Properly handle fonts with scripts without a default language.
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: 6c43ef46d85bb37de5ce65a8ae6634e891e5cb07 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6c43ef46d85bb37de5ce65a8a… Author: Aric Stewart <aric(a)codeweavers.com> Date: Mon Jun 11 07:15:11 2012 -0500 usp10: Properly handle fonts with scripts without a default language. --- dlls/usp10/opentype.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/dlls/usp10/opentype.c b/dlls/usp10/opentype.c index 74046fd..1a5d3e4 100644 --- a/dlls/usp10/opentype.c +++ b/dlls/usp10/opentype.c @@ -924,10 +924,15 @@ static void GSUB_initialize_language_cache(LoadedScript *script) if (!script->language_count) { + DWORD offset; const GSUB_Script* table = script->table; script->language_count = GET_BE_WORD(table->LangSysCount); - script->default_language.tag = MS_MAKE_TAG('d','f','l','t'); - script->default_language.table = (const BYTE*)table + GET_BE_WORD(table->DefaultLangSys); + offset = GET_BE_WORD(table->DefaultLangSys); + if (offset) + { + script->default_language.tag = MS_MAKE_TAG('d','f','l','t'); + script->default_language.table = (const BYTE*)table + offset; + } if (script->language_count) {

1 0

Józef Kucia : d3dx9/tests: Use inline functions instead of check_pixel_{1bpp, 2bpp, 4bpp} macros.
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: 0e532633e37bd066ceb1156f85e9075c79dea8e2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0e532633e37bd066ceb1156f8… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Sun Jun 10 21:39:48 2012 +0200 d3dx9/tests: Use inline functions instead of check_pixel_{1bpp, 2bpp, 4bpp} macros. --- dlls/d3dx9_36/tests/surface.c | 196 +++++++++++++++++++++------------------- 1 files changed, 103 insertions(+), 93 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=0e532633e37bd066ceb11…

1 0

Józef Kucia : d3dx9/tests: Improve test failure messages for check_{ref, release}.
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: cda4738f52152a805138b607726466836a73d22f URL: http://source.winehq.org/git/wine.git/?a=commit;h=cda4738f52152a805138b6077… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Sun Jun 10 21:39:47 2012 +0200 d3dx9/tests: Improve test failure messages for check_{ref, release}. --- dlls/d3dx9_36/tests/core.c | 10 ++++++---- dlls/d3dx9_36/tests/surface.c | 10 ++++++---- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/dlls/d3dx9_36/tests/core.c b/dlls/d3dx9_36/tests/core.c index fa259ab..7f9ab44 100644 --- a/dlls/d3dx9_36/tests/core.c +++ b/dlls/d3dx9_36/tests/core.c @@ -29,16 +29,18 @@ static inline int get_ref(IUnknown *obj) return IUnknown_Release(obj); } -static inline void check_ref(IUnknown *obj, int exp) +#define check_ref(obj, exp) _check_ref(__LINE__, obj, exp) +static inline void _check_ref(unsigned int line, IUnknown *obj, int exp) { int ref = get_ref(obj); - ok (exp == ref, "Invalid refcount. Expected %d, got %d\n", exp, ref); + ok_(__FILE__, line)(exp == ref, "Invalid refcount. Expected %d, got %d\n", exp, ref); } -static inline void check_release(IUnknown *obj, int exp) +#define check_release(obj, exp) _check_release(__LINE__, obj, exp) +static inline void _check_release(unsigned int line, IUnknown *obj, int exp) { int ref = IUnknown_Release(obj); - ok (ref == exp, "Invalid refcount. Expected %d, got %d\n", exp, ref); + ok_(__FILE__, line)(ref == exp, "Invalid refcount. Expected %d, got %d\n", exp, ref); } #define admitted_error 0.0001f diff --git a/dlls/d3dx9_36/tests/surface.c b/dlls/d3dx9_36/tests/surface.c index 4b3b511..72d2509 100644 --- a/dlls/d3dx9_36/tests/surface.c +++ b/dlls/d3dx9_36/tests/surface.c @@ -30,16 +30,18 @@ static inline int get_ref(IUnknown *obj) return IUnknown_Release(obj); } -static inline void check_ref(IUnknown *obj, int exp) +#define check_ref(obj, exp) _check_ref(__LINE__, obj, exp) +static inline void _check_ref(unsigned int line, IUnknown *obj, int exp) { int ref = get_ref(obj); - ok (exp == ref, "Invalid refcount. Expected %d, got %d\n", exp, ref); + ok_(__FILE__, line)(exp == ref, "Invalid refcount. Expected %d, got %d\n", exp, ref); } -static inline void check_release(IUnknown *obj, int exp) +#define check_release(obj, exp) _check_release(__LINE__, obj, exp) +static inline void _check_release(unsigned int line, IUnknown *obj, int exp) { int ref = IUnknown_Release(obj); - ok (ref == exp, "Invalid refcount. Expected %d, got %d\n", exp, ref); + ok_(__FILE__, line)(ref == exp, "Invalid refcount. Expected %d, got %d\n", exp, ref); } /* 1x1 bmp (1 bpp) */

1 0

Józef Kucia : d3dx9: Add ID3DXRenderToEnvMap interface stub.
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: 35c13b0879163a5b9de99e00aa29f4ffbc8d94d4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=35c13b0879163a5b9de99e00a… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Sun Jun 10 21:29:14 2012 +0200 d3dx9: Add ID3DXRenderToEnvMap interface stub. --- dlls/d3dx9_36/render.c | 197 +++++++++++++++++++++++++++++++++++++++++++- dlls/d3dx9_36/tests/core.c | 4 +- 2 files changed, 195 insertions(+), 6 deletions(-) diff --git a/dlls/d3dx9_36/render.c b/dlls/d3dx9_36/render.c index 9b691bc..7994a8b 100644 --- a/dlls/d3dx9_36/render.c +++ b/dlls/d3dx9_36/render.c @@ -315,7 +315,7 @@ static HRESULT WINAPI D3DXRenderToSurface_OnResetDevice(ID3DXRenderToSurface *if return D3D_OK; } -static const ID3DXRenderToSurfaceVtbl d3dx_render_to_surface_vtbl = +static const ID3DXRenderToSurfaceVtbl render_to_surface_vtbl = { /* IUnknown methods */ D3DXRenderToSurface_QueryInterface, @@ -354,7 +354,7 @@ HRESULT WINAPI D3DXCreateRenderToSurface(IDirect3DDevice9 *device, render = HeapAlloc(GetProcessHeap(), 0, sizeof(struct render_to_surface)); if (!render) return E_OUTOFMEMORY; - render->ID3DXRenderToSurface_iface.lpVtbl = &d3dx_render_to_surface_vtbl; + render->ID3DXRenderToSurface_iface.lpVtbl = &render_to_surface_vtbl; render->ref = 1; render->desc.Width = width; @@ -387,6 +387,172 @@ HRESULT WINAPI D3DXCreateRenderToSurface(IDirect3DDevice9 *device, return D3D_OK; } + +struct render_to_envmap +{ + ID3DXRenderToEnvMap ID3DXRenderToEnvMap_iface; + LONG ref; + + IDirect3DDevice9 *device; + D3DXRTE_DESC desc; +}; + +static inline struct render_to_envmap *impl_from_ID3DXRenderToEnvMap(ID3DXRenderToEnvMap *iface) +{ + return CONTAINING_RECORD(iface, struct render_to_envmap, ID3DXRenderToEnvMap_iface); +} + +static HRESULT WINAPI D3DXRenderToEnvMap_QueryInterface(ID3DXRenderToEnvMap *iface, + REFIID riid, + void **out) +{ + TRACE("iface %p, riid %s, out %p\n", iface, debugstr_guid(riid), out); + + if (IsEqualGUID(riid, &IID_ID3DXRenderToEnvMap) + || IsEqualGUID(riid, &IID_IUnknown)) + { + IUnknown_AddRef(iface); + *out = iface; + return S_OK; + } + + WARN("%s not implemented, returning E_NOINTERFACE\n", debugstr_guid(riid)); + + *out = NULL; + return E_NOINTERFACE; +} + +static ULONG WINAPI D3DXRenderToEnvMap_AddRef(ID3DXRenderToEnvMap *iface) +{ + struct render_to_envmap *render = impl_from_ID3DXRenderToEnvMap(iface); + ULONG ref = InterlockedIncrement(&render->ref); + + TRACE("%p increasing refcount to %u\n", iface, ref); + + return ref; +} + +static ULONG WINAPI D3DXRenderToEnvMap_Release(ID3DXRenderToEnvMap *iface) +{ + struct render_to_envmap *render = impl_from_ID3DXRenderToEnvMap(iface); + ULONG ref = InterlockedDecrement(&render->ref); + + TRACE("%p decreasing refcount to %u\n", iface, ref); + + if (!ref) + { + IDirect3DDevice9_Release(render->device); + + HeapFree(GetProcessHeap(), 0, render); + } + + return ref; +} + +static HRESULT WINAPI D3DXRenderToEnvMap_GetDevice(ID3DXRenderToEnvMap *iface, + IDirect3DDevice9 **device) +{ + struct render_to_envmap *render = impl_from_ID3DXRenderToEnvMap(iface); + + TRACE("(%p)->(%p)\n", iface, device); + + if (!device) return D3DERR_INVALIDCALL; + + IDirect3DDevice9_AddRef(render->device); + *device = render->device; + return D3D_OK; +} + +static HRESULT WINAPI D3DXRenderToEnvMap_GetDesc(ID3DXRenderToEnvMap *iface, + D3DXRTE_DESC *desc) +{ + struct render_to_envmap *render = impl_from_ID3DXRenderToEnvMap(iface); + + TRACE("(%p)->(%p)\n", iface, desc); + + if (!desc) return D3DERR_INVALIDCALL; + + *desc = render->desc; + return D3D_OK; +} + +static HRESULT WINAPI D3DXRenderToEnvMap_BeginCube(ID3DXRenderToEnvMap *iface, + IDirect3DCubeTexture9 *texture) +{ + FIXME("(%p)->(%p): stub\n", iface, texture); + return E_NOTIMPL; +} + +static HRESULT WINAPI D3DXRenderToEnvMap_BeginSphere(ID3DXRenderToEnvMap *iface, + IDirect3DTexture9 *texture) +{ + FIXME("(%p)->(%p): stub\n", iface, texture); + return E_NOTIMPL; +} + +static HRESULT WINAPI D3DXRenderToEnvMap_BeginHemisphere(ID3DXRenderToEnvMap *iface, + IDirect3DTexture9 *pos_z_texture, + IDirect3DTexture9 *neg_z_texture) +{ + FIXME("(%p)->(%p, %p): stub\n", iface, pos_z_texture, neg_z_texture); + return E_NOTIMPL; +} + +static HRESULT WINAPI D3DXRenderToEnvMap_BeginParabolic(ID3DXRenderToEnvMap *iface, + IDirect3DTexture9 *pos_z_texture, + IDirect3DTexture9 *neg_z_texture) +{ + FIXME("(%p)->(%p, %p): stub\n", iface, pos_z_texture, neg_z_texture); + return E_NOTIMPL; +} + +static HRESULT WINAPI D3DXRenderToEnvMap_Face(ID3DXRenderToEnvMap *iface, + D3DCUBEMAP_FACES face, + DWORD filter) +{ + FIXME("(%p)->(%u, %#x): stub\n", iface, face, filter); + return E_NOTIMPL; +} + +static HRESULT WINAPI D3DXRenderToEnvMap_End(ID3DXRenderToEnvMap *iface, + DWORD filter) +{ + FIXME("(%p)->(%#x): stub\n", iface, filter); + return E_NOTIMPL; +} + + +static HRESULT WINAPI D3DXRenderToEnvMap_OnLostDevice(ID3DXRenderToEnvMap *iface) +{ + FIXME("(%p)->(): stub\n", iface); + return D3D_OK; +} + +static HRESULT WINAPI D3DXRenderToEnvMap_OnResetDevice(ID3DXRenderToEnvMap *iface) +{ + FIXME("(%p)->(): stub\n", iface); + return D3D_OK; +} + +static const ID3DXRenderToEnvMapVtbl render_to_envmap_vtbl = +{ + /* IUnknown methods */ + D3DXRenderToEnvMap_QueryInterface, + D3DXRenderToEnvMap_AddRef, + D3DXRenderToEnvMap_Release, + /* ID3DXRenderToEnvMap methods */ + D3DXRenderToEnvMap_GetDevice, + D3DXRenderToEnvMap_GetDesc, + D3DXRenderToEnvMap_BeginCube, + D3DXRenderToEnvMap_BeginSphere, + D3DXRenderToEnvMap_BeginHemisphere, + D3DXRenderToEnvMap_BeginParabolic, + D3DXRenderToEnvMap_Face, + D3DXRenderToEnvMap_End, + D3DXRenderToEnvMap_OnLostDevice, + D3DXRenderToEnvMap_OnResetDevice +}; + HRESULT WINAPI D3DXCreateRenderToEnvMap(IDirect3DDevice9 *device, UINT size, UINT mip_levels, @@ -395,10 +561,33 @@ HRESULT WINAPI D3DXCreateRenderToEnvMap(IDirect3DDevice9 *device, D3DFORMAT depth_stencil_format, ID3DXRenderToEnvMap **out) { - FIXME("(%p, %u, %u, %#x, %d, %#x, %p): stub\n", device, size, mip_levels, + HRESULT hr; + struct render_to_envmap *render; + + TRACE("(%p, %u, %u, %#x, %d, %#x, %p)\n", device, size, mip_levels, format, depth_stencil, depth_stencil_format, out); if (!device || !out) return D3DERR_INVALIDCALL; - return E_NOTIMPL; + hr = D3DXCheckTextureRequirements(device, &size, &size, &mip_levels, + D3DUSAGE_RENDERTARGET, &format, D3DPOOL_DEFAULT); + if (FAILED(hr)) return hr; + + render = HeapAlloc(GetProcessHeap(), 0, sizeof(struct render_to_envmap)); + if (!render) return E_OUTOFMEMORY; + + render->ID3DXRenderToEnvMap_iface.lpVtbl = &render_to_envmap_vtbl; + render->ref = 1; + + render->desc.Size = size; + render->desc.MipLevels = mip_levels; + render->desc.Format = format; + render->desc.DepthStencil = depth_stencil; + render->desc.DepthStencilFormat = depth_stencil_format; + + IDirect3DDevice9_AddRef(device); + render->device = device; + + *out = &render->ID3DXRenderToEnvMap_iface; + return D3D_OK; } diff --git a/dlls/d3dx9_36/tests/core.c b/dlls/d3dx9_36/tests/core.c index 3009f50..fa259ab 100644 --- a/dlls/d3dx9_36/tests/core.c +++ b/dlls/d3dx9_36/tests/core.c @@ -908,7 +908,7 @@ void test_D3DXCreateRenderToEnvMap(IDirect3DDevice9 *device) const D3DXRTE_DESC *expected = &tests[i].expected_values; hr = D3DXCreateRenderToEnvMap(device, parameters->Size, parameters->MipLevels, parameters->Format, parameters->DepthStencil, parameters->DepthStencilFormat, &render); - todo_wine ok(hr == D3D_OK, "%d: D3DXCreateRenderToEnvMap returned %#x, expected %#x\n", i, hr, D3D_OK); + ok(hr == D3D_OK, "%d: D3DXCreateRenderToEnvMap returned %#x, expected %#x\n", i, hr, D3D_OK); if (SUCCEEDED(hr)) { hr = ID3DXRenderToEnvMap_GetDesc(render, &desc); @@ -930,7 +930,7 @@ void test_D3DXCreateRenderToEnvMap(IDirect3DDevice9 *device) /* check device ref count */ ref_count = get_ref((IUnknown *)device); hr = D3DXCreateRenderToEnvMap(device, 0, 0, D3DFMT_UNKNOWN, FALSE, D3DFMT_UNKNOWN, &render); - todo_wine check_ref((IUnknown *)device, ref_count + 1); + check_ref((IUnknown *)device, ref_count + 1); if (SUCCEEDED(hr)) ID3DXRenderToEnvMap_Release(render); }

1 0

Józef Kucia : d3dx9/tests: Add tests for D3DXCreateRenderToEnvMap.
by Alexandre Julliard 11 Jun '12

11 Jun '12

Module: wine Branch: master Commit: f072404f617ebc59de7d156ce338805202f74045 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f072404f617ebc59de7d156ce… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Sun Jun 10 21:29:13 2012 +0200 d3dx9/tests: Add tests for D3DXCreateRenderToEnvMap. --- dlls/d3dx9_36/d3dx9_36.spec | 2 +- dlls/d3dx9_36/render.c | 16 +++++++++++++ dlls/d3dx9_36/tests/core.c | 52 +++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 69 insertions(+), 1 deletions(-) diff --git a/dlls/d3dx9_36/d3dx9_36.spec b/dlls/d3dx9_36/d3dx9_36.spec index 8d7143e..20a8abf 100644 --- a/dlls/d3dx9_36/d3dx9_36.spec +++ b/dlls/d3dx9_36/d3dx9_36.spec @@ -81,7 +81,7 @@ @ stub D3DXCreatePRTBufferTex(long long long long ptr) @ stub D3DXCreatePRTCompBuffer(long long long ptr ptr ptr ptr) @ stub D3DXCreatePRTEngine(ptr ptr long ptr ptr) -@ stub D3DXCreateRenderToEnvMap(ptr long long long long long ptr) +@ stdcall D3DXCreateRenderToEnvMap(ptr long long long long long ptr) @ stdcall D3DXCreateRenderToSurface(ptr long long long long long ptr) @ stub D3DXCreateSPMesh(ptr ptr ptr ptr ptr) @ stdcall D3DXCreateSkinInfo(long ptr long ptr) diff --git a/dlls/d3dx9_36/render.c b/dlls/d3dx9_36/render.c index 35aed93..9b691bc 100644 --- a/dlls/d3dx9_36/render.c +++ b/dlls/d3dx9_36/render.c @@ -386,3 +386,19 @@ HRESULT WINAPI D3DXCreateRenderToSurface(IDirect3DDevice9 *device, *out = &render->ID3DXRenderToSurface_iface; return D3D_OK; } + +HRESULT WINAPI D3DXCreateRenderToEnvMap(IDirect3DDevice9 *device, + UINT size, + UINT mip_levels, + D3DFORMAT format, + BOOL depth_stencil, + D3DFORMAT depth_stencil_format, + ID3DXRenderToEnvMap **out) +{ + FIXME("(%p, %u, %u, %#x, %d, %#x, %p): stub\n", device, size, mip_levels, + format, depth_stencil, depth_stencil_format, out); + + if (!device || !out) return D3DERR_INVALIDCALL; + + return E_NOTIMPL; +} diff --git a/dlls/d3dx9_36/tests/core.c b/dlls/d3dx9_36/tests/core.c index a19e570..3009f50 100644 --- a/dlls/d3dx9_36/tests/core.c +++ b/dlls/d3dx9_36/tests/core.c @@ -883,6 +883,57 @@ static void test_ID3DXRenderToSurface(IDirect3DDevice9 *device) test_ID3DXRenderToSurface_device_state(device); } +void test_D3DXCreateRenderToEnvMap(IDirect3DDevice9 *device) +{ + int i; + HRESULT hr; + ULONG ref_count; + D3DXRTE_DESC desc; + ID3DXRenderToEnvMap *render; + static const struct { + D3DXRTE_DESC parameters; + D3DXRTE_DESC expected_values; + } tests[] = { + { { 0, 0, D3DFMT_A8R8G8B8, FALSE, D3DFMT_UNKNOWN }, { 1, 1, D3DFMT_A8R8G8B8, FALSE, D3DFMT_UNKNOWN } }, + { { 256, 0, D3DFMT_A8R8G8B8, FALSE, D3DFMT_UNKNOWN }, { 256, 9, D3DFMT_A8R8G8B8, FALSE, D3DFMT_UNKNOWN } }, + { { 256, 4, D3DFMT_A8R8G8B8, FALSE, D3DFMT_D24S8 }, { 256, 4, D3DFMT_A8R8G8B8, FALSE, D3DFMT_D24S8 } }, + { { 256, 256, D3DFMT_UNKNOWN, FALSE, D3DFMT_R8G8B8 }, { 256, 9, D3DFMT_A8R8G8B8, FALSE, D3DFMT_R8G8B8 } }, + { { -1, -1, D3DFMT_A8R8G8B8, TRUE, D3DFMT_DXT1 }, { 256, 9, D3DFMT_A8R8G8B8, TRUE, D3DFMT_DXT1 } }, + { { 256, 1, D3DFMT_X8R8G8B8, TRUE, D3DFMT_UNKNOWN }, { 256, 1, D3DFMT_X8R8G8B8, TRUE, D3DFMT_UNKNOWN } } + }; + + for (i = 0; i < sizeof(tests) / sizeof(tests[0]); i++) + { + const D3DXRTE_DESC *parameters = &tests[i].parameters; + const D3DXRTE_DESC *expected = &tests[i].expected_values; + hr = D3DXCreateRenderToEnvMap(device, parameters->Size, parameters->MipLevels, parameters->Format, + parameters->DepthStencil, parameters->DepthStencilFormat, &render); + todo_wine ok(hr == D3D_OK, "%d: D3DXCreateRenderToEnvMap returned %#x, expected %#x\n", i, hr, D3D_OK); + if (SUCCEEDED(hr)) + { + hr = ID3DXRenderToEnvMap_GetDesc(render, &desc); + ok(hr == D3D_OK, "%d: GetDesc failed %#x\n", i, hr); + if (SUCCEEDED(hr)) + { + ok(desc.Size == expected->Size, "%d: Got size %u, expected %u\n", i, desc.Size, expected->Size); + ok(desc.MipLevels == expected->MipLevels, "%d: Got miplevels %u, expected %u\n", i, desc.MipLevels, expected->MipLevels); + ok(desc.Format == expected->Format, "%d: Got format %#x, expected %#x\n", i, desc.Format, expected->Format); + ok(desc.DepthStencil == expected->DepthStencil, "%d: Got depth stencil %d, expected %d\n", + i, expected->DepthStencil, expected->DepthStencil); + ok(desc.DepthStencilFormat == expected->DepthStencilFormat, "%d: Got depth stencil format %#x, expected %#x\n", + i, expected->DepthStencilFormat, expected->DepthStencilFormat); + } + check_release((IUnknown *)render, 0); + } + } + + /* check device ref count */ + ref_count = get_ref((IUnknown *)device); + hr = D3DXCreateRenderToEnvMap(device, 0, 0, D3DFMT_UNKNOWN, FALSE, D3DFMT_UNKNOWN, &render); + todo_wine check_ref((IUnknown *)device, ref_count + 1); + if (SUCCEEDED(hr)) ID3DXRenderToEnvMap_Release(render); +} + START_TEST(core) { HWND wnd; @@ -919,6 +970,7 @@ START_TEST(core) test_ID3DXFont(device); test_D3DXCreateRenderToSurface(device); test_ID3DXRenderToSurface(device); + test_D3DXCreateRenderToEnvMap(device); check_release((IUnknown*)device, 0); check_release((IUnknown*)d3d, 0);

1 0

← Newer
1
...
59
60
61
62
63
64
65
...
83
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 June 2012