May 2012 - Wine-commits - list.winehq.org

Hans Leidekker : msi: Don' t create empty values for special registry entries.
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: e16bcda4862e7df0f266a42630030ad8c393c72b URL: http://source.winehq.org/git/wine.git/?a=commit;h=e16bcda4862e7df0f266a4263… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu May 31 10:06:02 2012 +0200 msi: Don't create empty values for special registry entries. --- dlls/msi/action.c | 49 +++++++++++++++++++++++++--------------------- dlls/msi/tests/action.c | 12 ++++++++-- 2 files changed, 36 insertions(+), 25 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index efdc79c..ec00886 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -2578,6 +2578,11 @@ static WCHAR *get_keypath( MSIPACKAGE *package, HKEY root, const WCHAR *path ) return strdupW( path ); } +static BOOL is_special_entry( const WCHAR *name, const WCHAR *value ) +{ + return (name && (name[0] == '*' || name[0] == '+') && !name[1] && !value); +} + static UINT ITERATE_WriteRegistryValues(MSIRECORD *row, LPVOID param) { MSIPACKAGE *package = param; @@ -2612,10 +2617,8 @@ static UINT ITERATE_WriteRegistryValues(MSIRECORD *row, LPVOID param) /* null values can have special meanings */ if (name[0]=='-' && name[1] == 0) return ERROR_SUCCESS; - else if ((name[0]=='+' && name[1] == 0) || - (name[0] == '*' && name[1] == 0)) - name = NULL; - check_first = TRUE; + if ((name[0] == '+' || name[0] == '*') && !name[1]) + check_first = TRUE; } root = MSI_RecordGetInteger(row,2); @@ -2652,28 +2655,30 @@ static UINT ITERATE_WriteRegistryValues(MSIRECORD *row, LPVOID param) } deformat_string(package, name, &deformated); - - 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); - } - else + if (!is_special_entry( name , value )) { - DWORD sz = 0; - rc = RegQueryValueExW(hkey, deformated, NULL, NULL, NULL, &sz); - if (rc == ERROR_SUCCESS || rc == ERROR_MORE_DATA) + if (!check_first) { - TRACE("value %s of %s checked already exists\n", - debugstr_w(deformated), debugstr_w(uikey)); + TRACE("Setting value %s of %s\n", debugstr_w(deformated), + debugstr_w(uikey)); + RegSetValueExW(hkey, deformated, 0, type, (LPBYTE)value_data, size); } else { - 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); + DWORD sz = 0; + rc = RegQueryValueExW(hkey, deformated, NULL, NULL, NULL, &sz); + if (rc == ERROR_SUCCESS || rc == ERROR_MORE_DATA) + { + TRACE("value %s of %s checked already exists\n", debugstr_w(deformated), + debugstr_w(uikey)); + } + else + { + 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); + } } } RegCloseKey(hkey); @@ -2773,7 +2778,7 @@ static UINT ITERATE_RemoveRegistryValuesOnUninstall( MSIRECORD *row, LPVOID para { if (name[0] == '+' && !name[1]) return ERROR_SUCCESS; - else if ((name[0] == '-' && !name[1]) || (name[0] == '*' && !name[1])) + if ((name[0] == '-' || name[0] == '*') && !name[1]) { delete_key = TRUE; name = NULL; diff --git a/dlls/msi/tests/action.c b/dlls/msi/tests/action.c index 2b482df..0bd65a0 100644 --- a/dlls/msi/tests/action.c +++ b/dlls/msi/tests/action.c @@ -487,7 +487,10 @@ static const char wrv_registry_dat[] = "Registry\tRoot\tKey\tName\tValue\tComponent_\n" "s72\ti2\tl255\tL255\tL0\ts72\n" "Registry\tRegistry\n" - "regdata\t2\tSOFTWARE\\Wine\\msitest\tValue\t[~]one[~]two[~]three\taugustus"; + "regdata\t2\tSOFTWARE\\Wine\\msitest\tValue\t[~]one[~]two[~]three\taugustus\n" + "regdata1\t2\tSOFTWARE\\Wine\\msitest\t*\t\taugustus\n" + "regdata2\t2\tSOFTWARE\\Wine\\msitest\t*\t#%\taugustus\n" + "regdata3\t2\tSOFTWARE\\Wine\\msitest\t*\t#x\taugustus\n"; static const char cf_directory_dat[] = "Directory\tDirectory_Parent\tDefaultDir\n" @@ -4572,7 +4575,7 @@ static void test_write_registry_values(void) ok(res == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", res); size = MAX_PATH; - type = REG_MULTI_SZ; + type = 0xdeadbeef; memset(path, 'a', MAX_PATH); res = RegQueryValueExA(hkey, "Value", NULL, &type, (LPBYTE)path, &size); ok(res == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", res); @@ -4580,9 +4583,12 @@ static void test_write_registry_values(void) ok(size == 15, "Expected 15, got %d\n", size); ok(type == REG_MULTI_SZ, "Expected REG_MULTI_SZ, got %d\n", type); + res = RegQueryValueExA(hkey, "", NULL, NULL, NULL, NULL); + ok(res == ERROR_FILE_NOT_FOUND, "Expected ERROR_FILE_NOT_FOUND, got %d\n", res); + RegDeleteValueA(hkey, "Value"); RegCloseKey(hkey); - RegDeleteKeyA(HKEY_CURRENT_USER, "SOFTWARE\\Wine\\msitest"); + RegDeleteKeyA(HKEY_LOCAL_MACHINE, "SOFTWARE\\Wine\\msitest"); error: DeleteFile(msifile);

