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

Vincent Povirk : windowscodecs: Add wrapper functions for IWICStream methods.
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: 73a4309234f4b2beedf866fb0dc12d66fa1fbabd URL: http://source.winehq.org/git/wine.git/?a=commit;h=73a4309234f4b2beedf866fb0… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Tue May 8 12:50:58 2012 -0500 windowscodecs: Add wrapper functions for IWICStream methods. --- dlls/windowscodecs/proxy.c | 12 ++++++++++++ dlls/windowscodecs/windowscodecs.spec | 4 ++-- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/dlls/windowscodecs/proxy.c b/dlls/windowscodecs/proxy.c index 1b298db..216bf19 100644 --- a/dlls/windowscodecs/proxy.c +++ b/dlls/windowscodecs/proxy.c @@ -608,6 +608,18 @@ HRESULT WINAPI IWICPalette_HasAlpha_Proxy_W(IWICPalette *iface, return IWICPalette_HasAlpha(iface, pfHasAlpha); } +HRESULT WINAPI IWICStream_InitializeFromIStream_Proxy_W(IWICStream *iface, + IStream *pIStream) +{ + return IWICStream_InitializeFromIStream(iface, pIStream); +} + +HRESULT WINAPI IWICStream_InitializeFromMemory_Proxy_W(IWICStream *iface, + BYTE *pbBuffer, DWORD cbBufferSize) +{ + return IWICStream_InitializeFromMemory(iface, pbBuffer, cbBufferSize); +} + HRESULT WINAPI WICCreateImagingFactory_Proxy(UINT SDKVersion, IWICImagingFactory **ppIImagingFactory) { TRACE("%x, %p\n", SDKVersion, ppIImagingFactory); diff --git a/dlls/windowscodecs/windowscodecs.spec b/dlls/windowscodecs/windowscodecs.spec index bd5b94c..453a84e 100644 --- a/dlls/windowscodecs/windowscodecs.spec +++ b/dlls/windowscodecs/windowscodecs.spec @@ -102,8 +102,8 @@ @ stub IWICPixelFormatInfo_GetBitsPerPixel_Proxy @ stub IWICPixelFormatInfo_GetChannelCount_Proxy @ stub IWICPixelFormatInfo_GetChannelMask_Proxy -@ stub IWICStream_InitializeFromIStream_Proxy -@ stub IWICStream_InitializeFromMemory_Proxy +@ stdcall IWICStream_InitializeFromIStream_Proxy(ptr ptr) IWICStream_InitializeFromIStream_Proxy_W +@ stdcall IWICStream_InitializeFromMemory_Proxy(ptr ptr long) IWICStream_InitializeFromMemory_Proxy_W @ stdcall WICConvertBitmapSource(ptr ptr ptr) @ stub WICCreateBitmapFromSection @ stub WICCreateColorContext_Proxy

1 0

Huw Davies : localspl: Calling AddPrinterDriver on an existing driver should succeed.
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: 25bb808cf104ee853173aefa6e1e685f62d28e5a URL: http://source.winehq.org/git/wine.git/?a=commit;h=25bb808cf104ee853173aefa6… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu May 17 13:12:03 2012 +0100 localspl: Calling AddPrinterDriver on an existing driver should succeed. --- dlls/localspl/provider.c | 7 ------- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/dlls/localspl/provider.c b/dlls/localspl/provider.c index 392b8ed..a599113 100644 --- a/dlls/localspl/provider.c +++ b/dlls/localspl/provider.c @@ -1361,13 +1361,6 @@ static BOOL myAddPrinterDriverEx(DWORD level, LPBYTE pDriverInfo, DWORD dwFileCo } RegCloseKey(hroot); - if (disposition == REG_OPENED_EXISTING_KEY) { - TRACE("driver %s already installed\n", debugstr_w(di.pName)); - RegCloseKey(hdrv); - SetLastError(ERROR_PRINTER_DRIVER_ALREADY_INSTALLED); - return FALSE; - } - /* Verified with the Adobe PS Driver, that w2k does not use di.Version */ RegSetValueExW(hdrv, versionW, 0, REG_DWORD, (const BYTE*) &env->driverversion, sizeof(DWORD));

1 0

