wine-commits June 2009

wine-commits@winehq.org

5 participants
1115 discussions

Alexandre Julliard : widl: Don' t make the proxy virtual tables const if they use delegation.
by Alexandre Julliard 09 Jun '09

09 Jun '09

Module: wine Branch: master Commit: d88f9e3cd9739dd29cd5325cc3b4aae7ff2c2d5e URL: http://source.winehq.org/git/wine.git/?a=commit;h=d88f9e3cd9739dd29cd5325cc… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Jun 9 12:10:20 2009 +0200 widl: Don't make the proxy virtual tables const if they use delegation. --- tools/widl/proxy.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/tools/widl/proxy.c b/tools/widl/proxy.c index b8ccdac..cc753bb 100644 --- a/tools/widl/proxy.c +++ b/tools/widl/proxy.c @@ -624,7 +624,8 @@ static void write_proxy(type_t *iface, unsigned int *proc_offset) count = count_methods(iface); /* proxy vtable */ - print_proxy( "static const CINTERFACE_PROXY_VTABLE(%d) _%sProxyVtbl =\n", count, iface->name); + print_proxy( "static %sCINTERFACE_PROXY_VTABLE(%d) _%sProxyVtbl =\n", + need_delegation_indirect(iface) ? "" : "const ", count, iface->name); print_proxy( "{\n"); indent++; print_proxy( "{\n");

1 0

Alexandre Julliard : comctl32: Avoid repainting a NULL link.
by Alexandre Julliard 09 Jun '09

09 Jun '09

Module: wine Branch: master Commit: 6d7ad663157fc4d1446e723bc3ce5e6e3d8e80ac URL: http://source.winehq.org/git/wine.git/?a=commit;h=6d7ad663157fc4d1446e723bc… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Jun 8 18:13:22 2009 +0200 comctl32: Avoid repainting a NULL link. --- dlls/comctl32/syslink.c | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/dlls/comctl32/syslink.c b/dlls/comctl32/syslink.c index 1b89bc7..f361779 100644 --- a/dlls/comctl32/syslink.c +++ b/dlls/comctl32/syslink.c @@ -1329,10 +1329,8 @@ static LRESULT SYSLINK_SetFocus (SYSLINK_INFO *infoPtr) if(Focus != NULL) { SYSLINK_SetFocusLink(infoPtr, Focus); + SYSLINK_RepaintLink(infoPtr, Focus); } - - SYSLINK_RepaintLink(infoPtr, Focus); - return 0; } @@ -1476,7 +1474,7 @@ static BOOL SYSKEY_SelectNextPrevLink (const SYSLINK_INFO *infoPtr, BOOL Prev) { OldFocus = SYSLINK_SetFocusLink(infoPtr, NewFocus); - if(OldFocus != NewFocus) + if(OldFocus && OldFocus != NewFocus) { SYSLINK_RepaintLink(infoPtr, OldFocus); }

1 0

Alexandre Julliard : kernel32: Fix handling of %ws format for Unicode.
by Alexandre Julliard 09 Jun '09

09 Jun '09

Module: wine Branch: master Commit: b4a9c99abcfc267af6caa0d0bd802ac27fcca39e URL: http://source.winehq.org/git/wine.git/?a=commit;h=b4a9c99abcfc267af6caa0d0b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Jun 8 18:13:13 2009 +0200 kernel32: Fix handling of %ws format for Unicode. --- dlls/kernel32/format_msg.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/format_msg.c b/dlls/kernel32/format_msg.c index f4baaae..e8f9fbc 100644 --- a/dlls/kernel32/format_msg.c +++ b/dlls/kernel32/format_msg.c @@ -485,7 +485,10 @@ DWORD WINAPI FormatMessageW( if (fmtstr[strlenW(fmtstr)-1]=='s' && argliststart[0]) { DWORD xarr[3]; + WCHAR *fmt_end = fmtstr + strlenW(fmtstr) - 1; + /* remap %ws to %ls */ + if (fmt_end > fmtstr && (fmt_end[-1] == 'w')) fmt_end[-1] = 'l'; xarr[0]=*(argliststart+0); /* possible invalid pointers */ xarr[1]=*(argliststart+1);

