Ros-diffs November 2024

ros-diffs@reactos.org

22 participants
103 discussions

[reactos] 03/10: [VCRUNTIME] Add isa_availability.h

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=be8bbcbe928a3eb1cd97c… commit be8bbcbe928a3eb1cd97c87cb13ba7b185c6d01c Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Sun May 26 13:33:07 2024 +0300 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Sat Nov 16 20:01:49 2024 +0200 [VCRUNTIME] Add isa_availability.h --- sdk/include/vcruntime/isa_availability.h | 33 ++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/sdk/include/vcruntime/isa_availability.h b/sdk/include/vcruntime/isa_availability.h new file mode 100644 index 00000000000..4b011fca3e2 --- /dev/null +++ b/sdk/include/vcruntime/isa_availability.h @@ -0,0 +1,33 @@ +/* + * PROJECT: ReactOS SDK + * LICENSE: MIT (https://spdx.org/licenses/MIT) + * PURPOSE: CRT - ISA availability + * COPYRIGHT: Copyright 2024 Timo Kreuzer (timo.kreuzer(a)reactos.org) + */ + +#pragma once + +#define __ISA_AVAILABILITY__H__ + +enum ISA_AVAILABILITY +{ + __ISA_AVAILABLE_X86 = 0, + __ISA_AVAILABLE_SSE2 = 1, + __ISA_AVAILABLE_SSE42 = 2, + __ISA_AVAILABLE_AVX = 3, + __ISA_AVAILABLE_ENFSTRG = 4, + __ISA_AVAILABLE_AVX2 = 5, + __ISA_AVAILABLE_AVX512 = 6, + + __ISA_AVAILABLE_ARMNT = 0, + __ISA_AVAILABLE_NEON = 1, + __ISA_AVAILABLE_NEON_ARM64 = 2, +}; + +#if defined(_M_IX86) +#define __FAVOR_ATOM 0 +#define __FAVOR_ENFSTRG 1 +#elif defined(_M_X64) +#define __FAVOR_ATOM 0 +#define __FAVOR_ENFSTRG 1 +#endif

1 month, 1 week

[reactos] 02/10: [VCRUNTIME] Fix eh.h

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=fb49e9126e9a9fb6246c3… commit fb49e9126e9a9fb6246c33283b51a4627a98a58d Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Fri Oct 11 15:31:56 2024 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Sat Nov 16 20:01:49 2024 +0200 [VCRUNTIME] Fix eh.h --- sdk/include/vcruntime/eh.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sdk/include/vcruntime/eh.h b/sdk/include/vcruntime/eh.h index a384707e8c7..d37f86e81a0 100644 --- a/sdk/include/vcruntime/eh.h +++ b/sdk/include/vcruntime/eh.h @@ -24,10 +24,10 @@ typedef void (__cdecl *unexpected_handler)(void); struct _EXCEPTION_POINTERS; typedef void (__cdecl *_se_translator_function)(unsigned int,struct _EXCEPTION_POINTERS *); -_CRTIMP __declspec(noreturn) void __cdecl terminate(void); +_CRTIMP __declspec(noreturn) void __cdecl terminate(void) throw(); _CRTIMP void __cdecl unexpected(void); _CRTIMP int __cdecl _is_exception_typeof(const type_info &_Type,struct _EXCEPTION_POINTERS *_ExceptionPtr); -_CRTIMP terminate_function __cdecl set_terminate(terminate_function _NewPtFunc); +_CRTIMP terminate_function __cdecl set_terminate(terminate_function _NewPtFunc) throw(); extern "C" _CRTIMP terminate_function __cdecl _get_terminate(void); _CRTIMP unexpected_function __cdecl set_unexpected(unexpected_function _NewPtFunc); extern "C" _CRTIMP unexpected_function __cdecl _get_unexpected(void);

1 month, 1 week

[reactos] 01/10: [VCRUNTIME] Add __nullptr definition for non-MSVC compilers

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=b75d476bb7b85aaf4e67a… commit b75d476bb7b85aaf4e67abae1582c2732c7bc136 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Tue May 14 09:18:43 2024 +0300 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Sat Nov 16 20:01:49 2024 +0200 [VCRUNTIME] Add __nullptr definition for non-MSVC compilers --- sdk/include/vcruntime/_mingw.h | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/sdk/include/vcruntime/_mingw.h b/sdk/include/vcruntime/_mingw.h index 06e0c62176b..988b64f1dbb 100644 --- a/sdk/include/vcruntime/_mingw.h +++ b/sdk/include/vcruntime/_mingw.h @@ -262,6 +262,10 @@ allow GCC to optimize away some EH unwind code, at least in DW2 case. */ # define __mingw_ovr static __cdecl #endif /* __cplusplus */ +#if defined(__cplusplus) && !defined(_MSC_VER) +#define __nullptr nullptr +#endif + #include "_mingw_mac.h" #endif /* !_INC_MINGW */

1 month, 1 week

