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

Alexandre Julliard : server: Expose areas revealed by changes to the window region.
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: 1767b4558a02e59757553b8ae183c3690879be95 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1767b4558a02e59757553b8ae… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 5 16:43:09 2007 +0100 server: Expose areas revealed by changes to the window region. --- dlls/user32/winpos.c | 2 ++ include/wine/server_protocol.h | 3 ++- server/protocol.def | 1 + server/trace.c | 1 + server/window.c | 30 ++++++++++++++++++++++++++++-- 5 files changed, 34 insertions(+), 3 deletions(-) diff --git a/dlls/user32/winpos.c b/dlls/user32/winpos.c index 46f9a26..635e968 100644 --- a/dlls/user32/winpos.c +++ b/dlls/user32/winpos.c @@ -250,6 +250,7 @@ int WINAPI SetWindowRgn( HWND hwnd, HRGN hrgn, BOOL bRedraw ) SERVER_START_REQ( set_window_region ) { req->window = hwnd; + req->redraw = (bRedraw != 0); if (data->rdh.nCount) wine_server_add_data( req, data->Buffer, data->rdh.nCount * sizeof(RECT) ); else @@ -263,6 +264,7 @@ int WINAPI SetWindowRgn( HWND hwnd, HRGN hrgn, BOOL bRedraw ) SERVER_START_REQ( set_window_region ) { req->window = hwnd; + req->redraw = (bRedraw != 0); ret = !wine_server_call_err( req ); } SERVER_END_REQ; diff --git a/include/wine/server_protocol.h b/include/wine/server_protocol.h index db13e0c..02851dc 100644 --- a/include/wine/server_protocol.h +++ b/include/wine/server_protocol.h @@ -3060,6 +3060,7 @@ struct set_window_region_request { struct request_header __header; user_handle_t window; + int redraw; /* VARARG(region,rectangles); */ }; struct set_window_region_reply @@ -4671,6 +4672,6 @@ union generic_reply struct get_token_impersonation_level_reply get_token_impersonation_level_reply; }; -#define SERVER_PROTOCOL_VERSION 278 +#define SERVER_PROTOCOL_VERSION 279 #endif /* __WINE_WINE_SERVER_PROTOCOL_H */ diff --git a/server/protocol.def b/server/protocol.def index 69daab3..d88bbe8 100644 --- a/server/protocol.def +++ b/server/protocol.def @@ -2233,6 +2233,7 @@ enum message_type /* Set the window region */ @REQ(set_window_region) user_handle_t window; /* handle to the window */ + int redraw; /* redraw the window? */ VARARG(region,rectangles); /* list of rectangles for the region */ @END diff --git a/server/trace.c b/server/trace.c index d4bcf08..56cc7f6 100644 --- a/server/trace.c +++ b/server/trace.c @@ -2708,6 +2708,7 @@ static void dump_get_window_region_reply( const struct get_window_region_reply * static void dump_set_window_region_request( const struct set_window_region_request *req ) { fprintf( stderr, " window=%p,", req->window ); + fprintf( stderr, " redraw=%d,", req->redraw ); fprintf( stderr, " region=" ); dump_varargs_rectangles( cur_size ); } diff --git a/server/window.c b/server/window.c index c526185..03ed61e 100644 --- a/server/window.c +++ b/server/window.c @@ -1443,6 +1443,33 @@ done: } +/* set the window region, updating the update region if necessary */ +static void set_window_region( struct window *win, struct region *region, int redraw ) +{ + struct region *old_vis_rgn = NULL, *new_vis_rgn; + struct window *top = get_top_clipping_window( win ); + + /* no need to redraw if window is not visible */ + if (redraw && !is_visible( win )) redraw = 0; + + if (redraw) old_vis_rgn = get_visible_region( win, top, DCX_WINDOW ); + + if (win->win_region) free_region( win->win_region ); + win->win_region = region; + + if (old_vis_rgn && (new_vis_rgn = get_visible_region( win, top, DCX_WINDOW ))) + { + /* expose anything revealed by the change */ + if (xor_region( new_vis_rgn, old_vis_rgn, new_vis_rgn )) + expose_window( win, top, new_vis_rgn ); + free_region( new_vis_rgn ); + } + + if (old_vis_rgn) free_region( old_vis_rgn ); + clear_error(); /* we ignore out of memory errors since the region has been set */ +} + + /* create a window */ DECL_HANDLER(create_window) { @@ -1900,8 +1927,7 @@ DECL_HANDLER(set_window_region) if (!(region = create_region_from_req_data( get_req_data(), get_req_data_size() ))) return; } - if (win->win_region) free_region( win->win_region ); - win->win_region = region; + set_window_region( win, region, req->redraw ); }

