wine-commits November 2015

wine-commits@winehq.org

2 participants
623 discussions

Andrew Eikum : shell32: Implement IShellDispatch2::ShellExecute.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: aaafeb4cec2c09630fac1ff1e75b448a23cb8421 URL: http://source.winehq.org/git/wine.git/?a=commit;h=aaafeb4cec2c09630fac1ff1e… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Oct 21 08:31:19 2015 -0500 shell32: Implement IShellDispatch2::ShellExecute. Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/shelldispatch.c | 42 ++++++++++++++++++++++++++++++++++---- dlls/shell32/tests/shelldispatch.c | 41 +++++++++++++++++++++++++++++++++++++ 2 files changed, 79 insertions(+), 4 deletions(-) diff --git a/dlls/shell32/shelldispatch.c b/dlls/shell32/shelldispatch.c index f5eca28..6dece6d 100644 --- a/dlls/shell32/shelldispatch.c +++ b/dlls/shell32/shelldispatch.c @@ -1569,11 +1569,45 @@ static HRESULT WINAPI ShellDispatch_IsRestricted(IShellDispatch6 *iface, BSTR gr return E_NOTIMPL; } -static HRESULT WINAPI ShellDispatch_ShellExecute(IShellDispatch6 *iface, BSTR file, VARIANT args, VARIANT dir, - VARIANT op, VARIANT show) +static HRESULT WINAPI ShellDispatch_ShellExecute(IShellDispatch6 *iface, + BSTR file, VARIANT v_args, VARIANT v_dir, VARIANT v_op, VARIANT v_show) { - FIXME("(%s): stub\n", debugstr_w(file)); - return E_NOTIMPL; + VARIANT args_str, dir_str, op_str, show_int; + WCHAR *args = NULL, *dir = NULL, *op = NULL; + INT show = 0; + HINSTANCE ret; + + TRACE("(%s, %s, %s, %s, %s)\n", debugstr_w(file), debugstr_variant(&v_args), + debugstr_variant(&v_dir), debugstr_variant(&v_op), debugstr_variant(&v_show)); + + VariantInit(&args_str); + VariantChangeType(&args_str, &v_args, 0, VT_BSTR); + if (V_VT(&args_str) == VT_BSTR) + args = V_BSTR(&args_str); + + VariantInit(&dir_str); + VariantChangeType(&dir_str, &v_dir, 0, VT_BSTR); + if (V_VT(&dir_str) == VT_BSTR) + dir = V_BSTR(&dir_str); + + VariantInit(&op_str); + VariantChangeType(&op_str, &v_op, 0, VT_BSTR); + if (V_VT(&op_str) == VT_BSTR) + op = V_BSTR(&op_str); + + VariantInit(&show_int); + VariantChangeType(&show_int, &v_show, 0, VT_I4); + if (V_VT(&show_int) == VT_I4) + show = V_I4(&show_int); + + ret = ShellExecuteW(NULL, op, file, args, dir, show); + + VariantClear(&args_str); + VariantClear(&dir_str); + VariantClear(&op_str); + VariantClear(&show_int); + + return (ULONG_PTR)ret > 32 ? S_OK : S_FALSE; } static HRESULT WINAPI ShellDispatch_FindPrinter(IShellDispatch6 *iface, BSTR name, BSTR location, BSTR model) diff --git a/dlls/shell32/tests/shelldispatch.c b/dlls/shell32/tests/shelldispatch.c index e4bda93..dfcbe6d 100644 --- a/dlls/shell32/tests/shelldispatch.c +++ b/dlls/shell32/tests/shelldispatch.c @@ -829,6 +829,46 @@ if (0) { /* crashes on winxp/win2k3 */ IShellDispatch_Release(sd); } +static void test_ShellExecute(void) +{ + HRESULT hr; + IShellDispatch2 *sd; + BSTR name; + VARIANT args, dir, op, show; + + static const WCHAR regW[] = {'r','e','g',0}; + + hr = CoCreateInstance(&CLSID_Shell, NULL, CLSCTX_INPROC_SERVER, + &IID_IShellDispatch2, (void**)&sd); + if (hr != S_OK) + { + win_skip("IShellDispatch2 not supported\n"); + return; + } + + VariantInit(&args); + VariantInit(&dir); + VariantInit(&op); + VariantInit(&show); + + V_VT(&show) = VT_I4; + V_I4(&show) = 0; + + name = SysAllocString(regW); + + hr = IShellDispatch2_ShellExecute(sd, name, args, dir, op, show); + ok(hr == S_OK, "ShellExecute failed: %08x\n", hr); + + /* test invalid value for show */ + V_VT(&show) = VT_BSTR; + V_BSTR(&show) = name; + + hr = IShellDispatch2_ShellExecute(sd, name, args, dir, op, show); + ok(hr == S_OK, "ShellExecute failed: %08x\n", hr); + + SysFreeString(name); +} + START_TEST(shelldispatch) { HRESULT r; @@ -845,6 +885,7 @@ START_TEST(shelldispatch) test_ShellWindows(); test_ParseName(); test_Verbs(); + test_ShellExecute(); CoUninitialize(); }

