April 2011 - Wine-commits - list.winehq.org

Frédéric Delanoy : po: Update French translation.
by Alexandre Julliard 05 Apr '11

05 Apr '11

Module: wine Branch: master Commit: 6fcad42e9a9982c4fdfb8a69051100a04cf1556e URL: http://source.winehq.org/git/wine.git/?a=commit;h=6fcad42e9a9982c4fdfb8a690… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Mon Apr 4 21:48:21 2011 +0200 po: Update French translation. --- po/fr.po | 21 ++++++--------------- 1 files changed, 6 insertions(+), 15 deletions(-) diff --git a/po/fr.po b/po/fr.po index a71f74f..3c67a9f 100644 --- a/po/fr.po +++ b/po/fr.po @@ -91,14 +91,12 @@ msgstr "" "l'installation d'un fichier corrompu." #: avifil32.rc:27 -#, fuzzy msgid "Waveform: %s" -msgstr "Waveform : %s" +msgstr "Forme d'onde : %s" #: avifil32.rc:28 -#, fuzzy msgid "Waveform" -msgstr "Waveform" +msgstr "Forme d'onde" #: avifil32.rc:29 msgid "All multimedia files" @@ -2025,13 +2023,12 @@ msgid "Default MidiOut Device" msgstr "Périphérique MidiOut par défaut" #: dxdiagn.rc:25 -#, fuzzy msgid "Regional Setting" -msgstr "Paramètres par défaut" +msgstr "Paramètres régionaux" #: dxdiagn.rc:26 msgid "%uMB used, %uMB available" -msgstr "" +msgstr "%u MB utilisés, %u MB disponibles" #: hhctrl.rc:67 hhctrl.rc:47 wordpad.rc:155 msgid "Options" @@ -5851,13 +5848,8 @@ msgid "Type the path where the file is located, and then click OK." msgstr "Saisissez le chemin du fichier, puis cliquez sur OK." #: shdoclc.rc:36 -#, fuzzy msgid "Default" -msgstr "" -"#-#-#-#-# fr.po (Wine) #-#-#-#-#\n" -"Standard\n" -"#-#-#-#-# fr.po (Wine) #-#-#-#-#\n" -"Par défaut" +msgstr "Standard" #: shdoclc.rc:39 msgid "F&orward" @@ -8539,9 +8531,8 @@ msgid "Could not stop service %s\n" msgstr "Impossible d'arrêter le service %s\n" #: net.rc:32 -#, fuzzy msgid "Could not get handle to service control manager.\n" -msgstr "Impossible d'obtenir le gestionnaire de contrôle de service.\n" +msgstr "Impossible d'accéder au gestionnaire de contrôle de services.\n" #: net.rc:33 msgid "Could not get handle to service.\n"

1 0

Dmitry Timoshkov : user32: Add a test for WM_QUIT handling by DefDlgProc.
by Alexandre Julliard 05 Apr '11

05 Apr '11