Huw Davies : wineps: Fix top-down dib pattern brushes.
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: c5697ff4b106cb08ea74ce6f7f94651fdf6ddc5b URL: http://source.winehq.org/git/wine.git/?a=commit;h=c5697ff4b106cb08ea74ce6f7… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu May 17 14:42:12 2012 +0100 wineps: Fix top-down dib pattern brushes. --- dlls/wineps.drv/ps.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/wineps.drv/ps.c b/dlls/wineps.drv/ps.c index 4a391cc..2e7597a 100644 --- a/dlls/wineps.drv/ps.c +++ b/dlls/wineps.drv/ps.c @@ -20,6 +20,7 @@ #include <ctype.h> #include <stdio.h> +#include <stdlib.h> #include <string.h> #include <stdarg.h> #include <locale.h> @@ -889,7 +890,7 @@ BOOL PSDRV_WriteDIBPatternDict(PHYSDEV dev, const BITMAPINFO *bmi, BYTE *bits, U " %d %d scale\n mypat image\n end\n }\n>>\n matrix makepattern setpattern\n"; PSDRV_PDEVICE *physDev = get_psdrv_dev( dev ); char *buf, *ptr; - INT w, h, x, y, w_mult, h_mult; + INT w, h, x, y, w_mult, h_mult, abs_height = abs( bmi->bmiHeader.biHeight ); COLORREF map[2]; TRACE( "size %dx%dx%d\n", @@ -901,7 +902,7 @@ BOOL PSDRV_WriteDIBPatternDict(PHYSDEV dev, const BITMAPINFO *bmi, BYTE *bits, U } w = bmi->bmiHeader.biWidth & ~0x7; - h = bmi->bmiHeader.biHeight & ~0x7; + h = abs_height & ~0x7; buf = HeapAlloc( GetProcessHeap(), 0, sizeof(do_pattern) + 100 ); ptr = buf; @@ -913,7 +914,7 @@ BOOL PSDRV_WriteDIBPatternDict(PHYSDEV dev, const BITMAPINFO *bmi, BYTE *bits, U } } PSDRV_WriteSpool(dev, mypat, sizeof(mypat) - 1); - PSDRV_WriteImageDict(dev, 1, 8, 8, buf, FALSE); + PSDRV_WriteImageDict(dev, 1, 8, 8, buf, bmi->bmiHeader.biHeight < 0); PSDRV_WriteSpool(dev, "def\n", 4); PSDRV_WriteIndexColorSpaceBegin(dev, 1);

1 0

Dmitry Timoshkov : gdiplus: Save total and current frame in the image structure.
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: 007d642e53b4ddc4ea75aba26fc35936b67fbc3f URL: http://source.winehq.org/git/wine.git/?a=commit;h=007d642e53b4ddc4ea75aba26… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Wed May 16 23:33:48 2012 +0900 gdiplus: Save total and current frame in the image structure. --- dlls/gdiplus/gdiplus_private.h | 1 + dlls/gdiplus/image.c | 12 +++++++++++- 2 files changed, 12 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h index df66903..753a162 100644 --- a/dlls/gdiplus/gdiplus_private.h +++ b/dlls/gdiplus/gdiplus_private.h @@ -266,6 +266,7 @@ struct GpImage{ ImageType type; GUID format; UINT flags; + UINT frame_count, current_frame; UINT palette_flags; UINT palette_count; UINT palette_size; diff --git a/dlls/gdiplus/image.c b/dlls/gdiplus/image.c index ba8b979..c6a72b4 100644 --- a/dlls/gdiplus/image.c +++ b/dlls/gdiplus/image.c @@ -1767,6 +1767,8 @@ GpStatus WINGDIPAPI GdipCreateBitmapFromScan0(INT width, INT height, INT stride, (*bitmap)->image.type = ImageTypeBitmap; memcpy(&(*bitmap)->image.format, &ImageFormatMemoryBMP, sizeof(GUID)); (*bitmap)->image.flags = ImageFlagsNone; + (*bitmap)->image.frame_count = 1; + (*bitmap)->image.current_frame = 0; (*bitmap)->image.palette_flags = 0; (*bitmap)->image.palette_count = 0; (*bitmap)->image.palette_size = 0; @@ -2591,7 +2593,7 @@ static GpStatus decode_image_wic(IStream* stream, REFCLSID clsid, GpImage **imag WICPixelFormatGUID wic_format; PixelFormat gdip_format=0; int i; - UINT width, height; + UINT width, height, frame_count; BitmapData lockeddata; WICRect wrc; HRESULT initresult; @@ -2604,7 +2606,11 @@ static GpStatus decode_image_wic(IStream* stream, REFCLSID clsid, GpImage **imag hr = IWICBitmapDecoder_Initialize(decoder, (IStream*)stream, WICDecodeMetadataCacheOnLoad); if (SUCCEEDED(hr)) + { + IWICBitmapDecoder_GetFrameCount(decoder, &frame_count); + /* FIXME: set current frame */ hr = IWICBitmapDecoder_GetFrame(decoder, 0, &frame); + } if (SUCCEEDED(hr)) /* got frame */ { @@ -2696,6 +2702,8 @@ end: { /* Native GDI+ used to be smarter, but since Win7 it just sets these flags. */ bitmap->image.flags |= ImageFlagsReadOnly|ImageFlagsHasRealPixelSize|ImageFlagsHasRealDPI|ImageFlagsColorSpaceRGB; + bitmap->image.frame_count = frame_count; + bitmap->image.current_frame = 0; } return status; @@ -2765,6 +2773,8 @@ static GpStatus decode_image_olepicture_metafile(IStream* stream, REFCLSID clsid (*image)->type = ImageTypeMetafile; (*image)->picture = pic; (*image)->flags = ImageFlagsNone; + (*image)->frame_count = 1; + (*image)->current_frame = 0; (*image)->palette_flags = 0; (*image)->palette_count = 0; (*image)->palette_size = 0;