1 0

Alexandre Julliard : user32: Disable assert() for the mingw build since mingw gets confused trying to import MessageBoxA .
by Alexandre Julliard 09 Jun '09

09 Jun '09

Module: wine Branch: master Commit: 031c73af3078b0c7fc45d0aa94c3852097940c65 URL: http://source.winehq.org/git/wine.git/?a=commit;h=031c73af3078b0c7fc45d0aa9… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Jun 8 16:45:54 2009 +0200 user32: Disable assert() for the mingw build since mingw gets confused trying to import MessageBoxA. --- dlls/user32/user_private.h | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/user32/user_private.h b/dlls/user32/user_private.h index f1a6585..1ff6ef4 100644 --- a/dlls/user32/user_private.h +++ b/dlls/user32/user_private.h @@ -354,4 +354,10 @@ typedef struct extern void CURSORICON_FreeModuleIcons( HMODULE16 hModule ) DECLSPEC_HIDDEN; +/* Mingw's assert() imports MessageBoxA and gets confused by user32 exporting it */ +#ifdef __MINGW32__ +#undef assert +#define assert(expr) ((void)0) +#endif + #endif /* __WINE_USER_PRIVATE_H */

1 0

Jeff Zaroyko : website: remove section about bugzilla FIXME keyword indicating a tasklist, this keyword is discouraged/ up for removal since nearly every application outputs fixmes.
by Jeremy Newman 08 Jun '09

08 Jun '09

Module: website Branch: master Commit: 19bf3e2df1bff2c6325a81c40f8f108cbc7e859b URL: http://source.winehq.org/git/website.git/?a=commit;h=19bf3e2df1bff2c6325a81… Author: Jeff Zaroyko <jeffzaroyko(a)gmail.com> Date: Mon Jun 8 09:50:34 2009 -0500 website: remove section about bugzilla FIXME keyword indicating a tasklist, this keyword is discouraged/up for removal since nearly every application outputs fixmes. --- templates/de/contributing.template | 9 --------- templates/en/contributing.template | 8 -------- templates/pt/contributing.template | 7 ------- 3 files changed, 0 insertions(+), 24 deletions(-) diff --git a/templates/de/contributing.template b/templates/de/contributing.template index 7c0ab2e..be57a75 100644 --- a/templates/de/contributing.template +++ b/templates/de/contributing.template @@ -165,15 +165,6 @@ verlangen, dass Du mit Lex/Yacc umgehen kannst. Falls du also spezielles Domänenwissen hast, dann guck doch mal über die Liste drüber.</li> - <li id="FIXMEs"><a href="http://bugs.winehq.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&a…">FIXMEs (en)</a>: - Diese Liste enthält Aufgaben, deren Komplexität zwischen - den Aufgäbchen und den Aufgaben liegt. Die hier aufgeführten - Probleme sind gut diagnostiziert; mit anderen Worten, wir wissen, - welche Funktion oder Funktionalität fehlt oder fehlerhaft ist. - Und auch wenn der Fehler nicht unbedingt einfach zu beheben ist, - sollte trotzdem weder extensives Wissen über mehrere Bereiche - von Wine noch eine umfängliche Reorganisation des Quellcodes - von Wine vonnöten sein.</li> <li id="tasklist"><a href="http://bugs.winehq.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&a…">Aufgaben (en)</a>: Hier sind die größeren Entwicklungsaufgaben aufgeführt. Guck hier nach, wenn du an einem der größeren / diff --git a/templates/en/contributing.template b/templates/en/contributing.template index 3aee073..a652f33 100644 --- a/templates/en/contributing.template +++ b/templates/en/contributing.template @@ -145,14 +145,6 @@ Some tasks also draw on non-Wine specific knowledge. For instance a task may require being knowledgeable in Lex/Yacc so if you have an expertise in a given domain you may want to scan it.</li> - <li id="FIXMEs"><a href="http://bugs.winehq.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&a…">FIXMEs</a>: - This list contains tasks that are intermediate between those found - in the Task list and the Task lets. Issues found in this list are - well diagnosed, that is, we know which function or functionality is - missing / buggy. And while the fix may not be very easy to - implement, it normally is not thought to require extensive - knowledge of multiple areas of Wine, or extensive reorganization - of the Wine code.</li> <li id="tasklist"><a href="http://bugs.winehq.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&a…">Tasklist</a>: The major development tasks are listed there. This is the place to check out if you want to get involved in one of the larger / diff --git a/templates/pt/contributing.template b/templates/pt/contributing.template index ed61e56..4009d74 100644 --- a/templates/pt/contributing.template +++ b/templates/pt/contributing.template @@ -151,13 +151,6 @@ também se focam em conhecimento não específico do Wine. Por exemplo uma tarefa pode necessitar que seja um conhecedor de Lex/Yacc, portanto se você tem vasta experiência num certo domínio pode querer procurar por ele.</li> - <li id="FIXMEs"><a href="http://bugs.winehq.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&a…">FIXMEs</a>: - Esta lista contém tarefas que são intermédias entre a Task List e a Task lets. - Assuntos encontrados nesta lista estão bem diagnosticados, isto é, sabe-se - qual a função ou funcionalidade em falta / incompleta. E enquanto o arranjo - possa não ser muito fácil de implementar, normalmente não é necessário um - conhecimento de múltiplas áreas do Wine ou uma reorganização extensa do - código Wine.</li> <li id="tasklist"><a href="http://bugs.winehq.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&a…">Tasklist</a>: As maiores tarefas de desenvolvimento estão listadas aqui. Este é o lugar a visitar se se quer envolver num dos projectos maiores, a longo prazo. Bugs