1 0

Rob Shearman : kernel32: Implement CallNamedPipeW.
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: 5267e36ad0abf1d5eb12d2a83ea4ec6af97b7b61 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5267e36ad0abf1d5eb12d2a83… Author: Rob Shearman <rob(a)codeweavers.com> Date: Mon Mar 5 12:06:02 2007 +0000 kernel32: Implement CallNamedPipeW. --- dlls/kernel32/sync.c | 37 +++++++++++++++++++++++++++++++++---- 1 files changed, 33 insertions(+), 4 deletions(-) diff --git a/dlls/kernel32/sync.c b/dlls/kernel32/sync.c index eaa0f70..d3f2f4b 100644 --- a/dlls/kernel32/sync.c +++ b/dlls/kernel32/sync.c @@ -1515,10 +1515,39 @@ BOOL WINAPI CallNamedPipeW( LPVOID lpOutput, DWORD lpOutputSize, LPDWORD lpBytesRead, DWORD nTimeout) { - FIXME("%s %p %d %p %d %p %d\n", - debugstr_w(lpNamedPipeName), lpInput, lpInputSize, - lpOutput, lpOutputSize, lpBytesRead, nTimeout); - return FALSE; + HANDLE pipe; + BOOL ret; + DWORD mode; + + TRACE("%s %p %d %p %d %p %d\n", + debugstr_w(lpNamedPipeName), lpInput, lpInputSize, + lpOutput, lpOutputSize, lpBytesRead, nTimeout); + + pipe = CreateFileW(lpNamedPipeName, GENERIC_READ|GENERIC_WRITE, 0, NULL, OPEN_EXISTING, 0, NULL); + if (pipe == INVALID_HANDLE_VALUE) + { + ret = WaitNamedPipeW(lpNamedPipeName, nTimeout); + if (!ret) + return FALSE; + pipe = CreateFileW(lpNamedPipeName, GENERIC_READ|GENERIC_WRITE, 0, NULL, OPEN_EXISTING, 0, NULL); + if (pipe == INVALID_HANDLE_VALUE) + return FALSE; + } + + mode = PIPE_READMODE_MESSAGE; + ret = SetNamedPipeHandleState(pipe, &mode, NULL, NULL); + if (!ret) + { + CloseHandle(pipe); + return FALSE; + } + + ret = TransactNamedPipe(pipe, lpInput, lpInputSize, lpOutput, lpOutputSize, lpBytesRead, NULL); + CloseHandle(pipe); + if (!ret) + return FALSE; + + return TRUE; } /******************************************************************

1 0

Michael Stefaniuc : hhctrl.ocx: Fix a format warning.
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: eff5b130436bace975dbe7c5b70664a1bf7f93e5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=eff5b130436bace975dbe7c5b… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Mar 5 13:48:23 2007 +0100 hhctrl.ocx: Fix a format warning. --- dlls/hhctrl.ocx/help.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/hhctrl.ocx/help.c b/dlls/hhctrl.ocx/help.c index f43cc0b..3bc9e8e 100644 --- a/dlls/hhctrl.ocx/help.c +++ b/dlls/hhctrl.ocx/help.c @@ -77,7 +77,7 @@ static HRESULT navigate_url(HHInfo *info, LPCWSTR surl) VariantClear(&url); if(FAILED(hres)) - TRACE("Navigation failed: %08lx\n", hres); + TRACE("Navigation failed: %08x\n", hres); return hres; }

1 0

Rob Shearman : wininet: Always make copies of strings passed to INTERNET_SendCallback.
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: 07daeff8f463d364deeff17a6a03439e9bfc9193 URL: http://source.winehq.org/git/wine.git/?a=commit;h=07daeff8f463d364deeff17a6… Author: Rob Shearman <rob(a)codeweavers.com> Date: Mon Mar 5 12:06:47 2007 +0000 wininet: Always make copies of strings passed to INTERNET_SendCallback. This ensures that applications that change the strings received in status callbacks don't affect the data needed to successfully connect to the correct server. --- dlls/wininet/tests/http.c | 5 +++++ dlls/wininet/utility.c | 12 ++++++++++++ 2 files changed, 17 insertions(+), 0 deletions(-) diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c index 91438c6..8926968 100644 --- a/dlls/wininet/tests/http.c +++ b/dlls/wininet/tests/http.c @@ -56,21 +56,25 @@ static VOID WINAPI callback( trace("%04x:Callback %p 0x%lx INTERNET_STATUS_RESOLVING_NAME \"%s\" %d\n", GetCurrentThreadId(), hInternet, dwContext, (LPCSTR)lpvStatusInformation,dwStatusInformationLength); + *(LPSTR)lpvStatusInformation = '\0'; break; case INTERNET_STATUS_NAME_RESOLVED: trace("%04x:Callback %p 0x%lx INTERNET_STATUS_NAME_RESOLVED \"%s\" %d\n", GetCurrentThreadId(), hInternet, dwContext, (LPCSTR)lpvStatusInformation,dwStatusInformationLength); + *(LPSTR)lpvStatusInformation = '\0'; break; case INTERNET_STATUS_CONNECTING_TO_SERVER: trace("%04x:Callback %p 0x%lx INTERNET_STATUS_CONNECTING_TO_SERVER \"%s\" %d\n", GetCurrentThreadId(), hInternet, dwContext, (LPCSTR)lpvStatusInformation,dwStatusInformationLength); + *(LPSTR)lpvStatusInformation = '\0'; break; case INTERNET_STATUS_CONNECTED_TO_SERVER: trace("%04x:Callback %p 0x%lx INTERNET_STATUS_CONNECTED_TO_SERVER \"%s\" %d\n", GetCurrentThreadId(), hInternet, dwContext, (LPCSTR)lpvStatusInformation,dwStatusInformationLength); + *(LPSTR)lpvStatusInformation = '\0'; break; case INTERNET_STATUS_SENDING_REQUEST: trace("%04x:Callback %p 0x%lx INTERNET_STATUS_SENDING_REQUEST %p %d\n", @@ -150,6 +154,7 @@ static VOID WINAPI callback( trace("%04x:Callback %p 0x%lx INTERNET_STATUS_REDIRECT \"%s\" %d\n", GetCurrentThreadId(), hInternet, dwContext, (LPCSTR)lpvStatusInformation, dwStatusInformationLength); + *(LPSTR)lpvStatusInformation = '\0'; break; case INTERNET_STATUS_INTERMEDIATE_RESPONSE: trace("%04x:Callback %p 0x%lx INTERNET_STATUS_INTERMEDIATE_RESPONSE %p %d\n", diff --git a/dlls/wininet/utility.c b/dlls/wininet/utility.c index 273177d..34e72d0 100644 --- a/dlls/wininet/utility.c +++ b/dlls/wininet/utility.c @@ -231,13 +231,25 @@ VOID INTERNET_SendCallback(LPWININETHANDLEHEADER hdr, DWORD dwContext, case INTERNET_STATUS_CONNECTING_TO_SERVER: case INTERNET_STATUS_CONNECTED_TO_SERVER: lpvNewInfo = WININET_strdup_AtoW(lpvStatusInfo); + break; + case INTERNET_STATUS_RESOLVING_NAME: + case INTERNET_STATUS_REDIRECT: + lpvNewInfo = WININET_strdupW(lpvStatusInfo); + break; } }else { switch(dwInternetStatus) { + case INTERNET_STATUS_NAME_RESOLVED: + case INTERNET_STATUS_CONNECTING_TO_SERVER: + case INTERNET_STATUS_CONNECTED_TO_SERVER: + lpvNewInfo = HeapAlloc(GetProcessHeap(), 0, strlen(lpvStatusInfo) + 1); + if (lpvNewInfo) strcpy(lpvNewInfo, lpvStatusInfo); + break; case INTERNET_STATUS_RESOLVING_NAME: case INTERNET_STATUS_REDIRECT: lpvNewInfo = WININET_strdup_WtoA(lpvStatusInfo); + break; } }

1 0

Rob Shearman : msi: Rewrite the second loop in ACTION_FinishCustomActions so that it always terminates .
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: e2972af43637d5e803d93b1cc89c8d0432bf8cb7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e2972af43637d5e803d93b1cc… Author: Rob Shearman <rob(a)codeweavers.com> Date: Mon Mar 5 12:03:59 2007 +0000 msi: Rewrite the second loop in ACTION_FinishCustomActions so that it always terminates. Create an array of handles to wait on so that we can wait without holding a critical section. --- dlls/msi/custom.c | 33 +++++++++++++++++---------------- 1 files changed, 17 insertions(+), 16 deletions(-) diff --git a/dlls/msi/custom.c b/dlls/msi/custom.c index c6c07cf..fc86691 100644 --- a/dlls/msi/custom.c +++ b/dlls/msi/custom.c @@ -859,6 +859,9 @@ static UINT HANDLE_CustomType34(MSIPACKAGE *package, LPCWSTR source, void ACTION_FinishCustomActions(MSIPACKAGE* package) { struct list *item; + HANDLE *wait_handles; + unsigned int handle_count, i; + msi_custom_action_info *info, *cursor; while ((item = list_head( &package->RunningActions ))) { @@ -874,27 +877,25 @@ void ACTION_FinishCustomActions(MSIPACKAGE* package) msi_free( action ); } - while (1) - { - HANDLE handle; - msi_custom_action_info *info; + EnterCriticalSection( &msi_custom_action_cs ); - EnterCriticalSection( &msi_custom_action_cs ); - item = list_head( &msi_pending_custom_actions ); - info = LIST_ENTRY( item, msi_custom_action_info, entry ); + handle_count = list_count( &msi_pending_custom_actions ); + wait_handles = HeapAlloc( GetProcessHeap(), 0, handle_count * sizeof(HANDLE) ); - if (item && info->package == package ) + handle_count = 0; + LIST_FOR_EACH_ENTRY_SAFE( info, cursor, &msi_pending_custom_actions, msi_custom_action_info, entry ) + { + if (info->package == package ) { - handle = info->handle; + wait_handles[handle_count++] = info->handle; free_custom_action_data( info ); } - else - handle = NULL; - LeaveCriticalSection( &msi_custom_action_cs ); + } - if (!item) - break; + LeaveCriticalSection( &msi_custom_action_cs ); - msi_dialog_check_messages( handle ); - } + for (i = 0; i < handle_count; i++) + msi_dialog_check_messages( wait_handles[i] ); + + HeapFree( GetProcessHeap(), 0, wait_handles ); }

1 0

Rob Shearman : msi: Set the PackageCode property based on the " revision number" data in the summary information.
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: 77f6bc4f8d46950ffcd9eb0d7ddd4b36fa16f9d3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=77f6bc4f8d46950ffcd9eb0d7… Author: Rob Shearman <rob(a)codeweavers.com> Date: Mon Mar 5 12:04:41 2007 +0000 msi: Set the PackageCode property based on the "revision number" data in the summary information. Add a test to show that PackageCode is set by MSI. --- dlls/msi/package.c | 41 ++++++++++++++++++++++++++++++++--------- dlls/msi/tests/package.c | 5 +++++ 2 files changed, 37 insertions(+), 9 deletions(-) diff --git a/dlls/msi/package.c b/dlls/msi/package.c index 74d18cb..73298af 100644 --- a/dlls/msi/package.c +++ b/dlls/msi/package.c @@ -459,12 +459,16 @@ static VOID set_installer_properties(MSIPACKAGE *package) CloseHandle( hkey ); } -static UINT msi_get_word_count( MSIPACKAGE *package ) +static UINT msi_load_summary_properties( MSIPACKAGE *package ) { UINT rc; - INT word_count; MSIHANDLE suminfo; MSIHANDLE hdb = alloc_msihandle( &package->db->hdr ); + INT word_count; + DWORD len; + LPWSTR package_code; + static const WCHAR szPackageCode[] = { + 'P','a','c','k','a','g','e','C','o','d','e',0}; if (!hdb) { ERR("Unable to allocate handle\n"); @@ -475,20 +479,38 @@ static UINT msi_get_word_count( MSIPACKAGE *package ) if (rc != ERROR_SUCCESS) { ERR("Unable to open Summary Information\n"); - return 0; + return rc; } + /* load package attributes */ rc = MsiSummaryInfoGetPropertyW( suminfo, PID_WORDCOUNT, NULL, &word_count, NULL, NULL, NULL ); - if (rc != ERROR_SUCCESS) + if (rc == ERROR_SUCCESS) + package->WordCount = word_count; + else + WARN("Unable to query word count\n"); + + /* load package code property */ + len = 0; + rc = MsiSummaryInfoGetPropertyW( suminfo, PID_REVNUMBER, NULL, + NULL, NULL, NULL, &len ); + if (rc == ERROR_MORE_DATA) { - ERR("Unable to query word count\n"); - MsiCloseHandle(suminfo); - return 0; + len++; + package_code = msi_alloc( len * sizeof(WCHAR) ); + rc = MsiSummaryInfoGetPropertyW( suminfo, PID_REVNUMBER, NULL, + NULL, NULL, package_code, &len ); + if (rc == ERROR_SUCCESS) + MSI_SetPropertyW( package, szPackageCode, package_code ); + else + WARN("Unable to query rev number, %d\n", rc); + msi_free( package_code ); } + else + WARN("Unable to query rev number, %d\n", rc); MsiCloseHandle(suminfo); - return word_count; + return ERROR_SUCCESS; } static MSIPACKAGE *msi_alloc_package( void ) @@ -542,7 +564,7 @@ MSIPACKAGE *MSI_CreatePackage( MSIDATABASE *db, LPWSTR base_url ) msiobj_addref( &db->hdr ); package->db = db; - package->WordCount = msi_get_word_count( package ); + package->WordCount = 0; package->PackagePath = strdupW( db->path ); package->BaseURL = strdupW( base_url ); @@ -553,6 +575,7 @@ MSIPACKAGE *MSI_CreatePackage( MSIDATABASE *db, LPWSTR base_url ) package->ProductCode = msi_dup_property( package, szProductCode ); set_installed_prop( package ); + msi_load_summary_properties( package ); } return package; diff --git a/dlls/msi/tests/package.c b/dlls/msi/tests/package.c index 536fa1e..f461cb6 100644 --- a/dlls/msi/tests/package.c +++ b/dlls/msi/tests/package.c @@ -3018,6 +3018,11 @@ static void test_installprops(void) ok( r == ERROR_SUCCESS, "failed to get property: %d\n", r); trace("Time = %s\n", buf); + size = MAX_PATH; + r = MsiGetProperty(hpkg, "PackageCode", buf, &size); + ok( r == ERROR_SUCCESS, "failed to get property: %d\n", r); + trace("PackageCode = %s\n", buf); + CloseHandle(hkey); MsiCloseHandle(hpkg); DeleteFile(msifile);