1 0

Stefan Leichter : kernel32: Fix OutputDebugStringA called with NULL pointer .
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: 68aa42a4191c50bb9bf2767c51a4b9c7b7aaef5e URL: http://source.winehq.org/git/wine.git/?a=commit;h=68aa42a4191c50bb9bf2767c5… Author: Stefan Leichter <Stefan.Leichter(a)camline.com> Date: Wed May 30 22:47:34 2012 +0200 kernel32: Fix OutputDebugStringA called with NULL pointer. --- dlls/kernel32/debugger.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/debugger.c b/dlls/kernel32/debugger.c index 7e1a860..528ba76 100644 --- a/dlls/kernel32/debugger.c +++ b/dlls/kernel32/debugger.c @@ -245,6 +245,8 @@ void WINAPI OutputDebugStringA( LPCSTR str ) static HANDLE DBWinMutex = NULL; static BOOL mutex_inited = FALSE; + if (!str) str = ""; + /* send string to attached debugger */ SERVER_START_REQ( output_debug_string ) {

1 0

Detlef Riekenberg : ntdll: Implement RtlGetProductInfo.
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: 0f4e9ffc22f32affcd6fb92c9e1925883c208de1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0f4e9ffc22f32affcd6fb92c9… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Wed May 30 18:40:33 2012 +0200 ntdll: Implement RtlGetProductInfo. --- dlls/ntdll/version.c | 18 +++++++++++++++--- include/winnt.h | 5 ++++- 2 files changed, 19 insertions(+), 4 deletions(-) diff --git a/dlls/ntdll/version.c b/dlls/ntdll/version.c index 0185d7f..745ee5d 100644 --- a/dlls/ntdll/version.c +++ b/dlls/ntdll/version.c @@ -544,7 +544,7 @@ done: } /*********************************************************************** - * GetProductInfo (NTDLL.@) + * RtlGetProductInfo (NTDLL.@) * * Gives info about the current Windows product type, in a format compatible * with the given Windows version @@ -554,10 +554,22 @@ done: BOOLEAN WINAPI RtlGetProductInfo(DWORD dwOSMajorVersion, DWORD dwOSMinorVersion, DWORD dwSpMajorVersion, DWORD dwSpMinorVersion, PDWORD pdwReturnedProductType) { - FIXME("(%d,%d,%d,%d,%p): stub\n", dwOSMajorVersion, dwOSMinorVersion, + TRACE("(%d, %d, %d, %d, %p)\n", dwOSMajorVersion, dwOSMinorVersion, dwSpMajorVersion, dwSpMinorVersion, pdwReturnedProductType); - *pdwReturnedProductType = PRODUCT_ULTIMATE_N; + if (!pdwReturnedProductType) + return FALSE; + + if (dwOSMajorVersion < 6) + { + *pdwReturnedProductType = PRODUCT_UNDEFINED; + return FALSE; + } + + if (current_version->wProductType == VER_NT_WORKSTATION) + *pdwReturnedProductType = PRODUCT_ULTIMATE_N; + else + *pdwReturnedProductType = PRODUCT_STANDARD_SERVER; return TRUE; } diff --git a/include/winnt.h b/include/winnt.h index 02c92e6..bed2a19 100644 --- a/include/winnt.h +++ b/include/winnt.h @@ -1976,7 +1976,7 @@ NTSYSAPI void WINAPI RtlCaptureContext(CONTEXT*); #define PRODUCT_DATACENTER_SERVER 0x00000008 #define PRODUCT_DATACENTER_SERVER_CORE 0x0000000C #define PRODUCT_DATACENTER_SERVER_CORE_V 0x00000027 -#define PRODUCT_SERVER_V 0x00000025 +#define PRODUCT_EMBEDDED 0x00000041 #define PRODUCT_ENTERPRISE 0x00000004 #define PRODUCT_ENTERPRISE_N 0x0000001B #define PRODUCT_ENTERPRISE_SERVER 0x0000000A @@ -1992,8 +1992,11 @@ NTSYSAPI void WINAPI RtlCaptureContext(CONTEXT*); #define PRODUCT_MEDIUMBUSINESS_SERVER_MANAGEMENT 0x0000001E #define PRODUCT_MEDIUMBUSINESS_SERVER_MESSAGING 0x00000020 #define PRODUCT_MEDIUMBUSINESS_SERVER_SECURITY 0x0000001F +#define PRODUCT_PROFESSIONAL 0x00000030 +#define PRODUCT_PROFESSIONAL_N 0x00000031 #define PRODUCT_SERVER_FOR_SMALLBUSINESS 0x00000018 #define PRODUCT_SERVER_FOR_SMALLBUSINESS_V 0x00000023 +#define PRODUCT_SERVER_V 0x00000025 #define PRODUCT_SMALLBUSINESS_SERVER 0x00000009 #define PRODUCT_STANDARD_SERVER 0x00000007 #define PRODUCT_STANDARD_SERVER_CORE 0x0000000D

1 0

Detlef Riekenberg : kernel32/tests: Add tests for GetProductInfo.
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: 64fbf5cac2d1c09e7c4dd25f42c74350f324ab19 URL: http://source.winehq.org/git/wine.git/?a=commit;h=64fbf5cac2d1c09e7c4dd25f4… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Wed May 30 18:40:34 2012 +0200 kernel32/tests: Add tests for GetProductInfo. --- dlls/kernel32/tests/version.c | 100 ++++++++++++++++++++++++++++++++++++++--- 1 files changed, 94 insertions(+), 6 deletions(-) diff --git a/dlls/kernel32/tests/version.c b/dlls/kernel32/tests/version.c index 9f9b72e..bb71cf8 100644 --- a/dlls/kernel32/tests/version.c +++ b/dlls/kernel32/tests/version.c @@ -18,31 +18,118 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +/* Needed for PRODUCT_* defines and GetProductInfo() */ +#define _WIN32_WINNT 0x0600 + #include <assert.h> #include "wine/test.h" #include "winbase.h" +static BOOL (WINAPI * pGetProductInfo)(DWORD, DWORD, DWORD, DWORD, DWORD *); static BOOL (WINAPI * pVerifyVersionInfoA)(LPOSVERSIONINFOEXA, DWORD, DWORDLONG); static ULONGLONG (WINAPI * pVerSetConditionMask)(ULONGLONG, DWORD, BYTE); #define KERNEL32_GET_PROC(func) \ - p##func = (void *)GetProcAddress(hKernel32, #func); \ - if(!p##func) trace("GetProcAddress(hKernel32, '%s') failed\n", #func); + p##func = (void *)GetProcAddress(hKernel32, #func); static void init_function_pointers(void) { HMODULE hKernel32; - pVerifyVersionInfoA = NULL; - pVerSetConditionMask = NULL; - hKernel32 = GetModuleHandleA("kernel32.dll"); - assert(hKernel32); + + KERNEL32_GET_PROC(GetProductInfo); KERNEL32_GET_PROC(VerifyVersionInfoA); KERNEL32_GET_PROC(VerSetConditionMask); } +#define PRODUCT2NAME(x) case x:\ + return #x; + +static const char * product_to_name(DWORD product) +{ + switch (product) + { + PRODUCT2NAME(PRODUCT_PROFESSIONAL) + PRODUCT2NAME(PRODUCT_STANDARD_SERVER) + PRODUCT2NAME(PRODUCT_BUSINESS) + PRODUCT2NAME(PRODUCT_ULTIMATE) + PRODUCT2NAME(PRODUCT_ULTIMATE_N) + PRODUCT2NAME(PRODUCT_HOME_PREMIUM) + } + return ""; +} +#undef PRODUCT2NAME + +static void test_GetProductInfo(void) +{ + OSVERSIONINFOEXA os; + DWORD product; + DWORD res; + DWORD table[] = {9,8,7,6, + 7,0,0,0, + 6,2,0,0, + 6,1,2,0, + 6,1,1,0, + 6,1,0,2, + 6,1,0,0, + 6,0,3,0, + 6,0,2,0, + 6,0,1,5, + 6,0,1,0, + 6,0,0,0, + 5,3,0,0, + 5,2,0,0, + 5,1,0,0, + 5,0,0,0, + 0}; + + DWORD *entry = table; + + if (!pGetProductInfo) + { + /* Not present before Vista */ + win_skip("GetProductInfo() not available\n"); + return; + } + + memset(&os, 0, sizeof(OSVERSIONINFOEXA)); + os.dwOSVersionInfoSize = sizeof(OSVERSIONINFOEXA); + res = GetVersionExA((OSVERSIONINFOA *)&os); + ok(res, "got %d (expected TRUE)\n", res); + + trace("%s %u.%u (SP: %u.%u)\n", os.wProductType == VER_NT_WORKSTATION ? "Workstation" : "Server", + os.dwMajorVersion, os.dwMinorVersion, os.wServicePackMajor, os.wServicePackMinor); + + while (*entry) + { + /* SetLastError() / GetLastError(): value is untouched */ + product = 0xdeadbeef; + SetLastError(0xdeadbeef); + res = pGetProductInfo(entry[0], entry[1], entry[2], entry[3], &product); + trace("%d.%d / %d.%d: got %d and 0x%x %s\n", + entry[0], entry[1], entry[2], entry[3], res, product, product_to_name(product)); + + if (entry[0] >= 6) + ok(res && (product > PRODUCT_UNDEFINED) && (product <= PRODUCT_EMBEDDED) && (GetLastError() == 0xdeadbeef), + "got %d and 0x%x with 0x%x (expected TRUE and a valid PRODUCT_* value with LastError untouched)\n", + res, product, GetLastError()); + else + ok(!res && !product && (GetLastError() == 0xdeadbeef), + "got %d and 0x%x with 0x%x (expected FALSE and PRODUCT_UNDEFINED with LastError untouched)\n", + res, product, GetLastError()); + + entry+= 4; + } + + /* NULL pointer is not a problem */ + SetLastError(0xdeadbeef); + res = pGetProductInfo(6, 1, 0, 0, NULL); + ok( (!res) && (GetLastError() == 0xdeadbeef), + "got %d with 0x%x (expected FALSE with LastError untouched\n", res, GetLastError()); +} + static void test_GetVersionEx(void) { OSVERSIONINFOA infoA; @@ -292,6 +379,7 @@ START_TEST(version) { init_function_pointers(); + test_GetProductInfo(); test_GetVersionEx(); test_VerifyVersionInfo(); }

