November 2007 - Wine-Commits

Rob Shearman : setupapi: Make the setupapi tests load on systems < Vista by providing a private implementation of RegDeleteTreeW.
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: 55aeb80aa3657623fdd474852d2930ca221ba53c URL: http://source.winehq.org/git/wine.git/?a=commit;h=55aeb80aa3657623fdd474852… Author: Rob Shearman <rob(a)codeweavers.com> Date: Thu Nov 1 22:04:03 2007 +0000 setupapi: Make the setupapi tests load on systems < Vista by providing a private implementation of RegDeleteTreeW. --- dlls/setupapi/tests/devinst.c | 70 ++++++++++++++++++++++++++++++++++++++++- 1 files changed, 69 insertions(+), 1 deletions(-) diff --git a/dlls/setupapi/tests/devinst.c b/dlls/setupapi/tests/devinst.c index 115da83..5dda333 100644 --- a/dlls/setupapi/tests/devinst.c +++ b/dlls/setupapi/tests/devinst.c @@ -70,6 +70,74 @@ static void init_function_pointers(void) pSetupDiRegisterDeviceInfo = (void *)GetProcAddress(hSetupAPI, "SetupDiRegisterDeviceInfo"); } +/* RegDeleteTreeW from dlls/advapi32/registry.c */ +LSTATUS WINAPI devinst_RegDeleteTreeW(HKEY hKey, LPCWSTR lpszSubKey) +{ + LONG ret; + DWORD dwMaxSubkeyLen, dwMaxValueLen; + DWORD dwMaxLen, dwSize; + WCHAR szNameBuf[MAX_PATH], *lpszName = szNameBuf; + HKEY hSubKey = hKey; + + if(lpszSubKey) + { + ret = RegOpenKeyExW(hKey, lpszSubKey, 0, KEY_READ, &hSubKey); + if (ret) return ret; + } + + /* Get highest length for keys, values */ + ret = RegQueryInfoKeyW(hSubKey, NULL, NULL, NULL, NULL, + &dwMaxSubkeyLen, NULL, NULL, &dwMaxValueLen, NULL, NULL, NULL); + if (ret) goto cleanup; + + dwMaxSubkeyLen++; + dwMaxValueLen++; + dwMaxLen = max(dwMaxSubkeyLen, dwMaxValueLen); + if (dwMaxLen > sizeof(szNameBuf)/sizeof(WCHAR)) + { + /* Name too big: alloc a buffer for it */ + if (!(lpszName = HeapAlloc( GetProcessHeap(), 0, dwMaxLen*sizeof(WCHAR)))) + { + ret = ERROR_NOT_ENOUGH_MEMORY; + goto cleanup; + } + } + + + /* Recursively delete all the subkeys */ + while (TRUE) + { + dwSize = dwMaxLen; + if (RegEnumKeyExW(hSubKey, 0, lpszName, &dwSize, NULL, + NULL, NULL, NULL)) break; + + ret = devinst_RegDeleteTreeW(hSubKey, lpszName); + if (ret) goto cleanup; + } + + if (lpszSubKey) + ret = RegDeleteKeyW(hKey, lpszSubKey); + else + while (TRUE) + { + dwSize = dwMaxLen; + if (RegEnumValueW(hKey, 0, lpszName, &dwSize, + NULL, NULL, NULL, NULL)) break; + + ret = RegDeleteValueW(hKey, lpszName); + if (ret) goto cleanup; + } + +cleanup: + /* Free buffer if allocated */ + if (lpszName != szNameBuf) + HeapFree( GetProcessHeap(), 0, lpszName); + if(lpszSubKey) + RegCloseKey(hSubKey); + return ret; +} + + static void test_SetupDiCreateDeviceInfoListEx(void) { HDEVINFO devlist; @@ -686,7 +754,7 @@ static void testDevRegKey(void) ret = pSetupDiCallClassInstaller(DIF_REMOVE, set, &devInfo); pSetupDiDestroyDeviceInfoList(set); } - RegDeleteTreeW(HKEY_LOCAL_MACHINE, classKey); + devinst_RegDeleteTreeW(HKEY_LOCAL_MACHINE, classKey); } START_TEST(devinst)