1 0

Dmitry Timoshkov : gdiplus: Implement GdipImageGetFrameCount.
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: eeee536ad6027086302b18bfc628c00c1894fccb URL: http://source.winehq.org/git/wine.git/?a=commit;h=eeee536ad6027086302b18bfc… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Wed May 16 23:33:55 2012 +0900 gdiplus: Implement GdipImageGetFrameCount. --- dlls/gdiplus/image.c | 17 +++++++++-------- dlls/gdiplus/tests/image.c | 2 +- 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/dlls/gdiplus/image.c b/dlls/gdiplus/image.c index c6a72b4..ce04c72 100644 --- a/dlls/gdiplus/image.c +++ b/dlls/gdiplus/image.c @@ -2456,23 +2456,24 @@ static struct image_format_dimension image_format_dimensions[] = {NULL} }; -/* FIXME: Need to handle multi-framed images */ GpStatus WINGDIPAPI GdipImageGetFrameCount(GpImage *image, GDIPCONST GUID* dimensionID, UINT* count) { - static int calls; - TRACE("(%p,%s,%p)\n", image, debugstr_guid(dimensionID), count); if(!image || !count) return InvalidParameter; - if(!(calls++)) - FIXME("returning frame count of 1\n"); - - *count = 1; + if (!dimensionID || + IsEqualGUID(dimensionID, &image->format) || + IsEqualGUID(dimensionID, &FrameDimensionPage) || + IsEqualGUID(dimensionID, &FrameDimensionTime)) + { + *count = image->frame_count; + return Ok; + } - return Ok; + return InvalidParameter; } GpStatus WINGDIPAPI GdipImageGetFrameDimensionsCount(GpImage *image, diff --git a/dlls/gdiplus/tests/image.c b/dlls/gdiplus/tests/image.c index b6262d1..e22d45a 100644 --- a/dlls/gdiplus/tests/image.c +++ b/dlls/gdiplus/tests/image.c @@ -2298,7 +2298,7 @@ static void test_multiframegif(void) count = 12345; stat = GdipImageGetFrameCount((GpImage*)bmp, &dimension, &count); expect(Ok, stat); - todo_wine expect(2, count); + expect(2, count); /* SelectActiveFrame overwrites our current data */ stat = GdipImageSelectActiveFrame((GpImage*)bmp, &dimension, 1);

1 0