1 0

Józef Kucia : windowscodecs: Implement IWICStream:: InitializeFromIStream.
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: d770297c37ec46bbff395e156464080f24da6f6f URL: http://source.winehq.org/git/wine.git/?a=commit;h=d770297c37ec46bbff395e156… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Tue May 29 11:29:21 2012 +0200 windowscodecs: Implement IWICStream::InitializeFromIStream. --- dlls/windowscodecs/stream.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/dlls/windowscodecs/stream.c b/dlls/windowscodecs/stream.c index cd235a3..a123b9d 100644 --- a/dlls/windowscodecs/stream.c +++ b/dlls/windowscodecs/stream.c @@ -735,8 +735,13 @@ static HRESULT WINAPI IWICStreamImpl_Clone(IWICStream *iface, static HRESULT WINAPI IWICStreamImpl_InitializeFromIStream(IWICStream *iface, IStream *pIStream) { - FIXME("(%p): stub\n", iface); - return E_NOTIMPL; + ULARGE_INTEGER offset, size; + TRACE("(%p): relay\n", iface); + + offset.QuadPart = 0; + size.u.LowPart = 0xffffffff; + size.u.HighPart = 0xffffffff; + return IWICStream_InitializeFromIStreamRegion(iface, pIStream, offset, size); } static HRESULT WINAPI IWICStreamImpl_InitializeFromFilename(IWICStream *iface,