1 0

Rob Shearman : credui: The pfSave parameter of CredUIPromptForCredentials is optional.
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: d72f741a33e25e50d85fff4e6ef201ed8a070a6d URL: http://source.winehq.org/git/wine.git/?a=commit;h=d72f741a33e25e50d85fff4e6… Author: Rob Shearman <rob(a)codeweavers.com> Date: Mon Mar 5 12:05:19 2007 +0000 credui: The pfSave parameter of CredUIPromptForCredentials is optional. --- dlls/credui/credui_main.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/credui/credui_main.c b/dlls/credui/credui_main.c index 714ec93..1d646f3 100644 --- a/dlls/credui/credui_main.c +++ b/dlls/credui/credui_main.c @@ -167,7 +167,7 @@ DWORD WINAPI CredUIPromptForCredentialsW(PCREDUI_INFOW pUIInfo, params.ulUsernameMaxChars = ulUsernameMaxChars; params.pszPassword = pszPassword; params.ulPasswordMaxChars = ulPasswordMaxChars; - params.fSave = *pfSave; + params.fSave = pfSave ? *pfSave : FALSE; ret = DialogBoxParamW(hinstCredUI, MAKEINTRESOURCEW(IDD_CREDDIALOG), pUIInfo->hwndParent, CredDialogProc, (LPARAM)&params); @@ -180,7 +180,8 @@ DWORD WINAPI CredUIPromptForCredentialsW(PCREDUI_INFOW pUIInfo, return ERROR_CANCELLED; } - *pfSave = params.fSave; + if (pfSave) + *pfSave = params.fSave; return ERROR_SUCCESS; }