Alex Henrie : kernel32: Avoid calling MultiByteToWideChar with invalid parameters.
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: 92b4979b3312f0d3a308c2c0a259fa0153612019 URL: http://source.winehq.org/git/wine.git/?a=commit;h=92b4979b3312f0d3a308c2c0a… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed May 16 19:24:07 2012 -0600 kernel32: Avoid calling MultiByteToWideChar with invalid parameters. --- dlls/kernel32/locale.c | 57 +++++++++++++++++++++++++++++++---------------- 1 files changed, 37 insertions(+), 20 deletions(-) diff --git a/dlls/kernel32/locale.c b/dlls/kernel32/locale.c index b506f15..380c616 100644 --- a/dlls/kernel32/locale.c +++ b/dlls/kernel32/locale.c @@ -2867,34 +2867,51 @@ INT WINAPI CompareStringA(LCID lcid, DWORD style, if (!(style & LOCALE_USE_CP_ACP)) locale_cp = get_lcid_codepage( lcid ); - len1W = MultiByteToWideChar(locale_cp, 0, str1, len1, buf1W, 130); - if (len1W) - str1W = buf1W; - else + if (len1) { - len1W = MultiByteToWideChar(locale_cp, 0, str1, len1, NULL, 0); - str1W = HeapAlloc(GetProcessHeap(), 0, len1W * sizeof(WCHAR)); - if (!str1W) + len1W = MultiByteToWideChar(locale_cp, 0, str1, len1, buf1W, 130); + if (len1W) + str1W = buf1W; + else { - SetLastError(ERROR_NOT_ENOUGH_MEMORY); - return 0; + len1W = MultiByteToWideChar(locale_cp, 0, str1, len1, NULL, 0); + str1W = HeapAlloc(GetProcessHeap(), 0, len1W * sizeof(WCHAR)); + if (!str1W) + { + SetLastError(ERROR_NOT_ENOUGH_MEMORY); + return 0; + } + MultiByteToWideChar(locale_cp, 0, str1, len1, str1W, len1W); } - MultiByteToWideChar(locale_cp, 0, str1, len1, str1W, len1W); } - len2W = MultiByteToWideChar(locale_cp, 0, str2, len2, buf2W, 130); - if (len2W) - str2W = buf2W; else { - len2W = MultiByteToWideChar(locale_cp, 0, str2, len2, NULL, 0); - str2W = HeapAlloc(GetProcessHeap(), 0, len2W * sizeof(WCHAR)); - if (!str2W) + len1W = 0; + str1W = buf1W; + } + + if (len2) + { + len2W = MultiByteToWideChar(locale_cp, 0, str2, len2, buf2W, 130); + if (len2W) + str2W = buf2W; + else { - if (str1W != buf1W) HeapFree(GetProcessHeap(), 0, str1W); - SetLastError(ERROR_NOT_ENOUGH_MEMORY); - return 0; + len2W = MultiByteToWideChar(locale_cp, 0, str2, len2, NULL, 0); + str2W = HeapAlloc(GetProcessHeap(), 0, len2W * sizeof(WCHAR)); + if (!str2W) + { + if (str1W != buf1W) HeapFree(GetProcessHeap(), 0, str1W); + SetLastError(ERROR_NOT_ENOUGH_MEMORY); + return 0; + } + MultiByteToWideChar(locale_cp, 0, str2, len2, str2W, len2W); } - MultiByteToWideChar(locale_cp, 0, str2, len2, str2W, len2W); + } + else + { + len2W = 0; + str2W = buf2W; } ret = CompareStringW(lcid, style, str1W, len1W, str2W, len2W);

1 0