[reactos] 01/01: [SHELL32] Correctly compare pidls for SHBrowseForFolder BFFM_SETEXPANDED (#7499)

by Whindmar Saksit

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=6d109254ababae6192aeb… commit 6d109254ababae6192aebca48e4743b0e1efb445 Author: Whindmar Saksit <whindsaks(a)proton.me> AuthorDate: Sat Nov 16 16:28:28 2024 +0100 Commit: GitHub <noreply(a)github.com> CommitDate: Sat Nov 16 16:28:28 2024 +0100 [SHELL32] Correctly compare pidls for SHBrowseForFolder BFFM_SETEXPANDED (#7499) _ILIsEqualSimple just does a memcmp and that does not work for all items. Should improve shell change notifications (CORE-13950). --- dll/win32/shell32/CDefView.cpp | 4 +- dll/win32/shell32/brfolder.cpp | 190 +++++++++++++++++--------------- dll/win32/shell32/folders/CFSFolder.cpp | 14 ++- dll/win32/shell32/precomp.h | 1 + dll/win32/shell32/shlfileop.cpp | 5 +- dll/win32/shell32/utils.cpp | 12 ++ dll/win32/shell32/wine/pidl.c | 98 +++++++++++++++- dll/win32/shell32/wine/pidl.h | 24 ++-- sdk/include/reactos/shellutils.h | 63 +++++++++-- 9 files changed, 297 insertions(+), 114 deletions(-) diff --git a/dll/win32/shell32/CDefView.cpp b/dll/win32/shell32/CDefView.cpp index 95f4a9bff14..d4c8ed82cab 100644 --- a/dll/win32/shell32/CDefView.cpp +++ b/dll/win32/shell32/CDefView.cpp @@ -4608,9 +4608,9 @@ HRESULT WINAPI CDefView::GetAdvise(DWORD *pAspects, DWORD *pAdvf, IAdviseSink ** HRESULT STDMETHODCALLTYPE CDefView::QueryService(REFGUID guidService, REFIID riid, void **ppvObject) { - if (IsEqualIID(guidService, SID_IShellBrowser)) + if (IsEqualIID(guidService, SID_IShellBrowser) && m_pShellBrowser) return m_pShellBrowser->QueryInterface(riid, ppvObject); - else if(IsEqualIID(guidService, SID_IFolderView)) + else if (IsEqualIID(guidService, SID_IFolderView)) return QueryInterface(riid, ppvObject); return E_NOINTERFACE; diff --git a/dll/win32/shell32/brfolder.cpp b/dll/win32/shell32/brfolder.cpp index c23eec56507..7b211939ef1 100644 --- a/dll/win32/shell32/brfolder.cpp +++ b/dll/win32/shell32/brfolder.cpp @@ -11,11 +11,36 @@ #include "precomp.h" #include <ui/layout.h> // Resizable window +#include <compat_undoc.h> // RosGetProcessEffectiveVersion WINE_DEFAULT_DEBUG_CHANNEL(shell); #define SHV_CHANGE_NOTIFY (WM_USER + 0x1111) +static LPITEMIDLIST +ILCloneToDepth(LPCITEMIDLIST pidlSrc, UINT depth) +{ + SIZE_T cb = 0; + for (LPCITEMIDLIST pidl = pidlSrc; pidl && depth--; pidl = ILGetNext(pidl)) + cb += pidl->mkid.cb; + + LPITEMIDLIST pidlOut = (LPITEMIDLIST)SHAlloc(cb + sizeof(WORD)); + if (pidlOut) + { + CopyMemory(pidlOut, pidlSrc, cb); + ZeroMemory(((BYTE*)pidlOut) + cb, sizeof(WORD)); + } + return pidlOut; +} + +static INT +GetIconIndex(PCIDLIST_ABSOLUTE pidl, UINT uFlags) +{ + SHFILEINFOW sfi; + SHGetFileInfoW((LPCWSTR)pidl, 0, &sfi, sizeof(sfi), uFlags); + return sfi.iIcon; +} + struct BrFolder { LPBROWSEINFOW lpBrowseInfo; @@ -145,12 +170,9 @@ BrFolder_InitTreeView(BrFolder *info) { HIMAGELIST hImageList; HRESULT hr; - CComPtr<IShellFolder> lpsfParent; - CComPtr<IEnumIDList> pEnumChildren; HTREEITEM hItem; Shell_GetImageLists(NULL, &hImageList); - if (hImageList) TreeView_SetImageList(info->hwndTreeView, hImageList, 0); @@ -171,54 +193,25 @@ BrFolder_InitTreeView(BrFolder *info) ILRemoveLastID(pidlParent); PCIDLIST_RELATIVE pidlChild = ILFindLastID(pidlRoot); - if (_ILIsDesktop(pidlParent)) - { - hr = SHGetDesktopFolder(&lpsfParent); - if (FAILED_UNEXPECTEDLY(hr)) - return; - } - else - { - CComPtr<IShellFolder> lpsfDesktop; - hr = SHGetDesktopFolder(&lpsfDesktop); - if (FAILED_UNEXPECTEDLY(hr)) - return; - - hr = lpsfDesktop->BindToObject(pidlParent, NULL, IID_PPV_ARG(IShellFolder, &lpsfParent)); - if (FAILED_UNEXPECTEDLY(hr)) - return; - } + CComPtr<IShellFolder> lpsfParent; + hr = SHBindToObject(NULL, pidlParent, /*NULL, */ IID_PPV_ARG(IShellFolder, &lpsfParent)); + if (FAILED_UNEXPECTEDLY(hr)) + return; TreeView_DeleteItem(info->hwndTreeView, TVI_ROOT); hItem = BrFolder_InsertItem(info, lpsfParent, pidlChild, pidlParent, TVI_ROOT); TreeView_Expand(info->hwndTreeView, hItem, TVE_EXPAND); } -static INT -BrFolder_GetIcon(PCIDLIST_ABSOLUTE pidl, UINT uFlags) -{ - SHFILEINFOW sfi; - SHGetFileInfoW((LPCWSTR)pidl, 0, &sfi, sizeof(sfi), uFlags); - return sfi.iIcon; -} - static void BrFolder_GetIconPair(PCIDLIST_ABSOLUTE pidl, LPTVITEMW pItem) { - DWORD flags; - - CComHeapPtr<ITEMIDLIST_ABSOLUTE> pidlDesktop; + static const ITEMIDLIST idlDesktop = { }; if (!pidl) - { - pidlDesktop.Attach(_ILCreateDesktop()); - pidl = pidlDesktop; - } - - flags = SHGFI_PIDL | SHGFI_SYSICONINDEX | SHGFI_SMALLICON; - pItem->iImage = BrFolder_GetIcon(pidl, flags); - - flags = SHGFI_PIDL | SHGFI_SYSICONINDEX | SHGFI_SMALLICON | SHGFI_OPENICON; - pItem->iSelectedImage = BrFolder_GetIcon(pidl, flags); + pidl = &idlDesktop; + DWORD flags = SHGFI_PIDL | SHGFI_SYSICONINDEX | SHGFI_SMALLICON; + pItem->iImage = GetIconIndex(pidl, flags); + pItem->iSelectedImage = GetIconIndex(pidl, flags | SHGFI_OPENICON); } /****************************************************************************** @@ -766,6 +759,19 @@ BrFolder_OnInitDialog(HWND hWnd, BrFolder *info) SHCNE_ALLEVENTS, SHV_CHANGE_NOTIFY, 1, &ntreg); + if (!lpBrowseInfo->pidlRoot) + { + UINT csidl = (lpBrowseInfo->ulFlags & BIF_NEWDIALOGSTYLE) ? CSIDL_PERSONAL : CSIDL_DRIVES; + LPITEMIDLIST pidl = SHCloneSpecialIDList(NULL, csidl, TRUE); + if (pidl) + { + SendMessageW(info->hWnd, BFFM_SETSELECTION, FALSE, (LPARAM)pidl); + if (csidl == CSIDL_DRIVES) + SendMessageW(info->hWnd, BFFM_SETEXPANDED, FALSE, (LPARAM)pidl); + ILFree(pidl); + } + } + BrFolder_Callback(info->lpBrowseInfo, hWnd, BFFM_INITIALIZED, 0); SHAutoComplete(GetDlgItem(hWnd, IDC_BROWSE_FOR_FOLDER_FOLDER_TEXT), @@ -985,8 +991,9 @@ BrFolder_OnContextMenu(BrFolder &info, LPARAM lParam) } static BOOL -BrFolder_OnSetExpandedPidl(BrFolder *info, LPITEMIDLIST pidlSelection, HTREEITEM *phItem) +BrFolder_ExpandToPidl(BrFolder *info, LPITEMIDLIST pidlSelection, HTREEITEM *phItem) { + LPITEMIDLIST pidlCurrent = pidlSelection; if (_ILIsDesktop(pidlSelection)) { if (phItem) @@ -994,24 +1001,6 @@ BrFolder_OnSetExpandedPidl(BrFolder *info, LPITEMIDLIST pidlSelection, HTREEITEM return TRUE; } - // Move pidlCurrent behind the SHITEMIDs in pidlSelection, which are the root of - // the sub-tree currently displayed. - PCIDLIST_ABSOLUTE pidlRoot = info->lpBrowseInfo->pidlRoot; - LPITEMIDLIST pidlCurrent = pidlSelection; - while (!_ILIsEmpty(pidlRoot) && _ILIsEqualSimple(pidlRoot, pidlCurrent)) - { - pidlRoot = ILGetNext(pidlRoot); - pidlCurrent = ILGetNext(pidlCurrent); - } - - // The given ID List is not part of the SHBrowseForFolder's current sub-tree. - if (!_ILIsEmpty(pidlRoot)) - { - if (phItem) - *phItem = NULL; - return FALSE; - } - // Initialize item to point to the first child of the root folder. TVITEMEXW item = { TVIF_PARAM }; item.hItem = TreeView_GetRoot(info->hwndTreeView); @@ -1019,18 +1008,28 @@ BrFolder_OnSetExpandedPidl(BrFolder *info, LPITEMIDLIST pidlSelection, HTREEITEM item.hItem = TreeView_GetChild(info->hwndTreeView, item.hItem); // Walk the tree along the nodes corresponding to the remaining ITEMIDLIST - while (item.hItem && !_ILIsEmpty(pidlCurrent)) + UINT depth = _ILGetDepth(info->lpBrowseInfo->pidlRoot); + while (item.hItem && pidlCurrent) { + LPITEMIDLIST pidlNeedle = ILCloneToDepth(pidlSelection, ++depth); + if (_ILIsEmpty(pidlNeedle)) + { + ILFree(pidlNeedle); + item.hItem = NULL; // Failure + break; + } +next: TreeView_GetItem(info->hwndTreeView, &item); - BrItemData *pItemData = (BrItemData *)item.lParam; - - if (_ILIsEqualSimple(pItemData->pidlChild, pidlCurrent)) + const BrItemData *pItemData = (BrItemData *)item.lParam; + if (ILIsEqual(pItemData->pidlFull, pidlNeedle)) { - pidlCurrent = ILGetNext(pidlCurrent); - if (!_ILIsEmpty(pidlCurrent)) + BOOL done = _ILGetDepth(pidlSelection) == _ILGetDepth(pidlNeedle); + if (done) + { + pidlCurrent = NULL; // Success + } + else { - // Only expand current node and move on to its first child, - // if we didn't already reach the last SHITEMID TreeView_Expand(info->hwndTreeView, item.hItem, TVE_EXPAND); item.hItem = TreeView_GetChild(info->hwndTreeView, item.hItem); } @@ -1038,7 +1037,10 @@ BrFolder_OnSetExpandedPidl(BrFolder *info, LPITEMIDLIST pidlSelection, HTREEITEM else { item.hItem = TreeView_GetNextSibling(info->hwndTreeView, item.hItem); + if (item.hItem) + goto next; } + ILFree(pidlNeedle); } if (phItem) @@ -1048,19 +1050,26 @@ BrFolder_OnSetExpandedPidl(BrFolder *info, LPITEMIDLIST pidlSelection, HTREEITEM } static BOOL -BrFolder_OnSetExpandedString(BrFolder *info, LPWSTR pszString, HTREEITEM *phItem) +BrFolder_ExpandToString(BrFolder *info, LPWSTR pszString, HTREEITEM *phItem) { - CComPtr<IShellFolder> psfDesktop; - HRESULT hr = SHGetDesktopFolder(&psfDesktop); - if (FAILED_UNEXPECTEDLY(hr)) - return FALSE; - CComHeapPtr<ITEMIDLIST_ABSOLUTE> pidlSelection; - hr = psfDesktop->ParseDisplayName(NULL, NULL, pszString, NULL, &pidlSelection, NULL); - if (FAILED_UNEXPECTEDLY(hr)) - return FALSE; + HRESULT hr = SHParseDisplayName(pszString, NULL, &pidlSelection, 0, NULL); + return SUCCEEDED(hr) && BrFolder_ExpandToPidl(info, pidlSelection, phItem); +} + +static BOOL +BrFolder_OnSetExpanded(BrFolder *info, LPITEMIDLIST pidlSelection, LPWSTR pszString) +{ + HTREEITEM hItem; + BOOL ret; + if (pszString) + ret = BrFolder_ExpandToString(info, pszString, &hItem); + else + ret = BrFolder_ExpandToPidl(info, pidlSelection, &hItem); - return BrFolder_OnSetExpandedPidl(info, pidlSelection, phItem); + if (ret) + TreeView_Expand(info->hwndTreeView, hItem, TVE_EXPAND); + return ret; } static BOOL @@ -1070,7 +1079,7 @@ BrFolder_OnSetSelectionPidl(BrFolder *info, LPITEMIDLIST pidlSelection) return FALSE; HTREEITEM hItem; - BOOL ret = BrFolder_OnSetExpandedPidl(info, pidlSelection, &hItem); + BOOL ret = BrFolder_ExpandToPidl(info, pidlSelection, &hItem); if (ret) TreeView_SelectItem(info->hwndTreeView, hItem); return ret; @@ -1083,7 +1092,7 @@ BrFolder_OnSetSelectionW(BrFolder *info, LPWSTR pszSelection) return FALSE; HTREEITEM hItem; - BOOL ret = BrFolder_OnSetExpandedString(info, pszSelection, &hItem); + BOOL ret = BrFolder_ExpandToString(info, pszSelection, &hItem); if (ret) TreeView_SelectItem(info->hwndTreeView, hItem); return ret; @@ -1304,9 +1313,9 @@ BrFolderDlgProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam) case BFFM_SETEXPANDED: // Unicode only if (wParam) // String - return BrFolder_OnSetExpandedString(info, (LPWSTR)lParam, NULL); + return BrFolder_OnSetExpanded(info, NULL, (LPWSTR)lParam); else // PIDL - return BrFolder_OnSetExpandedPidl(info, (LPITEMIDLIST)lParam, NULL); + return BrFolder_OnSetExpanded(info, (LPITEMIDLIST)lParam, NULL); case SHV_CHANGE_NOTIFY: BrFolder_OnChange(info, wParam, lParam); @@ -1362,16 +1371,23 @@ SHBrowseForFolderW(LPBROWSEINFOW lpbi) { TRACE("%p\n", lpbi); + // MSDN says the caller must initialize COM. We do it anyway in case the caller forgot. + COleInit OleInit; BrFolder info = { lpbi }; - HRESULT hr = OleInitialize(NULL); - INT id = ((lpbi->ulFlags & BIF_USENEWUI) ? IDD_BROWSE_FOR_FOLDER_NEW : IDD_BROWSE_FOR_FOLDER); INT_PTR ret = DialogBoxParamW(shell32_hInstance, MAKEINTRESOURCEW(id), lpbi->hwndOwner, BrFolderDlgProc, (LPARAM)&info); - if (SUCCEEDED(hr)) - OleUninitialize(); - + if (ret == IDOK && !(lpbi->ulFlags & BIF_NOTRANSLATETARGETS) && + RosGetProcessEffectiveVersion() >= _WIN32_WINNT_WINXP) + { + PIDLIST_ABSOLUTE pidlTarget; + if (SHELL_GetIDListTarget(info.pidlRet, &pidlTarget) == S_OK) + { + ILFree(info.pidlRet); + info.pidlRet = pidlTarget; + } + } if (ret != IDOK) { ILFree(info.pidlRet); diff --git a/dll/win32/shell32/folders/CFSFolder.cpp b/dll/win32/shell32/folders/CFSFolder.cpp index 0a209ade806..8ff94898d81 100644 --- a/dll/win32/shell32/folders/CFSFolder.cpp +++ b/dll/win32/shell32/folders/CFSFolder.cpp @@ -20,13 +20,20 @@ static LPCWSTR GetItemFileName(PCUITEMID_CHILD pidl, LPWSTR Buf, UINT cchMax) if (pDataW) return pDataW->wszName; LPPIDLDATA pdata = _ILGetDataPointer(pidl); - if ((pdata->type & PT_VALUEW) == PT_VALUEW) + if (_ILGetFSType(pidl) & PT_FS_UNICODE_FLAG) return (LPWSTR)pdata->u.file.szNames; if (_ILSimpleGetTextW(pidl, Buf, cchMax)) return Buf; return NULL; } +static BOOL IsRealItem(const ITEMIDLIST &idl) +{ + // PIDLs created with SHSimpleIDListFromPath contain no data, otherwise, the item is "real" + FileStruct &fsitem = ((PIDLDATA*)idl.mkid.abID)->u.file; + return fsitem.dwFileSize | fsitem.uFileDate; +} + static HKEY OpenKeyFromFileType(LPCWSTR pExtension, LPCWSTR KeyName) { HKEY hkey; @@ -1079,7 +1086,10 @@ HRESULT WINAPI CFSFolder::CompareIDs(LPARAM lParam, /* When sorting between a File and a Folder, the Folder gets sorted first */ if (bIsFolder1 != bIsFolder2) { - return MAKE_COMPARE_HRESULT(bIsFolder1 ? -1 : 1); + // ...but only if neither of them were generated by SHSimpleIDListFromPath + // because in that case we cannot tell if it's a file or a folder. + if (IsRealItem(*pidl1) && IsRealItem(*pidl2)) + return MAKE_COMPARE_HRESULT(bIsFolder1 ? -1 : 1); } int result = 0; diff --git a/dll/win32/shell32/precomp.h b/dll/win32/shell32/precomp.h index 97fd073ab6d..614478a3d08 100644 --- a/dll/win32/shell32/precomp.h +++ b/dll/win32/shell32/precomp.h @@ -219,6 +219,7 @@ SHBindToObjectEx( DWORD SHGetAttributes(_In_ IShellFolder *psf, _In_ LPCITEMIDLIST pidl, _In_ DWORD dwAttributes); +HRESULT SHELL_GetIDListTarget(_In_ LPCITEMIDLIST pidl, _Out_ PIDLIST_ABSOLUTE *ppidl); HRESULT SHCoInitializeAnyApartment(VOID); HRESULT diff --git a/dll/win32/shell32/shlfileop.cpp b/dll/win32/shell32/shlfileop.cpp index 53f209a8f4f..f26d54ecc68 100644 --- a/dll/win32/shell32/shlfileop.cpp +++ b/dll/win32/shell32/shlfileop.cpp @@ -1633,7 +1633,8 @@ static HRESULT delete_files(FILE_OPERATION *op, const FILE_LIST *flFrom) BOOL bDelete; if (TRASH_TrashFile(fileEntry->szFullPath)) { - SHChangeNotify(SHCNE_DELETE, SHCNF_PATHW, fileEntry->szFullPath, NULL); + UINT event = IsAttribFile(fileEntry->attributes) ? SHCNE_DELETE : SHCNE_RMDIR; + SHChangeNotify(event, SHCNF_PATHW, fileEntry->szFullPath, NULL); continue; } @@ -1652,9 +1653,7 @@ static HRESULT delete_files(FILE_OPERATION *op, const FILE_LIST *flFrom) /* delete the file or directory */ if (IsAttribFile(fileEntry->attributes)) - { bPathExists = (ERROR_SUCCESS == SHNotifyDeleteFileW(op, fileEntry->szFullPath)); - } else bPathExists = SHELL_DeleteDirectoryW(op, fileEntry->szFullPath, FALSE); diff --git a/dll/win32/shell32/utils.cpp b/dll/win32/shell32/utils.cpp index fe41af078c2..3fdce2eb5e3 100644 --- a/dll/win32/shell32/utils.cpp +++ b/dll/win32/shell32/utils.cpp @@ -277,6 +277,18 @@ SHGetAttributes(_In_ IShellFolder *psf, _In_ LPCITEMIDLIST pidl, _In_ DWORD dwAt return dwAttributes; } +HRESULT SHELL_GetIDListTarget(_In_ LPCITEMIDLIST pidl, _Out_ PIDLIST_ABSOLUTE *ppidl) +{ + IShellLink *pSL; + HRESULT hr = SHBindToObject(NULL, pidl, IID_PPV_ARG(IShellLink, &pSL)); + if (SUCCEEDED(hr)) + { + hr = pSL->GetIDList(ppidl); // Note: Returns S_FALSE if no target pidl + pSL->Release(); + } + return hr; +} + HRESULT SHCoInitializeAnyApartment(VOID) { HRESULT hr = CoInitializeEx(NULL, COINIT_APARTMENTTHREADED | COINIT_DISABLE_OLE1DDE); diff --git a/dll/win32/shell32/wine/pidl.c b/dll/win32/shell32/wine/pidl.c index 54751ded772..f6058d04ee5 100644 --- a/dll/win32/shell32/wine/pidl.c +++ b/dll/win32/shell32/wine/pidl.c @@ -485,6 +485,37 @@ LPITEMIDLIST WINAPI ILGlobalClone(LPCITEMIDLIST pidl) return newpidl; } +#ifdef __REACTOS__ +static inline LPITEMIDLIST _ILUnsafeNext(LPCITEMIDLIST pidl) +{ + return (LPITEMIDLIST)(((BYTE*)pidl) + pidl->mkid.cb); +} + +UINT _ILGetDepth(LPCITEMIDLIST pidl) +{ + for (UINT i = 0;; ++i) + { + if (!pidl || !pidl->mkid.cb) + return i; + pidl = _ILUnsafeNext(pidl); + } +} + +static BOOL _ILMemCmpEqualIDList(LPCITEMIDLIST p1, LPCITEMIDLIST p2) +{ + for (;; p1 = _ILUnsafeNext(p1), p2 = _ILUnsafeNext(p2)) + { + DWORD cb1 = p1 ? p1->mkid.cb : 0x80000000; /* Empty != NULL */ + DWORD cb2 = p2 ? p2->mkid.cb : 0x80000000; + if (cb1 != cb2) + return FALSE; + if (LOWORD(cb1) == 0) + return cb1 == cb2; + if (memcmp(p1, p2, cb1)) + return FALSE; + } +} +#else /* __REACTOS__ */ BOOL _ILHACKCompareSimpleIds(LPCITEMIDLIST pidltemp1, LPCITEMIDLIST pidltemp2) { LPPIDLDATA pdata1 = _ILGetDataPointer(pidltemp1); @@ -540,6 +571,7 @@ BOOL _ILHACKCompareSimpleIds(LPCITEMIDLIST pidltemp1, LPCITEMIDLIST pidltemp2) return TRUE; } +#endif /* __REACTOS__ */ /************************************************************************* * ILIsEqual [SHELL32.21] @@ -552,6 +584,21 @@ BOOL WINAPI ILIsEqual(LPCITEMIDLIST pidl1, LPCITEMIDLIST pidl2) TRACE("pidl1=%p pidl2=%p\n",pidl1, pidl2); +#ifdef __REACTOS__ + IShellFolder *psfDesktop; + UINT depth1; + + if (pidl1 == pidl2 || _ILMemCmpEqualIDList(pidltemp1, pidltemp2)) + return TRUE; + + depth1 = _ILGetDepth(pidl1); + if (depth1 && depth1 == _ILGetDepth(pidl2) && SUCCEEDED(SHGetDesktopFolder(&psfDesktop))) + { + HRESULT hr = IShellFolder_CompareIDs(psfDesktop, SHCIDS_CANONICALONLY, pidl1, pidl2); + IShellFolder_Release(psfDesktop); + return hr == 0; + } +#else /* __REACTOS__ */ /* * Explorer reads from registry directly (StreamMRU), * so we can only check here @@ -582,10 +629,47 @@ BOOL WINAPI ILIsEqual(LPCITEMIDLIST pidl1, LPCITEMIDLIST pidl2) if (!pidltemp1->mkid.cb && !pidltemp2->mkid.cb) return TRUE; - +#endif /* __REACTOS__ */ return FALSE; } +LPCITEMIDLIST _ILIsParentEx(LPCITEMIDLIST pidlParent, LPCITEMIDLIST pidlChild, BOOL bImmediate) +{ + LPCITEMIDLIST pParentRoot = pidlParent, pChildRoot = pidlChild, pResult = NULL; + LPITEMIDLIST pidl; + SIZE_T cb = 0; + + if (!pidlParent || !pidlChild) + return pResult; + + while (pidlParent->mkid.cb) + { + cb += pidlChild->mkid.cb; + if (!pidlChild->mkid.cb) + { + if (pidlParent->mkid.cb) + return pResult; /* The child is shorter than the parent */ + else + break; + } + pidlChild = _ILUnsafeNext(pidlChild); + pidlParent = _ILUnsafeNext(pidlParent); + } + + if (bImmediate && (!pidlChild->mkid.cb || _ILUnsafeNext(pidlChild)->mkid.cb)) + return pResult; /* Same as parent or a deeper grandchild */ + + if ((pidl = SHAlloc(cb + sizeof(WORD))) != NULL) + { + CopyMemory(pidl, pChildRoot, cb); + ZeroMemory((BYTE*)pidl + cb, sizeof(WORD)); + if (ILIsEqual(pParentRoot, pidl)) + pResult = pidlChild; + ILFree(pidl); + } + return pResult; +} + /************************************************************************* * ILIsParent [SHELL32.23] * @@ -615,6 +699,9 @@ BOOL WINAPI ILIsParent(LPCITEMIDLIST pidlParent, LPCITEMIDLIST pidlChild, BOOL b TRACE("%p %p %x\n", pidlParent, pidlChild, bImmediate); +#ifdef __REACTOS__ + return _ILIsParentEx(pParent, pChild, bImmediate) != NULL; +#else /* __REACTOS__ */ if (!pParent || !pChild) return FALSE; @@ -636,6 +723,7 @@ BOOL WINAPI ILIsParent(LPCITEMIDLIST pidlParent, LPCITEMIDLIST pidlChild, BOOL b return FALSE; return TRUE; +#endif /* __REACTOS__ */ } /************************************************************************* @@ -659,6 +747,13 @@ BOOL WINAPI ILIsParent(LPCITEMIDLIST pidlParent, LPCITEMIDLIST pidlChild, BOOL b */ PUIDLIST_RELATIVE WINAPI ILFindChild(PIDLIST_ABSOLUTE pidl1, PCIDLIST_ABSOLUTE pidl2) { +#ifdef __REACTOS__ + TRACE("pidl1=%p pidl2=%p\n", pidl1, pidl2); + + if (_ILIsDesktop(pidl1)) + return (PUIDLIST_RELATIVE)pidl2; + return (PUIDLIST_RELATIVE)_ILIsParentEx(pidl1, pidl2, FALSE); +#else /* __REACTOS__ */ LPCITEMIDLIST pidltemp1 = pidl1; LPCITEMIDLIST pidltemp2 = pidl2; LPCITEMIDLIST ret=NULL; @@ -694,6 +789,7 @@ PUIDLIST_RELATIVE WINAPI ILFindChild(PIDLIST_ABSOLUTE pidl1, PCIDLIST_ABSOLUTE p } TRACE_(shell)("--- %p\n", ret); return (PUIDLIST_RELATIVE)ret; /* pidl 1 is shorter */ +#endif /* __REACTOS__ */ } /************************************************************************* diff --git a/dll/win32/shell32/wine/pidl.h b/dll/win32/shell32/wine/pidl.h index 304ce1cc19b..ef0002941f7 100644 --- a/dll/win32/shell32/wine/pidl.h +++ b/dll/win32/shell32/wine/pidl.h @@ -105,8 +105,9 @@ extern "C" { #define PT_SHARE 0xc3 #ifdef __REACTOS__ +#define PT_FOLDERTYPEMASK 0x70 #define PT_DESKTOP_REGITEM 0x1F // => SHDID_ROOT_REGITEM -#define PT_COMPUTER_REGITEM 0x2E // => SHDID_COMPUTER_OTHER +#define PT_COMPUTER_REGITEM 0x2E // => SHDID_COMPUTER_? #define PT_FS 0x30 // Win95 SHSimpleIDListFromPath #define PT_FS_FOLDER_FLAG 0x01 #define PT_FS_FILE_FLAG 0x02 @@ -116,6 +117,17 @@ extern "C" { #define PT_CONTROLS_NEWREGITEM 0x71 #endif +static inline BYTE _ILGetType(LPCITEMIDLIST pidl) +{ + return pidl && pidl->mkid.cb >= 3 ? pidl->mkid.abID[0] : 0; +} + +static inline BYTE _ILGetFSType(LPCITEMIDLIST pidl) +{ + const BYTE type = _ILGetType(pidl); + return (type & PT_FOLDERTYPEMASK) == PT_FS ? type : 0; +} + #include "pshpack1.h" typedef BYTE PIDLTYPE; @@ -257,14 +269,8 @@ BOOL _ILIsValue (LPCITEMIDLIST pidl) DECLSPEC_HIDDEN; BOOL _ILIsSpecialFolder (LPCITEMIDLIST pidl) DECLSPEC_HIDDEN; BOOL _ILIsPidlSimple (LPCITEMIDLIST pidl) DECLSPEC_HIDDEN; BOOL _ILIsCPanelStruct (LPCITEMIDLIST pidl) DECLSPEC_HIDDEN; -static inline -BOOL _ILIsEqualSimple (LPCITEMIDLIST pidlA, LPCITEMIDLIST pidlB) -{ - return (pidlA->mkid.cb > 0 && !memcmp(pidlA, pidlB, pidlA->mkid.cb)) || - (!pidlA->mkid.cb && !pidlB->mkid.cb); -} -static inline -BOOL _ILIsEmpty (LPCITEMIDLIST pidl) { return _ILIsDesktop(pidl); } +static inline BOOL _ILIsEmpty(LPCITEMIDLIST pidl) { return _ILIsDesktop(pidl); } +UINT _ILGetDepth(LPCITEMIDLIST pidl); /* * simple pidls diff --git a/sdk/include/reactos/shellutils.h b/sdk/include/reactos/shellutils.h index 825e834897b..756c57c9d73 100644 --- a/sdk/include/reactos/shellutils.h +++ b/sdk/include/reactos/shellutils.h @@ -597,21 +597,19 @@ void DumpIdList(LPCITEMIDLIST pcidl) DbgPrint("End IDList Dump.\n"); } -struct CCoInit +template <HRESULT (WINAPI *InitFunc)(void*), void (WINAPI *UninitFunc)()> +struct CCoInitBase { - CCoInit() - { - hr = CoInitialize(NULL); - } - ~CCoInit() + HRESULT hr; + CCoInitBase() : hr(InitFunc(NULL)) { } + ~CCoInitBase() { if (SUCCEEDED(hr)) - { - CoUninitialize(); - } + UninitFunc(); } - HRESULT hr; }; +typedef CCoInitBase<CoInitialize, CoUninitialize> CCoInit; +typedef CCoInitBase<OleInitialize, OleUninitialize> COleInit; #endif /* __cplusplus */ @@ -828,5 +826,50 @@ struct SHELL_GetSettingImpl #define SHELL_GetSetting(pss, ssf, field) ( SHGetSetSettings((pss), (ssf), FALSE), (pss)->field ) #endif +static inline void DumpIdListOneLine(LPCITEMIDLIST pidl) +{ + char buf[1024], *data, drive = 0; + for (UINT depth = 0, type; ; pidl = ILGetNext(pidl), ++depth) + { + if (!pidl || !pidl->mkid.cb) + { + if (!depth) + { + wsprintfA(buf, "%p [] (%s)\n", pidl, pidl ? "Empty/Desktop" : "NULL"); + OutputDebugStringA(buf); + } + break; + } + else if (!depth) + { + wsprintfA(buf, "%p", pidl); + OutputDebugStringA(buf); + } + type = pidl->mkid.abID[0] & 0x7f; + data = (char*)&pidl->mkid.abID[0]; + if (depth == 0 && type == 0x1f && pidl->mkid.cb == 20 && *(UINT*)(&data[2]) == 0x20D04FE0) + { + wsprintfA(buf, " [%.2x ThisPC?]", type); /* "?" because we did not check the full GUID */ + } + else if (depth == 1 && type >= 0x20 && type < 0x30 && type != 0x2E && pidl->mkid.cb > 4) + { + drive = data[1]; + wsprintfA(buf, " [%.2x %c: %ub]", type, drive, pidl->mkid.cb); + } + else if (depth >= 2 && drive && (type & 0x70) == 0x30) /* PT_FS */ + { + if (type & 4) + wsprintfA(buf, " [%.2x FS %.256ls %ub]", type, data + 12, pidl->mkid.cb); + else + wsprintfA(buf, " [%.2x FS %.256hs %ub]", type, data + 12, pidl->mkid.cb); + } + else + { + wsprintfA(buf, " [%.2x ? %ub]", type, pidl->mkid.cb); + } + OutputDebugStringA(buf); + } + OutputDebugStringA("\n"); +} #endif /* __ROS_SHELL_UTILS_H */

1 month, 1 week

[reactos] 01/01: [CMBATT] Fix battery Tag

by Hervé Poussineau

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=81860b482a745b86572fb… commit 81860b482a745b86572fb4804aa28c3481117fac Author: Hervé Poussineau <hpoussin(a)reactos.org> AuthorDate: Tue Nov 12 20:39:48 2024 +0100 Commit: Hervé Poussineau <hpoussin(a)reactos.org> CommitDate: Tue Nov 12 21:26:42 2024 +0100 [CMBATT] Fix battery Tag 0 (ie BATTERY_TAG_INVALID) is not a valid battery tag. First battery must have a tag of 1. --- drivers/bus/acpi/cmbatt/cmbatt.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/bus/acpi/cmbatt/cmbatt.c b/drivers/bus/acpi/cmbatt/cmbatt.c index 09d9c81d837..3935a9c8150 100644 --- a/drivers/bus/acpi/cmbatt/cmbatt.c +++ b/drivers/bus/acpi/cmbatt/cmbatt.c @@ -568,7 +568,6 @@ CmBattQueryTag(IN PCMBATT_DEVICE_EXTENSION DeviceExtension, { PDEVICE_OBJECT PdoDevice; ULONG StaData; - ULONG NewTag; NTSTATUS Status; PAGED_CODE(); if (CmBattDebug & (CMBATT_ACPI_WARNING | CMBATT_GENERIC_INFO)) @@ -587,12 +586,12 @@ CmBattQueryTag(IN PCMBATT_DEVICE_EXTENSION DeviceExtension, if (StaData & ACPI_STA_BATTERY_PRESENT) { /* Do we not have a tag yet? */ - if (!DeviceExtension->Tag) + if (DeviceExtension->Tag == BATTERY_TAG_INVALID) { /* Set the new tag value, reset tags if we reached the maximum */ - NewTag = DeviceExtension->TagData; - if (DeviceExtension->TagData++ == 0xFFFFFFFF) NewTag = 1; - DeviceExtension->Tag = NewTag; + if (++DeviceExtension->TagData == BATTERY_TAG_INVALID) + DeviceExtension->TagData = 1; + DeviceExtension->Tag = DeviceExtension->TagData; if (CmBattDebug & CMBATT_GENERIC_INFO) DbgPrint("CmBattQueryTag - New Tag: (%d)\n", DeviceExtension->Tag); @@ -608,7 +607,7 @@ CmBattQueryTag(IN PCMBATT_DEVICE_EXTENSION DeviceExtension, else { /* No battery, so no tag */ - DeviceExtension->Tag = 0; + DeviceExtension->Tag = BATTERY_TAG_INVALID; Status = STATUS_NO_SUCH_DEVICE; } }

1 month, 1 week

[reactos] 03/03: [USETUP] Deduplicate code in SelectPartitionPage()

by Hermès Bélusca-Maïto

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=6aa11ac494ec2cc127196… commit 6aa11ac494ec2cc127196f17247fbb3fb973b86a Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Sun Nov 10 17:43:31 2024 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Sun Nov 10 18:45:58 2024 +0100 [USETUP] Deduplicate code in SelectPartitionPage() --- base/setup/usetup/usetup.c | 207 ++++++++++++++++++++++----------------------- 1 file changed, 103 insertions(+), 104 deletions(-) diff --git a/base/setup/usetup/usetup.c b/base/setup/usetup/usetup.c index ee02faaca8f..d5d569ede42 100644 --- a/base/setup/usetup/usetup.c +++ b/base/setup/usetup/usetup.c @@ -1493,23 +1493,20 @@ LayoutSettingsPage(PINPUT_RECORD Ir) static BOOLEAN -IsPartitionLargeEnough( - _In_ PPARTENTRY PartEntry) +IsMediumLargeEnough( + _In_ ULONGLONG SizeInBytes) { /* Retrieve the maximum size in MB (rounded up) */ - ULONGLONG PartSize = RoundingDivide(GetPartEntrySizeInBytes(PartEntry), MB); + ULONGLONG SizeInMB = RoundingDivide(SizeInBytes, MB); - if (PartSize < USetupData.RequiredPartitionDiskSpace) + /* Check the medium size */ + if (SizeInMB < USetupData.RequiredPartitionDiskSpace) { - /* Partition is too small so ask for another one */ - DPRINT1("Partition is too small (size: %I64u MB), required disk space is %lu MB\n", - PartSize, USetupData.RequiredPartitionDiskSpace); + DPRINT1("Partition/Volume is too small (size: %I64u MB), required space is %lu MB\n", + SizeInMB, USetupData.RequiredPartitionDiskSpace); return FALSE; } - else - { - return TRUE; - } + return TRUE; } @@ -1535,6 +1532,7 @@ SelectPartitionPage(PINPUT_RECORD Ir) { PARTLIST_UI ListUi; ULONG Error; + ULONGLONG MaxTargetSize; if (PartitionList == NULL) { @@ -1555,7 +1553,9 @@ SelectPartitionPage(PINPUT_RECORD Ir) { ASSERT(CurrentInstallation); - /* Determine the selected installation disk & partition */ + /* Determine the selected installation disk & partition. + * It must exist and be valid, since this is the partition + * where the existing installation already resides. */ InstallPartition = SelectPartition(PartitionList, CurrentInstallation->DiskNumber, CurrentInstallation->PartitionNumber); @@ -1564,6 +1564,8 @@ SelectPartitionPage(PINPUT_RECORD Ir) DPRINT1("RepairUpdateFlag == TRUE, SelectPartition() returned FALSE, assert!\n"); ASSERT(FALSE); } + ASSERT(InstallPartition->IsPartitioned); + ASSERT(InstallPartition->Volume); return START_PARTITION_OPERATIONS_PAGE; } @@ -1577,59 +1579,46 @@ SelectPartitionPage(PINPUT_RECORD Ir) yScreen - 3); DrawPartitionList(&ListUi); - if (IsUnattendedSetup) + if (IsUnattendedSetup) do { + /* If DestinationDiskNumber or DestinationPartitionNumber are invalid + * (see below), don't select the partition and show the list instead */ + if (USetupData.DestinationDiskNumber == -1 || + USetupData.DestinationPartitionNumber == -1) + { + break; + } + /* Determine the selected installation disk & partition */ - InstallPartition = SelectPartition(PartitionList, + CurrentPartition = SelectPartition(PartitionList, USetupData.DestinationDiskNumber, USetupData.DestinationPartitionNumber); - if (!InstallPartition) - { - CurrentPartition = ListUi.CurrentPartition; - - if (USetupData.AutoPartition) - { - ASSERT(CurrentPartition != NULL); - ASSERT(!IsContainerPartition(CurrentPartition->PartitionType)); - - /* Automatically create the partition on the whole empty space; - * it will be formatted later with default parameters */ - CreatePartition(PartitionList, - CurrentPartition, - 0ULL, - 0); - if (CurrentPartition->Volume) - CurrentPartition->Volume->New |= VOLUME_NEW_AUTOCREATE; - -// FIXME?? Aren't we going to enter an infinite loop, if this test fails?? - if (!IsPartitionLargeEnough(CurrentPartition)) - { - MUIDisplayError(ERROR_INSUFFICIENT_PARTITION_SIZE, Ir, POPUP_WAIT_ANY_KEY, - USetupData.RequiredPartitionDiskSpace); - return SELECT_PARTITION_PAGE; /* let the user select another partition */ - } - InstallPartition = CurrentPartition; - return START_PARTITION_OPERATIONS_PAGE; - } - } - else + /* Now reset DestinationDiskNumber and DestinationPartitionNumber + * to *invalid* values, so that if the corresponding partition is + * determined to be invalid by the code below or in CreateInstallPartition, + * we don't reselect it when SelectPartitionPage() is called again */ + USetupData.DestinationDiskNumber = -1; + USetupData.DestinationPartitionNumber = -1; + + // FIXME: Here and in the AutoPartition case below, the CurrentPartition + // may actually be unsuitable (MBR-extended, non-simple volume...). + // More checks need to be made here! + // + // NOTE: We don't check for CurrentPartition->Volume in case + // the partition doesn't contain a recognized volume/none exists. + // We also don't check whether IsPartitioned is TRUE, because if + // the partition is still empty space, we'll try to partition it. + if (CurrentPartition && !IsContainerPartition(CurrentPartition->PartitionType)) + goto CreateInstallPartition; + + if (USetupData.AutoPartition) { - ASSERT(!IsContainerPartition(InstallPartition->PartitionType)); - - DrawPartitionList(&ListUi); // FIXME: Doesn't make much sense... - -// FIXME?? Aren't we going to enter an infinite loop, if this test fails?? - if (!IsPartitionLargeEnough(InstallPartition)) - { - MUIDisplayError(ERROR_INSUFFICIENT_PARTITION_SIZE, Ir, POPUP_WAIT_ANY_KEY, - USetupData.RequiredPartitionDiskSpace); - return SELECT_PARTITION_PAGE; /* let the user select another partition */ - } - - return START_PARTITION_OPERATIONS_PAGE; + CurrentPartition = ListUi.CurrentPartition; + // TODO: Do more checks, and loop until we find a valid partition. + goto CreateInstallPartition; } - } + } while (0); while (TRUE) { @@ -1638,7 +1627,7 @@ SelectPartitionPage(PINPUT_RECORD Ir) CurrentPartition = ListUi.CurrentPartition; /* Update status text */ - if (CurrentPartition == NULL) + if (!CurrentPartition) { // FIXME: If we get a NULL current partition, this means that // the current disk is of unrecognized type. So we should display @@ -1679,7 +1668,7 @@ SelectPartitionPage(PINPUT_RECORD Ir) PartitionList = NULL; return QUIT_PAGE; } - break; + return SELECT_PARTITION_PAGE; } else if ((Ir->Event.KeyEvent.uChar.AsciiChar == 0x00) && (Ir->Event.KeyEvent.wVirtualKeyCode == VK_DOWN)) /* DOWN */ @@ -1693,10 +1682,11 @@ SelectPartitionPage(PINPUT_RECORD Ir) } else if (Ir->Event.KeyEvent.wVirtualKeyCode == VK_RETURN) /* ENTER */ { - ASSERT(CurrentPartition != NULL); + ASSERT(CurrentPartition); + /* Don't select an extended partition for OS installation */ if (IsContainerPartition(CurrentPartition->PartitionType)) - continue; // return SELECT_PARTITION_PAGE; + continue; /* * Check whether the user wants to install ReactOS on a disk that @@ -1715,38 +1705,11 @@ SelectPartitionPage(PINPUT_RECORD Ir) // return SELECT_PARTITION_PAGE; } - if (!CurrentPartition->IsPartitioned) - { - Error = PartitionCreationChecks(CurrentPartition); - if (Error != NOT_AN_ERROR) - { - MUIDisplayError(Error, Ir, POPUP_WAIT_ANY_KEY); - return SELECT_PARTITION_PAGE; - } - - /* Automatically create the partition on the whole empty space; - * it will be formatted later with default parameters */ - CreatePartition(PartitionList, - CurrentPartition, - 0ULL, - 0); - if (CurrentPartition->Volume) - CurrentPartition->Volume->New |= VOLUME_NEW_AUTOCREATE; - } - - if (!IsPartitionLargeEnough(CurrentPartition)) - { - MUIDisplayError(ERROR_INSUFFICIENT_PARTITION_SIZE, Ir, POPUP_WAIT_ANY_KEY, - USetupData.RequiredPartitionDiskSpace); - return SELECT_PARTITION_PAGE; /* let the user select another partition */ - } - - InstallPartition = CurrentPartition; - return START_PARTITION_OPERATIONS_PAGE; + goto CreateInstallPartition; } else if (Ir->Event.KeyEvent.wVirtualKeyCode == 'C') /* C */ { - ASSERT(CurrentPartition != NULL); + ASSERT(CurrentPartition); Error = PartitionCreationChecks(CurrentPartition); if (Error != NOT_AN_ERROR) @@ -1760,24 +1723,25 @@ SelectPartitionPage(PINPUT_RECORD Ir) } else if (Ir->Event.KeyEvent.wVirtualKeyCode == 'E') /* E */ { - ASSERT(CurrentPartition != NULL); + ASSERT(CurrentPartition); - if (CurrentPartition->LogicalPartition == FALSE) - { - Error = ExtendedPartitionCreationChecks(CurrentPartition); - if (Error != NOT_AN_ERROR) - { - MUIDisplayError(Error, Ir, POPUP_WAIT_ANY_KEY); - return SELECT_PARTITION_PAGE; - } + /* Don't create an extended partition within a logical partition */ + if (CurrentPartition->LogicalPartition) + continue; - PartCreateType = PartTypeExtended; - return CREATE_PARTITION_PAGE; + Error = ExtendedPartitionCreationChecks(CurrentPartition); + if (Error != NOT_AN_ERROR) + { + MUIDisplayError(Error, Ir, POPUP_WAIT_ANY_KEY); + return SELECT_PARTITION_PAGE; } + + PartCreateType = PartTypeExtended; + return CREATE_PARTITION_PAGE; } else if (Ir->Event.KeyEvent.wVirtualKeyCode == 'D') /* D */ { - ASSERT(CurrentPartition != NULL); + ASSERT(CurrentPartition); /* Ignore deletion in case this is not a partitioned entry */ if (!CurrentPartition->IsPartitioned) @@ -1818,7 +1782,42 @@ SelectPartitionPage(PINPUT_RECORD Ir) } } - return SELECT_PARTITION_PAGE; +CreateInstallPartition: + ASSERT(CurrentPartition); + ASSERT(!IsContainerPartition(CurrentPartition->PartitionType)); + + /* Create the partition if the selected region is empty */ + if (!CurrentPartition->IsPartitioned) + { + Error = PartitionCreationChecks(CurrentPartition); + if (Error != NOT_AN_ERROR) + { + MUIDisplayError(Error, Ir, POPUP_WAIT_ANY_KEY); + return SELECT_PARTITION_PAGE; + } + + /* Automatically create the partition on the whole empty space; + * it will be formatted later with default parameters */ + CreatePartition(PartitionList, + CurrentPartition, + 0ULL, + 0); + ASSERT(CurrentPartition->IsPartitioned); + if (CurrentPartition->Volume) + CurrentPartition->Volume->New |= VOLUME_NEW_AUTOCREATE; + } + + /* Verify the target medium size */ + MaxTargetSize = GetPartEntrySizeInBytes(CurrentPartition); + if (!IsMediumLargeEnough(MaxTargetSize)) + { + MUIDisplayError(ERROR_INSUFFICIENT_PARTITION_SIZE, Ir, POPUP_WAIT_ANY_KEY, + USetupData.RequiredPartitionDiskSpace); + return SELECT_PARTITION_PAGE; /* Let the user select another partition */ + } + + InstallPartition = CurrentPartition; + return START_PARTITION_OPERATIONS_PAGE; }

1 month, 2 weeks

[reactos] 02/03: [USETUP] Deduplicate code in InstallDirectoryPage()

by Hermès Bélusca-Maïto

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=50668182844bb50c89e80… commit 50668182844bb50c89e801cb72f6bf8d6741e4da Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Sat Nov 9 19:15:23 2024 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Sun Nov 10 18:45:57 2024 +0100 [USETUP] Deduplicate code in InstallDirectoryPage() --- base/setup/usetup/usetup.c | 85 +++++++++++++++++++--------------------------- 1 file changed, 34 insertions(+), 51 deletions(-) diff --git a/base/setup/usetup/usetup.c b/base/setup/usetup/usetup.c index db788518a55..ee02faaca8f 100644 --- a/base/setup/usetup/usetup.c +++ b/base/setup/usetup/usetup.c @@ -2878,28 +2878,14 @@ InstallDirectoryPage(PINPUT_RECORD Ir) * of an invalid path, or we are in regular setup), display the UI and allow * the user to specify a new installation path. */ - if ((RepairUpdateFlag || IsUnattendedSetup) && IsValidInstallDirectory(InstallDir)) + if (RepairUpdateFlag || IsUnattendedSetup) { - Status = InitDestinationPaths(&USetupData, InstallDir, InstallVolume); - if (!NT_SUCCESS(Status)) - { - DPRINT1("InitDestinationPaths() failed: Status 0x%lx\n", Status); - MUIDisplayError(ERROR_NO_BUILD_PATH, Ir, POPUP_WAIT_ENTER); - return QUIT_PAGE; - } - - /* - * Check whether the user attempts to install ReactOS within the - * installation source directory, or in a subdirectory thereof. - * If so, fail with an error. - */ - if (RtlPrefixUnicodeString(&USetupData.SourcePath, &USetupData.DestinationPath, TRUE)) - { - MUIDisplayError(ERROR_SOURCE_DIR, Ir, POPUP_WAIT_ENTER); - return INSTALL_DIRECTORY_PAGE; - } + /* Check for the validity of the installation directory and pop up + * an error if it is not the case. Then the user can fix it. */ + if (IsValidInstallDirectory(InstallDir)) + goto InitInstallDir; - return PREPARE_COPY_PAGE; + MUIDisplayError(ERROR_DIRECTORY_NAME, Ir, POPUP_WAIT_ENTER); } Length = wcslen(InstallDir); @@ -2921,7 +2907,7 @@ InstallDirectoryPage(PINPUT_RECORD Ir) if (ConfirmQuit(Ir)) return QUIT_PAGE; - break; + return INSTALL_DIRECTORY_PAGE; } else if ((Ir->Event.KeyEvent.uChar.AsciiChar == 0x00) && (Ir->Event.KeyEvent.wVirtualKeyCode == VK_DELETE)) /* DEL */ @@ -2980,36 +2966,13 @@ InstallDirectoryPage(PINPUT_RECORD Ir) { CONSOLE_SetCursorType(TRUE, FALSE); - /* - * Check for the validity of the installation directory and pop up - * an error if it is not the case. Then the user can fix its input. - */ - if (!IsValidInstallDirectory(InstallDir)) - { - MUIDisplayError(ERROR_DIRECTORY_NAME, Ir, POPUP_WAIT_ENTER); - return INSTALL_DIRECTORY_PAGE; - } - - Status = InitDestinationPaths(&USetupData, InstallDir, InstallVolume); - if (!NT_SUCCESS(Status)) - { - DPRINT1("InitDestinationPaths() failed: Status 0x%lx\n", Status); - MUIDisplayError(ERROR_NO_BUILD_PATH, Ir, POPUP_WAIT_ENTER); - return QUIT_PAGE; - } - - /* - * Check whether the user attempts to install ReactOS within the - * installation source directory, or in a subdirectory thereof. - * If so, fail with an error. - */ - if (RtlPrefixUnicodeString(&USetupData.SourcePath, &USetupData.DestinationPath, TRUE)) - { - MUIDisplayError(ERROR_SOURCE_DIR, Ir, POPUP_WAIT_ENTER); - return INSTALL_DIRECTORY_PAGE; - } + /* Check for the validity of the installation directory and pop up + * an error if it is not the case. Then the user can fix it. */ + if (IsValidInstallDirectory(InstallDir)) + goto InitInstallDir; - return PREPARE_COPY_PAGE; + MUIDisplayError(ERROR_DIRECTORY_NAME, Ir, POPUP_WAIT_ENTER); + return INSTALL_DIRECTORY_PAGE; } else if (Ir->Event.KeyEvent.uChar.AsciiChar == 0x08) /* BACKSPACE */ { @@ -3051,7 +3014,27 @@ InstallDirectoryPage(PINPUT_RECORD Ir) } } - return INSTALL_DIRECTORY_PAGE; +InitInstallDir: + Status = InitDestinationPaths(&USetupData, InstallDir, InstallVolume); + if (!NT_SUCCESS(Status)) + { + DPRINT1("InitDestinationPaths() failed: Status 0x%lx\n", Status); + MUIDisplayError(ERROR_NO_BUILD_PATH, Ir, POPUP_WAIT_ENTER); + return QUIT_PAGE; + } + + /* + * Check whether the user attempts to install ReactOS within the + * installation source directory, or in a subdirectory thereof. + * If so, fail with an error. + */ + if (RtlPrefixUnicodeString(&USetupData.SourcePath, &USetupData.DestinationPath, TRUE)) + { + MUIDisplayError(ERROR_SOURCE_DIR, Ir, POPUP_WAIT_ENTER); + return INSTALL_DIRECTORY_PAGE; + } + + return PREPARE_COPY_PAGE; }

1 month, 2 weeks

[reactos] 01/03: [USETUP] Minor simplifications in some branching code

by Hermès Bélusca-Maïto

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=8bb3e5a017a47ae32311d… commit 8bb3e5a017a47ae32311d4e23b8f2c25a73706ea Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Sat Nov 9 16:18:39 2024 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Sun Nov 10 18:45:40 2024 +0100 [USETUP] Minor simplifications in some branching code --- base/setup/usetup/usetup.c | 37 +++++++------------------------------ 1 file changed, 7 insertions(+), 30 deletions(-) diff --git a/base/setup/usetup/usetup.c b/base/setup/usetup/usetup.c index 806353223a8..db788518a55 100644 --- a/base/setup/usetup/usetup.c +++ b/base/setup/usetup/usetup.c @@ -727,8 +727,7 @@ LanguagePage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - else - RedrawGenericList(&ListUi); + RedrawGenericList(&ListUi); } else if (Ir->Event.KeyEvent.uChar.AsciiChar == 0x0D) /* ENTER */ { @@ -818,7 +817,6 @@ WelcomePage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - break; } else if (Ir->Event.KeyEvent.uChar.AsciiChar == 0x0D) /* ENTER */ @@ -995,8 +993,7 @@ UpgradeRepairPage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - else - RedrawGenericList(&ListUi); + RedrawGenericList(&ListUi); break; } #if 1 @@ -1105,7 +1102,6 @@ InstallIntroPage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - break; } else if (Ir->Event.KeyEvent.uChar.AsciiChar == 0x0D) /* ENTER */ @@ -1142,7 +1138,6 @@ ScsiControllerPage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - break; } else if (Ir->Event.KeyEvent.uChar.AsciiChar == 0x0D) /* ENTER */ @@ -1174,7 +1169,6 @@ OemDriverPage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - break; } else if (Ir->Event.KeyEvent.uChar.AsciiChar == 0x0D) /* ENTER */ @@ -1311,7 +1305,6 @@ DeviceSettingsPage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - break; } else if (Ir->Event.KeyEvent.uChar.AsciiChar == 0x0D) /* ENTER */ @@ -1375,8 +1368,7 @@ HandleGenericList(PGENERIC_LIST_UI ListUi, { if (ConfirmQuit(Ir)) return QUIT_PAGE; - else - RedrawGenericList(ListUi); + RedrawGenericList(ListUi); } else if (Ir->Event.KeyEvent.wVirtualKeyCode == VK_ESCAPE) /* ESC */ { @@ -1687,7 +1679,6 @@ SelectPartitionPage(PINPUT_RECORD Ir) PartitionList = NULL; return QUIT_PAGE; } - break; } else if ((Ir->Event.KeyEvent.uChar.AsciiChar == 0x00) && @@ -1905,12 +1896,10 @@ ShowPartitionSizeInputBox(SHORT Left, *Quit = TRUE; InputBuffer[0] = UNICODE_NULL; - CONSOLE_SetCursorType(TRUE, FALSE); break; } else if (Ir.Event.KeyEvent.wVirtualKeyCode == VK_RETURN) /* ENTER */ { - CONSOLE_SetCursorType(TRUE, FALSE); break; } else if (Ir.Event.KeyEvent.wVirtualKeyCode == VK_ESCAPE) /* ESC */ @@ -1919,7 +1908,6 @@ ShowPartitionSizeInputBox(SHORT Left, *Cancel = TRUE; InputBuffer[0] = UNICODE_NULL; - CONSOLE_SetCursorType(TRUE, FALSE); break; } else if ((Ir.Event.KeyEvent.uChar.AsciiChar == 0x00) && @@ -2015,6 +2003,8 @@ ShowPartitionSizeInputBox(SHORT Left, } } } + + CONSOLE_SetCursorType(TRUE, FALSE); } @@ -2152,7 +2142,6 @@ ConfirmDeleteSystemPartitionPage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - break; } else if (Ir->Event.KeyEvent.wVirtualKeyCode == VK_RETURN) /* ENTER */ @@ -2213,7 +2202,6 @@ DeletePartitionPage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - break; } else if (Ir->Event.KeyEvent.wVirtualKeyCode == VK_ESCAPE) /* ESC */ @@ -2470,7 +2458,6 @@ Restart: FsVolContext->NextPageOnAbort = QUIT_PAGE; return FSVOL_ABORT; } - break; } else if (Ir->Event.KeyEvent.wVirtualKeyCode == VK_ESCAPE) /* ESC */ @@ -2550,7 +2537,6 @@ Restart: FsVolContext->NextPageOnAbort = QUIT_PAGE; return FSVOL_ABORT; } - goto Restart; } else if (Ir->Event.KeyEvent.wVirtualKeyCode == VK_RETURN || IsUnattendedSetup) /* ENTER */ @@ -2724,10 +2710,7 @@ FsVolCallback( FsVolContext->NextPageOnAbort = QUIT_PAGE; return FSVOL_ABORT; } - else - { - return FSVOL_RETRY; - } + return FSVOL_RETRY; } else if (Ir->Event.KeyEvent.uChar.AsciiChar == VK_RETURN) /* ENTER */ { @@ -2777,10 +2760,7 @@ FsVolCallback( FsVolContext->NextPageOnAbort = QUIT_PAGE; return FSVOL_ABORT; } - else - { - return FSVOL_SKIP; - } + return FSVOL_SKIP; } else if (Ir->Event.KeyEvent.uChar.AsciiChar == VK_RETURN) /* ENTER */ { @@ -2941,8 +2921,6 @@ InstallDirectoryPage(PINPUT_RECORD Ir) if (ConfirmQuit(Ir)) return QUIT_PAGE; - - CONSOLE_SetCursorType(TRUE, TRUE); break; } else if ((Ir->Event.KeyEvent.uChar.AsciiChar == 0x00) && @@ -3559,7 +3537,6 @@ BootLoaderSelectPage(PINPUT_RECORD Ir) { if (ConfirmQuit(Ir)) return QUIT_PAGE; - break; } else if (Ir->Event.KeyEvent.uChar.AsciiChar == 0x0D) /* ENTER */

1 month, 2 weeks

[reactos] 01/01: [CALC] Fix copy command when output is NaN (#7496)

by Carlo Bramini

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=30b40247a303365acd83b… commit 30b40247a303365acd83b21cae417344c079f070 Author: Carlo Bramini <carlo_bramini(a)users.sourceforge.net> AuthorDate: Sun Nov 10 17:01:34 2024 +0100 Commit: GitHub <noreply(a)github.com> CommitDate: Sun Nov 10 17:01:34 2024 +0100 [CALC] Fix copy command when output is NaN (#7496) CORE-19745 --- base/applications/calc/winmain.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/base/applications/calc/winmain.c b/base/applications/calc/winmain.c index b2ab16cd0e2..54025b64999 100644 --- a/base/applications/calc/winmain.c +++ b/base/applications/calc/winmain.c @@ -1015,10 +1015,16 @@ static void handle_copy_command(HWND hWnd) TCHAR display[MAX_CALC_SIZE]; UINT n; + // Read current text from output display n = GetDlgItemText(hWnd, IDC_TEXT_OUTPUT, display, SIZEOF(display)); - if (calc.base == IDC_RADIO_DEC && _tcschr(calc.buffer, _T('.')) == NULL) - display[n - calc.sDecimal_len] = _T('\0'); + // Check if result is a true number + if (!calc.is_nan) + { + // Remove trailing decimal point if no decimal digits exist + if (calc.base == IDC_RADIO_DEC && _tcschr(calc.buffer, _T('.')) == NULL) + display[n - calc.sDecimal_len] = _T('\0'); + } CopyMemToClipboard(display); }

1 month, 2 weeks

[reactos] 01/01: [CMAKE][LWIP] Fix build with newer CMake versions

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=0d6316b008c90cb3ce139… commit 0d6316b008c90cb3ce139fe126a518f1bdda93c5 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Mon Nov 4 13:16:08 2024 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Sun Nov 10 11:46:47 2024 +0200 [CMAKE][LWIP] Fix build with newer CMake versions --- drivers/network/tcpip/lwip/src/Filelists.cmake | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/network/tcpip/lwip/src/Filelists.cmake b/drivers/network/tcpip/lwip/src/Filelists.cmake index 3a0430e457d..21bbddfb04c 100644 --- a/drivers/network/tcpip/lwip/src/Filelists.cmake +++ b/drivers/network/tcpip/lwip/src/Filelists.cmake @@ -262,19 +262,19 @@ set(DOXYGEN_IN ${LWIP_DIR}/doc/doxygen/lwip.Doxyfile.cmake.in) set(DOXYGEN_OUT ${LWIP_DIR}/doc/doxygen/lwip.Doxyfile) configure_file(${DOXYGEN_IN} ${DOXYGEN_OUT}) -find_package(Doxygen) -if (DOXYGEN_FOUND) - message(STATUS "Doxygen build started") - - add_custom_target(lwipdocs - COMMAND ${CMAKE_COMMAND} -E remove_directory ${DOXYGEN_DIR}/${DOXYGEN_OUTPUT_DIR}/html - COMMAND ${DOXYGEN_EXECUTABLE} ${DOXYGEN_OUT} - WORKING_DIRECTORY ${DOXYGEN_DIR} - COMMENT "Generating API documentation with Doxygen" - VERBATIM) -else (DOXYGEN_FOUND) - message(STATUS "Doxygen needs to be installed to generate the doxygen documentation") -endif (DOXYGEN_FOUND) +#find_package(Doxygen) +#if (DOXYGEN_FOUND) +# message(STATUS "Doxygen build started") +# +# add_custom_target(lwipdocs +# COMMAND ${CMAKE_COMMAND} -E remove_directory ${DOXYGEN_DIR}/${DOXYGEN_OUTPUT_DIR}/html +# COMMAND ${DOXYGEN_EXECUTABLE} ${DOXYGEN_OUT} +# WORKING_DIRECTORY ${DOXYGEN_DIR} +# COMMENT "Generating API documentation with Doxygen" +# VERBATIM) +#else (DOXYGEN_FOUND) +# message(STATUS "Doxygen needs to be installed to generate the doxygen documentation") +#endif (DOXYGEN_FOUND) # lwIP libraries add_library(lwipcore EXCLUDE_FROM_ALL ${lwipnoapps_SRCS})

1 month, 2 weeks

← Newer
1
2
3
4
5
6
7
8
...
11
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs November 2024