1 0

Jeff Zaroyko : website: fix capitalisation of GIT -> Git, consistent with usage on the Git homepage since it's not really an acronym
by Jeremy Newman 08 Jun '09

08 Jun '09

Module: website Branch: master Commit: 86180f8d954647fdc6782fbb92448719d6ec9dcc URL: http://source.winehq.org/git/website.git/?a=commit;h=86180f8d954647fdc6782f… Author: Jeff Zaroyko <jeffzaroyko(a)gmail.com> Date: Mon Jun 8 09:49:48 2009 -0500 website: fix capitalisation of GIT -> Git, consistent with usage on the Git homepage since it's not really an acronym --- templates/de/devel.template | 2 +- templates/en/cvs.template | 4 ++-- templates/en/devel.template | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/templates/de/devel.template b/templates/de/devel.template index 087eba6..4957064 100644 --- a/templates/de/devel.template +++ b/templates/de/devel.template @@ -62,7 +62,7 @@ Hilfsmittel, die wir unseren Entwicklern zur Verfügung stellen.</p> <tr bgcolor="#ffffff"> <td> - <a href="http://wiki.winehq.org/GitWine">GIT</a> + <a href="http://wiki.winehq.org/GitWine">Git</a> </td> <td valign="top"> Wir nutzen Git zur Quellcodeverwaltung. Auf dieser Seite findest Du Informationen über den Umgang damit. diff --git a/templates/en/cvs.template b/templates/en/cvs.template index f742f86..358ba0b 100644 --- a/templates/en/cvs.template +++ b/templates/en/cvs.template @@ -3,8 +3,8 @@ <h1 class="title">The Wine CVS Tree</h1> -<p><strong>Note: We are switching development from CVS to <a href="http://git.or.cz">Git</a>. - Instructions for using the <a href="{$root}/site/git">Wine GIT tree</a> +<p><strong>Note: We have switched development from CVS to <a href="http://git.or.cz">Git</a>. + Instructions for using the <a href="{$root}/site/git">Wine Git tree</a> can be found on the <a href="http://wiki.winehq.org/GitWine">GitWine</a> page on the <a href="http://wiki.winehq.org">Wine Wiki</a>.</strong></p> diff --git a/templates/en/devel.template b/templates/en/devel.template index 82b9f1d..ddb54a8 100644 --- a/templates/en/devel.template +++ b/templates/en/devel.template @@ -64,7 +64,7 @@ various resources available for our developers.</p> <tr bgcolor="#ffffff"> <td> - <a href="http://wiki.winehq.org/GitWine">GIT</a> + <a href="http://wiki.winehq.org/GitWine">Git</a> </td> <td valign="top"> We use the source management tool Git. More information on how to use it can be found on this page.