Alex Henrie : kernel32: Correct WideCharToMultiByte and MultiByteToWideChar error codes and conditions .
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: 8ccf747d6a7c05f64559949705548e2158b85eea URL: http://source.winehq.org/git/wine.git/?a=commit;h=8ccf747d6a7c05f6455994970… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed May 16 19:24:03 2012 -0600 kernel32: Correct WideCharToMultiByte and MultiByteToWideChar error codes and conditions. --- dlls/kernel32/locale.c | 38 ++++++++++++++++++--- dlls/kernel32/tests/codepage.c | 69 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 101 insertions(+), 6 deletions(-) diff --git a/dlls/kernel32/locale.c b/dlls/kernel32/locale.c index 380c616..2970856 100644 --- a/dlls/kernel32/locale.c +++ b/dlls/kernel32/locale.c @@ -1878,7 +1878,7 @@ INT WINAPI MultiByteToWideChar( UINT page, DWORD flags, LPCSTR src, INT srclen, const union cptable *table; int ret; - if (!src || (!dst && dstlen)) + if (!src || !srclen || (!dst && dstlen)) { SetLastError( ERROR_INVALID_PARAMETER ); return 0; @@ -1889,14 +1889,19 @@ INT WINAPI MultiByteToWideChar( UINT page, DWORD flags, LPCSTR src, INT srclen, switch(page) { case CP_SYMBOL: - if( flags) + if (flags) { - SetLastError( ERROR_INVALID_PARAMETER ); + SetLastError( ERROR_INVALID_FLAGS ); return 0; } ret = wine_cpsymbol_mbstowcs( src, srclen, dst, dstlen ); break; case CP_UTF7: + if (flags) + { + SetLastError( ERROR_INVALID_FLAGS ); + return 0; + } FIXME("UTF-7 not supported\n"); SetLastError( ERROR_CALL_NOT_IMPLEMENTED ); return 0; @@ -1969,7 +1974,7 @@ INT WINAPI WideCharToMultiByte( UINT page, DWORD flags, LPCWSTR src, INT srclen, const union cptable *table; int ret, used_tmp; - if (!src || (!dst && dstlen)) + if (!src || !srclen || (!dst && dstlen)) { SetLastError( ERROR_INVALID_PARAMETER ); return 0; @@ -1980,7 +1985,13 @@ INT WINAPI WideCharToMultiByte( UINT page, DWORD flags, LPCWSTR src, INT srclen, switch(page) { case CP_SYMBOL: - if( flags || defchar || used) + /* when using CP_SYMBOL, ERROR_INVALID_FLAGS takes precedence */ + if (flags) + { + SetLastError( ERROR_INVALID_FLAGS ); + return 0; + } + if (defchar || used) { SetLastError( ERROR_INVALID_PARAMETER ); return 0; @@ -1988,6 +1999,17 @@ INT WINAPI WideCharToMultiByte( UINT page, DWORD flags, LPCWSTR src, INT srclen, ret = wine_cpsymbol_wcstombs( src, srclen, dst, dstlen ); break; case CP_UTF7: + /* when using CP_UTF7, ERROR_INVALID_PARAMETER takes precedence */ + if (defchar || used) + { + SetLastError( ERROR_INVALID_PARAMETER ); + return 0; + } + if (flags) + { + SetLastError( ERROR_INVALID_FLAGS ); + return 0; + } FIXME("UTF-7 not supported\n"); SetLastError( ERROR_CALL_NOT_IMPLEMENTED ); return 0; @@ -2000,7 +2022,11 @@ INT WINAPI WideCharToMultiByte( UINT page, DWORD flags, LPCWSTR src, INT srclen, } /* fall through */ case CP_UTF8: - if (used) *used = FALSE; /* all chars are valid for UTF-8 */ + if (defchar || used) + { + SetLastError( ERROR_INVALID_PARAMETER ); + return 0; + } ret = wine_utf8_wcstombs( flags, src, srclen, dst, dstlen ); break; default: diff --git a/dlls/kernel32/tests/codepage.c b/dlls/kernel32/tests/codepage.c index 61bcf54..c980348 100644 --- a/dlls/kernel32/tests/codepage.c +++ b/dlls/kernel32/tests/codepage.c @@ -189,6 +189,74 @@ static void test_negative_dest_length(void) } +static void test_other_invalid_parameters(void) +{ + char c_string[] = "Hello World"; + size_t c_string_len = sizeof(c_string); + WCHAR w_string[] = {'H','e','l','l','o',' ','W','o','r','l','d',0}; + size_t w_string_len = sizeof(w_string) / sizeof(WCHAR); + BOOL used; + INT len; + + /* srclen=0 => ERROR_INVALID_PARAMETER */ + SetLastError(0xdeadbeef); + len = WideCharToMultiByte(CP_ACP, 0, w_string, 0, c_string, c_string_len, NULL, NULL); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + SetLastError(0xdeadbeef); + len = MultiByteToWideChar(CP_ACP, 0, c_string, 0, w_string, w_string_len); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + + /* dst=NULL but dstlen not 0 => ERROR_INVALID_PARAMETER */ + SetLastError(0xdeadbeef); + len = WideCharToMultiByte(CP_ACP, 0, w_string, w_string_len, NULL, c_string_len, NULL, NULL); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + SetLastError(0xdeadbeef); + len = MultiByteToWideChar(CP_ACP, 0, c_string, c_string_len, NULL, w_string_len); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + + /* CP_UTF7, CP_UTF8, or CP_SYMBOL and defchar not NULL => ERROR_INVALID_PARAMETER */ + /* CP_SYMBOL's behavior here is undocumented */ + SetLastError(0xdeadbeef); + len = WideCharToMultiByte(CP_UTF7, 0, w_string, w_string_len, c_string, c_string_len, c_string, NULL); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + SetLastError(0xdeadbeef); + len = WideCharToMultiByte(CP_UTF8, 0, w_string, w_string_len, c_string, c_string_len, c_string, NULL); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + SetLastError(0xdeadbeef); + len = WideCharToMultiByte(CP_SYMBOL, 0, w_string, w_string_len, c_string, c_string_len, c_string, NULL); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + + /* CP_UTF7, CP_UTF8, or CP_SYMBOL and used not NULL => ERROR_INVALID_PARAMETER */ + /* CP_SYMBOL's behavior here is undocumented */ + SetLastError(0xdeadbeef); + len = WideCharToMultiByte(CP_UTF7, 0, w_string, w_string_len, c_string, c_string_len, NULL, &used); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + SetLastError(0xdeadbeef); + len = WideCharToMultiByte(CP_UTF8, 0, w_string, w_string_len, c_string, c_string_len, NULL, &used); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + SetLastError(0xdeadbeef); + len = WideCharToMultiByte(CP_SYMBOL, 0, w_string, w_string_len, c_string, c_string_len, NULL, &used); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); + + + /* CP_UTF7, flags not 0 and used not NULL => ERROR_INVALID_PARAMETER */ + /* (tests precedence of ERROR_INVALID_PARAMETER over ERROR_INVALID_FLAGS) */ + /* The same test with CP_SYMBOL instead of CP_UTF7 gives ERROR_INVALID_FLAGS + instead except on Windows NT4 */ + SetLastError(0xdeadbeef); + len = WideCharToMultiByte(CP_UTF7, 1, w_string, w_string_len, c_string, c_string_len, NULL, &used); + ok(len == 0 && GetLastError() == ERROR_INVALID_PARAMETER, "len=%d error=%x\n", len, GetLastError()); +} + static void test_overlapped_buffers(void) { static const WCHAR strW[] = {'j','u','s','t',' ','a',' ','t','e','s','t',0}; @@ -402,6 +470,7 @@ START_TEST(codepage) test_null_source(); test_negative_source_length(); test_negative_dest_length(); + test_other_invalid_parameters(); test_overlapped_buffers(); /* WideCharToMultiByte has two code paths, test both here */