1 0

Józef Kucia : d3dx9: Implement D3DXSaveSurfaceToFileInMemory.
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: 70b1bff69520c2ba82bb54b12338a76664c9e60a URL: http://source.winehq.org/git/wine.git/?a=commit;h=70b1bff69520c2ba82bb54b12… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Tue May 29 11:29:22 2012 +0200 d3dx9: Implement D3DXSaveSurfaceToFileInMemory. --- dlls/d3dx9_36/d3dx9_36.spec | 2 +- dlls/d3dx9_36/d3dx9_36_private.h | 2 + dlls/d3dx9_36/surface.c | 96 +++++++++++++++++++++++++++++--------- dlls/d3dx9_36/tests/surface.c | 27 +++++++++++ dlls/d3dx9_36/util.c | 19 ++++++++ include/d3dx9tex.h | 6 ++ 6 files changed, 128 insertions(+), 24 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=70b1bff69520c2ba82bb5…

1 0

Józef Kucia : d3dx9: Forward D3DXSaveTextureToFile to D3DXSaveTextureToFileInMemory.
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: dfc5ab140c3211d73239352253ee0f80a960dfd7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dfc5ab140c3211d7323935225… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Tue May 29 11:29:23 2012 +0200 d3dx9: Forward D3DXSaveTextureToFile to D3DXSaveTextureToFileInMemory. --- dlls/d3dx9_36/d3dx9_36.spec | 6 ++-- dlls/d3dx9_36/texture.c | 61 +++++++++++++++++++++++++++++++++++++++++++ include/d3dx9tex.h | 5 +++ 3 files changed, 69 insertions(+), 3 deletions(-) diff --git a/dlls/d3dx9_36/d3dx9_36.spec b/dlls/d3dx9_36/d3dx9_36.spec index 44efaf5..aec2a5c 100644 --- a/dlls/d3dx9_36/d3dx9_36.spec +++ b/dlls/d3dx9_36/d3dx9_36.spec @@ -266,9 +266,9 @@ @ stdcall D3DXSaveSurfaceToFileA(ptr long ptr ptr ptr) @ stdcall D3DXSaveSurfaceToFileInMemory(ptr long ptr ptr ptr) @ stdcall D3DXSaveSurfaceToFileW(ptr long ptr ptr ptr) -@ stub D3DXSaveTextureToFileA(ptr long ptr ptr) -@ stub D3DXSaveTextureToFileInMemory(ptr long ptr ptr) -@ stub D3DXSaveTextureToFileW(ptr long ptr ptr) +@ stdcall D3DXSaveTextureToFileA(ptr long ptr ptr) +@ stdcall D3DXSaveTextureToFileInMemory(ptr long ptr ptr) +@ stdcall D3DXSaveTextureToFileW(ptr long ptr ptr) @ stub D3DXSaveVolumeToFileA(ptr long ptr ptr ptr) @ stub D3DXSaveVolumeToFileInMemory(ptr long ptr ptr ptr) @ stub D3DXSaveVolumeToFileW(ptr long ptr ptr ptr) diff --git a/dlls/d3dx9_36/texture.c b/dlls/d3dx9_36/texture.c index c17a327..97d26d9 100644 --- a/dlls/d3dx9_36/texture.c +++ b/dlls/d3dx9_36/texture.c @@ -1606,3 +1606,64 @@ HRESULT WINAPI D3DXFillVolumeTexture(LPDIRECT3DVOLUMETEXTURE9 texture, return D3D_OK; } + +HRESULT WINAPI D3DXSaveTextureToFileA(const char *dst_filename, D3DXIMAGE_FILEFORMAT file_format, + IDirect3DBaseTexture9 *src_texture, const PALETTEENTRY *src_palette) +{ + int len; + WCHAR *filename; + HRESULT hr; + ID3DXBuffer *buffer; + + TRACE("(%s, %#x, %p, %p): relay\n", + wine_dbgstr_a(dst_filename), file_format, src_texture, src_palette); + + if (!dst_filename) return D3DERR_INVALIDCALL; + + len = MultiByteToWideChar(CP_ACP, 0, dst_filename, -1, NULL, 0); + filename = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (!filename) return E_OUTOFMEMORY; + MultiByteToWideChar(CP_ACP, 0, dst_filename, -1, filename, len); + + hr = D3DXSaveTextureToFileInMemory(&buffer, file_format, src_texture, src_palette); + if (SUCCEEDED(hr)) + { + hr = write_buffer_to_file(filename, buffer); + ID3DXBuffer_Release(buffer); + } + + HeapFree(GetProcessHeap(), 0, filename); + return hr; +} + +HRESULT WINAPI D3DXSaveTextureToFileW(const WCHAR *dst_filename, D3DXIMAGE_FILEFORMAT file_format, + IDirect3DBaseTexture9 *src_texture, const PALETTEENTRY *src_palette) +{ + HRESULT hr; + ID3DXBuffer *buffer; + + TRACE("(%s, %#x, %p, %p): relay\n", + wine_dbgstr_w(dst_filename), file_format, src_texture, src_palette); + + if (!dst_filename) return D3DERR_INVALIDCALL; + + hr = D3DXSaveTextureToFileInMemory(&buffer, file_format, src_texture, src_palette); + if (SUCCEEDED(hr)) + { + hr = write_buffer_to_file(dst_filename, buffer); + ID3DXBuffer_Release(buffer); + } + + return hr; +} + +HRESULT WINAPI D3DXSaveTextureToFileInMemory(ID3DXBuffer **dst_buffer, D3DXIMAGE_FILEFORMAT file_format, + IDirect3DBaseTexture9 *src_texture, const PALETTEENTRY *src_palette) +{ + FIXME("(%p, %#x, %p, %p): stub\n", + dst_buffer, file_format, src_texture, src_palette); + + if (!dst_buffer || !src_texture) return D3DERR_INVALIDCALL; + + return E_NOTIMPL; +} diff --git a/include/d3dx9tex.h b/include/d3dx9tex.h index 0e89021..38441ce 100644 --- a/include/d3dx9tex.h +++ b/include/d3dx9tex.h @@ -638,6 +638,11 @@ HRESULT WINAPI D3DXCreateVolumeTextureFromFileInMemoryEx(LPDIRECT3DDEVICE9 devic PALETTEENTRY *palette, LPDIRECT3DVOLUMETEXTURE9 *volume); +HRESULT WINAPI D3DXSaveTextureToFileInMemory(LPD3DXBUFFER *destbuffer, + D3DXIMAGE_FILEFORMAT destformat, + LPDIRECT3DBASETEXTURE9 srctexture, + CONST PALETTEENTRY *srcpalette); + HRESULT WINAPI D3DXSaveTextureToFileA(LPCSTR destfile, D3DXIMAGE_FILEFORMAT destformat, LPDIRECT3DBASETEXTURE9 srctexture,

1 0

Józef Kucia : d3dx9: Implement D3DXSaveTextureToFileInMemory.
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: 38b8949fea926a0afce627537f29df413397f34f URL: http://source.winehq.org/git/wine.git/?a=commit;h=38b8949fea926a0afce627537… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Tue May 29 11:29:24 2012 +0200 d3dx9: Implement D3DXSaveTextureToFileInMemory. --- dlls/d3dx9_36/texture.c | 34 ++++++++++++++++++++++++++++++++-- 1 files changed, 32 insertions(+), 2 deletions(-) diff --git a/dlls/d3dx9_36/texture.c b/dlls/d3dx9_36/texture.c index 97d26d9..1e87a00 100644 --- a/dlls/d3dx9_36/texture.c +++ b/dlls/d3dx9_36/texture.c @@ -1660,10 +1660,40 @@ HRESULT WINAPI D3DXSaveTextureToFileW(const WCHAR *dst_filename, D3DXIMAGE_FILEF HRESULT WINAPI D3DXSaveTextureToFileInMemory(ID3DXBuffer **dst_buffer, D3DXIMAGE_FILEFORMAT file_format, IDirect3DBaseTexture9 *src_texture, const PALETTEENTRY *src_palette) { - FIXME("(%p, %#x, %p, %p): stub\n", + HRESULT hr; + D3DRESOURCETYPE type; + IDirect3DSurface9 *surface; + + TRACE("(%p, %#x, %p, %p)\n", dst_buffer, file_format, src_texture, src_palette); if (!dst_buffer || !src_texture) return D3DERR_INVALIDCALL; - return E_NOTIMPL; + if (file_format == D3DXIFF_DDS) + { + FIXME("DDS file format isn't supported yet\n"); + return E_NOTIMPL; + } + + type = IDirect3DBaseTexture9_GetType(src_texture); + switch (type) + { + case D3DRTYPE_TEXTURE: + case D3DRTYPE_CUBETEXTURE: + hr = get_surface(type, src_texture, D3DCUBEMAP_FACE_POSITIVE_X, 0, &surface); + break; + case D3DRTYPE_VOLUMETEXTURE: + FIXME("Volume textures aren't supported yet\n"); + return E_NOTIMPL; + default: + return D3DERR_INVALIDCALL; + } + + if (SUCCEEDED(hr)) + { + hr = D3DXSaveSurfaceToFileInMemory(dst_buffer, file_format, surface, src_palette, NULL); + IDirect3DSurface9_Release(surface); + } + + return hr; }

1 0

Józef Kucia : d3dx9/tests: Add tests for D3DXSaveTextureToFileInMemory.
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: 13b2a3b9bca935e1d0d8896fb9a735a46becbbde URL: http://source.winehq.org/git/wine.git/?a=commit;h=13b2a3b9bca935e1d0d8896fb… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Tue May 29 11:29:25 2012 +0200 d3dx9/tests: Add tests for D3DXSaveTextureToFileInMemory. --- dlls/d3dx9_36/tests/texture.c | 191 +++++++++++++++++++++++++++++++++++++++++ 1 files changed, 191 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=13b2a3b9bca935e1d0d88…

1 0

Alexandre Julliard : gdi32: Don' t implement the gamma ramp functions in the DIB driver.
by Alexandre Julliard 31 May '12

31 May '12

Module: wine Branch: master Commit: 61f4767adeee0b02786faeec4a2fb583f3439154 URL: http://source.winehq.org/git/wine.git/?a=commit;h=61f4767adeee0b02786faeec4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu May 31 13:44:32 2012 +0200 gdi32: Don't implement the gamma ramp functions in the DIB driver. --- dlls/gdi32/dc.c | 16 ++++++++++++---- dlls/gdi32/dibdrv/dc.c | 22 ++-------------------- 2 files changed, 14 insertions(+), 24 deletions(-) diff --git a/dlls/gdi32/dc.c b/dlls/gdi32/dc.c index 3caa120..e128e53 100644 --- a/dlls/gdi32/dc.c +++ b/dlls/gdi32/dc.c @@ -1245,8 +1245,12 @@ BOOL WINAPI GetDeviceGammaRamp(HDC hDC, LPVOID ptr) TRACE("%p, %p\n", hDC, ptr); if( dc ) { - PHYSDEV physdev = GET_DC_PHYSDEV( dc, pGetDeviceGammaRamp ); - ret = physdev->funcs->pGetDeviceGammaRamp( physdev, ptr ); + if (GetObjectType( hDC ) != OBJ_MEMDC) + { + PHYSDEV physdev = GET_DC_PHYSDEV( dc, pGetDeviceGammaRamp ); + ret = physdev->funcs->pGetDeviceGammaRamp( physdev, ptr ); + } + else SetLastError( ERROR_INVALID_PARAMETER ); release_dc_ptr( dc ); } return ret; @@ -1263,8 +1267,12 @@ BOOL WINAPI SetDeviceGammaRamp(HDC hDC, LPVOID ptr) TRACE("%p, %p\n", hDC, ptr); if( dc ) { - PHYSDEV physdev = GET_DC_PHYSDEV( dc, pSetDeviceGammaRamp ); - ret = physdev->funcs->pSetDeviceGammaRamp( physdev, ptr ); + if (GetObjectType( hDC ) != OBJ_MEMDC) + { + PHYSDEV physdev = GET_DC_PHYSDEV( dc, pSetDeviceGammaRamp ); + ret = physdev->funcs->pSetDeviceGammaRamp( physdev, ptr ); + } + else SetLastError( ERROR_INVALID_PARAMETER ); release_dc_ptr( dc ); } return ret; diff --git a/dlls/gdi32/dibdrv/dc.c b/dlls/gdi32/dibdrv/dc.c index 5c59041..798a9cd 100644 --- a/dlls/gdi32/dibdrv/dc.c +++ b/dlls/gdi32/dibdrv/dc.c @@ -392,24 +392,6 @@ static UINT dibdrv_SetBoundsRect( PHYSDEV dev, RECT *rect, UINT flags ) return DCB_RESET; /* we don't have device-specific bounds */ } -/*********************************************************************** - * dibdrv_GetDeviceGammaRamp - */ -static BOOL dibdrv_GetDeviceGammaRamp( PHYSDEV dev, void *ramp ) -{ - SetLastError( ERROR_INVALID_PARAMETER ); - return FALSE; -} - -/*********************************************************************** - * dibdrv_SetDeviceGammaRamp - */ -static BOOL dibdrv_SetDeviceGammaRamp( PHYSDEV dev, void *ramp ) -{ - SetLastError( ERROR_INVALID_PARAMETER ); - return FALSE; -} - const struct gdi_dc_funcs dib_driver = { NULL, /* pAbortDoc */ @@ -453,7 +435,7 @@ const struct gdi_dc_funcs dib_driver = NULL, /* pGetCharABCWidthsI */ NULL, /* pGetCharWidth */ NULL, /* pGetDeviceCaps */ - dibdrv_GetDeviceGammaRamp, /* pGetDeviceGammaRamp */ + NULL, /* pGetDeviceGammaRamp */ NULL, /* pGetFontData */ NULL, /* pGetFontUnicodeRanges */ NULL, /* pGetGlyphIndices */ @@ -515,7 +497,7 @@ const struct gdi_dc_funcs dib_driver = dibdrv_SetDCPenColor, /* pSetDCPenColor */ NULL, /* pSetDIBitsToDevice */ dibdrv_SetDeviceClipping, /* pSetDeviceClipping */ - dibdrv_SetDeviceGammaRamp, /* pSetDeviceGammaRamp */ + NULL, /* pSetDeviceGammaRamp */ NULL, /* pSetLayout */ NULL, /* pSetMapMode */ NULL, /* pSetMapperFlags */

1 0