1 0

Jeff Zaroyko : website: mention bugzilla registration on sending_patches
by Jeremy Newman 08 Jun '09

08 Jun '09

Module: website Branch: master Commit: 690d5fb3489022ac5a4405a1d4784fb39cdaeb76 URL: http://source.winehq.org/git/website.git/?a=commit;h=690d5fb3489022ac5a4405… Author: Jeff Zaroyko <jeffzaroyko(a)gmail.com> Date: Mon Jun 8 09:49:18 2009 -0500 website: mention bugzilla registration on sending_patches --- templates/en/sending_patches.template | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/templates/en/sending_patches.template b/templates/en/sending_patches.template index f185117..d4b2ac6 100644 --- a/templates/en/sending_patches.template +++ b/templates/en/sending_patches.template @@ -38,6 +38,8 @@ or anywhere else to the Git tree.) <li>Use standard C, avoid C++ comments and inline variable declarations. <li>Read the diff and make sure it contains only what you want in it. + + <li>Register an account on the Wine bugzilla with the email address you use to send patches. </ul> <p>Accepted patches will appear in the <a href="{$root}/site/forums">wine-cvs</a> mailing list.

1 0

Paul Vriens : setupapi/tests: pSetupGetField returns A or W string depending on OS.
by Alexandre Julliard 08 Jun '09

08 Jun '09