1 0

Stefan Dösinger : wined3d: Adjust the rhw transformation for offscreen rendering.
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: 2cdced8193fbf4693298f84d6a121d4ce4607fd5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2cdced8193fbf4693298f84d6… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sat Mar 3 02:35:17 2007 +0100 wined3d: Adjust the rhw transformation for offscreen rendering. When drawing processed vertices with the fixed function pipeline the projection matrix is set up to map y values from 0 to height to 1.0; -1.0(gl and d3d coord systems are flipped). This moves the y axis to the bottom of the drawing area. When later on the y inversion matrix is applied for offscreen rendering, the coordinate system will get flipped out of the viewport. This patch sets the Y range up upside down when using offscreen rendering, so the invymat will flip it to the correct position. This has to happen before the 0.375 pixel correction. --- dlls/wined3d/state.c | 12 ++++++++++-- 1 files changed, 10 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/state.c b/dlls/wined3d/state.c index 64bce1c..ba421a1 100644 --- a/dlls/wined3d/state.c +++ b/dlls/wined3d/state.c @@ -2154,7 +2154,11 @@ static void transform_projection(DWORD state, IWineD3DStateBlockImpl *stateblock * the Z coordinate does not affect the size of the primitives */ TRACE("Calling glOrtho with %f, %f, %f, %f\n", width, height, -minZ, -maxZ); - glOrtho(X, X + width, Y + height, Y, -minZ, -maxZ); + if(stateblock->wineD3DDevice->render_offscreen) { + glOrtho(X, X + width, Y, Y - height, -minZ, -maxZ); + } else { + glOrtho(X, X + width, Y + height, Y, -minZ, -maxZ); + } } else { /* If the app mixes transformed and untransformed primitives we can't use the coordinate system * trick above because this would mess up transformed and untransformed Z order. Pass the z position @@ -2164,7 +2168,11 @@ static void transform_projection(DWORD state, IWineD3DStateBlockImpl *stateblock * replacement shader. */ TRACE("Calling glOrtho with %f, %f, %f, %f\n", width, height, 1.0, -1.0); - glOrtho(X, X + width, Y + height, Y, 1.0, -1.0); + if(stateblock->wineD3DDevice->render_offscreen) { + glOrtho(X, X + width, Y, Y - height, 1.0, -1.0); + } else { + glOrtho(X, X + width, Y + height, Y, 1.0, -1.0); + } } checkGLcall("glOrtho");