1 0

Dmitry Timoshkov : ole32/tests: Add the tests to show that IEnumSTATSTG:: Next should zero out returned stats when it reaches end of enumeration.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: 263b06ef8ddcfe64061928459083da41c9111f1d URL: http://source.winehq.org/git/wine.git/?a=commit;h=263b06ef8ddcfe64061928459… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Nov 2 11:34:51 2015 +0800 ole32/tests: Add the tests to show that IEnumSTATSTG::Next should zero out returned stats when it reaches end of enumeration. Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ole32/tests/storage32.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/dlls/ole32/tests/storage32.c b/dlls/ole32/tests/storage32.c index edd0b3f..afe9332 100644 --- a/dlls/ole32/tests/storage32.c +++ b/dlls/ole32/tests/storage32.c @@ -1224,6 +1224,7 @@ static void test_streamenum(void) static const WCHAR stmname[] = { 'C','O','N','T','E','N','T','S',0 }; static const WCHAR stmname2[] = { 'A','B','C','D','E','F','G','H','I',0 }; static const WCHAR stmname3[] = { 'A','B','C','D','E','F','G','H','I','J',0 }; + static const STATSTG stat_null; STATSTG stat; IEnumSTATSTG *ee = NULL; ULONG count; @@ -1269,10 +1270,12 @@ static void test_streamenum(void) r = IStorage_DestroyElement(stg, stmname); ok(r==S_OK, "IStorage->DestroyElement failed\n"); + memset(&stat, 0xad, sizeof(stat)); count = 0xf00; r = IEnumSTATSTG_Next(ee, 1, &stat, &count); ok(r==S_FALSE, "IEnumSTATSTG->Next failed\n"); ok(count == 0, "count wrong\n"); + ok(memcmp(&stat, &stat_null, sizeof(stat)) == 0, "stat is not zeroed\n"); /* reset and try again */ r = IEnumSTATSTG_Reset(ee); @@ -2418,6 +2421,7 @@ static void test_fmtusertypestg(void) static const WCHAR fileW[] = {'f','m','t','t','e','s','t',0}; static WCHAR userTypeW[] = {'S','t','g','U','s','r','T','y','p','e',0}; static const WCHAR strmNameW[] = {1,'C','o','m','p','O','b','j',0}; + static const STATSTG statstg_null; hr = StgCreateDocfile( fileW, STGM_CREATE | STGM_SHARE_EXCLUSIVE | STGM_READWRITE, 0, &stg); ok(hr == S_OK, "should succeed, res=%x\n", hr); @@ -2436,6 +2440,7 @@ static void test_fmtusertypestg(void) BOOL found = FALSE; STATSTG statstg; DWORD got; + memset(&statstg, 0xad, sizeof(statstg)); while ((hr = IEnumSTATSTG_Next(stat, 1, &statstg, &got)) == S_OK && got == 1) { if (strcmp_ww(statstg.pwcsName, strmNameW) == 0) @@ -2444,6 +2449,7 @@ static void test_fmtusertypestg(void) ok(0, "found unexpected stream or storage\n"); CoTaskMemFree(statstg.pwcsName); } + ok(memcmp(&statstg, &statstg_null, sizeof(statstg)) == 0, "statstg is not zeroed\n"); ok(found == TRUE, "expected storage to contain stream \\0001CompObj\n"); IEnumSTATSTG_Release(stat); } @@ -2460,6 +2466,7 @@ static void test_fmtusertypestg(void) BOOL found = FALSE; STATSTG statstg; DWORD got; + memset(&statstg, 0xad, sizeof(statstg)); while ((hr = IEnumSTATSTG_Next(stat, 1, &statstg, &got)) == S_OK && got == 1) { if (strcmp_ww(statstg.pwcsName, strmNameW) == 0) @@ -2468,6 +2475,7 @@ static void test_fmtusertypestg(void) ok(0, "found unexpected stream or storage\n"); CoTaskMemFree(statstg.pwcsName); } + ok(memcmp(&statstg, &statstg_null, sizeof(statstg)) == 0, "statstg is not zeroed\n"); ok(found == TRUE, "expected storage to contain stream \\0001CompObj\n"); IEnumSTATSTG_Release(stat); }