Module: wine Branch: master Commit: de945384a4c1f593820e91811c1c04ff0263ca48 URL: http://source.winehq.org/git/wine.git/?a=commit;h=de945384a4c1f593820e91811… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Mon Jun 8 16:09:51 2009 +0200 setupapi/tests: pSetupGetField returns A or W string depending on OS. --- dlls/setupapi/tests/parser.c | 83 ++++++++++++++++++++++++++++++++---------- 1 files changed, 64 insertions(+), 19 deletions(-) diff --git a/dlls/setupapi/tests/parser.c b/dlls/setupapi/tests/parser.c index a67a5cd..f12d02d 100644 --- a/dlls/setupapi/tests/parser.c +++ b/dlls/setupapi/tests/parser.c @@ -32,14 +32,18 @@ /* function pointers */ static HMODULE hSetupAPI; -static LPCWSTR (WINAPI *pSetupGetField)(PINFCONTEXT,DWORD); +static LPCSTR (WINAPI *pSetupGetFieldA)(PINFCONTEXT,DWORD); +static LPCWSTR (WINAPI *pSetupGetFieldW)(PINFCONTEXT,DWORD); static BOOL (WINAPI *pSetupEnumInfSectionsA)( HINF hinf, UINT index, PSTR buffer, DWORD size, UINT *need ); static void init_function_pointers(void) { hSetupAPI = GetModuleHandleA("setupapi.dll"); - pSetupGetField = (void *)GetProcAddress(hSetupAPI, "pSetupGetField"); + /* Nice, pSetupGetField is either A or W depending on the Windows version! The actual test + * takes care of this difference */ + pSetupGetFieldA = (void *)GetProcAddress(hSetupAPI, "pSetupGetField"); + pSetupGetFieldW = (void *)GetProcAddress(hSetupAPI, "pSetupGetField"); pSetupEnumInfSectionsA = (void *)GetProcAddress(hSetupAPI, "SetupEnumInfSectionsA" ); } @@ -497,7 +501,14 @@ static const char *contents = "[Version]\n" "[Strings]\n" "RTMQFE_NAME = \"RTMQFE\"\n"; -static const WCHAR getfield_res[][20] = +static const CHAR getfield_resA[][20] = +{ + "RTMQFE", + "%RTMGFE_NAME%", + "SP1RTM", +}; + +static const WCHAR getfield_resW[][20] = { {'R','T','M','Q','F','E',0}, {'%','R','T','M','G','F','E','_','N','A','M','E','%',0}, @@ -509,10 +520,20 @@ static void test_pSetupGetField(void) UINT err; BOOL ret; HINF hinf; - LPCWSTR field; + LPCSTR fieldA; + LPCWSTR fieldW; INFCONTEXT context; int i; int len; + BOOL unicode = TRUE; + + SetLastError(0xdeadbeef); + lstrcmpW(NULL, NULL); + if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED) + { + win_skip("Using A-functions instead of W\n"); + unicode = FALSE; + } hinf = test_file_contents( contents, &err ); ok( hinf != NULL, "Expected valid INF file\n" ); @@ -524,23 +545,47 @@ static void test_pSetupGetField(void) for ( i = 0; i < 3; i++ ) { - field = pSetupGetField( &context, i ); - ok( field != NULL, "Failed to get field %i\n", i ); - ok( !lstrcmpW( getfield_res[i], field ), "Wrong string returned\n" ); + if (unicode) + { + fieldW = pSetupGetFieldW( &context, i ); + ok( fieldW != NULL, "Failed to get field %i\n", i ); + ok( !lstrcmpW( getfield_resW[i], fieldW ), "Wrong string returned\n" ); + } + else + { + fieldA = pSetupGetFieldA( &context, i ); + ok( fieldA != NULL, "Failed to get field %i\n", i ); + ok( !lstrcmpA( getfield_resA[i], fieldA ), "Wrong string returned\n" ); + } } - field = pSetupGetField( &context, 3 ); - ok( field != NULL, "Failed to get field 3\n" ); - len = lstrlenW( field ); - ok( len == 511 /* NT4, W2K, XP and W2K3 */ || - len == 4096 /* Vista */ || - len == 256 /* Win9x and WinME */, - "Unexpected length, got %d\n", len ); - - field = pSetupGetField( &context, 4 ); - ok( field == NULL, "Expected NULL, got %p\n", field ); - ok( GetLastError() == ERROR_INVALID_PARAMETER, - "Expected ERROR_INVALID_PARAMETER, got %u\n", GetLastError() ); + if (unicode) + { + fieldW = pSetupGetFieldW( &context, 3 ); + ok( fieldW != NULL, "Failed to get field 3\n" ); + len = lstrlenW( fieldW ); + ok( len == 511 || /* NT4, W2K, XP and W2K3 */ + len == 4096, /* Vista */ + "Unexpected length, got %d\n", len ); + + fieldW = pSetupGetFieldW( &context, 4 ); + ok( fieldW == NULL, "Expected NULL, got %p\n", fieldW ); + ok( GetLastError() == ERROR_INVALID_PARAMETER, + "Expected ERROR_INVALID_PARAMETER, got %u\n", GetLastError() ); + } + else + { + fieldA = pSetupGetFieldA( &context, 3 ); + ok( fieldA != NULL, "Failed to get field 3\n" ); + len = lstrlenA( fieldA ); + ok( len == 511, /* Win9x, WinME */ + "Unexpected length, got %d\n", len ); + + fieldA = pSetupGetFieldA( &context, 4 ); + ok( fieldA == NULL, "Expected NULL, got %p\n", fieldA ); + ok( GetLastError() == ERROR_INVALID_PARAMETER, + "Expected ERROR_INVALID_PARAMETER, got %u\n", GetLastError() ); + } SetupCloseInfFile( hinf ); }