1 0

Joris Huizer : winebuild: sign-compare fixes.
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: c9f145223368da9f663aaad415aa776eb2b297c2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c9f145223368da9f663aaad41… Author: Joris Huizer <joris_huizer(a)yahoo.com> Date: Sat Mar 3 05:30:03 2007 -0800 winebuild: sign-compare fixes. --- tools/winebuild/import.c | 12 +++++++----- tools/winebuild/spec32.c | 5 +++-- tools/winebuild/utils.c | 2 +- 3 files changed, 11 insertions(+), 8 deletions(-) diff --git a/tools/winebuild/import.c b/tools/winebuild/import.c index 2241fa5..80c6c12 100644 --- a/tools/winebuild/import.c +++ b/tools/winebuild/import.c @@ -168,7 +168,7 @@ static void free_imports( struct import *imp ) /* check whether a given dll is imported in delayed mode */ static int is_delayed_import( const char *name ) { - int i; + unsigned int i; for (i = 0; i < delayed_imports.count; i++) { @@ -468,7 +468,8 @@ static void check_undefined_exports( DLLSPEC *spec ) static char *create_undef_symbols_file( DLLSPEC *spec ) { char *as_file, *obj_file; - unsigned int i; + int i; + unsigned int j; FILE *f; as_file = get_temp_file_name( output_file_name, ".s" ); @@ -482,8 +483,8 @@ static char *create_undef_symbols_file( DLLSPEC *spec ) if (odp->flags & FLAG_FORWARD) continue; fprintf( f, "\t%s %s\n", get_asm_ptr_keyword(), asm_name(odp->link_name) ); } - for (i = 0; i < extra_ld_symbols.count; i++) - fprintf( f, "\t%s %s\n", get_asm_ptr_keyword(), asm_name(extra_ld_symbols.names[i]) ); + for (j = 0; j < extra_ld_symbols.count; j++) + fprintf( f, "\t%s %s\n", get_asm_ptr_keyword(), asm_name(extra_ld_symbols.names[j]) ); fclose( f ); obj_file = get_temp_file_name( output_file_name, ".o" ); @@ -557,7 +558,8 @@ void read_undef_symbols( DLLSPEC *spec, char **argv ) /* resolve the imports for a Win32 module */ int resolve_imports( DLLSPEC *spec ) { - unsigned int i, j, removed; + int i; + unsigned int j, removed; ORDDEF *odp; sort_names( &ignore_symbols ); diff --git a/tools/winebuild/spec32.c b/tools/winebuild/spec32.c index 6fcc42e..26e66b8 100644 --- a/tools/winebuild/spec32.c +++ b/tools/winebuild/spec32.c @@ -51,7 +51,7 @@ static inline int needs_relay( const ORDDEF *odp ) /* check if dll will output relay thunks */ int has_relays( DLLSPEC *spec ) { - unsigned int i; + int i; if (target_cpu != CPU_x86) return 0; @@ -70,7 +70,8 @@ int has_relays( DLLSPEC *spec ) */ static void output_relay_debug( DLLSPEC *spec ) { - unsigned int i, j, args, flags; + int i; + unsigned int j, args, flags; /* first the table of entry point offsets */ diff --git a/tools/winebuild/utils.c b/tools/winebuild/utils.c index 3458eb7..656e375 100644 --- a/tools/winebuild/utils.c +++ b/tools/winebuild/utils.c @@ -454,7 +454,7 @@ unsigned int get_alignment(unsigned int align) case CPU_POWERPC: case CPU_ALPHA: n = 0; - while ((1 << n) != align) n++; + while ((1u << n) != align) n++; return n; } /* unreached */

1 0

Joris Huizer : preloader: sign-compare fix.
by Alexandre Julliard 05 Mar '07

05 Mar '07

Module: wine Branch: master Commit: e1e54d1ee7892457e697aa5d78a202befb909c54 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e1e54d1ee7892457e697aa5d7… Author: Joris Huizer <joris_huizer(a)yahoo.com> Date: Sat Mar 3 05:30:55 2007 -0800 preloader: sign-compare fix. --- loader/preloader.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/loader/preloader.c b/loader/preloader.c index 5648158..0034e9a 100644 --- a/loader/preloader.c +++ b/loader/preloader.c @@ -1006,7 +1006,8 @@ static int is_in_preload_range( const ElfW(auxv_t) *av, int type ) /* set the process name if supported */ static void set_process_name( int argc, char *argv[] ) { - unsigned int i, off; + int i; + unsigned int off; char *p, *name, *end; /* set the process short name */

1 0