1 0

Dmitry Timoshkov : ole32: Zero out returned stats when IEnumSTATSTG:: Next reaches end of enumeration.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: 9f2bd8503d7fc65a194a12ec5b9245fb027e1ce1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9f2bd8503d7fc65a194a12ec5… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Nov 2 11:34:41 2015 +0800 ole32: Zero out returned stats when IEnumSTATSTG::Next reaches end of enumeration. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ole32/storage32.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c index 4d775b4..f75e097 100644 --- a/dlls/ole32/storage32.c +++ b/dlls/ole32/storage32.c @@ -1031,7 +1031,10 @@ static HRESULT WINAPI IEnumSTATSTGImpl_Next( hr = IEnumSTATSTGImpl_GetNextRef(This, &currentSearchNode); if (FAILED(hr) || currentSearchNode == DIRENTRY_NULL) + { + memset(currentReturnStruct, 0, sizeof(*currentReturnStruct)); break; + } /* * Read the entry from the storage.

1 0

Dmitry Timoshkov : ole32: Handle failure of reading directory entry in IEnumSTATSTG::Next.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: 7bc1005f56da97df89dfbf3262175a7a92eeb446 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7bc1005f56da97df89dfbf326… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Nov 2 11:34:31 2015 +0800 ole32: Handle failure of reading directory entry in IEnumSTATSTG::Next. Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ole32/storage32.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c index bfef4a2..4d775b4 100644 --- a/dlls/ole32/storage32.c +++ b/dlls/ole32/storage32.c @@ -1036,9 +1036,10 @@ static HRESULT WINAPI IEnumSTATSTGImpl_Next( /* * Read the entry from the storage. */ - StorageBaseImpl_ReadDirEntry(This->parentStorage, + hr = StorageBaseImpl_ReadDirEntry(This->parentStorage, currentSearchNode, &currentEntry); + if (FAILED(hr)) break; /* * Copy the information to the return buffer.

1 0

Huw Davies : user32: Restore the dc's colours after sending WM_DRAWITEM.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: e79684874d6be3e28504480ca434f593919ac421 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e79684874d6be3e28504480ca… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Nov 2 10:10:54 2015 +0000 user32: Restore the dc's colours after sending WM_DRAWITEM. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/menu.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/dlls/user32/menu.c b/dlls/user32/menu.c index 847540b..2db18b4 100644 --- a/dlls/user32/menu.c +++ b/dlls/user32/menu.c @@ -1458,6 +1458,7 @@ static void MENU_DrawMenuItem( HWND hwnd, HMENU hmenu, HWND hwndOwner, HDC hdc, ** the menu owner has finished drawing. */ DRAWITEMSTRUCT dis; + COLORREF old_bk, old_text; dis.CtlType = ODT_MENU; dis.CtlID = 0; @@ -1475,8 +1476,12 @@ static void MENU_DrawMenuItem( HWND hwnd, HMENU hmenu, HWND hwndOwner, HDC hdc, "hwndItem=%p, hdc=%p, rcItem=%s\n", hwndOwner, dis.itemID, dis.itemState, dis.itemAction, dis.hwndItem, dis.hDC, wine_dbgstr_rect( &dis.rcItem)); + old_bk = GetBkColor( hdc ); + old_text = GetTextColor( hdc ); SendMessageW( hwndOwner, WM_DRAWITEM, 0, (LPARAM)&dis ); /* Draw the popup-menu arrow */ + SetBkColor( hdc, old_bk ); + SetTextColor( hdc, old_text ); if (lpitem->fType & MF_POPUP) draw_popup_arrow( hdc, rect, arrow_bitmap_width, arrow_bitmap_height);

1 0

Martin Storsjo : ucrtbase: Add defines for the printf/ scanf options constants.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: fcf9ec80800d9b2bc6b874642a2f3f1d151009d0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fcf9ec80800d9b2bc6b874642… Author: Martin Storsjo <martin(a)martin.st> Date: Mon Nov 2 12:08:37 2015 +0200 ucrtbase: Add defines for the printf/scanf options constants. Avoid using hardcoded magic numbers for these options. Signed-off-by: Martin Storsjo <martin(a)martin.st> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcrt/msvcrt.h | 10 ++++++++++ dlls/msvcrt/scanf.c | 2 +- dlls/msvcrt/wcs.c | 10 ++++++---- dlls/ucrtbase/tests/printf.c | 14 ++++++++++---- 4 files changed, 27 insertions(+), 9 deletions(-) diff --git a/dlls/msvcrt/msvcrt.h b/dlls/msvcrt/msvcrt.h index 8c07b94..a3b2595 100644 --- a/dlls/msvcrt/msvcrt.h +++ b/dlls/msvcrt/msvcrt.h @@ -1124,4 +1124,14 @@ extern char* __cdecl __unDName(char *,const char*,int,malloc_func_t,free_func_t, #define UNDNAME_NO_SPECIAL_SYMS (0x4000) #define UNDNAME_NO_COMPLEX_TYPE (0x8000) +#define UCRTBASE_PRINTF_LEGACY_VSPRINTF_NULL_TERMINATION (0x0001) +#define UCRTBASE_PRINTF_STANDARD_SNPRINTF_BEHAVIOUR (0x0002) +#define UCRTBASE_PRINTF_LEGACY_WIDE_SPECIFIERS (0x0004) +#define UCRTBASE_PRINTF_LEGACY_MSVCRT_COMPATIBILITY (0x0008) +#define UCRTBASE_PRINTF_LEGACY_THREE_DIGIT_EXPONENTS (0x0010) + +#define UCRTBASE_SCANF_SECURECRT (0x0001) +#define UCRTBASE_SCANF_LEGACY_WIDE_SPECIFIERS (0x0002) +#define UCRTBASE_SCANF_LEGACY_MSVCRT_COMPATIBILITY (0x0004) + #endif /* __WINE_MSVCRT_H */ diff --git a/dlls/msvcrt/scanf.c b/dlls/msvcrt/scanf.c index 9a0150f..b02977f 100644 --- a/dlls/msvcrt/scanf.c +++ b/dlls/msvcrt/scanf.c @@ -671,7 +671,7 @@ int CDECL MSVCRT__stdio_common_vsscanf(unsigned __int64 options, MSVCRT__locale_t locale, __ms_va_list valist) { - if (options != 2) + if (options != UCRTBASE_SCANF_LEGACY_WIDE_SPECIFIERS) FIXME("options %s not handled\n", wine_dbgstr_longlong(options)); return MSVCRT_vsnscanf_l(input, length, format, locale, valist); } diff --git a/dlls/msvcrt/wcs.c b/dlls/msvcrt/wcs.c index 321e4df..0bfddbb 100644 --- a/dlls/msvcrt/wcs.c +++ b/dlls/msvcrt/wcs.c @@ -724,9 +724,10 @@ int CDECL MSVCRT__stdio_common_vsprintf( unsigned __int64 options, char *str, MS struct _str_ctx_a ctx = {len, str}; int ret; - if (options != 1 && options != 2) + if (options != UCRTBASE_PRINTF_LEGACY_VSPRINTF_NULL_TERMINATION && + options != UCRTBASE_PRINTF_STANDARD_SNPRINTF_BEHAVIOUR) FIXME("options %s not handled\n", wine_dbgstr_longlong(options)); - ret = pf_printf_a(options & 2 ? puts_clbk_str_c99_a : puts_clbk_str_a, + ret = pf_printf_a(options & UCRTBASE_PRINTF_STANDARD_SNPRINTF_BEHAVIOUR ? puts_clbk_str_c99_a : puts_clbk_str_a, &ctx, format, locale, FALSE, FALSE, arg_clbk_valist, NULL, &valist); puts_clbk_str_a(&ctx, 1, &nullbyte); return ret; @@ -1170,9 +1171,10 @@ int CDECL MSVCRT__stdio_common_vswprintf( unsigned __int64 options, MSVCRT_wchar struct _str_ctx_w ctx = {len, str}; int ret; - if (options != 1 && options != 2) + if (options != UCRTBASE_PRINTF_LEGACY_VSPRINTF_NULL_TERMINATION && + options != UCRTBASE_PRINTF_STANDARD_SNPRINTF_BEHAVIOUR) FIXME("options %s not handled\n", wine_dbgstr_longlong(options)); - ret = pf_printf_w(options & 2 ? puts_clbk_str_c99_w : puts_clbk_str_w, + ret = pf_printf_w(options & UCRTBASE_PRINTF_STANDARD_SNPRINTF_BEHAVIOUR ? puts_clbk_str_c99_w : puts_clbk_str_w, &ctx, format, locale, FALSE, FALSE, arg_clbk_valist, NULL, &valist); puts_clbk_str_w(&ctx, 1, &nullbyte); return ret; diff --git a/dlls/ucrtbase/tests/printf.c b/dlls/ucrtbase/tests/printf.c index 96f22c1..99d756e 100644 --- a/dlls/ucrtbase/tests/printf.c +++ b/dlls/ucrtbase/tests/printf.c @@ -30,6 +30,12 @@ #include "wine/test.h" +#define UCRTBASE_PRINTF_LEGACY_VSPRINTF_NULL_TERMINATION (0x0001) +#define UCRTBASE_PRINTF_STANDARD_SNPRINTF_BEHAVIOUR (0x0002) +#define UCRTBASE_PRINTF_LEGACY_WIDE_SPECIFIERS (0x0004) +#define UCRTBASE_PRINTF_LEGACY_MSVCRT_COMPATIBILITY (0x0008) +#define UCRTBASE_PRINTF_LEGACY_THREE_DIGIT_EXPONENTS (0x0010) + static int (__cdecl *p_vfprintf)(unsigned __int64 options, FILE *file, const char *format, void *locale, __ms_va_list valist); static int (__cdecl *p_vsprintf)(unsigned __int64 options, char *str, size_t len, const char *format, @@ -91,7 +97,7 @@ static void test_snprintf (void) for (i = 0; i < sizeof tests / sizeof tests[0]; i++) { const char *fmt = tests[i]; const int expect = strlen(fmt) > bufsiz ? -1 : strlen(fmt); - const int n = vsprintf_wrapper (1, buffer, bufsiz, fmt); + const int n = vsprintf_wrapper (UCRTBASE_PRINTF_LEGACY_VSPRINTF_NULL_TERMINATION, buffer, bufsiz, fmt); const int valid = n < 0 ? bufsiz : (n == bufsiz ? n : n+1); ok (n == expect, "\"%s\": expected %d, returned %d\n", @@ -104,7 +110,7 @@ static void test_snprintf (void) for (i = 0; i < sizeof tests / sizeof tests[0]; i++) { const char *fmt = tests[i]; const int expect = strlen(fmt); - const int n = vsprintf_wrapper (2, buffer, bufsiz, fmt); + const int n = vsprintf_wrapper (UCRTBASE_PRINTF_STANDARD_SNPRINTF_BEHAVIOUR, buffer, bufsiz, fmt); const int valid = n >= bufsiz ? bufsiz - 1 : n < 0 ? 0 : n; ok (n == expect, "\"%s\": expected %d, returned %d\n", @@ -144,7 +150,7 @@ static void test_swprintf (void) for (i = 0; i < sizeof tests / sizeof tests[0]; i++) { const wchar_t *fmt = tests[i]; const int expect = wcslen(fmt) > bufsiz ? -1 : wcslen(fmt); - const int n = vswprintf_wrapper (1, buffer, bufsiz, fmt); + const int n = vswprintf_wrapper (UCRTBASE_PRINTF_LEGACY_VSPRINTF_NULL_TERMINATION, buffer, bufsiz, fmt); const int valid = n < 0 ? bufsiz : (n == bufsiz ? n : n+1); WideCharToMultiByte (CP_ACP, 0, buffer, -1, narrow, sizeof(narrow), NULL, NULL); @@ -159,7 +165,7 @@ static void test_swprintf (void) for (i = 0; i < sizeof tests / sizeof tests[0]; i++) { const wchar_t *fmt = tests[i]; const int expect = wcslen(fmt); - const int n = vswprintf_wrapper (2, buffer, bufsiz, fmt); + const int n = vswprintf_wrapper (UCRTBASE_PRINTF_STANDARD_SNPRINTF_BEHAVIOUR, buffer, bufsiz, fmt); const int valid = n >= bufsiz ? bufsiz - 1 : n < 0 ? 0 : n; WideCharToMultiByte (CP_ACP, 0, buffer, -1, narrow, sizeof(narrow), NULL, NULL);

1 0

Martin Storsjo : ucrtbase: Simplify the snprintf/swprintf tests.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: 4b47da3ad7f94b01af53e0b0fa49c86a22967336 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4b47da3ad7f94b01af53e0b0f… Author: Martin Storsjo <martin(a)martin.st> Date: Mon Nov 2 12:08:36 2015 +0200 ucrtbase: Simplify the snprintf/swprintf tests. Avoid repeating the test strings, calculate the expected return value with code. Signed-off-by: Martin Storsjo <martin(a)martin.st> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ucrtbase/tests/printf.c | 60 +++++++++++++++----------------------------- 1 file changed, 20 insertions(+), 40 deletions(-) diff --git a/dlls/ucrtbase/tests/printf.c b/dlls/ucrtbase/tests/printf.c index 4393300..96f22c1 100644 --- a/dlls/ucrtbase/tests/printf.c +++ b/dlls/ucrtbase/tests/printf.c @@ -82,27 +82,15 @@ static int __cdecl vsprintf_wrapper(unsigned __int64 options, char *str, static void test_snprintf (void) { - struct snprintf_test { - const char *format; - int expected; - }; - /* Legacy behaviour, not C99 compliant. */ - const struct snprintf_test tests_legacy[] = {{"short", 5}, - {"justfit", 7}, - {"justfits", 8}, - {"muchlonger", -1}}; - /* New C99 compliant behaviour. */ - const struct snprintf_test tests_standard[] = {{"short", 5}, - {"justfit", 7}, - {"justfits", 8}, - {"muchlonger", 10}}; + const char *tests[] = {"short", "justfit", "justfits", "muchlonger"}; char buffer[8]; const int bufsiz = sizeof buffer; unsigned int i; - for (i = 0; i < sizeof tests_legacy / sizeof tests_legacy[0]; i++) { - const char *fmt = tests_legacy[i].format; - const int expect = tests_legacy[i].expected; + /* Legacy _snprintf style termination */ + for (i = 0; i < sizeof tests / sizeof tests[0]; i++) { + const char *fmt = tests[i]; + const int expect = strlen(fmt) > bufsiz ? -1 : strlen(fmt); const int n = vsprintf_wrapper (1, buffer, bufsiz, fmt); const int valid = n < 0 ? bufsiz : (n == bufsiz ? n : n+1); @@ -112,9 +100,10 @@ static void test_snprintf (void) "\"%s\": rendered \"%.*s\"\n", fmt, valid, buffer); } - for (i = 0; i < sizeof tests_standard / sizeof tests_standard[0]; i++) { - const char *fmt = tests_standard[i].format; - const int expect = tests_standard[i].expected; + /* C99 snprintf style termination */ + for (i = 0; i < sizeof tests / sizeof tests[0]; i++) { + const char *fmt = tests[i]; + const int expect = strlen(fmt); const int n = vsprintf_wrapper (2, buffer, bufsiz, fmt); const int valid = n >= bufsiz ? bufsiz - 1 : n < 0 ? 0 : n; @@ -137,34 +126,24 @@ static int __cdecl vswprintf_wrapper(unsigned __int64 options, wchar_t *str, __ms_va_end(valist); return ret; } + static void test_swprintf (void) { - struct swprintf_test { - const wchar_t *format; - int expected; - }; const wchar_t str_short[] = {'s','h','o','r','t',0}; const wchar_t str_justfit[] = {'j','u','s','t','f','i','t',0}; const wchar_t str_justfits[] = {'j','u','s','t','f','i','t','s',0}; const wchar_t str_muchlonger[] = {'m','u','c','h','l','o','n','g','e','r',0}; - /* Legacy behaviour, not C99 compliant. */ - const struct swprintf_test tests_legacy[] = {{str_short, 5}, - {str_justfit, 7}, - {str_justfits, 8}, - {str_muchlonger, -1}}; - /* New C99 compliant behaviour. */ - const struct swprintf_test tests_standard[] = {{str_short, 5}, - {str_justfit, 7}, - {str_justfits, 8}, - {str_muchlonger, 10}}; + const wchar_t *tests[] = {str_short, str_justfit, str_justfits, str_muchlonger}; + wchar_t buffer[8]; char narrow[8], narrow_fmt[16]; const int bufsiz = sizeof buffer / sizeof buffer[0]; unsigned int i; - for (i = 0; i < sizeof tests_legacy / sizeof tests_legacy[0]; i++) { - const wchar_t *fmt = tests_legacy[i].format; - const int expect = tests_legacy[i].expected; + /* Legacy _snprintf style termination */ + for (i = 0; i < sizeof tests / sizeof tests[0]; i++) { + const wchar_t *fmt = tests[i]; + const int expect = wcslen(fmt) > bufsiz ? -1 : wcslen(fmt); const int n = vswprintf_wrapper (1, buffer, bufsiz, fmt); const int valid = n < 0 ? bufsiz : (n == bufsiz ? n : n+1); @@ -176,9 +155,10 @@ static void test_swprintf (void) "\"%s\": rendered \"%.*s\"\n", narrow_fmt, valid, narrow); } - for (i = 0; i < sizeof tests_standard / sizeof tests_standard[0]; i++) { - const wchar_t *fmt = tests_standard[i].format; - const int expect = tests_standard[i].expected; + /* C99 snprintf style termination */ + for (i = 0; i < sizeof tests / sizeof tests[0]; i++) { + const wchar_t *fmt = tests[i]; + const int expect = wcslen(fmt); const int n = vswprintf_wrapper (2, buffer, bufsiz, fmt); const int valid = n >= bufsiz ? bufsiz - 1 : n < 0 ? 0 : n;

1 0

Martin Storsjo : ucrtbase: Fix the function name in a comment.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: 2a74c9a8eda818c1245a4a245d4d087e46d28fac URL: http://source.winehq.org/git/wine.git/?a=commit;h=2a74c9a8eda818c1245a4a245… Author: Martin Storsjo <martin(a)martin.st> Date: Mon Nov 2 12:08:35 2015 +0200 ucrtbase: Fix the function name in a comment. Signed-off-by: Martin Storsjo <martin(a)martin.st> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcrt/wcs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/msvcrt/wcs.c b/dlls/msvcrt/wcs.c index d3114c6..321e4df 100644 --- a/dlls/msvcrt/wcs.c +++ b/dlls/msvcrt/wcs.c @@ -1161,7 +1161,7 @@ static int puts_clbk_str_c99_w(void *ctx, int len, const MSVCRT_wchar_t *str) } /********************************************************************* - * __stdio_common_vsprintf (MSVCRT.@) + * __stdio_common_vswprintf (MSVCRT.@) */ int CDECL MSVCRT__stdio_common_vswprintf( unsigned __int64 options, MSVCRT_wchar_t *str, MSVCRT_size_t len, const MSVCRT_wchar_t *format, MSVCRT__locale_t locale, __ms_va_list valist )

1 0

Stefan Dösinger : comctl32: Re-enable the parent before destroying the dialog.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: 4e2d25c7cd22ae4c7ea1be9126c1f7183ebcd722 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4e2d25c7cd22ae4c7ea1be912… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Fri Oct 30 19:33:11 2015 +0100 comctl32: Re-enable the parent before destroying the dialog. Prevent 1602 AD from minimizing when the multiplayer dialog is closed. When DestroyWindow is called, user32 tries to find a window to activate, finds the main game window disabled and decides to go to the desktop. Wined3d will help this process by minimizing the game out of the way. Signed-off-by: Stefan Dösinger <stefan(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/comctl32/propsheet.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/comctl32/propsheet.c b/dlls/comctl32/propsheet.c index ce7001c..ed5f7d4 100644 --- a/dlls/comctl32/propsheet.c +++ b/dlls/comctl32/propsheet.c @@ -2706,6 +2706,7 @@ static INT do_loop(const PropSheetInfo *psInfo) MSG msg; INT ret = -1; HWND hwnd = psInfo->hwnd; + HWND parent = psInfo->ppshheader.hwndParent; while(IsWindow(hwnd) && !psInfo->ended && (ret = GetMessageW(&msg, NULL, 0, 0))) { @@ -2728,6 +2729,9 @@ static INT do_loop(const PropSheetInfo *psInfo) if(ret != -1) ret = psInfo->result; + if(parent) + EnableWindow(parent, TRUE); + DestroyWindow(hwnd); return ret; } @@ -2754,10 +2758,7 @@ static INT_PTR PROPSHEET_PropertySheet(PropSheetInfo* psInfo, BOOL unicode) } bRet = PROPSHEET_CreateDialog(psInfo); if(!psInfo->isModeless) - { bRet = do_loop(psInfo); - if (parent) EnableWindow(parent, TRUE); - } return bRet; }

1 0

Dmitry Timoshkov : widl: Avoid warnings when being added dispatch guids already present in a typelib.
by Alexandre Julliard 02 Nov '15

02 Nov '15

Module: wine Branch: master Commit: f16396ceecc811fad38480215f438122a877c1bc URL: http://source.winehq.org/git/wine.git/?a=commit;h=f16396ceecc811fad38480215… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Sat Oct 31 14:31:32 2015 +0800 widl: Avoid warnings when being added dispatch guids already present in a typelib. Silence warnings about adding a duplicate uuid when generating stdole2.tlb. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/write_msft.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/tools/widl/write_msft.c b/tools/widl/write_msft.c index 98081f2..daa0366 100644 --- a/tools/widl/write_msft.c +++ b/tools/widl/write_msft.c @@ -1978,7 +1978,7 @@ static msft_typeinfo_t *create_msft_typeinfo(msft_typelib_t *typelib, enum type_ static void add_dispatch(msft_typelib_t *typelib) { - int guid_offset, impfile_offset; + int guid_offset, impfile_offset, hash_key; MSFT_GuidEntry guidentry; MSFT_ImpInfo impinfo; GUID stdole = {0x00020430,0x0000,0x0000,{0xc0,0x00,0x00,0x00,0x00,0x00,0x00,0x46}}; @@ -1989,7 +1989,10 @@ static void add_dispatch(msft_typelib_t *typelib) guidentry.guid = stdole; guidentry.hreftype = 2; guidentry.next_hash = -1; - guid_offset = ctl2_alloc_guid(typelib, &guidentry); + hash_key = ctl2_hash_guid(&guidentry.guid); + guid_offset = ctl2_find_guid(typelib, hash_key, &guidentry.guid); + if (guid_offset == -1) + guid_offset = ctl2_alloc_guid(typelib, &guidentry); impfile_offset = alloc_importfile(typelib, guid_offset, 2, 0, "stdole2.tlb"); guidentry.guid = iid_idispatch; @@ -1997,7 +2000,11 @@ static void add_dispatch(msft_typelib_t *typelib) guidentry.next_hash = -1; impinfo.flags = TKIND_INTERFACE << 24 | MSFT_IMPINFO_OFFSET_IS_GUID; impinfo.oImpFile = impfile_offset; - impinfo.oGuid = ctl2_alloc_guid(typelib, &guidentry); + hash_key = ctl2_hash_guid(&guidentry.guid); + guid_offset = ctl2_find_guid(typelib, hash_key, &guidentry.guid); + if (guid_offset == -1) + guid_offset = ctl2_alloc_guid(typelib, &guidentry); + impinfo.oGuid = guid_offset; typelib->typelib_header.dispatchpos = alloc_msft_importinfo(typelib, &impinfo) | 0x01; }

1 0

← Newer
1
...
56
57
58
59
60
61
62
63
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 November 2015