June 2007 - Wine-commits - list.winehq.org

Alexander Nicolaysen Sørnes : wordpad: Split code into set_caption function.
by Alexandre Julliard 04 Jun '07

04 Jun '07

Module: wine Branch: master Commit: e173504ccd14f30cfac4662b7594bad34d193e33 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e173504ccd14f30cfac4662b7… Author: Alexander Nicolaysen Sørnes <alex(a)thehandofagony.com> Date: Wed May 30 14:42:36 2007 +0200 wordpad: Split code into set_caption function. --- programs/wordpad/wordpad.c | 36 +++++++++++++++++++++--------------- 1 files changed, 21 insertions(+), 15 deletions(-) diff --git a/programs/wordpad/wordpad.c b/programs/wordpad/wordpad.c index 4655b2d..e131527 100644 --- a/programs/wordpad/wordpad.c +++ b/programs/wordpad/wordpad.c @@ -114,7 +114,24 @@ static DWORD CALLBACK stream_in(DWORD_PTR cookie, LPBYTE buffer, LONG cb, LONG * return 0; } -static void DoOpenFile(LPCWSTR szFileName) +static void set_caption(LPCWSTR wszNewFileName) +{ + static const WCHAR wszSeparator[] = {' ','-',' ','\0'}; + WCHAR wszCaption[MAX_PATH]; + + if(wszNewFileName) + { + lstrcpyW(wszCaption, wszNewFileName); + lstrcatW(wszCaption, wszSeparator); + lstrcatW(wszCaption, wszAppTitle); + SetWindowTextW(hMainWnd, wszCaption); + } else + { + SetWindowTextW(hMainWnd, wszAppTitle); + } +} + +static void DoOpenFile(LPCWSTR szOpenFileName) { HANDLE hFile; LPSTR pTemp; @@ -122,11 +139,7 @@ static void DoOpenFile(LPCWSTR szFileName) DWORD dwNumRead; EDITSTREAM es; - char szCaption[MAX_PATH]; - char szAppTitle[sizeof(wszAppTitle)]; - char szSeparator[] = " - "; - - hFile = CreateFileW(szFileName, GENERIC_READ, FILE_SHARE_READ, NULL, + hFile = CreateFileW(szOpenFileName, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); if (hFile == INVALID_HANDLE_VALUE) return; @@ -166,14 +179,7 @@ static void DoOpenFile(LPCWSTR szFileName) SetFocus(hEditorWnd); - WideCharToMultiByte(CP_ACP, 0, wszAppTitle, -1, szAppTitle, sizeof(wszAppTitle), NULL, NULL); - - WideCharToMultiByte(CP_ACP, 0, szFileName, -1, szCaption, MAX_PATH, NULL, NULL); - - lstrcat(szCaption, szSeparator); - lstrcat(szCaption, szAppTitle); - - SetWindowText(hMainWnd, szCaption); + set_caption(szOpenFileName); } static void DialogOpenFile(void) @@ -435,7 +441,7 @@ static LRESULT OnCommand( HWND hWnd, WPARAM wParam, LPARAM lParam) case ID_FILE_NEW: SetWindowTextA(hwndEditor, ""); - SetWindowTextW(hMainWnd, wszAppTitle); + set_caption(NULL); /* FIXME: set default format too */ break;

1 0

Alexandre Julliard : ws2_32: Reimplement WSASendTo along the lines of NtWriteFile.
by Alexandre Julliard 04 Jun '07

04 Jun '07

Module: wine Branch: master Commit: 7bee1d358366b5845a59b822e043a8017c10e626 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7bee1d358366b5845a59b822e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu May 31 17:02:21 2007 +0200 ws2_32: Reimplement WSASendTo along the lines of NtWriteFile. --- dlls/ws2_32/socket.c | 144 ++++++++++++++++++++++++++----------------------- 1 files changed, 76 insertions(+), 68 deletions(-) diff --git a/dlls/ws2_32/socket.c b/dlls/ws2_32/socket.c index 2bd4c14..a616172 100644 --- a/dlls/ws2_32/socket.c +++ b/dlls/ws2_32/socket.c @@ -2691,9 +2691,8 @@ INT WINAPI WSASendTo( SOCKET s, LPWSABUF lpBuffers, DWORD dwBufferCount, LPWSAOVERLAPPED_COMPLETION_ROUTINE lpCompletionRoutine ) { unsigned int i, options; - int n, fd, err = WSAENOTSOCK, ret; + int n, fd, err; struct iovec iovec[WS_MSG_MAXIOVLEN]; - struct ws2_async *wsa; IO_STATUS_BLOCK* iosb; TRACE("socket %04lx, wsabuf %p, nbufs %d, flags %d, to %p, tolen %d, ovl %p, func %p\n", @@ -2723,105 +2722,114 @@ INT WINAPI WSASendTo( SOCKET s, LPWSABUF lpBuffers, DWORD dwBufferCount, iovec[i].iov_len = lpBuffers[i].len; } - if ( (lpOverlapped || lpCompletionRoutine) && - !(options & (FILE_SYNCHRONOUS_IO_ALERT | FILE_SYNCHRONOUS_IO_NONALERT))) + for (;;) { - wsa = WS2_make_async( s, ws2m_write, iovec, dwBufferCount, - &dwFlags, (struct WS_sockaddr*) to, &tolen, - lpOverlapped, lpCompletionRoutine, &iosb ); - if ( !wsa ) + n = WS2_send( fd, iovec, dwBufferCount, to, tolen, dwFlags ); + if (n != -1 || errno != EINTR) break; + } + if (n == -1) + { + if (errno != EAGAIN) { - err = WSAEFAULT; + err = wsaErrno(); goto error; } - if ((ret = ws2_queue_async( wsa, iosb )) != STATUS_PENDING) + if ((lpOverlapped || lpCompletionRoutine) && + !(options & (FILE_SYNCHRONOUS_IO_ALERT | FILE_SYNCHRONOUS_IO_NONALERT))) { - err = NtStatusToWSAError( ret ); + struct ws2_async *wsa = WS2_make_async( s, ws2m_write, iovec, dwBufferCount, + &dwFlags, (struct WS_sockaddr*) to, &tolen, + lpOverlapped, lpCompletionRoutine, &iosb ); + if ( !wsa ) + { + err = WSAEFAULT; + goto error; + } + release_sock_fd( s, fd ); - HeapFree( GetProcessHeap(), 0, wsa ); - goto error; - } - release_sock_fd( s, fd ); + iosb->u.Status = STATUS_PENDING; + iosb->Information = 0; - /* Try immediate completion */ - if ( lpOverlapped ) - { - if ( WSAGetOverlappedResult( s, lpOverlapped, - lpNumberOfBytesSent, FALSE, &dwFlags) ) - return 0; + SERVER_START_REQ( register_async ) + { + req->handle = wsa->hSocket; + req->type = ASYNC_TYPE_WRITE; + req->async.callback = WS2_async_send; + req->async.iosb = iosb; + req->async.arg = wsa; + req->async.event = lpCompletionRoutine ? 0 : lpOverlapped->hEvent; + err = wine_server_call( req ); + } + SERVER_END_REQ; - if ( (err = WSAGetLastError()) != WSA_IO_INCOMPLETE ) - return SOCKET_ERROR; - } + if (err == STATUS_PENDING) + NtCurrentTeb()->num_async_io++; + else + ws2_async_terminate(wsa, iosb, err, 0); - WSASetLastError( WSA_IO_PENDING ); - return SOCKET_ERROR; + WSASetLastError( NtStatusToWSAError( err )); + return SOCKET_ERROR; + } } - *lpNumberOfBytesSent = 0; if ( _is_blocking(s) ) { /* On a blocking non-overlapped stream socket, * sending blocks until the entire buffer is sent. */ - struct iovec *piovec = iovec; - int finish_time = GET_SNDTIMEO(fd); - if (finish_time >= 0) - finish_time += GetTickCount(); - while ( dwBufferCount > 0 ) + DWORD timeout_start = GetTickCount(); + unsigned int first_buff = 0; + + *lpNumberOfBytesSent = 0; + + while (first_buff < dwBufferCount) { - int timeout; - if ( finish_time >= 0 ) + struct pollfd pfd; + int timeout = GET_SNDTIMEO(fd); + + if (n > 0) { - timeout = finish_time - GetTickCount(); - if ( timeout < 0 ) - timeout = 0; + *lpNumberOfBytesSent += n; + if (iovec[first_buff].iov_len > n) + iovec[first_buff].iov_len -= n; + else + { + while (n > 0) n -= iovec[first_buff++].iov_len; + if (first_buff >= dwBufferCount) break; + } } - else - timeout = finish_time; - /* FIXME: exceptfds? */ - if( !do_block(fd, POLLOUT, timeout)) { + + if (timeout != -1) + { + timeout -= timeout_start - GetTickCount(); + if (timeout < 0) timeout = 0; + } + + pfd.fd = fd; + pfd.events = POLLOUT; + + if (!timeout || !poll( &pfd, 1, timeout )) + { err = WSAETIMEDOUT; goto error; /* msdn says a timeout in send is fatal */ } - n = WS2_send( fd, piovec, dwBufferCount, to, tolen, dwFlags ); - if ( n == -1 ) + n = WS2_send( fd, iovec + first_buff, dwBufferCount - first_buff, to, tolen, dwFlags ); + if (n == -1 && errno != EAGAIN && errno != EINTR) { err = wsaErrno(); goto error; } - *lpNumberOfBytesSent += n; - - while ( n > 0 ) - { - if ( piovec->iov_len > n ) - { - piovec->iov_base = (char*)piovec->iov_base + n; - piovec->iov_len -= n; - n = 0; - } - else - { - n -= piovec->iov_len; - --dwBufferCount; - ++piovec; - } - } } } - else + else /* non-blocking */ { - n = WS2_send( fd, iovec, dwBufferCount, to, tolen, dwFlags ); - if ( n == -1 ) + _enable_event(SOCKET2HANDLE(s), FD_WRITE, 0, 0); + if (n == -1) { - err = wsaErrno(); - if ( err == WSAEWOULDBLOCK ) - _enable_event(SOCKET2HANDLE(s), FD_WRITE, 0, 0); + err = WSAEWOULDBLOCK; goto error; } - else - _enable_event(SOCKET2HANDLE(s), FD_WRITE, 0, 0); *lpNumberOfBytesSent = n; }

1 0

wiki/winehq/css common.css
by Dimi Paun 04 Jun '07

04 Jun '07

ChangeSet ID: 1180923848893135093226196 CVSROOT: /cvsroot/wine Module name: wiki Changes by: dimi(a)sc8-pr-cvs9.sourceforge.net 2007/06/03 19:24:08 Modified files: winehq/css : common.css Log message: Vitaliy Margolen <wine-patches(a)kievinfo.com> Fix table cell alignment and different size of text in bullet lists. Old revision New revision Changes Path 1.2 1.3 +1 -1 wiki/winehq/css/common.css Index: wiki/winehq/css/common.css diff -u -p wiki/winehq/css/common.css:1.2 wiki/winehq/css/common.css:1.3 --- wiki/winehq/css/common.css 4 Jun 2007 2:24: 8 -0000 +++ /dev/null 4 Jun 2007 2:24: 8 -0000 @@ -102,7 +102,7 @@ td border: 1px solid tan; } -td { +td p { margin: 0; padding: 0; }

1 0

lostwages/ include/winehq.conf news/2007060101.xml
by Alexandre Julliard 01 Jun '07

01 Jun '07

ChangeSet ID: 31156 CVSROOT: /opt/cvs-commit Module name: lostwages Changes by: julliard(a)winehq.org 2007/06/01 13:11:11 Modified files: include : winehq.conf Added files: news : 2007060101.xml Log message: Release 0.9.38 Patch: http://cvs.winehq.org/patch.py?id=31156 Old revision New revision Changes Path 1.60 1.61 +1 -1 lostwages/include/winehq.conf Added 1.1 +0 -0 lostwages/news/2007060101.xml Index: lostwages/include/winehq.conf diff -u -p lostwages/include/winehq.conf:1.60 lostwages/include/winehq.conf:1.61 --- lostwages/include/winehq.conf:1.60 1 Jun 2007 18:11:11 -0000 +++ lostwages/include/winehq.conf 1 Jun 2007 18:11:11 -0000 @@ -29,7 +29,7 @@ news_xml_path: news wwn_xml_path: wwn # Snapshot Date -snapshot_date: 0.9.37 +snapshot_date: 0.9.38 # Current RH Packages Snapshot Date snapshot_date_rh: 0.9.2 Index: lostwages/news/2007060101.xml diff -u -p /dev/null lostwages/news/2007060101.xml:1.1 --- /dev/null 1 Jun 2007 18:11:11 -0000 +++ lostwages/news/2007060101.xml 1 Jun 2007 18:11:11 -0000 @@ -0,0 +1,25 @@ +<news> +<date>June 1, 2007</date> +<title>Wine 0.9.38 Released</title> +<body> +<p>Wine 0.9.38 was released today, with the following main changes:</p> + +<ul> +<li>Beginnings of support for copy protection kernel drivers.</li> +<li>More MSI automation support.</li> +<li>Many 64-bit compilation fixes.</li> +<li>A number of OLE fixes.</li> +<li>Lots of bug fixes.</li> +</ul> + +<p> +Binary packages are in the process of being built and it may take a +few days for them to appear, but the source is +<a href="http://prdownloads.sourceforge.net/wine/wine-0.9.38.tar.bz2">available now</a>. +You can find out more about this release in the +<a href="?announce=0.9.38">announcement</a>. Check out our +<a href="/site/download">download page</a> for packages for your +distribution. +</p> +</body> +</news>

1 0

Alexandre Julliard : Release 0.9.38.
by Alexandre Julliard 01 Jun '07

01 Jun '07

Module: wine Branch: master Commit: c0bf8e7db5f3817c2cc8a4c48fa28f6414b03942 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c0bf8e7db5f3817c2cc8a4c48… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jun 1 19:49:52 2007 +0200 Release 0.9.38. --- ANNOUNCE | 910 ++++++++++++++++------------ ChangeLog | 2057 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ VERSION | 2 +- configure | 18 +- 4 files changed, 2593 insertions(+), 394 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=c0bf8e7db5f3817c2cc8a…

1 0

Tag wine-0.9.38 : Alexandre Julliard : Release 0.9.38
by Alexandre Julliard 01 Jun '07

01 Jun '07

Module: wine Branch: refs/tags/wine-0.9.38 Tag: d3ce69c08f0bed7db571c49048d4a34ddf05ed83 URL: http://source.winehq.org/git/wine.git/?a=tag;h=d3ce69c08f0bed7db571c49048d4… Tagger: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jun 1 19:49:53 2007 +0200 Release 0.9.38

1 0

lostwages/. winehq_styles.css
by Jeremy Newman 01 Jun '07

01 Jun '07

ChangeSet ID: 31155 CVSROOT: /opt/cvs-commit Module name: lostwages Changes by: jnewman(a)winehq.org 2007/06/01 09:19:04 Modified files: . : winehq_styles.css Log message: Alexander Q. Waite <awaite2(a)uiuc.edu> Fix CSS and implement more rules Patch: http://cvs.winehq.org/patch.py?id=31155 Old revision New revision Changes Path 1.17 1.18 +9 -1 lostwages/winehq_styles.css Index: lostwages/winehq_styles.css diff -u -p lostwages/winehq_styles.css:1.17 lostwages/winehq_styles.css:1.18 --- lostwages/winehq_styles.css:1.17 1 Jun 2007 14:19: 4 -0000 +++ lostwages/winehq_styles.css 1 Jun 2007 14:19: 4 -0000 @@ -8,6 +8,7 @@ BODY { } img { border: 0; } +dt { margin: 10px 0; } /* Link Colors */ A { color: #A50D0D; } @@ -55,6 +56,7 @@ p.indent { padding-left: 1em; .strike { text-decoration: line-through; } .bold { font-weight: bold; } .newstitle { font-size: 14px; font-weight: bold; color: #000000; } +.underline { text-decoration: underline; } /* Menu Config */ .menuTitle { font-size: 12px; color: #ffffff; font-weight: bold; text-decoration: none; } @@ -74,7 +76,7 @@ p.indent { padding-left: 1em; .small { font-size: 10px; } .big { font-size: 14px; } .rowtitle { background-color: black; color: white; font-weight: bold; } -.commandWidth { width: 500px; } +.commandWidth { width: 500px; padding: 5px; } /* Generic Buttons */ .button { background-color: #A50D0D; color: #ffffff; font-family: Sans-Serif; font-size: 12px; padding: 0; } @@ -83,6 +85,12 @@ p.indent { padding-left: 1em; /* Note Blip */ .blip { color: #ff0000; font-size: 10px; } +/* template navigation links */ +.link-container { position: relative; height: 1.2em; } +.link-container p { position: absolute; width: 50%; margin: 0; } +.left-link { left: 0; } +.right-link { right: 0; text-align: right; } + /* Footer */ .footer { font-size: 8pt; color: #777777; }

1 0

Andrew Talbot : msi: Constify some variables.
by Alexandre Julliard 01 Jun '07

01 Jun '07

Module: wine Branch: master Commit: 1486d851da87927a8f6bc13235eb36e928933fe3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1486d851da87927a8f6bc1323… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Thu May 31 22:19:28 2007 +0100 msi: Constify some variables. --- dlls/msi/appsearch.c | 6 +++--- dlls/msi/classes.c | 8 ++++---- dlls/msi/cond.y | 8 ++++---- dlls/msi/custom.c | 6 +++--- dlls/msi/msipriv.h | 2 +- 5 files changed, 15 insertions(+), 15 deletions(-) diff --git a/dlls/msi/appsearch.c b/dlls/msi/appsearch.c index ea2f7cf..8db9cb9 100644 --- a/dlls/msi/appsearch.c +++ b/dlls/msi/appsearch.c @@ -474,7 +474,7 @@ static void ACTION_ExpandAnyPath(MSIPACKAGE *package, WCHAR *src, WCHAR *dst, * Return ERROR_SUCCESS in case of success (whether or not the file matches), * something else if an install-halting error occurs. */ -static UINT ACTION_FileVersionMatches(MSISIGNATURE *sig, LPCWSTR filePath, +static UINT ACTION_FileVersionMatches(const MSISIGNATURE *sig, LPCWSTR filePath, BOOL *matches) { UINT rc = ERROR_SUCCESS; @@ -549,8 +549,8 @@ static UINT ACTION_FileVersionMatches(MSISIGNATURE *sig, LPCWSTR filePath, * Return ERROR_SUCCESS in case of success (whether or not the file matches), * something else if an install-halting error occurs. */ -static UINT ACTION_FileMatchesSig(MSISIGNATURE *sig, - LPWIN32_FIND_DATAW findData, LPCWSTR fullFilePath, BOOL *matches) +static UINT ACTION_FileMatchesSig(const MSISIGNATURE *sig, + const WIN32_FIND_DATAW *findData, LPCWSTR fullFilePath, BOOL *matches) { UINT rc = ERROR_SUCCESS; diff --git a/dlls/msi/classes.c b/dlls/msi/classes.c index e2bb34c..92c333d 100644 --- a/dlls/msi/classes.c +++ b/dlls/msi/classes.c @@ -735,7 +735,7 @@ static void mark_mime_for_install( MSIMIME *mime ) mime->InstallMe = TRUE; } -static UINT register_appid(MSIAPPID *appid, LPCWSTR app ) +static UINT register_appid(const MSIAPPID *appid, LPCWSTR app ) { static const WCHAR szAppID[] = { 'A','p','p','I','D',0 }; static const WCHAR szRemoteServerName[] = @@ -965,7 +965,7 @@ UINT ACTION_RegisterClassInfo(MSIPACKAGE *package) return rc; } -static LPCWSTR get_clsid_of_progid( MSIPROGID *progid ) +static LPCWSTR get_clsid_of_progid( const MSIPROGID *progid ) { while (progid) { @@ -976,7 +976,7 @@ static LPCWSTR get_clsid_of_progid( MSIPROGID *progid ) return NULL; } -static UINT register_progid( MSIPROGID* progid ) +static UINT register_progid( const MSIPROGID* progid ) { static const WCHAR szCLSID[] = { 'C','L','S','I','D',0 }; static const WCHAR szDefaultIcon[] = @@ -1048,7 +1048,7 @@ UINT ACTION_RegisterProgIdInfo(MSIPACKAGE *package) } static UINT register_verb(MSIPACKAGE *package, LPCWSTR progid, - MSICOMPONENT* component, MSIEXTENSION* extension, + MSICOMPONENT* component, const MSIEXTENSION* extension, MSIVERB* verb, INT* Sequence ) { LPWSTR keyname; diff --git a/dlls/msi/cond.y b/dlls/msi/cond.y index 9cfb001..ee42ede 100644 --- a/dlls/msi/cond.y +++ b/dlls/msi/cond.y @@ -56,8 +56,8 @@ struct cond_str { INT len; }; -static LPWSTR COND_GetString( struct cond_str *str ); -static LPWSTR COND_GetLiteral( struct cond_str *str ); +static LPWSTR COND_GetString( const struct cond_str *str ); +static LPWSTR COND_GetLiteral( const struct cond_str *str ); static int cond_lex( void *COND_lval, COND_input *info); static const WCHAR szEmpty[] = { 0 }; @@ -678,7 +678,7 @@ static int cond_lex( void *COND_lval, COND_input *cond ) return rc; } -static LPWSTR COND_GetString( struct cond_str *str ) +static LPWSTR COND_GetString( const struct cond_str *str ) { LPWSTR ret; @@ -692,7 +692,7 @@ static LPWSTR COND_GetString( struct cond_str *str ) return ret; } -static LPWSTR COND_GetLiteral( struct cond_str *str ) +static LPWSTR COND_GetLiteral( const struct cond_str *str ) { LPWSTR ret; diff --git a/dlls/msi/custom.c b/dlls/msi/custom.c index 0216f03..e516256 100644 --- a/dlls/msi/custom.c +++ b/dlls/msi/custom.c @@ -124,7 +124,7 @@ static BOOL check_execution_scheduling_options(MSIPACKAGE *package, LPCWSTR acti /* stores the CustomActionData before the action: * [CustomActionData]Action */ -static LPWSTR msi_get_deferred_action(LPCWSTR action, LPWSTR actiondata) +static LPWSTR msi_get_deferred_action(LPCWSTR action, LPCWSTR actiondata) { LPWSTR deferred; DWORD len; @@ -1016,7 +1016,7 @@ static UINT HANDLE_CustomType34(MSIPACKAGE *package, LPCWSTR source, return wait_process_handle(package, type, info.hProcess, action); } -static DWORD WINAPI ACTION_CallScript( const LPGUID guid ) +static DWORD WINAPI ACTION_CallScript( const GUID *guid ) { msi_custom_action_info *info; MSIHANDLE hPackage; @@ -1235,7 +1235,7 @@ static UINT HANDLE_CustomType53_54(MSIPACKAGE *package, LPCWSTR source, return wait_thread_handle( info ); } -void ACTION_FinishCustomActions(MSIPACKAGE* package) +void ACTION_FinishCustomActions(const MSIPACKAGE* package) { struct list *item; HANDLE *wait_handles; diff --git a/dlls/msi/msipriv.h b/dlls/msi/msipriv.h index f8f1d3d..2d05d76 100644 --- a/dlls/msi/msipriv.h +++ b/dlls/msi/msipriv.h @@ -726,7 +726,7 @@ extern HINSTANCE msi_hInstance; /* action related functions */ extern UINT ACTION_PerformAction(MSIPACKAGE *package, const WCHAR *action, BOOL force); extern UINT ACTION_PerformUIAction(MSIPACKAGE *package, const WCHAR *action); -extern void ACTION_FinishCustomActions( MSIPACKAGE* package); +extern void ACTION_FinishCustomActions( const MSIPACKAGE* package); extern UINT ACTION_CustomAction(MSIPACKAGE *package,const WCHAR *action, BOOL execute); static inline void msi_feature_set_state( MSIFEATURE *feature, INSTALLSTATE state )

1 0

Alexandre Julliard : user32: Fix a printf format warning.
by Alexandre Julliard 01 Jun '07

01 Jun '07

Module: wine Branch: master Commit: 6797f15a64dc0d45d90d41c54de37fce13c40bb1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6797f15a64dc0d45d90d41c54… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jun 1 13:42:37 2007 +0200 user32: Fix a printf format warning. --- dlls/user32/tests/menu.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/user32/tests/menu.c b/dlls/user32/tests/menu.c index b270a25..3037099 100644 --- a/dlls/user32/tests/menu.c +++ b/dlls/user32/tests/menu.c @@ -2036,7 +2036,7 @@ static void test_menu_resource_layout(void) count = GetMenuItemCount(hmenu); ok(count == sizeof(menu_data)/sizeof(menu_data[0]), "expected %u menu items, got %u\n", - (UINT)sizeof(menu_data)/sizeof(menu_data[0]), count); + (UINT)(sizeof(menu_data)/sizeof(menu_data[0])), count); for (i = 0; i < count; i++) {

1 0

Misha Koshelev : comctl32: imagelist: Fix simultaneous selection of one bitmap into two device contexts.
by Alexandre Julliard 01 Jun '07

01 Jun '07

Module: wine Branch: master Commit: fc5a746104bf8eedee25b686d9639b17d521c37a URL: http://source.winehq.org/git/wine.git/?a=commit;h=fc5a746104bf8eedee25b686d… Author: Misha Koshelev <mk144210(a)bcm.edu> Date: Thu May 31 23:32:41 2007 -0500 comctl32: imagelist: Fix simultaneous selection of one bitmap into two device contexts. --- dlls/comctl32/imagelist.c | 8 +++++--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/dlls/comctl32/imagelist.c b/dlls/comctl32/imagelist.c index 3c9e89b..ee1ef0e 100644 --- a/dlls/comctl32/imagelist.c +++ b/dlls/comctl32/imagelist.c @@ -2563,7 +2563,7 @@ BOOL WINAPI ImageList_SetImageCount (HIMAGELIST himl, UINT iImageCount) { HDC hdcBitmap; - HBITMAP hbmNewBitmap; + HBITMAP hbmNewBitmap, hbmOld; INT nNewCount, nCopyCount; TRACE("%p %d\n",himl,iImageCount); @@ -2588,8 +2588,9 @@ ImageList_SetImageCount (HIMAGELIST himl, UINT iImageCount) if (hbmNewBitmap != 0) { - SelectObject (hdcBitmap, hbmNewBitmap); + hbmOld = SelectObject (hdcBitmap, hbmNewBitmap); imagelist_copy_images( himl, himl->hdcImage, hdcBitmap, 0, nCopyCount, 0 ); + SelectObject (hdcBitmap, hbmOld); /* FIXME: delete 'empty' image space? */ @@ -2607,8 +2608,9 @@ ImageList_SetImageCount (HIMAGELIST himl, UINT iImageCount) hbmNewBitmap = CreateBitmap (sz.cx, sz.cy, 1, 1, NULL); if (hbmNewBitmap != 0) { - SelectObject (hdcBitmap, hbmNewBitmap); + hbmOld = SelectObject (hdcBitmap, hbmNewBitmap); imagelist_copy_images( himl, himl->hdcMask, hdcBitmap, 0, nCopyCount, 0 ); + SelectObject (hdcBitmap, hbmOld); /* FIXME: delete 'empty' image space? */

1 0