Module: wine Branch: master Commit: 1a65e209f8c638fde7d7b3b8928fdc365a7379f7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1a65e209f8c638fde7d7b3b89… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Mon Apr 4 20:40:42 2011 +0900 user32: Add a test for WM_QUIT handling by DefDlgProc. --- dlls/user32/tests/msg.c | 60 +++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 60 insertions(+), 0 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 712f564..65e417a 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -10026,6 +10026,50 @@ done: flush_events(); } +static INT_PTR CALLBACK wm_quit_dlg_proc(HWND hwnd, UINT message, WPARAM wp, LPARAM lp) +{ + struct recvd_message msg; + + if (ignore_message( message )) return 0; + + msg.hwnd = hwnd; + msg.message = message; + msg.flags = sent|wparam|lparam; + msg.wParam = wp; + msg.lParam = lp; + msg.descr = "dialog"; + add_message(&msg); + + switch (message) + { + case WM_INITDIALOG: + PostMessage(hwnd, WM_QUIT, 0x1234, 0x5678); + PostMessage(hwnd, WM_USER, 0xdead, 0xbeef); + return 0; + + case WM_GETDLGCODE: + return 0; + + case WM_USER: + EndDialog(hwnd, 0); + break; + } + + return 1; +} + +static const struct message WmQuitDialogSeq[] = { + { HCBT_CREATEWND, hook }, + { WM_SETFONT, sent }, + { WM_INITDIALOG, sent }, + { WM_CHANGEUISTATE, sent|optional }, + { HCBT_DESTROYWND, hook }, + { 0x0090, sent|optional }, /* Vista */ + { WM_DESTROY, sent }, + { WM_NCDESTROY, sent }, + { 0 } +}; + static void test_quit_message(void) { MSG msg; @@ -10076,6 +10120,22 @@ static void test_quit_message(void) ret = GetMessage(&msg, NULL, 0, 0); ok(ret > 0, "GetMessage failed with error %d\n", GetLastError()); ok(msg.message == WM_USER, "Received message 0x%04x instead of WM_USER\n", msg.message); + + flush_events(); + flush_sequence(); + ret = DialogBoxParam(GetModuleHandle(0), "TEST_EMPTY_DIALOG", 0, wm_quit_dlg_proc, 0); +todo_wine + ok(ret == 1, "expected 1, got %d\n", ret); + ok_sequence(WmQuitDialogSeq, "WmQuitDialogSeq", TRUE); + memset(&msg, 0xab, sizeof(msg)); + ret = PeekMessage(&msg, NULL, 0, 0, PM_REMOVE); +todo_wine + ok(ret, "PeekMessage failed\n"); +if (ret) { + ok(msg.message == WM_QUIT, "Received message 0x%04x instead of WM_QUIT\n", msg.message); + ok(msg.wParam == 0x1234, "wParam was 0x%lx instead of 0x1234\n", msg.wParam); + ok(msg.lParam == 0, "lParam was 0x%lx instead of 0\n", msg.lParam); +} } static const struct message WmMouseHoverSeq[] = {

1 0

Akihiro Sagawa : kernel32/tests: Add a test for undefined byte character conversion.
by Alexandre Julliard 05 Apr '11

05 Apr '11

Module: wine Branch: master Commit: 79d7cd76f9bcfb8885d0dd4a3760afa767380ffa URL: http://source.winehq.org/git/wine.git/?a=commit;h=79d7cd76f9bcfb8885d0dd4a3… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Mon Apr 4 19:00:58 2011 +0900 kernel32/tests: Add a test for undefined byte character conversion. --- dlls/kernel32/tests/codepage.c | 53 ++++++++++++++++++++++++++++++++++++++++ 1 files changed, 53 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/tests/codepage.c b/dlls/kernel32/tests/codepage.c index 402e4bf..96dd45d 100644 --- a/dlls/kernel32/tests/codepage.c +++ b/dlls/kernel32/tests/codepage.c @@ -354,6 +354,57 @@ static void test_string_conversion(LPBOOL bUsedDefaultChar) ok(GetLastError() == 0xdeadbeef, "GetLastError() is %u\n", GetLastError()); } +static void test_undefined_byte_char(void) +{ + struct tag_testset { + INT codepage; + LPCSTR str; + BOOL is_error; + } testset[] = { + { 874, "\xdd", TRUE }, + { 932, "\xfe", TRUE }, + { 932, "\x80", FALSE }, + { 936, "\xff", TRUE }, + { 949, "\xff", TRUE }, + { 950, "\xff", TRUE }, + { 1252, "\x90", FALSE }, + { 1253, "\xaa", TRUE }, + { 1255, "\xff", TRUE }, + { 1257, "\xa5", TRUE }, + }; + INT i, ret; + + for (i = 0; i < (sizeof(testset) / sizeof(testset[0])); i++) { + if (! IsValidCodePage(testset[i].codepage)) + { + skip("Codepage %d not available\n", testset[i].codepage); + continue; + } + + SetLastError(0xdeadbeef); + ret = MultiByteToWideChar(testset[i].codepage, MB_ERR_INVALID_CHARS, + testset[i].str, -1, NULL, 0); + if (testset[i].is_error) { + todo_wine + ok(ret == 0 && GetLastError() == ERROR_NO_UNICODE_TRANSLATION, + "ret is %d, GetLastError is %u (cp %d)\n", + ret, GetLastError(), testset[i].codepage); + } + else { + ok(ret == strlen(testset[i].str)+1 && GetLastError() == 0xdeadbeef, + "ret is %d, GetLastError is %u (cp %d)\n", + ret, GetLastError(), testset[i].codepage); + } + + SetLastError(0xdeadbeef); + ret = MultiByteToWideChar(testset[i].codepage, 0, + testset[i].str, -1, NULL, 0); + ok(ret == strlen(testset[i].str)+1 && GetLastError() == 0xdeadbeef, + "ret is %d, GetLastError is %u (cp %d)\n", + ret, GetLastError(), testset[i].codepage); + } +} + START_TEST(codepage) { BOOL bUsedDefaultChar; @@ -367,4 +418,6 @@ START_TEST(codepage) /* WideCharToMultiByte has two code paths, test both here */ test_string_conversion(NULL); test_string_conversion(&bUsedDefaultChar); + + test_undefined_byte_char(); }

1 0

Akihiro Sagawa : gdi32: Remove redundant charset assignments.
by Alexandre Julliard 04 Apr '11

04 Apr '11

Module: wine Branch: master Commit: f0881854712e55b9b1e85b939c65f3737d197fd3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f0881854712e55b9b1e85b939… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Fri Apr 1 21:19:57 2011 +0900 gdi32: Remove redundant charset assignments. --- dlls/gdi32/freetype.c | 5 +---- 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index 31fe4f2..0b50330 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -4121,10 +4121,7 @@ static BOOL enum_face_charsets(Face *face, FONTENUMPROCW proc, LPARAM lparam) else { fs.fsCsb[0] = 1L << i; fs.fsCsb[1] = 0; - if(!TranslateCharsetInfo(fs.fsCsb, &csi, TCI_SRCFONTSIG)) - csi.ciCharset = DEFAULT_CHARSET; - if(i == 31) csi.ciCharset = SYMBOL_CHARSET; - if(csi.ciCharset != DEFAULT_CHARSET) { + if(TranslateCharsetInfo(fs.fsCsb, &csi, TCI_SRCFONTSIG)) elf.elfLogFont.lfCharSet = ntm.ntmTm.tmCharSet = csi.ciCharset; if(ElfScriptsW[i]) strcpyW(elf.elfScript, ElfScriptsW[i]);

1 0

Akihiro Sagawa : gdi32: Create charset enumeration order list for EnumFontFamiliesEx.
by Alexandre Julliard 04 Apr '11

04 Apr '11

Module: wine Branch: master Commit: cb77f9a2395b31079ab72f99706be7f6b107dd69 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cb77f9a2395b31079ab72f997… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Fri Apr 1 21:20:00 2011 +0900 gdi32: Create charset enumeration order list for EnumFontFamiliesEx. --- dlls/gdi32/freetype.c | 96 ++++++++++++++++++++++++++++++++++++++-------- dlls/gdi32/tests/font.c | 14 +----- 2 files changed, 82 insertions(+), 28 deletions(-) diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index 0b50330..aefb0e8 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -354,6 +354,17 @@ typedef struct { struct list links; } SYSTEM_LINKS; +struct enum_charset_element { + DWORD mask; + DWORD charset; + LPCWSTR name; +}; + +struct enum_charset_list { + DWORD total; + struct enum_charset_element element[32]; +}; + #define GM_BLOCK_SIZE 128 #define FONT_GM(font,idx) (&(font)->gm[(idx) / GM_BLOCK_SIZE][(idx) % GM_BLOCK_SIZE]) @@ -3951,6 +3962,59 @@ BOOL WineEngDestroyFontInstance(HFONT handle) return ret; } +/*************************************************** + * create_enum_charset_list + * + * This function creates charset enumeration list because in DEFAULT_CHARSET + * case, the ANSI codepage's charset takes precedence over other charsets. + * This function works as a filter other than DEFAULT_CHARSET case. + */ +static DWORD create_enum_charset_list(DWORD charset, struct enum_charset_list *list) +{ + CHARSETINFO csi; + DWORD n = 0; + + if (TranslateCharsetInfo((DWORD*)charset, &csi, TCI_SRCCHARSET) && + csi.fs.fsCsb[0] != 0) { + list->element[n].mask = csi.fs.fsCsb[0]; + list->element[n].charset = csi.ciCharset; + list->element[n].name = ElfScriptsW[ffs(csi.fs.fsCsb[0]) - 1]; + n++; + } + else { /* charset is DEFAULT_CHARSET or invalid. */ + INT acp, i; + + /* Set the current codepage's charset as the first element. */ + acp = GetACP(); + if (TranslateCharsetInfo((DWORD*)(INT_PTR)acp, &csi, TCI_SRCCODEPAGE) && + csi.fs.fsCsb[0] != 0) { + list->element[n].mask = csi.fs.fsCsb[0]; + list->element[n].charset = csi.ciCharset; + list->element[n].name = ElfScriptsW[ffs(csi.fs.fsCsb[0]) - 1]; + n++; + } + + /* Fill out left elements. */ + for (i = 0; i < 32; i++) { + FONTSIGNATURE fs; + fs.fsCsb[0] = 1L << i; + fs.fsCsb[1] = 0; + if (n > 0 && fs.fsCsb[0] == list->element[0].mask) + continue; /* skip, already added. */ + if (!TranslateCharsetInfo(fs.fsCsb, &csi, TCI_SRCFONTSIG)) + continue; /* skip, this is an invalid fsCsb bit. */ + + list->element[n].mask = fs.fsCsb[0]; + list->element[n].charset = csi.ciCharset; + list->element[n].name = ElfScriptsW[i]; + n++; + } + } + list->total = n; + + return n; +} + static void GetEnumStructs(Face *face, LPENUMLOGFONTEXW pelf, NEWTEXTMETRICEXW *pntm, LPDWORD ptype) { @@ -4101,38 +4165,33 @@ static BOOL face_matches(Face *face, const LOGFONTW *lf) return !strcmpiW(lf->lfFaceName, full_family_name); } -static BOOL enum_face_charsets(Face *face, FONTENUMPROCW proc, LPARAM lparam) +static BOOL enum_face_charsets(Face *face, struct enum_charset_list *list, + FONTENUMPROCW proc, LPARAM lparam) { ENUMLOGFONTEXW elf; NEWTEXTMETRICEXW ntm; DWORD type = 0; - FONTSIGNATURE fs; - CHARSETINFO csi; int i; GetEnumStructs(face, &elf, &ntm, &type); - for(i = 0; i < 32; i++) { + for(i = 0; i < list->total; i++) { if(!face->scalable && face->fs.fsCsb[0] == 0) { /* OEM bitmap */ elf.elfLogFont.lfCharSet = ntm.ntmTm.tmCharSet = OEM_CHARSET; strcpyW(elf.elfScript, OEM_DOSW); i = 32; /* break out of loop */ - } else if(!(face->fs.fsCsb[0] & (1L << i))) + } else if(!(face->fs.fsCsb[0] & list->element[i].mask)) continue; else { - fs.fsCsb[0] = 1L << i; - fs.fsCsb[1] = 0; - if(TranslateCharsetInfo(fs.fsCsb, &csi, TCI_SRCFONTSIG)) - elf.elfLogFont.lfCharSet = ntm.ntmTm.tmCharSet = csi.ciCharset; - if(ElfScriptsW[i]) - strcpyW(elf.elfScript, ElfScriptsW[i]); - else - FIXME("Unknown elfscript for bit %d\n", i); - } + elf.elfLogFont.lfCharSet = ntm.ntmTm.tmCharSet = list->element[i].charset; + if(list->element[i].name) + strcpyW(elf.elfScript, list->element[i].name); + else + FIXME("Unknown elfscript for bit %d\n", ffs(list->element[i].mask) - 1); } TRACE("enuming face %s full %s style %s charset = %d type %d script %s it %d weight %d ntmflags %08x\n", debugstr_w(elf.elfLogFont.lfFaceName), debugstr_w(elf.elfFullName), debugstr_w(elf.elfStyle), - csi.ciCharset, type, debugstr_w(elf.elfScript), + list->element[i].charset, type, debugstr_w(elf.elfScript), elf.elfLogFont.lfItalic, elf.elfLogFont.lfWeight, ntm.ntmTm.ntmFlags); /* release section before callback (FIXME) */ @@ -4153,6 +4212,7 @@ DWORD WineEngEnumFonts(LPLOGFONTW plf, FONTENUMPROCW proc, LPARAM lparam) Face *face; struct list *family_elem_ptr, *face_elem_ptr; LOGFONTW lf; + struct enum_charset_list enum_charsets; if (!plf) { @@ -4164,6 +4224,8 @@ DWORD WineEngEnumFonts(LPLOGFONTW plf, FONTENUMPROCW proc, LPARAM lparam) TRACE("facename = %s charset %d\n", debugstr_w(plf->lfFaceName), plf->lfCharSet); + create_enum_charset_list(plf->lfCharSet, &enum_charsets); + GDI_CheckNotLock(); EnterCriticalSection( &freetype_cs ); if(plf->lfFaceName[0]) { @@ -4184,7 +4246,7 @@ DWORD WineEngEnumFonts(LPLOGFONTW plf, FONTENUMPROCW proc, LPARAM lparam) LIST_FOR_EACH(face_elem_ptr, &family->faces) { face = LIST_ENTRY(face_elem_ptr, Face, entry); if (!face_matches(face, plf)) continue; - if (!enum_face_charsets(face, proc, lparam)) return 0; + if (!enum_face_charsets(face, &enum_charsets, proc, lparam)) return 0; } } } @@ -4193,7 +4255,7 @@ DWORD WineEngEnumFonts(LPLOGFONTW plf, FONTENUMPROCW proc, LPARAM lparam) family = LIST_ENTRY(family_elem_ptr, Family, entry); face_elem_ptr = list_head(&family->faces); face = LIST_ENTRY(face_elem_ptr, Face, entry); - if (!enum_face_charsets(face, proc, lparam)) return 0; + if (!enum_face_charsets(face, &enum_charsets, proc, lparam)) return 0; } } LeaveCriticalSection( &freetype_cs ); diff --git a/dlls/gdi32/tests/font.c b/dlls/gdi32/tests/font.c index b0f2e9f..2d4efff 100644 --- a/dlls/gdi32/tests/font.c +++ b/dlls/gdi32/tests/font.c @@ -2210,17 +2210,9 @@ static void test_EnumFontFamiliesEx_default_charset(void) } trace("'%s' has %d charsets.\n", gui_font.lfFaceName, efd.total); - if (gui_font.lfCharSet != ANSI_CHARSET) { - todo_wine - ok(efd.lf[0].lfCharSet == gui_font.lfCharSet, - "(%s) got charset %d expected %d\n", - efd.lf[0].lfFaceName, efd.lf[0].lfCharSet, gui_font.lfCharSet); - } - else { - ok(efd.lf[0].lfCharSet == gui_font.lfCharSet, - "(%s) got charset %d expected %d\n", - efd.lf[0].lfFaceName, efd.lf[0].lfCharSet, gui_font.lfCharSet); - } + ok(efd.lf[0].lfCharSet == gui_font.lfCharSet, + "(%s) got charset %d expected %d\n", + efd.lf[0].lfFaceName, efd.lf[0].lfCharSet, gui_font.lfCharSet); return; }

1 0

Jörg Höhle : winmm: Check callback flags prior to opening a driver.
by Alexandre Julliard 04 Apr '11

04 Apr '11

Module: wine Branch: master Commit: c3774b664ad3cfd3b53eae19999489bc16904ff8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c3774b664ad3cfd3b53eae199… Author: Jörg Höhle <hoehle(a)users.sourceforge.net> Date: Tue Mar 15 23:00:04 2011 +0100 winmm: Check callback flags prior to opening a driver. --- dlls/winmm/tests/mixer.c | 35 ++++++++++++++++- dlls/winmm/winmm.c | 98 ++++++++++++++++++++++++++++------------------ 2 files changed, 94 insertions(+), 39 deletions(-) diff --git a/dlls/winmm/tests/mixer.c b/dlls/winmm/tests/mixer.c index 59627a7..da27b16 100644 --- a/dlls/winmm/tests/mixer.c +++ b/dlls/winmm/tests/mixer.c @@ -1004,6 +1004,7 @@ static void mixer_testsW(void) static void test_mixerOpen(void) { HMIXER mix; + HANDLE event; MMRESULT rc; UINT ndev, d; @@ -1031,7 +1032,7 @@ static void test_mixerOpen(void) rc = mixerOpen(&mix, d, 0xdeadbeef, 0, CALLBACK_WINDOW); ok(rc == MMSYSERR_INVALPARAM || - rc == MMSYSERR_NOERROR, /* 98 */ + broken(rc == MMSYSERR_NOERROR /* 98 */), "mixerOpen: MMSYSERR_INVALPARAM expected, got %s\n", mmsys_error(rc)); if (rc == MMSYSERR_NOERROR) @@ -1042,7 +1043,35 @@ static void test_mixerOpen(void) ok(rc == MMSYSERR_NOERROR, "mixerOpen: MMSYSERR_NOERROR expected, got %s\n", mmsys_error(rc)); + if (rc == MMSYSERR_NOERROR) + test_mixerClose(mix); + + rc = mixerOpen(&mix, d, 0, 0, CALLBACK_THREAD); + ok(rc == MMSYSERR_NOERROR /* since w2k */ || + rc == MMSYSERR_NOTSUPPORTED, /* 98 */ + "mixerOpen: MMSYSERR_NOERROR expected, got %s\n", + mmsys_error(rc)); + if (rc == MMSYSERR_NOERROR) + test_mixerClose(mix); + + rc = mixerOpen(&mix, d, 0, 0, CALLBACK_EVENT); + ok(rc == MMSYSERR_NOERROR /* since w2k */ || + rc == MMSYSERR_NOTSUPPORTED, /* 98 */ + "mixerOpen: MMSYSERR_NOERROR expected, got %s\n", + mmsys_error(rc)); + if (rc == MMSYSERR_NOERROR) + test_mixerClose(mix); + event = CreateEvent(NULL, FALSE, FALSE, NULL); + ok(event != NULL, "CreateEvent(): error=%d\n", GetLastError()); + + /* NOTSUPPORTED is not broken, but it enables the todo_wine marker. */ + rc = mixerOpen(&mix, d, (DWORD_PTR)event, 0, CALLBACK_EVENT); + todo_wine + ok(rc == MMSYSERR_NOERROR /* since w2k */ || + broken(rc == MMSYSERR_NOTSUPPORTED), /* 98 */ + "mixerOpen: MMSYSERR_NOERROR expected, got %s\n", + mmsys_error(rc)); if (rc == MMSYSERR_NOERROR) test_mixerClose(mix); @@ -1054,6 +1083,10 @@ static void test_mixerOpen(void) if (rc == MMSYSERR_NOERROR) test_mixerClose(mix); + + rc = WaitForSingleObject(event, 0); + ok(rc == WAIT_TIMEOUT, "WaitEvent %d\n", rc); + CloseHandle(event); } } diff --git a/dlls/winmm/winmm.c b/dlls/winmm/winmm.c index 6d5950b..29f068b 100644 --- a/dlls/winmm/winmm.c +++ b/dlls/winmm/winmm.c @@ -173,6 +173,35 @@ BOOL WINAPI DllMain(HINSTANCE hInstDLL, DWORD fdwReason, LPVOID fImpLoad) } /************************************************************************** + * WINMM_CheckCallback [internal] + */ +static MMRESULT WINMM_CheckCallback(DWORD_PTR dwCallback, DWORD fdwOpen, BOOL mixer) +{ + switch (fdwOpen & CALLBACK_TYPEMASK) { + case CALLBACK_NULL: /* dwCallback need not be NULL */ + break; + case CALLBACK_WINDOW: + if (dwCallback && !IsWindow((HWND)dwCallback)) + return MMSYSERR_INVALPARAM; + break; + + case CALLBACK_FUNCTION: + /* a NULL cb is acceptable since w2k, MMSYSERR_INVALPARAM earlier */ + if (mixer) + return MMSYSERR_INVALFLAG; /* since w2k, MMSYSERR_NOTSUPPORTED earlier */ + break; + case CALLBACK_THREAD: + case CALLBACK_EVENT: + if (mixer) /* FIXME: mixer supports THREAD+EVENT since w2k */ + return MMSYSERR_NOTSUPPORTED; /* w9X */ + break; + default: + WARN("Unknown callback type %d\n", HIWORD(fdwOpen)); + } + return MMSYSERR_NOERROR; +} + +/************************************************************************** * Mixer devices. New to Win95 */ @@ -297,35 +326,27 @@ UINT WINAPI mixerOpen(LPHMIXER lphMix, UINT uDeviceID, DWORD_PTR dwCallback, { HANDLE hMix; LPWINE_MLD wmld; - DWORD dwRet = 0; + DWORD dwRet; MIXEROPENDESC mod; TRACE("(%p, %d, %08lx, %08lx, %08x)\n", lphMix, uDeviceID, dwCallback, dwInstance, fdwOpen); - mod.dwCallback = (DWORD_PTR)MIXER_WCallback; - mod.dwInstance = 0; + dwRet = WINMM_CheckCallback(dwCallback, fdwOpen, TRUE); + if (dwRet != MMSYSERR_NOERROR) + return dwRet; -/* If callback is a function, - * dwCallback contains function pointer - * dwInstance private data - * - * if callback is a window - * dwCallback contains a window handle - */ - switch (fdwOpen & CALLBACK_TYPEMASK) { - default: - return MMSYSERR_INVALFLAG; - - case CALLBACK_NULL: - break; - - case CALLBACK_WINDOW: + mod.dwCallback = (DWORD_PTR)MIXER_WCallback; + if ((fdwOpen & CALLBACK_TYPEMASK) == CALLBACK_WINDOW) mod.dwInstance = dwCallback; - if (dwCallback && !IsWindow((HWND)dwCallback)) - return MMSYSERR_INVALPARAM; - break; - } + else + mod.dwInstance = 0; + + /* We're remapping to CALLBACK_FUNCTION because that's what old winmm is + * documented to do when opening the mixer driver. + * FIXME: Native supports CALLBACK_EVENT + CALLBACK_THREAD flags since w2k. + * FIXME: The non ALSA drivers ignore callback requests - bug. + */ wmld = MMDRV_Alloc(sizeof(WINE_MIXER), MMDRV_MIXER, &hMix, &fdwOpen, &dwCallback, &dwInstance); @@ -901,19 +922,16 @@ UINT WINAPI midiOutOpen(LPHMIDIOUT lphMidiOut, UINT uDeviceID, { HMIDIOUT hMidiOut; LPWINE_MIDI lpwm; - UINT dwRet = 0; + UINT dwRet; TRACE("(%p, %d, %08lX, %08lX, %08X);\n", lphMidiOut, uDeviceID, dwCallback, dwInstance, dwFlags); if (lphMidiOut != NULL) *lphMidiOut = 0; - switch (dwFlags & CALLBACK_TYPEMASK) { - case CALLBACK_WINDOW: - if (dwCallback && !IsWindow((HWND)dwCallback)) - return MMSYSERR_INVALPARAM; - break; - } + dwRet = WINMM_CheckCallback(dwCallback, dwFlags, FALSE); + if (dwRet != MMSYSERR_NOERROR) + return dwRet; lpwm = MIDI_OutAlloc(&hMidiOut, &dwCallback, &dwInstance, &dwFlags, 0, NULL); @@ -1200,13 +1218,17 @@ UINT WINAPI midiInOpen(HMIDIIN* lphMidiIn, UINT uDeviceID, { HANDLE hMidiIn; LPWINE_MIDI lpwm; - DWORD dwRet = 0; + DWORD dwRet; TRACE("(%p, %d, %08lX, %08lX, %08X);\n", lphMidiIn, uDeviceID, dwCallback, dwInstance, dwFlags); if (lphMidiIn != NULL) *lphMidiIn = 0; + dwRet = WINMM_CheckCallback(dwCallback, dwFlags, FALSE); + if (dwRet != MMSYSERR_NOERROR) + return dwRet; + lpwm = (LPWINE_MIDI)MMDRV_Alloc(sizeof(WINE_MIDI), MMDRV_MIDIIN, &hMidiIn, &dwFlags, &dwCallback, &dwInstance); @@ -1758,12 +1780,9 @@ MMRESULT WINAPI midiStreamOpen(HMIDISTRM* lphMidiStrm, LPUINT lpuDeviceID, if (cMidi != 1 || lphMidiStrm == NULL || lpuDeviceID == NULL) return MMSYSERR_INVALPARAM; - switch (fdwOpen & CALLBACK_TYPEMASK) { - case CALLBACK_WINDOW: - if (dwCallback && !IsWindow((HWND)dwCallback)) - return MMSYSERR_INVALPARAM; - break; - } + ret = WINMM_CheckCallback(dwCallback, fdwOpen, FALSE); + if (ret != MMSYSERR_NOERROR) + return ret; lpMidiStrm = HeapAlloc(GetProcessHeap(), 0, sizeof(WINE_MIDIStream)); if (!lpMidiStrm) @@ -2012,7 +2031,7 @@ static UINT WAVE_Open(HANDLE* lphndl, UINT uDeviceID, UINT uType, { HANDLE handle; LPWINE_MLD wmld; - DWORD dwRet = MMSYSERR_NOERROR; + DWORD dwRet; WAVEOPENDESC wod; TRACE("(%p, %d, %s, %p, %08lX, %08lX, %08X);\n", @@ -2022,6 +2041,10 @@ static UINT WAVE_Open(HANDLE* lphndl, UINT uDeviceID, UINT uType, if (dwFlags & WAVE_FORMAT_QUERY) TRACE("WAVE_FORMAT_QUERY requested !\n"); + dwRet = WINMM_CheckCallback(dwCallback, dwFlags, FALSE); + if (dwRet != MMSYSERR_NOERROR) + return dwRet; + if (lpFormat == NULL) { WARN("bad format\n"); return WAVERR_BADFORMAT; @@ -2039,7 +2062,6 @@ static UINT WAVE_Open(HANDLE* lphndl, UINT uDeviceID, UINT uType, if ((wmld = MMDRV_Alloc(sizeof(WINE_WAVE), uType, &handle, &dwFlags, &dwCallback, &dwInstance)) == NULL) { - WARN("no memory\n"); return MMSYSERR_NOMEM; }

1 0

Gerald Pfeifer : winex11.drv: Initialize a variable in create_xcursor_frame to silence a warning.
by Alexandre Julliard 04 Apr '11

04 Apr '11

Module: wine Branch: master Commit: 4d936753321fb40a3eedd527f1b548e556e51af4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4d936753321fb40a3eedd527f… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Sun Apr 3 21:23:38 2011 +0200 winex11.drv: Initialize a variable in create_xcursor_frame to silence a warning. --- dlls/winex11.drv/mouse.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/winex11.drv/mouse.c b/dlls/winex11.drv/mouse.c index ab500ef..1a2d618 100644 --- a/dlls/winex11.drv/mouse.c +++ b/dlls/winex11.drv/mouse.c @@ -295,7 +295,7 @@ static XcursorImage *create_xcursor_frame( HDC hdc, const ICONINFOEXW *iinfo, HA { XcursorImage *image, *ret = NULL; DWORD delay_jiffies, num_steps; - int x, y, i, has_alpha; + int x, y, i, has_alpha = FALSE; XcursorPixel *ptr; wine_tsx11_lock();

1 0

Gerald Pfeifer : gdiplus: Initialize a variable in SOFTWARE_GdipFillRegion to avoid a compiler warning .
by Alexandre Julliard 04 Apr '11

04 Apr '11

Module: wine Branch: master Commit: 25764d79c01a0f01e2ece5f48e0ae09590eca1a8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=25764d79c01a0f01e2ece5f48… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Sat Apr 2 22:35:17 2011 +0200 gdiplus: Initialize a variable in SOFTWARE_GdipFillRegion to avoid a compiler warning. --- dlls/gdiplus/graphics.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 4fdfa9c..cecc880 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -3673,7 +3673,7 @@ static GpStatus SOFTWARE_GdipFillRegion(GpGraphics *graphics, GpBrush *brush, GpRectF graphics_bounds; UINT scans_count, i; INT dummy; - GpRect *scans; + GpRect *scans = NULL; DWORD *pixel_data; if (!brush_can_fill_pixels(brush))

1 0

Andrew Nguyen : msvcrt: Ensure that old buffer contents are copied when allocating a growable pf_output buffer for the first time .
by Alexandre Julliard 04 Apr '11

04 Apr '11

Module: wine Branch: master Commit: 5c57185060ce9863119d310ed380772866cb21bb URL: http://source.winehq.org/git/wine.git/?a=commit;h=5c57185060ce9863119d310ed… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Sun Apr 3 23:05:20 2011 -0500 msvcrt: Ensure that old buffer contents are copied when allocating a growable pf_output buffer for the first time. --- dlls/msvcrt/wcs.c | 32 ++++++++++++++++++++++++++------ 1 files changed, 26 insertions(+), 6 deletions(-) diff --git a/dlls/msvcrt/wcs.c b/dlls/msvcrt/wcs.c index a7c7a77..fc77656 100644 --- a/dlls/msvcrt/wcs.c +++ b/dlls/msvcrt/wcs.c @@ -429,19 +429,39 @@ static inline int pf_check_auto_grow(pf_output *out, unsigned delta) out->len = max(out->len * 2, out->used + delta); if (out->unicode) { + WCHAR *ptr; + if (out->buf.W != out->grow.W) - out->buf.W = MSVCRT_realloc(out->buf.W, out->len * sizeof(WCHAR)); + { + if (!(ptr = MSVCRT_realloc(out->buf.W, out->len * sizeof(WCHAR)))) + return -1; + } else - out->buf.W = MSVCRT_malloc(out->len * sizeof(WCHAR)); - if (!out->buf.W) return -1; + { + if (!(ptr = MSVCRT_malloc(out->len * sizeof(WCHAR)))) + return -1; + memcpy(ptr, out->buf.W, out->used * sizeof(WCHAR)); + } + + out->buf.W = ptr; } else { + char *ptr; + if (out->buf.A != out->grow.A) - out->buf.A = MSVCRT_realloc(out->buf.A, out->len * sizeof(char)); + { + if (!(ptr = MSVCRT_realloc(out->buf.A, out->len * sizeof(char)))) + return -1; + } else - out->buf.A = MSVCRT_malloc(out->len * sizeof(char)); - if (!out->buf.A) return -1; + { + if (!(ptr = MSVCRT_malloc(out->len * sizeof(char)))) + return -1; + memcpy(ptr, out->buf.A, out->used * sizeof(char)); + } + + out->buf.A = ptr; } } return 0;

1 0

Rico Schüller : d3dx9: Add argument check to D3DXCreateBuffer().
by Alexandre Julliard 04 Apr '11

04 Apr '11

Module: wine Branch: master Commit: 1fcce55894c6a79e2f85def1875784a4f4baa40a URL: http://source.winehq.org/git/wine.git/?a=commit;h=1fcce55894c6a79e2f85def18… Author: Rico Schüller <kgbricola(a)web.de> Date: Fri Apr 1 15:00:11 2011 +0200 d3dx9: Add argument check to D3DXCreateBuffer(). --- dlls/d3dx9_36/core.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/d3dx9_36/core.c b/dlls/d3dx9_36/core.c index 06a8305..8947970 100644 --- a/dlls/d3dx9_36/core.c +++ b/dlls/d3dx9_36/core.c @@ -137,6 +137,12 @@ HRESULT WINAPI D3DXCreateBuffer(DWORD size, LPD3DXBUFFER *buffer) struct ID3DXBufferImpl *object; HRESULT hr; + if (!buffer) + { + WARN("Invalid buffer specified.\n"); + return D3DERR_INVALIDCALL; + } + object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)); if (!object) {

1 0