1 0

Józef Kucia : d3dx9/tests: Do not release a cube texture if it wasn't created.
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: 5ab8c2fdf8af0589bc9b2ffed152ebd6e0d0a5a9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5ab8c2fdf8af0589bc9b2ffed… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Thu May 17 16:51:22 2012 +0200 d3dx9/tests: Do not release a cube texture if it wasn't created. --- dlls/d3dx9_36/tests/texture.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/dlls/d3dx9_36/tests/texture.c b/dlls/d3dx9_36/tests/texture.c index 1314fa2..143d45b 100644 --- a/dlls/d3dx9_36/tests/texture.c +++ b/dlls/d3dx9_36/tests/texture.c @@ -734,11 +734,10 @@ static void test_D3DXFilterTexture(IDirect3DDevice9 *device) hr = D3DXFilterTexture((IDirect3DBaseTexture9*) cubetex, NULL, 5, D3DX_FILTER_NONE); /* Invalid miplevel */ ok(hr == D3DERR_INVALIDCALL, "D3DXFilterTexture returned %#x, expected %#x\n", hr, D3DERR_INVALIDCALL); + IDirect3DCubeTexture9_Release(cubetex); } else skip("Failed to create texture\n"); - - IDirect3DCubeTexture9_Release(cubetex); } static BOOL color_match(const DWORD *value, const DWORD *expected)

1 0