1 0

Gerald Pfeifer : shell32: Avoid checking for <0 for unsigned variables.
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: e403292e5af8e5a1bc18de74a28d2fee6c482b19 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e403292e5af8e5a1bc18de74a… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Thu Nov 1 17:58:32 2007 +0100 shell32: Avoid checking for <0 for unsigned variables. --- dlls/shell32/recyclebin.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/shell32/recyclebin.c b/dlls/shell32/recyclebin.c index 441068f..ddf2fc3 100644 --- a/dlls/shell32/recyclebin.c +++ b/dlls/shell32/recyclebin.c @@ -381,7 +381,7 @@ static HRESULT WINAPI RecycleBin_GetDefaultColumnState(IShellFolder2 *iface, UIN { RecycleBin *This = (RecycleBin *)iface; TRACE("(%p, %d, %p)\n", This, iColumn, pcsFlags); - if (iColumn < 0 || iColumn >= COLUMNS_COUNT) + if (iColumn >= COLUMNS_COUNT) return E_INVALIDARG; *pcsFlags = RecycleBinColumns[iColumn].pcsFlags; return S_OK; @@ -400,7 +400,7 @@ static HRESULT WINAPI RecycleBin_GetDetailsOf(IShellFolder2 *iface, LPCITEMIDLIS WCHAR buffer[MAX_PATH]; TRACE("(%p, %p, %d, %p)\n", This, pidl, iColumn, pDetails); - if (iColumn < 0 || iColumn >= COLUMNS_COUNT) + if (iColumn >= COLUMNS_COUNT) return E_FAIL; pDetails->fmt = RecycleBinColumns[iColumn].fmt; pDetails->cxChar = RecycleBinColumns[iColumn].cxChars; @@ -446,7 +446,7 @@ static HRESULT WINAPI RecycleBin_MapColumnToSCID(IShellFolder2 *iface, UINT iCol { RecycleBin *This = (RecycleBin *)iface; TRACE("(%p, %d, %p)\n", This, iColumn, pscid); - if (iColumn<0 || iColumn>=COLUMNS_COUNT) + if (iColumn>=COLUMNS_COUNT) return E_INVALIDARG; pscid->fmtid = *RecycleBinColumns[iColumn].fmtId; pscid->pid = RecycleBinColumns[iColumn].pid;

1 0

Gerald Pfeifer : dinput: Fix type of loop variable in alloc_device().
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: 79ccb7889c86367c40c994b5ef73a0f4adcce118 URL: http://source.winehq.org/git/wine.git/?a=commit;h=79ccb7889c86367c40c994b5e… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Thu Nov 1 18:05:39 2007 +0100 dinput: Fix type of loop variable in alloc_device(). --- dlls/dinput/mouse.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/dinput/mouse.c b/dlls/dinput/mouse.c index 214bbd8..35ddc0b 100644 --- a/dlls/dinput/mouse.c +++ b/dlls/dinput/mouse.c @@ -173,7 +173,7 @@ static SysMouseImpl *alloc_device(REFGUID rguid, const void *mvt, IDirectInputIm { SysMouseImpl* newDevice; LPDIDATAFORMAT df = NULL; - int i; + unsigned i; newDevice = HeapAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY,sizeof(SysMouseImpl)); if (!newDevice) return NULL;

1 0

Gerald Pfeifer : user32: Don't check unsigned types for >= 0.
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: c7adf5f4b296d266bf06ed6c1a638595b00e13fc URL: http://source.winehq.org/git/wine.git/?a=commit;h=c7adf5f4b296d266bf06ed6c1… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Thu Nov 1 17:42:50 2007 +0100 user32: Don't check unsigned types for >= 0. --- dlls/user32/tests/dde.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/user32/tests/dde.c b/dlls/user32/tests/dde.c index b6f1d94..ca9fe20 100644 --- a/dlls/user32/tests/dde.c +++ b/dlls/user32/tests/dde.c @@ -1286,7 +1286,7 @@ static HDDEDATA CALLBACK client_dde_callback(UINT uType, UINT uFmt, HCONV hconv, const char *cmd_name; type = (uType & XTYP_MASK) >> XTYP_SHIFT; - cmd_name = (type >= 0 && type <= 14) ? cmd_type[type] : "unknown"; + cmd_name = (type <= 14) ? cmd_type[type] : "unknown"; trace("client_dde_callback: %04x (%s) %d %p %p %p %p %08lx %08lx\n", uType, cmd_name, uFmt, hconv, hsz1, hsz2, hdata, dwData1, dwData2);

1 0

Gerald Pfeifer : wined3d: Fix type of loop variable in IWineD3DCubeTextureImpl_Destroy().
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: d59ca347949535db169bc516dcd2c2529735ae79 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d59ca347949535db169bc516d… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Thu Nov 1 17:15:45 2007 +0100 wined3d: Fix type of loop variable in IWineD3DCubeTextureImpl_Destroy(). --- dlls/wined3d/cubetexture.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/cubetexture.c b/dlls/wined3d/cubetexture.c index 28677c6..3cb44e6 100644 --- a/dlls/wined3d/cubetexture.c +++ b/dlls/wined3d/cubetexture.c @@ -272,7 +272,7 @@ static void WINAPI IWineD3DCubeTextureImpl_ApplyStateChanges(IWineD3DCubeTexture ******************************************* */ static void WINAPI IWineD3DCubeTextureImpl_Destroy(IWineD3DCubeTexture *iface, D3DCB_DESTROYSURFACEFN D3DCB_DestroySurface) { IWineD3DCubeTextureImpl *This = (IWineD3DCubeTextureImpl *)iface; - int i,j; + unsigned int i,j; TRACE("(%p) : Cleaning up\n",This); for (i = 0; i < This->baseTexture.levels; i++) { for (j = 0; j < 6; j++) {

1 0

Chris Robinson : setupapi: Add stub for SetupInstallServicesFromInfSectionW .
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: 4d721de4d2e413787c5a11a50ca2478617ef4154 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4d721de4d2e413787c5a11a50… Author: Chris Robinson <chris.kcat(a)gmail.com> Date: Fri Nov 2 02:27:40 2007 -0700 setupapi: Add stub for SetupInstallServicesFromInfSectionW. --- dlls/setupapi/install.c | 9 +++++++++ dlls/setupapi/setupapi.spec | 2 +- 2 files changed, 10 insertions(+), 1 deletions(-) diff --git a/dlls/setupapi/install.c b/dlls/setupapi/install.c index 412b1fd..ad44640 100644 --- a/dlls/setupapi/install.c +++ b/dlls/setupapi/install.c @@ -1067,6 +1067,15 @@ void WINAPI InstallHinfSectionA( HWND hwnd, HINSTANCE handle, LPCSTR cmdline, IN } /*********************************************************************** + * SetupInstallServicesFromInfSectionW (SETUPAPI.@) + */ +BOOL WINAPI SetupInstallServicesFromInfSectionW( HINF Inf, PCWSTR SectionName, DWORD Flags) +{ + FIXME("(%p, %s, %d) stub!\n", Inf, debugstr_w(SectionName), Flags); + return FALSE; +} + +/*********************************************************************** * SetupInstallServicesFromInfSectionA (SETUPAPI.@) */ BOOL WINAPI SetupInstallServicesFromInfSectionA( HINF Inf, PCSTR SectionName, DWORD Flags) diff --git a/dlls/setupapi/setupapi.spec b/dlls/setupapi/setupapi.spec index 7f64edf..328a23e 100644 --- a/dlls/setupapi/setupapi.spec +++ b/dlls/setupapi/setupapi.spec @@ -445,7 +445,7 @@ @ stdcall SetupInstallServicesFromInfSectionA(long str long) @ stub SetupInstallServicesFromInfSectionExA @ stub SetupInstallServicesFromInfSectionExW -@ stub SetupInstallServicesFromInfSectionW +@ stdcall SetupInstallServicesFromInfSectionW(long wstr long) @ stdcall SetupIterateCabinetA(str long ptr ptr) @ stdcall SetupIterateCabinetW(wstr long ptr ptr) @ stub SetupLogErrorA

1 0

Dmitry Timoshkov : winex11.drv: Initialize buffer size.
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: 407db36684b879c15247b5a5aed6610da40a647c URL: http://source.winehq.org/git/wine.git/?a=commit;h=407db36684b879c15247b5a5a… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Thu Nov 1 21:09:03 2007 +0800 winex11.drv: Initialize buffer size. --- dlls/winex11.drv/settings.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/winex11.drv/settings.c b/dlls/winex11.drv/settings.c index d530f9b..116bf15 100644 --- a/dlls/winex11.drv/settings.c +++ b/dlls/winex11.drv/settings.c @@ -176,6 +176,7 @@ static BOOL read_registry_settings(DEVMODEW *dm) return FALSE; #define query_value(name, data) \ + size = sizeof(DWORD); \ if (RegQueryValueExA(hkey, name, 0, &type, (LPBYTE)(data), &size) || \ type != REG_DWORD || size != sizeof(DWORD)) \ ret = FALSE

1 0

Alexandre Julliard : user32/tests: Don' t test the contents of uninitialized buffers.
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: 5c5e333284a832c8ef0f40ad1a55e7abc40da095 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5c5e333284a832c8ef0f40ad1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 2 12:27:27 2007 +0100 user32/tests: Don't test the contents of uninitialized buffers. --- dlls/user32/tests/dde.c | 3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/dlls/user32/tests/dde.c b/dlls/user32/tests/dde.c index e78c8c9..b6f1d94 100644 --- a/dlls/user32/tests/dde.c +++ b/dlls/user32/tests/dde.c @@ -1512,7 +1512,6 @@ static void test_DdeCreateDataHandle(void) ptr = DdeAccessData(hdata, &size); ok(ptr != NULL, "Expected non-NULL ptr\n"); - ok(lstrlenA((LPSTR)ptr) == 0, "Expected 0, got %d\n", lstrlenA((LPSTR)ptr)); ok(size == 260, "Expected 260, got %d\n", size); ret = DdeUnaccessData(hdata); @@ -1536,8 +1535,6 @@ static void test_DdeCreateDataHandle(void) ptr = DdeAccessData(hdata, &size); ok(ptr != NULL, "Expected non-NULL ptr\n"); - ok(lstrlenA((LPSTR)ptr) != 0, "Expected non-empty string\n"); - ok(lstrcmpA((LPSTR)ptr, "data"), "Did not expect data\n"); ok(size == 0, "Expected 0, got %d\n", size); ret = DdeUnaccessData(hdata);

1 0

Marcel Partap : include: Add winddiui.h header.
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: 91b3eac2f21d7b2f531b120f6e9aa8646c8a9295 URL: http://source.winehq.org/git/wine.git/?a=commit;h=91b3eac2f21d7b2f531b120f6… Author: Marcel Partap <mpartap(a)gmx.net> Date: Thu Nov 1 19:12:19 2007 +0100 include: Add winddiui.h header. --- include/Makefile.in | 1 + include/ddk/winddiui.h | 50 ++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 51 insertions(+), 0 deletions(-) diff --git a/include/Makefile.in b/include/Makefile.in index 7ae17d7..9835f09 100644 --- a/include/Makefile.in +++ b/include/Makefile.in @@ -125,6 +125,7 @@ SRCDIR_INCLUDES = \ ddk/ntddser.h \ ddk/ntddtape.h \ ddk/wdm.h \ + ddk/winddiui.h \ ddk/winsplp.h \ ddraw.h \ ddrawi.h \ diff --git a/include/ddk/winddiui.h b/include/ddk/winddiui.h new file mode 100644 index 0000000..7c6e775 --- /dev/null +++ b/include/ddk/winddiui.h @@ -0,0 +1,50 @@ +/* + * Header for the Device Driver Interface - User Interface library + * + * Copyright 2007 Marcel Partap + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#ifndef __WINE_WINDDIUI_H +#define __WINE_WINDDIUI_H + +#include <ddk/compstui.h> + +#ifdef __cplusplus +extern "C" { +#endif + +#define DRIVER_EVENT_INITIALIZE 1 +#define DRIVER_EVENT_DELETE 2 + +#define PRINTER_EVENT_ADD_CONNECTION 1 +#define PRINTER_EVENT_DELETE_CONNECTION 2 +#define PRINTER_EVENT_INITIALIZE 3 +#define PRINTER_EVENT_DELETE 4 +#define PRINTER_EVENT_CACHE_REFRESH 5 +#define PRINTER_EVENT_CACHE_DELETE 6 +#define PRINTER_EVENT_ATTRIBUTES_CHANGED 7 + +#define PRINTER_EVENT_FLAG_NO_UI 1 + +BOOL WINAPI DrvDriverEvent(DWORD, DWORD, LPBYTE, LPARAM); +BOOL WINAPI DrvPrinterEvent(LPWSTR, INT, DWORD, LPARAM); + +#ifdef __cplusplus +} /* extern "C" */ +#endif + +#endif /* __WINE_WINDDIUI_H */

1 0

Andrew Talbot : shell32: Constify some variables.
by Alexandre Julliard Nov. 2, 2007

Nov. 2, 2007

Module: wine Branch: master Commit: 36d32df5e60dcf6b6f3eea5bcf3f0c40fb451552 URL: http://source.winehq.org/git/wine.git/?a=commit;h=36d32df5e60dcf6b6f3eea5bc… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Thu Nov 1 22:15:02 2007 +0000 shell32: Constify some variables. --- dlls/shell32/shlfileop.c | 51 +++++++++++++++++++++++---------------------- 1 files changed, 26 insertions(+), 25 deletions(-) diff --git a/dlls/shell32/shlfileop.c b/dlls/shell32/shlfileop.c index 4f89c4c..0d9f737 100644 --- a/dlls/shell32/shlfileop.c +++ b/dlls/shell32/shlfileop.c @@ -931,10 +931,10 @@ static inline void grow_list(FILE_LIST *list) /* adds a file to the FILE_ENTRY struct */ -static void add_file_to_entry(FILE_ENTRY *feFile, LPWSTR szFile) +static void add_file_to_entry(FILE_ENTRY *feFile, LPCWSTR szFile) { DWORD dwLen = lstrlenW(szFile) + 1; - LPWSTR ptr; + LPCWSTR ptr; feFile->szFullPath = HeapAlloc(GetProcessHeap(), 0, dwLen * sizeof(WCHAR)); lstrcpyW(feFile->szFullPath, szFile); @@ -953,9 +953,10 @@ static void add_file_to_entry(FILE_ENTRY *feFile, LPWSTR szFile) feFile->bFromWildcard = FALSE; } -static LPWSTR wildcard_to_file(LPWSTR szWildCard, LPWSTR szFileName) +static LPWSTR wildcard_to_file(LPCWSTR szWildCard, LPCWSTR szFileName) { - LPWSTR szFullPath, ptr; + LPCWSTR ptr; + LPWSTR szFullPath; DWORD dwDirLen, dwFullLen; ptr = StrRChrW(szWildCard, NULL, '\\'); @@ -970,7 +971,7 @@ static LPWSTR wildcard_to_file(LPWSTR szWildCard, LPWSTR szFileName) return szFullPath; } -static void parse_wildcard_files(FILE_LIST *flList, LPWSTR szFile, LPDWORD pdwListIndex) +static void parse_wildcard_files(FILE_LIST *flList, LPCWSTR szFile, LPDWORD pdwListIndex) { WIN32_FIND_DATAW wfd; HANDLE hFile = FindFirstFileW(szFile, &wfd); @@ -1080,7 +1081,7 @@ static void destroy_file_list(FILE_LIST *flList) HeapFree(GetProcessHeap(), 0, flList->feFiles); } -static void copy_dir_to_dir(FILE_OPERATION *op, FILE_ENTRY *feFrom, LPWSTR szDestPath) +static void copy_dir_to_dir(FILE_OPERATION *op, const FILE_ENTRY *feFrom, LPCWSTR szDestPath) { WCHAR szFrom[MAX_PATH], szTo[MAX_PATH]; SHFILEOPSTRUCTW fileOp; @@ -1124,7 +1125,7 @@ static void copy_dir_to_dir(FILE_OPERATION *op, FILE_ENTRY *feFrom, LPWSTR szDes SHFileOperationW(&fileOp); } -static BOOL copy_file_to_file(FILE_OPERATION *op, WCHAR *szFrom, WCHAR *szTo) +static BOOL copy_file_to_file(FILE_OPERATION *op, const WCHAR *szFrom, const WCHAR *szTo) { if (!(op->req->fFlags & FOF_NOCONFIRMATION) && PathFileExistsW(szTo)) { @@ -1136,7 +1137,7 @@ static BOOL copy_file_to_file(FILE_OPERATION *op, WCHAR *szFrom, WCHAR *szTo) } /* copy a file or directory to another directory */ -static void copy_to_dir(FILE_OPERATION *op, FILE_ENTRY *feFrom, FILE_ENTRY *feTo) +static void copy_to_dir(FILE_OPERATION *op, const FILE_ENTRY *feFrom, const FILE_ENTRY *feTo) { if (!PathFileExistsW(feTo->szFullPath)) SHNotifyCreateDirectoryW(feTo->szFullPath, NULL); @@ -1152,10 +1153,10 @@ static void copy_to_dir(FILE_OPERATION *op, FILE_ENTRY *feFrom, FILE_ENTRY *feTo copy_dir_to_dir(op, feFrom, feTo->szFullPath); } -static void create_dest_dirs(LPWSTR szDestDir) +static void create_dest_dirs(LPCWSTR szDestDir) { WCHAR dir[MAX_PATH]; - LPWSTR ptr = StrChrW(szDestDir, '\\'); + LPCWSTR ptr = StrChrW(szDestDir, '\\'); /* make sure all directories up to last one are created */ while (ptr && (ptr = StrChrW(ptr + 1, '\\'))) @@ -1172,11 +1173,11 @@ static void create_dest_dirs(LPWSTR szDestDir) } /* the FO_COPY operation */ -static HRESULT copy_files(FILE_OPERATION *op, FILE_LIST *flFrom, FILE_LIST *flTo) +static HRESULT copy_files(FILE_OPERATION *op, const FILE_LIST *flFrom, const FILE_LIST *flTo) { DWORD i; - FILE_ENTRY *entryToCopy; - FILE_ENTRY *fileDest = &flTo->feFiles[0]; + const FILE_ENTRY *entryToCopy; + const FILE_ENTRY *fileDest = &flTo->feFiles[0]; BOOL bCancelIfAnyDirectories = FALSE; if (flFrom->bAnyDontExist) @@ -1269,7 +1270,7 @@ static HRESULT copy_files(FILE_OPERATION *op, FILE_LIST *flFrom, FILE_LIST *flTo return ERROR_SUCCESS; } -static BOOL confirm_delete_list(HWND hWnd, DWORD fFlags, BOOL fTrash, FILE_LIST *flFrom) +static BOOL confirm_delete_list(HWND hWnd, DWORD fFlags, BOOL fTrash, const FILE_LIST *flFrom) { if (flFrom->dwNumFiles > 1) { @@ -1281,7 +1282,7 @@ static BOOL confirm_delete_list(HWND hWnd, DWORD fFlags, BOOL fTrash, FILE_LIST } else { - FILE_ENTRY *fileEntry = &flFrom->feFiles[0]; + const FILE_ENTRY *fileEntry = &flFrom->feFiles[0]; if (IsAttribFile(fileEntry->attributes)) return SHELL_ConfirmDialogW(hWnd, (fTrash?ASK_TRASH_FILE:ASK_DELETE_FILE), fileEntry->szFullPath, NULL); @@ -1292,9 +1293,9 @@ static BOOL confirm_delete_list(HWND hWnd, DWORD fFlags, BOOL fTrash, FILE_LIST } /* the FO_DELETE operation */ -static HRESULT delete_files(LPSHFILEOPSTRUCTW lpFileOp, FILE_LIST *flFrom) +static HRESULT delete_files(LPSHFILEOPSTRUCTW lpFileOp, const FILE_LIST *flFrom) { - FILE_ENTRY *fileEntry; + const FILE_ENTRY *fileEntry; DWORD i; BOOL bPathExists; BOOL bTrash; @@ -1354,7 +1355,7 @@ static HRESULT delete_files(LPSHFILEOPSTRUCTW lpFileOp, FILE_LIST *flFrom) return ERROR_SUCCESS; } -static void move_dir_to_dir(LPSHFILEOPSTRUCTW lpFileOp, FILE_ENTRY *feFrom, LPWSTR szDestPath) +static void move_dir_to_dir(LPSHFILEOPSTRUCTW lpFileOp, const FILE_ENTRY *feFrom, LPCWSTR szDestPath) { WCHAR szFrom[MAX_PATH], szTo[MAX_PATH]; SHFILEOPSTRUCTW fileOp; @@ -1380,7 +1381,7 @@ static void move_dir_to_dir(LPSHFILEOPSTRUCTW lpFileOp, FILE_ENTRY *feFrom, LPWS } /* moves a file or directory to another directory */ -static void move_to_dir(LPSHFILEOPSTRUCTW lpFileOp, FILE_ENTRY *feFrom, FILE_ENTRY *feTo) +static void move_to_dir(LPSHFILEOPSTRUCTW lpFileOp, const FILE_ENTRY *feFrom, const FILE_ENTRY *feTo) { WCHAR szDestPath[MAX_PATH]; @@ -1393,11 +1394,11 @@ static void move_to_dir(LPSHFILEOPSTRUCTW lpFileOp, FILE_ENTRY *feFrom, FILE_ENT } /* the FO_MOVE operation */ -static HRESULT move_files(LPSHFILEOPSTRUCTW lpFileOp, FILE_LIST *flFrom, FILE_LIST *flTo) +static HRESULT move_files(LPSHFILEOPSTRUCTW lpFileOp, const FILE_LIST *flFrom, const FILE_LIST *flTo) { DWORD i; - FILE_ENTRY *entryToMove; - FILE_ENTRY *fileDest; + const FILE_ENTRY *entryToMove; + const FILE_ENTRY *fileDest; if (!flFrom->dwNumFiles || !flTo->dwNumFiles) return ERROR_CANCELLED; @@ -1445,10 +1446,10 @@ static HRESULT move_files(LPSHFILEOPSTRUCTW lpFileOp, FILE_LIST *flFrom, FILE_LI } /* the FO_RENAME files */ -static HRESULT rename_files(LPSHFILEOPSTRUCTW lpFileOp, FILE_LIST *flFrom, FILE_LIST *flTo) +static HRESULT rename_files(LPSHFILEOPSTRUCTW lpFileOp, const FILE_LIST *flFrom, const FILE_LIST *flTo) { - FILE_ENTRY *feFrom; - FILE_ENTRY *feTo; + const FILE_ENTRY *feFrom; + const FILE_ENTRY *feTo; if (flFrom->dwNumFiles != 1) return ERROR_GEN_FAILURE;

1 0