1 0

Vincent Povirk : comctl32: Use NMTVDISPINFOEX in the TVN_GETDISPINFO notification.
by Alexandre Julliard 08 Jun '09

08 Jun '09

Module: wine Branch: master Commit: 49308e8da0705f8207f889b06d6f33919199aebf URL: http://source.winehq.org/git/wine.git/?a=commit;h=49308e8da0705f8207f889b06… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Tue May 26 14:40:04 2009 -0500 comctl32: Use NMTVDISPINFOEX in the TVN_GETDISPINFO notification. --- dlls/comctl32/treeview.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/comctl32/treeview.c b/dlls/comctl32/treeview.c index d01a127..8bea973 100644 --- a/dlls/comctl32/treeview.c +++ b/dlls/comctl32/treeview.c @@ -710,7 +710,7 @@ static void TREEVIEW_UpdateDispInfo(const TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *wineItem, UINT mask) { - NMTVDISPINFOW callback; + NMTVDISPINFOEXW callback; HWND hwnd = infoPtr->hwnd; TRACE("mask %x callbackMask %x\n", mask, wineItem->callbackMask);

1 0

Vincent Povirk : include: Extend TVITEMEX and add NMTVDISPINFOEX.
by Alexandre Julliard 08 Jun '09

08 Jun '09

Module: wine Branch: master Commit: 715928301182dfc8bc38a00201a463f45368043b URL: http://source.winehq.org/git/wine.git/?a=commit;h=715928301182dfc8bc38a0020… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Tue May 26 14:33:18 2009 -0500 include: Extend TVITEMEX and add NMTVDISPINFOEX. --- include/commctrl.h | 18 ++++++++++++++++++ 1 files changed, 18 insertions(+), 0 deletions(-) diff --git a/include/commctrl.h b/include/commctrl.h index e39f4a2..7d77faa 100644 --- a/include/commctrl.h +++ b/include/commctrl.h @@ -2506,6 +2506,9 @@ typedef struct { INT cChildren; LPARAM lParam; INT iIntegral; + UINT uStateEx; /* _WIN32_IE >= 0x600 */ + HWND hwnd; /* _WIN32_IE >= 0x600 */ + INT iExpandedImage; /* _WIN32_IE >= 0x600 */ } TVITEMEXA, *LPTVITEMEXA; typedef struct { @@ -2520,6 +2523,9 @@ typedef struct { INT cChildren; LPARAM lParam; INT iIntegral; + UINT uStateEx; /* _WIN32_IE >= 0x600 */ + HWND hwnd; /* _WIN32_IE >= 0x600 */ + INT iExpandedImage; /* _WIN32_IE >= 0x600 */ } TVITEMEXW, *LPTVITEMEXW; #define TVITEMEX WINELIB_NAME_AW(TVITEMEX) @@ -2595,8 +2601,20 @@ typedef struct tagTVDISPINFOW { TVITEMW item; } NMTVDISPINFOW, *LPNMTVDISPINFOW; +typedef struct tagTVDISPINFOEXA { + NMHDR hdr; + TVITEMEXA item; +} NMTVDISPINFOEXA, *LPNMTVDISPINFOEXA; + +typedef struct tagTVDISPINFOEXW { + NMHDR hdr; + TVITEMEXW item; +} NMTVDISPINFOEXW, *LPNMTVDISPINFOEXW; + #define NMTVDISPINFO WINELIB_NAME_AW(NMTVDISPINFO) #define LPNMTVDISPINFO WINELIB_NAME_AW(LPNMTVDISPINFO) +#define NMTVDISPINFOEX WINELIB_NAME_AW(NMTVDISPINFOEX) +#define LPNMTVDISPINFOEX WINELIB_NAME_AW(LPNMTVDISPINFOEX) #define TV_DISPINFOA NMTVDISPINFOA #define TV_DISPINFOW NMTVDISPINFOW #define TV_DISPINFO NMTVDISPINFO

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits June 2009