Józef Kucia : d3dx9/tests: Fix messed up parameters.
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: 091a4269ef682b90df77603681ba56168b7546f4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=091a4269ef682b90df7760368… Author: Józef Kucia <joseph.kucia(a)gmail.com> Date: Thu May 17 16:51:23 2012 +0200 d3dx9/tests: Fix messed up parameters. --- dlls/d3dx9_36/tests/texture.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/d3dx9_36/tests/texture.c b/dlls/d3dx9_36/tests/texture.c index 143d45b..5663bfa 100644 --- a/dlls/d3dx9_36/tests/texture.c +++ b/dlls/d3dx9_36/tests/texture.c @@ -473,7 +473,7 @@ static void test_D3DXCreateTexture(IDirect3DDevice9 *device) IDirect3DDevice9_GetDeviceCaps(device, &caps); - hr = D3DXCreateTexture(NULL, 0, 0, 0, 0, D3DX_DEFAULT, 0, D3DPOOL_DEFAULT); + hr = D3DXCreateTexture(NULL, 0, 0, 0, 0, D3DX_DEFAULT, D3DPOOL_DEFAULT, NULL); ok(hr == D3DERR_INVALIDCALL, "D3DXCreateTexture returned %#x, expected %#x\n", hr, D3DERR_INVALIDCALL); /* width and height tests */

1 0

Vincent Povirk : appwiz.cpl: Move dialog template id to addon_info_t.
by Alexandre Julliard 17 May '12

17 May '12

Module: wine Branch: master Commit: ad7279a5a605ac738d5712105739f1d2fd802fa8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ad7279a5a605ac738d5712105… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu May 17 09:19:10 2012 -0500 appwiz.cpl: Move dialog template id to addon_info_t. --- dlls/appwiz.cpl/addons.c | 6 ++++-- dlls/appwiz.cpl/appwiz.rc | 2 +- dlls/appwiz.cpl/res.h | 2 +- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/dlls/appwiz.cpl/addons.c b/dlls/appwiz.cpl/addons.c index 1cff5c4..ab70ed1 100644 --- a/dlls/appwiz.cpl/addons.c +++ b/dlls/appwiz.cpl/addons.c @@ -72,6 +72,7 @@ typedef struct { const char *config_key; const char *url_config_key; const char *dir_config_key; + LPCWSTR dialog_template; } addon_info_t; static const addon_info_t addons_info[] = { @@ -80,7 +81,8 @@ static const addon_info_t addons_info[] = { "wine_gecko-" GECKO_VERSION "-" ARCH_STRING ".msi", "gecko", GECKO_SHA, - "MSHTML", "GeckoUrl", "GeckoCabDir" + "MSHTML", "GeckoUrl", "GeckoCabDir", + MAKEINTRESOURCEW(ID_DWL_GECKO_DIALOG) } }; @@ -571,7 +573,7 @@ BOOL install_addon(addon_t addon_type) if(!install_from_registered_dir() && !install_from_default_dir() && (url = get_url())) - DialogBoxW(hInst, MAKEINTRESOURCEW(ID_DWL_DIALOG), 0, installer_proc); + DialogBoxW(hInst, addon->dialog_template, 0, installer_proc); heap_free(url); url = NULL; diff --git a/dlls/appwiz.cpl/appwiz.rc b/dlls/appwiz.cpl/appwiz.rc index 8107212..af41d6a 100644 --- a/dlls/appwiz.cpl/appwiz.rc +++ b/dlls/appwiz.cpl/appwiz.rc @@ -92,7 +92,7 @@ FONT 8, "MS Shell Dlg" CONTROL "", IDC_INFO_COMMENTS, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 100, 105, 150, 8 } -ID_DWL_DIALOG DIALOG 0, 0, 260, 95 +ID_DWL_GECKO_DIALOG DIALOG 0, 0, 260, 95 STYLE DS_MODALFRAME | DS_CENTER | WS_POPUP | WS_VISIBLE | WS_CAPTION | WS_SYSMENU CAPTION "Wine Gecko Installer" FONT 8, "MS Shell Dlg" diff --git a/dlls/appwiz.cpl/res.h b/dlls/appwiz.cpl/res.h index 7dfbf1f..96cd62d 100644 --- a/dlls/appwiz.cpl/res.h +++ b/dlls/appwiz.cpl/res.h @@ -45,7 +45,7 @@ #define IDC_INFO_COMMENTS 1107 #define IDC_INFO_LABEL 1108 -#define ID_DWL_DIALOG 1200 +#define ID_DWL_GECKO_DIALOG 1200 #define ID_DWL_PROGRESS 1201 #define ID_DWL_INSTALL 1202 #define ID_DWL_STATUS 1203

1 0