Ros-diffs

ros-diffs@reactos.org

81390 discussions

[reactos] 01/01: [NTUSER][USER32] Refactor NtUserLoadKeyboardLayoutEx (#5107)
by Katayama Hirofumi MZ 05 Mar '23

05 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=77e6348f5fce69070852d… commit 77e6348f5fce69070852df0e516f649012930d30 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Sun Mar 5 11:41:32 2023 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Sun Mar 5 11:41:32 2023 +0900 [NTUSER][USER32] Refactor NtUserLoadKeyboardLayoutEx (#5107) - Split some code of NtUserLoadKeyboardLayoutEx to newly-added co_IntLoadKeyboardLayoutEx helper function. - Modify NtUserLoadKeyboardLayoutEx prototype. - Move co_UserImmLoadLayout code. - Implement KLF_REORDER. - Rename UserLoadKbdLayout as co_UserLoadKbdLayout. - Improve LoadKeyboardLayoutEx. CORE-11700 --- win32ss/include/ntuser.h | 8 +- win32ss/user/ntuser/kbdlayout.c | 257 ++++++++++++++++++++++-------------- win32ss/user/user32/windows/input.c | 10 +- 3 files changed, 165 insertions(+), 110 deletions(-) diff --git a/win32ss/include/ntuser.h b/win32ss/include/ntuser.h index cd7fcdf7034..656d3b443fa 100644 --- a/win32ss/include/ntuser.h +++ b/win32ss/include/ntuser.h @@ -2736,12 +2736,12 @@ NtUserKillTimer( HKL NTAPI NtUserLoadKeyboardLayoutEx( - IN HANDLE Handle, + IN HANDLE hFile, IN DWORD offTable, - IN PUNICODE_STRING puszKeyboardName, - IN HKL hKL, + IN PVOID pTables, + IN HKL hOldKL, IN PUNICODE_STRING puszKLID, - IN DWORD dwKLID, + IN DWORD dwNewKL, IN UINT Flags); BOOL diff --git a/win32ss/user/ntuser/kbdlayout.c b/win32ss/user/ntuser/kbdlayout.c index 6ce1523d208..9a5c43d60a4 100644 --- a/win32ss/user/ntuser/kbdlayout.c +++ b/win32ss/user/ntuser/kbdlayout.c @@ -369,12 +369,12 @@ cleanup: } /* - * UserLoadKbdLayout + * co_UserLoadKbdLayout * * Loads keyboard layout and creates KL object */ static PKL -UserLoadKbdLayout(PUNICODE_STRING pustrKLID, HKL hKL) +co_UserLoadKbdLayout(PUNICODE_STRING pustrKLID, HKL hKL) { LCID lCid; CHARSETINFO cs; @@ -846,6 +846,129 @@ IntUnloadKeyboardLayout(_Inout_ PWINSTATION_OBJECT pWinSta, _In_ HKL hKL) return co_IntUnloadKeyboardLayoutEx(pWinSta, pKL, 0); } +PIMEINFOEX FASTCALL co_UserImmLoadLayout(_In_ HKL hKL) +{ + PIMEINFOEX piiex; + + if (!IS_IME_HKL(hKL) && !IS_CICERO_MODE()) + return NULL; + + piiex = ExAllocatePoolWithTag(PagedPool, sizeof(IMEINFOEX), USERTAG_IME); + if (!piiex) + return NULL; + + if (!co_ClientImmLoadLayout(hKL, piiex)) + { + ExFreePoolWithTag(piiex, USERTAG_IME); + return NULL; + } + + return piiex; +} + +HKL APIENTRY +co_IntLoadKeyboardLayoutEx( + IN OUT PWINSTATION_OBJECT pWinSta, + IN HANDLE hSafeFile, + IN HKL hOldKL, + IN PUNICODE_STRING puszSafeKLID, + IN HKL hNewKL, + IN UINT Flags) +{ + PKL pOldKL, pNewKL; + + UNREFERENCED_PARAMETER(hSafeFile); + + if (hNewKL == NULL || (pWinSta->Flags & WSS_NOIO)) + return NULL; + + /* If hOldKL is specified, unload it and load new layput as default */ + if (hOldKL && hOldKL != hNewKL) + { + pOldKL = UserHklToKbl(hOldKL); + if (pOldKL) + UserUnloadKbl(pOldKL); + } + + /* FIXME: It seems KLF_RESET is only supported for WINLOGON */ + + /* Let's see if layout was already loaded. */ + pNewKL = UserHklToKbl(hNewKL); + if (!pNewKL) + { + /* It wasn't, so load it. */ + pNewKL = co_UserLoadKbdLayout(puszSafeKLID, hNewKL); + if (!pNewKL) + return NULL; + + if (gspklBaseLayout) + { + /* Find last not unloaded layout */ + PKL pLastKL = gspklBaseLayout->pklPrev; + while (pLastKL != gspklBaseLayout && (pLastKL->dwKL_Flags & KLF_UNLOAD)) + pLastKL = pLastKL->pklPrev; + + /* Add new layout to the list */ + pNewKL->pklNext = pLastKL->pklNext; + pNewKL->pklPrev = pLastKL; + pNewKL->pklNext->pklPrev = pNewKL; + pNewKL->pklPrev->pklNext = pNewKL; + } + else + { + /* This is the first layout */ + pNewKL->pklNext = pNewKL; + pNewKL->pklPrev = pNewKL; + gspklBaseLayout = pNewKL; + } + + pNewKL->piiex = co_UserImmLoadLayout(hNewKL); + } + + /* If this layout was prepared to unload, undo it */ + pNewKL->dwKL_Flags &= ~KLF_UNLOAD; + + /* Reorder if necessary */ + if (Flags & KLF_REORDER) + IntReorderKeyboardLayouts(pWinSta, pNewKL); + + /* Activate this layout in current thread */ + if (Flags & KLF_ACTIVATE) + co_UserActivateKbl(PsGetCurrentThreadWin32Thread(), pNewKL, Flags); + + /* Send shell message */ + if (!(Flags & KLF_NOTELLSHELL)) + co_IntShellHookNotify(HSHELL_LANGUAGE, 0, (LPARAM)hNewKL); + + /* FIXME: KLF_REPLACELANG */ + + return hNewKL; +} + +HANDLE FASTCALL IntVerifyKeyboardFileHandle(HANDLE hFile) +{ + PFILE_OBJECT FileObject; + NTSTATUS Status; + + if (hFile == INVALID_HANDLE_VALUE) + return NULL; + + Status = ObReferenceObjectByHandle(hFile, FILE_READ_DATA, NULL, UserMode, + (PVOID*)&FileObject, NULL); + if (!NT_SUCCESS(Status)) + { + ERR("0x%08X\n", Status); + return NULL; + } + + /* FIXME: Is the file in the system directory? */ + + if (FileObject) + ObDereferenceObject(FileObject); + + return hFile; +} + /* EXPORTS *******************************************************************/ /* @@ -1009,50 +1132,34 @@ cleanup: return bRet; } -/* Win: xxxImmLoadLayout */ -PIMEINFOEX FASTCALL co_UserImmLoadLayout(_In_ HKL hKL) -{ - PIMEINFOEX piiex; - - if (!IS_IME_HKL(hKL) && !IS_CICERO_MODE()) - return NULL; - - piiex = ExAllocatePoolWithTag(PagedPool, sizeof(IMEINFOEX), USERTAG_IME); - if (!piiex) - return NULL; - - if (!co_ClientImmLoadLayout(hKL, piiex)) - { - ExFreePoolWithTag(piiex, USERTAG_IME); - return NULL; - } - - return piiex; -} - /* * NtUserLoadKeyboardLayoutEx * * Loads keyboard layout with given locale id * - * NOTE: We adopt a different design from Microsoft's one for security reason. - * We don't use the 1st and 3rd parameters of NtUserLoadKeyboardLayoutEx. + * NOTE: We adopt a different design from Microsoft's one due to security reason. + * We don't use the 3rd parameter of NtUserLoadKeyboardLayoutEx. + * See https://bugtraq.securityfocus.com/detail/50056B96.6040306 */ HKL -APIENTRY +NTAPI NtUserLoadKeyboardLayoutEx( - IN HANDLE Handle, // hFile (See downloads.securityfocus.com/vulnerabilities/exploits/43774.c) - IN DWORD offTable, // Offset to KbdTables - IN PUNICODE_STRING puszKeyboardName, // Not used? - IN HKL hklUnload, - IN PUNICODE_STRING pustrKLID, - IN DWORD hkl, + IN HANDLE hFile, + IN DWORD offTable, + IN PVOID pTables, + IN HKL hOldKL, + IN PUNICODE_STRING puszKLID, + IN DWORD dwNewKL, IN UINT Flags) { - HKL hklRet = NULL; - PKL pKl = NULL, pklLast; + HKL hRetKL; WCHAR Buffer[KL_NAMELENGTH]; - UNICODE_STRING ustrSafeKLID; + UNICODE_STRING uszSafeKLID; + PWINSTATION_OBJECT pWinSta; + HANDLE hSafeFile; + + UNREFERENCED_PARAMETER(offTable); + UNREFERENCED_PARAMETER(pTables); if (Flags & ~(KLF_ACTIVATE|KLF_NOTELLSHELL|KLF_REORDER|KLF_REPLACELANG| KLF_SUBSTITUTE_OK|KLF_SETFORPROCESS|KLF_UNLOADPREVIOUS| @@ -1063,14 +1170,12 @@ NtUserLoadKeyboardLayoutEx( return NULL; } - /* FIXME: It seems KLF_RESET is only supported for WINLOGON */ - - RtlInitEmptyUnicodeString(&ustrSafeKLID, Buffer, sizeof(Buffer)); + RtlInitEmptyUnicodeString(&uszSafeKLID, Buffer, sizeof(Buffer)); _SEH2_TRY { - ProbeForRead(pustrKLID, sizeof(*pustrKLID), 1); - ProbeForRead(pustrKLID->Buffer, sizeof(pustrKLID->Length), 1); - RtlCopyUnicodeString(&ustrSafeKLID, pustrKLID); + ProbeForRead(puszKLID, sizeof(*puszKLID), 1); + ProbeForRead(puszKLID->Buffer, sizeof(puszKLID->Length), 1); + RtlCopyUnicodeString(&uszSafeKLID, puszKLID); } _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) { @@ -1081,67 +1186,19 @@ NtUserLoadKeyboardLayoutEx( UserEnterExclusive(); - /* If hklUnload is specified, unload it and load new layput as default */ - if (hklUnload && (hklUnload != UlongToHandle(hkl))) - { - pKl = UserHklToKbl(hklUnload); - if (pKl) - UserUnloadKbl(pKl); - } - - /* Let's see if layout was already loaded. */ - pKl = UserHklToKbl(UlongToHandle(hkl)); - if (!pKl) - { - /* It wasn't, so load it. */ - pKl = UserLoadKbdLayout(&ustrSafeKLID, UlongToHandle(hkl)); - if (!pKl) - goto cleanup; - - if (gspklBaseLayout) - { - /* Find last not unloaded layout */ - pklLast = gspklBaseLayout->pklPrev; - while (pklLast != gspklBaseLayout && pklLast->dwKL_Flags & KLF_UNLOAD) - pklLast = pklLast->pklPrev; - - /* Add new layout to the list */ - pKl->pklNext = pklLast->pklNext; - pKl->pklPrev = pklLast; - pKl->pklNext->pklPrev = pKl; - pKl->pklPrev->pklNext = pKl; - } - else - { - /* This is the first layout */ - pKl->pklNext = pKl; - pKl->pklPrev = pKl; - gspklBaseLayout = pKl; - } - - pKl->piiex = co_UserImmLoadLayout(UlongToHandle(hkl)); - } - - /* If this layout was prepared to unload, undo it */ - pKl->dwKL_Flags &= ~KLF_UNLOAD; - - /* Activate this layout in current thread */ - if (Flags & KLF_ACTIVATE) - co_UserActivateKbl(PsGetCurrentThreadWin32Thread(), pKl, Flags); - - /* Send shell message */ - if (!(Flags & KLF_NOTELLSHELL)) - co_IntShellHookNotify(HSHELL_LANGUAGE, 0, (LPARAM)hkl); - - /* Return hkl on success */ - hklRet = UlongToHandle(hkl); - - /* FIXME: KLF_REPLACELANG - KLF_REORDER */ + hSafeFile = (hFile ? IntVerifyKeyboardFileHandle(hFile) : NULL); + pWinSta = IntGetProcessWindowStation(NULL); + hRetKL = co_IntLoadKeyboardLayoutEx(pWinSta, + hSafeFile, + hOldKL, + &uszSafeKLID, + (HKL)(DWORD_PTR)dwNewKL, + Flags); + if (hSafeFile) + ZwClose(hSafeFile); -cleanup: UserLeave(); - return hklRet; + return hRetKL; } /* diff --git a/win32ss/user/user32/windows/input.c b/win32ss/user/user32/windows/input.c index d3b7e114ba9..48b18b069da 100644 --- a/win32ss/user/user32/windows/input.c +++ b/win32ss/user/user32/windows/input.c @@ -721,9 +721,9 @@ VOID GetSystemLibraryPath(LPWSTR pszPath, INT cchPath, LPCWSTR pszFileName) /* * @unimplemented * - * NOTE: We adopt a different design from Microsoft's one for security reason. + * NOTE: We adopt a different design from Microsoft's one due to security reason. + * See NtUserLoadKeyboardLayoutEx. */ -/* Win: LoadKeyboardLayoutWorker */ HKL APIENTRY IntLoadKeyboardLayout( _In_ HKL hklUnload, @@ -733,7 +733,6 @@ IntLoadKeyboardLayout( _In_ BOOL unknown5) { DWORD dwKLID, dwHKL, dwType, dwSize; - UNICODE_STRING ustrKbdName; UNICODE_STRING ustrKLID; WCHAR wszRegKey[256] = L"SYSTEM\\CurrentControlSet\\Control\\Keyboard Layouts\\"; WCHAR wszLayoutId[10], wszNewKLID[KL_NAMELENGTH], szImeFileName[80]; @@ -807,7 +806,7 @@ IntLoadKeyboardLayout( szImeFileName[_countof(szImeFileName) - 1] = UNICODE_NULL; GetSystemLibraryPath(szPath, _countof(szPath), szImeFileName); - /* We don't allow the invalid "IME File" values for security reason */ + /* We don't allow the invalid "IME File" values due to security reason */ if (dwType != REG_SZ || szImeFileName[0] == 0 || wcscspn(szImeFileName, L":\\/") != wcslen(szImeFileName) || GetFileAttributesW(szPath) == INVALID_FILE_ATTRIBUTES) /* Does not exist? */ @@ -833,9 +832,8 @@ IntLoadKeyboardLayout( dwHKL = MAKELONG(wLow, wHigh); - ZeroMemory(&ustrKbdName, sizeof(ustrKbdName)); RtlInitUnicodeString(&ustrKLID, pwszKLID); - hNewKL = NtUserLoadKeyboardLayoutEx(NULL, 0, &ustrKbdName, NULL, &ustrKLID, dwHKL, Flags); + hNewKL = NtUserLoadKeyboardLayoutEx(NULL, 0, NULL, hklUnload, &ustrKLID, dwHKL, Flags); CliImmInitializeHotKeys(SETIMEHOTKEY_ADD, hNewKL); return hNewKL; }

1 0

[reactos] 01/01: [SDK] Improve gen_baseaddress.py and its output files (#4964)
by Joachim Henze 04 Mar '23

04 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=0fb580a8553e1028fb942… commit 0fb580a8553e1028fb942ed8c061f9cd88d319e9 Author: Joachim Henze <joachim.henze(a)reactos.org> AuthorDate: Sat Mar 4 11:45:06 2023 +0100 Commit: GitHub <noreply(a)github.com> CommitDate: Sat Mar 4 11:45:06 2023 +0100 [SDK] Improve gen_baseaddress.py and its output files (#4964) * [SDK:TOOLS] Improve gen_baseaddress.py Adds some kbd layout modules into the excludes-section: that we do lack for todays master: e.g._ kbdeo, kbdsf, kbdrost and also some that we lacked in the past (for improving backwards compatibility of the script to older rls-branches): kbdgm, kbdes, kbdgrist, kbdja, kbdko, kbdsk, kbdsk1 * [SDK:CMAKE] Highlight in the output files where the script does misbehave --- sdk/cmake/baseaddress.cmake | 8 ++++---- sdk/cmake/baseaddress_dwarf.cmake | 6 +++--- sdk/cmake/baseaddress_msvc.cmake | 6 +++--- sdk/cmake/baseaddress_msvc_x64.cmake | 4 ++-- sdk/tools/gen_baseaddress.py | 10 ++++++++++ 5 files changed, 22 insertions(+), 12 deletions(-) diff --git a/sdk/cmake/baseaddress.cmake b/sdk/cmake/baseaddress.cmake index dbaad8bb9ba..02ad613b9b9 100644 --- a/sdk/cmake/baseaddress.cmake +++ b/sdk/cmake/baseaddress.cmake @@ -181,9 +181,9 @@ set(baseaddress_itss 0x745e0000) set(baseaddress_joy 0x745a0000) set(baseaddress_jscript 0x74400000) set(baseaddress_jsproxy 0x743d0000) -set(baseaddress_kbdeo 0x743b0000) -set(baseaddress_kbdrost 0x74390000) -set(baseaddress_kbdsf 0x74370000) +#set(baseaddress_kbdeo 0x743b0000) #unwanted, should vanish when rerunning the script +#set(baseaddress_kbdrost 0x74390000) #unwanted, should vanish when rerunning the script +#set(baseaddress_kbdsf 0x74370000) #unwanted, should vanish when rerunning the script set(baseaddress_kbsdll 0x74340000) set(baseaddress_kernel32_vista 0x74320000) set(baseaddress_ksuser 0x742f0000) @@ -196,7 +196,7 @@ set(baseaddress_load_notifications 0x73cd0000) set(baseaddress_loadperf 0x73ca0000) set(baseaddress_localmon 0x73c40000) set(baseaddress_localspl 0x73b50000) -set(baseaddress_localspl_apitest 0x73b20000) +set(baseaddress_localspl_apitest.dll 0x73b20000) #module name corrected manually set(baseaddress_localui 0x73ad0000) set(baseaddress_lpk 0x73aa0000) set(baseaddress_lsasrv 0x73a20000) diff --git a/sdk/cmake/baseaddress_dwarf.cmake b/sdk/cmake/baseaddress_dwarf.cmake index 00b8d491251..3ddc970ad77 100644 --- a/sdk/cmake/baseaddress_dwarf.cmake +++ b/sdk/cmake/baseaddress_dwarf.cmake @@ -177,8 +177,8 @@ set(baseaddress_itss 0x738e0000) set(baseaddress_joy 0x73890000) set(baseaddress_jscript 0x735b0000) set(baseaddress_jsproxy 0x73560000) -set(baseaddress_kbdeo 0x73540000) -set(baseaddress_kbdrost 0x73520000) +#set(baseaddress_kbdeo 0x73540000) #unwanted, should vanish when rerunning the script +#set(baseaddress_kbdrost 0x73520000) #unwanted, should vanish when rerunning the script set(baseaddress_kbsdll 0x734f0000) set(baseaddress_kernel32_vista 0x734c0000) set(baseaddress_ksuser 0x73490000) @@ -191,7 +191,7 @@ set(baseaddress_load_notifications 0x72b70000) set(baseaddress_loadperf 0x72b30000) set(baseaddress_localmon 0x72ae0000) set(baseaddress_localspl 0x72a60000) -set(baseaddress_localspl_apitest 0x72a20000) +set(baseaddress_localspl_apitest.dll 0x72a20000) #module name corrected manually set(baseaddress_localui 0x729e0000) set(baseaddress_lpk 0x72990000) set(baseaddress_lsasrv 0x728d0000) diff --git a/sdk/cmake/baseaddress_msvc.cmake b/sdk/cmake/baseaddress_msvc.cmake index a4dee3b4993..13860cfef0c 100644 --- a/sdk/cmake/baseaddress_msvc.cmake +++ b/sdk/cmake/baseaddress_msvc.cmake @@ -177,8 +177,8 @@ set(baseaddress_itss 0x78080000) set(baseaddress_joy 0x78040000) set(baseaddress_jscript 0x77f90000) set(baseaddress_jsproxy 0x77f70000) -set(baseaddress_kbdeo 0x77f50000) -set(baseaddress_kbdrost 0x77f30000) +#set(baseaddress_kbdeo 0x77f50000) #unwanted, should vanish when rerunning the script +#set(baseaddress_kbdrost 0x77f30000) #unwanted, should vanish when rerunning the script set(baseaddress_kbsdll 0x77f10000) set(baseaddress_kernel32_vista 0x77ef0000) set(baseaddress_ksuser 0x77ed0000) @@ -191,7 +191,7 @@ set(baseaddress_load_notifications 0x77bf0000) set(baseaddress_loadperf 0x77bd0000) set(baseaddress_localmon 0x77bb0000) set(baseaddress_localspl 0x77b80000) -set(baseaddress_localspl_apitest 0x77b60000) +set(baseaddress_localspl_apitest.dll 0x77b60000) #module name corrected manually set(baseaddress_localui 0x77b30000) set(baseaddress_lpk 0x779f0000) set(baseaddress_lsasrv 0x77990000) diff --git a/sdk/cmake/baseaddress_msvc_x64.cmake b/sdk/cmake/baseaddress_msvc_x64.cmake index a9a251aef1d..3f69f4e018f 100644 --- a/sdk/cmake/baseaddress_msvc_x64.cmake +++ b/sdk/cmake/baseaddress_msvc_x64.cmake @@ -174,7 +174,7 @@ set(baseaddress_itss 0x7FF19500000) set(baseaddress_joy 0x7FF19000000) set(baseaddress_jscript 0x7FF18500000) # 000A4000 set(baseaddress_jsproxy 0x7FF17500000) -set(baseaddress_kbdeo 0x7FF17000000) +#set(baseaddress_kbdeo 0x7FF17000000) #unwanted, should vanish when rerunning the script set(baseaddress_kbsdll 0x7FF16500000) set(baseaddress_kernel32_vista 0x7FF15500000) set(baseaddress_ksuser 0x7FF14500000) @@ -187,7 +187,7 @@ set(baseaddress_load_notifications 0x7FF10000000) set(baseaddress_loadperf 0x7FF0F500000) set(baseaddress_localmon 0x7FF0F000000) set(baseaddress_localspl 0x7FF0E500000) -set(baseaddress_localspl_apitest 0x7FF0E000000) +set(baseaddress_localspl_apitest.dll 0x7FF0E000000) #module name corrected manually set(baseaddress_localui 0x7FF0D500000) set(baseaddress_lpk 0x7FF0C500000) set(baseaddress_lsasrv 0x7FF0B500000) diff --git a/sdk/tools/gen_baseaddress.py b/sdk/tools/gen_baseaddress.py index 51e89d0870d..e7a691424da 100644 --- a/sdk/tools/gen_baseaddress.py +++ b/sdk/tools/gen_baseaddress.py @@ -120,6 +120,7 @@ EXCLUDE = ( 'kbdazel.dll', 'kbdbe.dll', 'kbdbga.dll', + 'kbdbgm.dll', 'kbdbgt.dll', 'kbdblr.dll', 'kbdbr.dll', @@ -131,6 +132,8 @@ EXCLUDE = ( 'kbdcz1.dll', 'kbdda.dll', 'kbddv.dll', + 'kbdeo.dll', + 'kbdes.dll', 'kbdest.dll', 'kbdfc.dll', 'kbdfi.dll', @@ -140,6 +143,7 @@ EXCLUDE = ( 'kbdgneo.dll', 'kbdgr.dll', 'kbdgr1.dll', + 'kbdgrist.dll', 'kbdhe.dll', 'kbdheb.dll', 'kbdhu.dll', @@ -151,8 +155,10 @@ EXCLUDE = ( 'kbdinmal.dll', 'kbdir.dll', 'kbdit.dll', + 'kbdja.dll', 'kbdjpn.dll', 'kbdkaz.dll', + 'kbdko.dll', 'kbdkor.dll', 'kbdla.dll', 'kbdlt1.dll', @@ -164,9 +170,13 @@ EXCLUDE = ( 'kbdpl1.dll', 'kbdpo.dll', 'kbdro.dll', + 'kbdrost.dll', 'kbdru.dll', 'kbdru1.dll', + 'kbdsf.dll', 'kbdsg.dll', + 'kbdsk.dll', + 'kbdsk1.dll', 'kbdsl.dll', 'kbdsl1.dll', 'kbdsp.dll',

1 0

[reactos] 01/01: [CMDUTILS] Add Italian (it-IT) translation for WHERE command
by Carlo Bramini 01 Mar '23

01 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=6d57c6c91cdf2854b73b9… commit 6d57c6c91cdf2854b73b9696cd45f41322449368 Author: Carlo Bramini <30959007+carlo-bramini(a)users.noreply.github.com> AuthorDate: Thu Feb 23 10:57:28 2023 +0100 Commit: George Bișoc <george.bisoc(a)reactos.org> CommitDate: Wed Mar 1 22:05:04 2023 +0100 [CMDUTILS] Add Italian (it-IT) translation for WHERE command --- base/applications/cmdutils/where/lang/it-IT.rc | 61 ++++++++++++++++++++++++++ base/applications/cmdutils/where/where.rc | 3 ++ 2 files changed, 64 insertions(+) diff --git a/base/applications/cmdutils/where/lang/it-IT.rc b/base/applications/cmdutils/where/lang/it-IT.rc new file mode 100644 index 00000000000..d98e722b157 --- /dev/null +++ b/base/applications/cmdutils/where/lang/it-IT.rc @@ -0,0 +1,61 @@ +/* + * PROJECT: ReactOS WHERE command + * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) + * PURPOSE: Italian resource file + * TRANSLATOR: Copyright 2023 Carlo Bramini + */ + +LANGUAGE LANG_ITALIAN, SUBLANG_NEUTRAL + +STRINGTABLE +BEGIN + IDS_USAGE "Sintassi: WHERE [opzioni] criteri...\n\ +\n\ +Descrizione:\n\ + Visualizza il percorso dei file corrispondenti ai criteri di ricerca.\n\ + Per impostazione predefinita, questo strumento esegue la ricerca\n\ + utilizzando i criteri e i percorsi della variabile d'ambiente PATH.\n\ +\n\ +Opzioni:\n\ + /F Visualizza tutti i file corrispondenti tra virgolette.\n\ + /Q Effettua la ricerca senza visualizzare l'elenco dei file e\n\ + i messaggi.\n\ + /R dir Avvia la ricerca dalla directory specificata ed esegue la ricerca\n\ + in modo ricorsivo.\n\ + /T Visualizza la dimensione dei file e la data dell'ultima modifica\n\ + di tutti di file corrispondenti ai criteri di ricerca.\n\ + criteri Specifica i criteri di ricerca per i file. I caratteri jolly * e ?\n\ + possono essere usati per la ricerca.\n\ + Anche i formati ""$env:pattern"" e ""path:pattern"" possono essere\n\ + utilizzati, dove ""env"" rappresenta una variabile d'ambiente\n\ + che contiene i percorsi di ricerca. Non usare questi formati con\n\ + l'opzione /R. La ricerca viene eseguita anche aggiungendo\n\ + l'estensione della variabile PATHEXT allo schema.\n\ + /? Visualizza questo messaggio.\n\ +\n\ +NOTA: Questo strumento ritorna un livello di errore 0 se la ricerca viene\n\ + completata con dei file trovati, 1 se nessun file corrisponde ai criteri\n\ + oppure 2 se si verifica un errore.\n\ +\n\ +Esempi:\n\ + WHERE myfile*.exe\n\ + WHERE /F /T mspaint\n\ + WHERE $WINDIR:notepad myfile???\n\ + WHERE C:\\ReactOS;C:\\ReactOS\\system32:exp*.exe\n\ + WHERE /R ""C:\\Program Files"" *.dll\n" + + IDS_BAD_ARG "ERRORE: argomento non valido - '%ls'.\n" + IDS_NOT_FOUND "INFO: Impossibile trovare i file per i criteri specificati.\n" + IDS_FILE_INFO "%10I64u %-12ls %-12ls %ls\n" + IDS_WANT_VALUE "ERRORE: Valore necessario per '%ls'.\n" + IDS_TYPE_HELP "Digitare ""WHERE /?"" per la guida di utilizzo.\n" + IDS_ENVPAT_WITH_R "ERRORE: ""$env:pattern"" non utilizzabile con /R.\n" + IDS_PATHPAT_WITH_R "ERRORE: formato ""path:pattern"" non utilizzabile con /R.\n" + IDS_BAD_PATHPAT "ERRORE: criteri errati specificati in ""path:pattern"".\n" + IDS_OUTOFMEMORY "ERRORE: Memoria esaurita.\n" + IDS_BAD_ENVVAR "ERRORE: variabile d'ambiente ""%ls"" non trovata.\n" + IDS_CANT_FOUND "ERRORE: il sistema non ha trovato il file specificato.\n" + IDS_BAD_DIR "ERRORE: percorso specificato non valido.\n" + IDS_BAD_NAME "ERRORE: il nome del file, il nome della directory o la sintassi del nome del volume sono errati.\n" + IDS_TOO_MANY "ERRORE: opzione '%ls' non consentita oltre '%u' volte.\n" +END diff --git a/base/applications/cmdutils/where/where.rc b/base/applications/cmdutils/where/where.rc index b3593949726..115d90a45b6 100644 --- a/base/applications/cmdutils/where/where.rc +++ b/base/applications/cmdutils/where/where.rc @@ -14,6 +14,9 @@ #ifdef LANGUAGE_EN_US #include "lang/en-US.rc" #endif +#ifdef LANGUAGE_IT_IT + #include "lang/it-IT.rc" +#endif #ifdef LANGUAGE_TR_TR #include "lang/tr-TR.rc" #endif

1 0

[reactos] 01/01: [NOTEPAD] Don't reset Find/Replace if lpstrFindWhat is set (#5104)
by Katayama Hirofumi MZ 01 Mar '23

01 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=f8c7bd6898ec15bb8420b… commit f8c7bd6898ec15bb8420b7c7a2683759b67ddcd7 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Wed Mar 1 20:17:27 2023 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Wed Mar 1 20:17:27 2023 +0900 [NOTEPAD] Don't reset Find/Replace if lpstrFindWhat is set (#5104) We don't want to reset the status of the checkboxes even if the Find/Replace dialog was closed. If lpstrFindWhat is set, do not reset the Find/Replace settings at DIALOG_SearchDialog. CORE-18837 --- base/applications/notepad/dialog.c | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/base/applications/notepad/dialog.c b/base/applications/notepad/dialog.c index 72bb17a88dd..0d4272b1bb5 100644 --- a/base/applications/notepad/dialog.c +++ b/base/applications/notepad/dialog.c @@ -1048,15 +1048,17 @@ static VOID DIALOG_SearchDialog(FINDPROC pfnProc) return; } - ZeroMemory(&Globals.find, sizeof(Globals.find)); - Globals.find.lStructSize = sizeof(Globals.find); - Globals.find.hwndOwner = Globals.hMainWnd; - Globals.find.hInstance = Globals.hInstance; - Globals.find.lpstrFindWhat = Globals.szFindText; - Globals.find.wFindWhatLen = ARRAY_SIZE(Globals.szFindText); - Globals.find.lpstrReplaceWith = Globals.szReplaceText; - Globals.find.wReplaceWithLen = ARRAY_SIZE(Globals.szReplaceText); - Globals.find.Flags = FR_DOWN; + if (!Globals.find.lpstrFindWhat) + { + ZeroMemory(&Globals.find, sizeof(Globals.find)); + Globals.find.lStructSize = sizeof(Globals.find); + Globals.find.hwndOwner = Globals.hMainWnd; + Globals.find.lpstrFindWhat = Globals.szFindText; + Globals.find.wFindWhatLen = ARRAY_SIZE(Globals.szFindText); + Globals.find.lpstrReplaceWith = Globals.szReplaceText; + Globals.find.wReplaceWithLen = ARRAY_SIZE(Globals.szReplaceText); + Globals.find.Flags = FR_DOWN; + } /* We only need to create the modal FindReplace dialog which will */ /* notify us of incoming events using hMainWnd Window Messages */

1 0

[reactos] 01/01: [NOTEPAD] Fix NOTEPAD_FindTextAt (#5103)
by Katayama Hirofumi MZ 01 Mar '23

01 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=8c7233e01587afde3d7b5… commit 8c7233e01587afde3d7b53961857dc8a1454a411 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Wed Mar 1 19:57:10 2023 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Wed Mar 1 19:57:10 2023 +0900 [NOTEPAD] Fix NOTEPAD_FindTextAt (#5103) The whole-word search of Notepad had a bug around punctuation. For example, the text "Windows" didn't match in the text "MS-DOS,Windows,ReactOS". Use _istalnum instead of _istspace. Fix the position to check. _istalnum matches an alphabet letter or numeric digit. CORE-18837 --- base/applications/notepad/main.c | 28 +++++++++++++++++----------- 1 file changed, 17 insertions(+), 11 deletions(-) diff --git a/base/applications/notepad/main.c b/base/applications/notepad/main.c index 52370736318..f4312159731 100644 --- a/base/applications/notepad/main.c +++ b/base/applications/notepad/main.c @@ -109,30 +109,36 @@ static int NOTEPAD_MenuCommand(WPARAM wParam) */ static BOOL -NOTEPAD_FindTextAt(FINDREPLACE *pFindReplace, LPCTSTR pszText, int iTextLength, DWORD dwPosition) +NOTEPAD_FindTextAt(FINDREPLACE *pFindReplace, LPCTSTR pszText, INT iTextLength, DWORD dwPosition) { BOOL bMatches; size_t iTargetLength; + LPCTSTR pchPosition; - if ((!pFindReplace) || (!pszText)) - { + if (!pFindReplace || !pszText) return FALSE; - } iTargetLength = _tcslen(pFindReplace->lpstrFindWhat); + pchPosition = &pszText[dwPosition]; /* Make proper comparison */ if (pFindReplace->Flags & FR_MATCHCASE) - bMatches = !_tcsncmp(&pszText[dwPosition], pFindReplace->lpstrFindWhat, iTargetLength); + bMatches = !_tcsncmp(pchPosition, pFindReplace->lpstrFindWhat, iTargetLength); else - bMatches = !_tcsnicmp(&pszText[dwPosition], pFindReplace->lpstrFindWhat, iTargetLength); + bMatches = !_tcsnicmp(pchPosition, pFindReplace->lpstrFindWhat, iTargetLength); - if (bMatches && pFindReplace->Flags & FR_WHOLEWORD) + if (bMatches && (pFindReplace->Flags & FR_WHOLEWORD)) { - if ((dwPosition > 0) && !_istspace(pszText[dwPosition-1])) - bMatches = FALSE; - if ((dwPosition < (DWORD) iTextLength - 1) && !_istspace(pszText[dwPosition+1])) - bMatches = FALSE; + if (dwPosition > 0) + { + if (_istalnum(*(pchPosition - 1)) || *(pchPosition - 1) == _T('_')) + bMatches = FALSE; + } + if ((INT)dwPosition + iTargetLength < iTextLength) + { + if (_istalnum(pchPosition[iTargetLength]) || pchPosition[iTargetLength] == _T('_')) + bMatches = FALSE; + } } return bMatches;

1 0

[reactos] 01/01: [NOTEPAD] Shift+F3 for backward search (#5101)
by Katayama Hirofumi MZ 28 Feb '23

28 Feb '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=55e3feba539fa06753365… commit 55e3feba539fa067533650aaf57d40752e54b805 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Wed Mar 1 07:16:28 2023 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Wed Mar 1 07:16:28 2023 +0900 [NOTEPAD] Shift+F3 for backward search (#5101) - Add Shift+F3 accelerator as command CMD_SEARCH_PREV. - Extend DIALOG_SearchNext function with bDown argument for the action of CMD_SEARCH_PREV. - Modify the message loop for effective F3 key on Find/Replace dialog. CORE-17064 --- base/applications/notepad/dialog.c | 7 ++++++- base/applications/notepad/dialog.h | 2 +- base/applications/notepad/lang/bg-BG.rc | 1 + base/applications/notepad/lang/cs-CZ.rc | 1 + base/applications/notepad/lang/da-DK.rc | 1 + base/applications/notepad/lang/de-DE.rc | 1 + base/applications/notepad/lang/el-GR.rc | 1 + base/applications/notepad/lang/en-US.rc | 1 + base/applications/notepad/lang/es-ES.rc | 1 + base/applications/notepad/lang/et-EE.rc | 1 + base/applications/notepad/lang/eu-ES.rc | 1 + base/applications/notepad/lang/fi-FI.rc | 1 + base/applications/notepad/lang/fr-FR.rc | 1 + base/applications/notepad/lang/he-IL.rc | 1 + base/applications/notepad/lang/hi-IN.rc | 1 + base/applications/notepad/lang/hr-HR.rc | 1 + base/applications/notepad/lang/hu-HU.rc | 1 + base/applications/notepad/lang/hy-AM.rc | 1 + base/applications/notepad/lang/id-ID.rc | 1 + base/applications/notepad/lang/it-IT.rc | 1 + base/applications/notepad/lang/ja-JP.rc | 1 + base/applications/notepad/lang/lt-LT.rc | 1 + base/applications/notepad/lang/ms-MY.rc | 1 + base/applications/notepad/lang/nl-NL.rc | 1 + base/applications/notepad/lang/no-NO.rc | 1 + base/applications/notepad/lang/pl-PL.rc | 1 + base/applications/notepad/lang/pt-BR.rc | 1 + base/applications/notepad/lang/pt-PT.rc | 1 + base/applications/notepad/lang/ro-RO.rc | 1 + base/applications/notepad/lang/ru-RU.rc | 1 + base/applications/notepad/lang/sk-SK.rc | 1 + base/applications/notepad/lang/sl-SI.rc | 1 + base/applications/notepad/lang/sq-AL.rc | 1 + base/applications/notepad/lang/sv-SE.rc | 1 + base/applications/notepad/lang/th-TH.rc | 1 + base/applications/notepad/lang/tr-TR.rc | 1 + base/applications/notepad/lang/uk-UA.rc | 1 + base/applications/notepad/lang/uz-UZ.rc | 1 + base/applications/notepad/lang/zh-CN.rc | 1 + base/applications/notepad/lang/zh-HK.rc | 1 + base/applications/notepad/lang/zh-TW.rc | 1 + base/applications/notepad/main.c | 15 ++++++++++----- base/applications/notepad/main.h | 2 +- base/applications/notepad/notepad_res.h | 1 + 44 files changed, 58 insertions(+), 8 deletions(-) diff --git a/base/applications/notepad/dialog.c b/base/applications/notepad/dialog.c index b62c2768b09..72bb17a88dd 100644 --- a/base/applications/notepad/dialog.c +++ b/base/applications/notepad/dialog.c @@ -1070,8 +1070,13 @@ VOID DIALOG_Search(VOID) DIALOG_SearchDialog(FindText); } -VOID DIALOG_SearchNext(VOID) +VOID DIALOG_SearchNext(BOOL bDown) { + if (bDown) + Globals.find.Flags |= FR_DOWN; + else + Globals.find.Flags &= ~FR_DOWN; + if (Globals.find.lpstrFindWhat != NULL) NOTEPAD_FindNext(&Globals.find, FALSE, TRUE); else diff --git a/base/applications/notepad/dialog.h b/base/applications/notepad/dialog.h index 5b9967af0ad..5ebe3b227b2 100644 --- a/base/applications/notepad/dialog.h +++ b/base/applications/notepad/dialog.h @@ -39,7 +39,7 @@ VOID DIALOG_EditTimeDate(VOID); VOID DIALOG_EditWrap(VOID); VOID DIALOG_Search(VOID); -VOID DIALOG_SearchNext(VOID); +VOID DIALOG_SearchNext(BOOL bDown); VOID DIALOG_Replace(VOID); VOID DIALOG_GoTo(VOID); diff --git a/base/applications/notepad/lang/bg-BG.rc b/base/applications/notepad/lang/bg-BG.rc index 955ea84c56f..6f638b9ead2 100644 --- a/base/applications/notepad/lang/bg-BG.rc +++ b/base/applications/notepad/lang/bg-BG.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/cs-CZ.rc b/base/applications/notepad/lang/cs-CZ.rc index c50c65a8b54..6b3a41a8673 100644 --- a/base/applications/notepad/lang/cs-CZ.rc +++ b/base/applications/notepad/lang/cs-CZ.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/da-DK.rc b/base/applications/notepad/lang/da-DK.rc index f0b8b8e7acb..50e250611cf 100644 --- a/base/applications/notepad/lang/da-DK.rc +++ b/base/applications/notepad/lang/da-DK.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/de-DE.rc b/base/applications/notepad/lang/de-DE.rc index 6143b98bbb9..367c4bf1c29 100644 --- a/base/applications/notepad/lang/de-DE.rc +++ b/base/applications/notepad/lang/de-DE.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/el-GR.rc b/base/applications/notepad/lang/el-GR.rc index c362a1c76d5..5aab21d86ea 100644 --- a/base/applications/notepad/lang/el-GR.rc +++ b/base/applications/notepad/lang/el-GR.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/en-US.rc b/base/applications/notepad/lang/en-US.rc index 2dd54fc24f9..50f5578e83d 100644 --- a/base/applications/notepad/lang/en-US.rc +++ b/base/applications/notepad/lang/en-US.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/es-ES.rc b/base/applications/notepad/lang/es-ES.rc index b4d68c439cb..5e84bf21445 100644 --- a/base/applications/notepad/lang/es-ES.rc +++ b/base/applications/notepad/lang/es-ES.rc @@ -18,6 +18,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/et-EE.rc b/base/applications/notepad/lang/et-EE.rc index 08101d5ac6b..a0bcf0bf1c6 100644 --- a/base/applications/notepad/lang/et-EE.rc +++ b/base/applications/notepad/lang/et-EE.rc @@ -24,6 +24,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/eu-ES.rc b/base/applications/notepad/lang/eu-ES.rc index 55fce4b4ff5..7378bd87a34 100644 --- a/base/applications/notepad/lang/eu-ES.rc +++ b/base/applications/notepad/lang/eu-ES.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/fi-FI.rc b/base/applications/notepad/lang/fi-FI.rc index fc140b42480..af260b19698 100644 --- a/base/applications/notepad/lang/fi-FI.rc +++ b/base/applications/notepad/lang/fi-FI.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/fr-FR.rc b/base/applications/notepad/lang/fr-FR.rc index bbdaa8511c2..dc4f17032bb 100644 --- a/base/applications/notepad/lang/fr-FR.rc +++ b/base/applications/notepad/lang/fr-FR.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/he-IL.rc b/base/applications/notepad/lang/he-IL.rc index a1a320a3850..c611b674a89 100644 --- a/base/applications/notepad/lang/he-IL.rc +++ b/base/applications/notepad/lang/he-IL.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/hi-IN.rc b/base/applications/notepad/lang/hi-IN.rc index 15f70a0e759..67d9c87684d 100644 --- a/base/applications/notepad/lang/hi-IN.rc +++ b/base/applications/notepad/lang/hi-IN.rc @@ -23,6 +23,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/hr-HR.rc b/base/applications/notepad/lang/hr-HR.rc index 7cd1fea757e..6e201827d4d 100644 --- a/base/applications/notepad/lang/hr-HR.rc +++ b/base/applications/notepad/lang/hr-HR.rc @@ -23,6 +23,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/hu-HU.rc b/base/applications/notepad/lang/hu-HU.rc index ff0ed7099b0..6ee0bdaee9a 100644 --- a/base/applications/notepad/lang/hu-HU.rc +++ b/base/applications/notepad/lang/hu-HU.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/hy-AM.rc b/base/applications/notepad/lang/hy-AM.rc index 78501b77a81..66574a6c560 100644 --- a/base/applications/notepad/lang/hy-AM.rc +++ b/base/applications/notepad/lang/hy-AM.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/id-ID.rc b/base/applications/notepad/lang/id-ID.rc index c690e7d0c87..3c00fb80c68 100644 --- a/base/applications/notepad/lang/id-ID.rc +++ b/base/applications/notepad/lang/id-ID.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/it-IT.rc b/base/applications/notepad/lang/it-IT.rc index 2bf7cb3c202..1c5c0766b95 100644 --- a/base/applications/notepad/lang/it-IT.rc +++ b/base/applications/notepad/lang/it-IT.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/ja-JP.rc b/base/applications/notepad/lang/ja-JP.rc index ff0fbb033e6..cb5f34450aa 100644 --- a/base/applications/notepad/lang/ja-JP.rc +++ b/base/applications/notepad/lang/ja-JP.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/lt-LT.rc b/base/applications/notepad/lang/lt-LT.rc index 55ea57b7134..f3f30f389c3 100644 --- a/base/applications/notepad/lang/lt-LT.rc +++ b/base/applications/notepad/lang/lt-LT.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/ms-MY.rc b/base/applications/notepad/lang/ms-MY.rc index a261729f19c..c218b23f6ca 100644 --- a/base/applications/notepad/lang/ms-MY.rc +++ b/base/applications/notepad/lang/ms-MY.rc @@ -18,6 +18,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/nl-NL.rc b/base/applications/notepad/lang/nl-NL.rc index f1451f7edd0..a5767cb7e0d 100644 --- a/base/applications/notepad/lang/nl-NL.rc +++ b/base/applications/notepad/lang/nl-NL.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/no-NO.rc b/base/applications/notepad/lang/no-NO.rc index bfd23c526cb..1612b9913c0 100644 --- a/base/applications/notepad/lang/no-NO.rc +++ b/base/applications/notepad/lang/no-NO.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/pl-PL.rc b/base/applications/notepad/lang/pl-PL.rc index 6c064cfd089..fb9af935eb7 100644 --- a/base/applications/notepad/lang/pl-PL.rc +++ b/base/applications/notepad/lang/pl-PL.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/pt-BR.rc b/base/applications/notepad/lang/pt-BR.rc index 9bf26297c29..ffc6ef44cce 100644 --- a/base/applications/notepad/lang/pt-BR.rc +++ b/base/applications/notepad/lang/pt-BR.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/pt-PT.rc b/base/applications/notepad/lang/pt-PT.rc index 6eb677293c2..d9a5b8576e0 100644 --- a/base/applications/notepad/lang/pt-PT.rc +++ b/base/applications/notepad/lang/pt-PT.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/ro-RO.rc b/base/applications/notepad/lang/ro-RO.rc index f8ee0a8a557..cd10206770f 100644 --- a/base/applications/notepad/lang/ro-RO.rc +++ b/base/applications/notepad/lang/ro-RO.rc @@ -21,6 +21,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/ru-RU.rc b/base/applications/notepad/lang/ru-RU.rc index 1b52362c6f8..1d27885d2f4 100644 --- a/base/applications/notepad/lang/ru-RU.rc +++ b/base/applications/notepad/lang/ru-RU.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/sk-SK.rc b/base/applications/notepad/lang/sk-SK.rc index 7b7d740e85d..a27501ed73f 100644 --- a/base/applications/notepad/lang/sk-SK.rc +++ b/base/applications/notepad/lang/sk-SK.rc @@ -24,6 +24,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/sl-SI.rc b/base/applications/notepad/lang/sl-SI.rc index d3c655f8828..6a87df05d1c 100644 --- a/base/applications/notepad/lang/sl-SI.rc +++ b/base/applications/notepad/lang/sl-SI.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/sq-AL.rc b/base/applications/notepad/lang/sq-AL.rc index 9ad098f2948..adc546e0411 100644 --- a/base/applications/notepad/lang/sq-AL.rc +++ b/base/applications/notepad/lang/sq-AL.rc @@ -20,6 +20,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/sv-SE.rc b/base/applications/notepad/lang/sv-SE.rc index 120eefdc5cc..55c6ff59785 100644 --- a/base/applications/notepad/lang/sv-SE.rc +++ b/base/applications/notepad/lang/sv-SE.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/th-TH.rc b/base/applications/notepad/lang/th-TH.rc index 3af4eef7ea1..aacedd7305b 100644 --- a/base/applications/notepad/lang/th-TH.rc +++ b/base/applications/notepad/lang/th-TH.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/tr-TR.rc b/base/applications/notepad/lang/tr-TR.rc index 78e609ca893..0401ea4a169 100644 --- a/base/applications/notepad/lang/tr-TR.rc +++ b/base/applications/notepad/lang/tr-TR.rc @@ -19,6 +19,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/uk-UA.rc b/base/applications/notepad/lang/uk-UA.rc index 854d1407c5a..a64d6e92276 100644 --- a/base/applications/notepad/lang/uk-UA.rc +++ b/base/applications/notepad/lang/uk-UA.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/uz-UZ.rc b/base/applications/notepad/lang/uz-UZ.rc index 42a4d14afd0..3eaea09f591 100644 --- a/base/applications/notepad/lang/uz-UZ.rc +++ b/base/applications/notepad/lang/uz-UZ.rc @@ -16,6 +16,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/zh-CN.rc b/base/applications/notepad/lang/zh-CN.rc index 4808a2e015b..28fad41e6e6 100644 --- a/base/applications/notepad/lang/zh-CN.rc +++ b/base/applications/notepad/lang/zh-CN.rc @@ -24,6 +24,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/zh-HK.rc b/base/applications/notepad/lang/zh-HK.rc index 2ceb617aaed..2ed9942d2b5 100644 --- a/base/applications/notepad/lang/zh-HK.rc +++ b/base/applications/notepad/lang/zh-HK.rc @@ -24,6 +24,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/lang/zh-TW.rc b/base/applications/notepad/lang/zh-TW.rc index f83d9d8c80f..b4974b34d83 100644 --- a/base/applications/notepad/lang/zh-TW.rc +++ b/base/applications/notepad/lang/zh-TW.rc @@ -24,6 +24,7 @@ BEGIN "^X", CMD_CUT "^Z", CMD_UNDO VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F3, CMD_SEARCH_PREV, VIRTKEY, SHIFT VK_F5, CMD_TIME_DATE, VIRTKEY END diff --git a/base/applications/notepad/main.c b/base/applications/notepad/main.c index e42b0c749e0..52370736318 100644 --- a/base/applications/notepad/main.c +++ b/base/applications/notepad/main.c @@ -5,7 +5,7 @@ * Copyright 1997,98 Marcel Baur <mbaur(a)g26.ethz.ch> * Copyright 2002 Sylvain Petreolle <spetreolle(a)yahoo.fr> * Copyright 2002 Andriy Palamarchuk - * Copyright 2020 Katayama Hirofumi MZ + * Copyright 2020-2023 Katayama Hirofumi MZ * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -37,6 +37,7 @@ VOID NOTEPAD_EnableSearchMenu() UINT uEnable = MF_BYCOMMAND | (bEmpty ? MF_GRAYED : MF_ENABLED); EnableMenuItem(Globals.hMenu, CMD_SEARCH, uEnable); EnableMenuItem(Globals.hMenu, CMD_SEARCH_NEXT, uEnable); + EnableMenuItem(Globals.hMenu, CMD_SEARCH_PREV, uEnable); } /*********************************************************************** @@ -83,9 +84,10 @@ static int NOTEPAD_MenuCommand(WPARAM wParam) case CMD_TIME_DATE: DIALOG_EditTimeDate(); break; case CMD_SEARCH: DIALOG_Search(); break; - case CMD_SEARCH_NEXT: DIALOG_SearchNext(); break; + case CMD_SEARCH_NEXT: DIALOG_SearchNext(TRUE); break; case CMD_REPLACE: DIALOG_Replace(); break; case CMD_GOTO: DIALOG_GoTo(); break; + case CMD_SEARCH_PREV: DIALOG_SearchNext(FALSE); break; case CMD_WRAP: DIALOG_EditWrap(); break; case CMD_FONT: DIALOG_SelectFont(); break; @@ -647,14 +649,17 @@ int WINAPI _tWinMain(HINSTANCE hInstance, HINSTANCE prev, LPTSTR cmdline, int sh hAccel = LoadAccelerators(hInstance, MAKEINTRESOURCE(ID_ACCEL)); - while (GetMessage(&msg, 0, 0, 0)) + while (GetMessage(&msg, NULL, 0, 0)) { - if (!IsDialogMessage(Globals.hFindReplaceDlg, &msg) && - !TranslateAccelerator(Globals.hMainWnd, hAccel, &msg)) + if (!TranslateAccelerator(Globals.hMainWnd, hAccel, &msg) && + !IsDialogMessage(Globals.hFindReplaceDlg, &msg)) { TranslateMessage(&msg); DispatchMessage(&msg); } } + + DestroyAcceleratorTable(hAccel); + return (int) msg.wParam; } diff --git a/base/applications/notepad/main.h b/base/applications/notepad/main.h index 34d7e542670..a1ecae7e641 100644 --- a/base/applications/notepad/main.h +++ b/base/applications/notepad/main.h @@ -99,5 +99,5 @@ void NOTEPAD_LoadSettingsFromRegistry(void); void NOTEPAD_SaveSettingsToRegistry(void); /* from main.c */ -BOOL NOTEPAD_FindNext(FINDREPLACE *, BOOL , BOOL ); +BOOL NOTEPAD_FindNext(FINDREPLACE *pFindReplace, BOOL bReplace, BOOL bShowAlert); VOID NOTEPAD_EnableSearchMenu(VOID); diff --git a/base/applications/notepad/notepad_res.h b/base/applications/notepad/notepad_res.h index 460a7c8d3d4..179a1c73573 100644 --- a/base/applications/notepad/notepad_res.h +++ b/base/applications/notepad/notepad_res.h @@ -54,6 +54,7 @@ #define CMD_SEARCH_NEXT 0x121 #define CMD_REPLACE 0x122 #define CMD_GOTO 0x123 +#define CMD_SEARCH_PREV 0x124 #define CMD_WRAP 0x119 #define CMD_FONT 0x140

1 0

[reactos] 01/01: [RAPPS] Style-addendum, no functional change (#5091)
by Joachim Henze 28 Feb '23

28 Feb '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=a9b1e059f422ecb6d02f4… commit a9b1e059f422ecb6d02f46720f8cc1e5b50decb1 Author: Joachim Henze <joachim.henze(a)reactos.org> AuthorDate: Tue Feb 28 01:00:29 2023 +0100 Commit: GitHub <noreply(a)github.com> CommitDate: Tue Feb 28 01:00:29 2023 +0100 [RAPPS] Style-addendum, no functional change (#5091) * Just rename the 4 new files. Absolutely no other change. * CMakeLists.txt, adapt the filenames * Adapt all includes to make it compile * CApplicationDB->CAppDB * CApplicationInfo->CAppInfo * Follow Mark Jansens good example and antipad the headers --- base/applications/rapps/CMakeLists.txt | 8 ++--- .../rapps/{applicationdb.cpp => appdb.cpp} | 36 +++++++++++----------- .../rapps/{applicationinfo.cpp => appinfo.cpp} | 12 ++++---- base/applications/rapps/appview.cpp | 10 +++--- base/applications/rapps/asyncinet.cpp | 2 +- base/applications/rapps/cabinet.cpp | 2 +- base/applications/rapps/gui.cpp | 28 ++++++++--------- base/applications/rapps/include/appdb.h | 35 +++++++++++++++++++++ .../rapps/include/{applicationinfo.h => appinfo.h} | 10 +++--- base/applications/rapps/include/applicationdb.h | 35 --------------------- base/applications/rapps/include/appview.h | 6 ++-- base/applications/rapps/include/dialogs.h | 8 ++--- base/applications/rapps/include/gui.h | 10 +++--- base/applications/rapps/include/rapps.h | 4 +-- base/applications/rapps/include/rosui.h | 2 +- base/applications/rapps/loaddlg.cpp | 18 +++++------ base/applications/rapps/misc.cpp | 4 +-- base/applications/rapps/settings.cpp | 2 +- base/applications/rapps/settingsdlg.cpp | 4 +-- base/applications/rapps/unattended.cpp | 26 ++++++++-------- base/applications/rapps/winmain.cpp | 4 +-- 21 files changed, 133 insertions(+), 133 deletions(-) diff --git a/base/applications/rapps/CMakeLists.txt b/base/applications/rapps/CMakeLists.txt index 0e22cea8134..5f3bbcc3cca 100644 --- a/base/applications/rapps/CMakeLists.txt +++ b/base/applications/rapps/CMakeLists.txt @@ -5,8 +5,8 @@ include_directories(${REACTOS_SOURCE_DIR}/sdk/lib/conutils) include_directories(include) list(APPEND SOURCE - applicationdb.cpp - applicationinfo.cpp + appdb.cpp + appinfo.cpp appview.cpp asyncinet.cpp cabinet.cpp @@ -19,8 +19,8 @@ list(APPEND SOURCE settingsdlg.cpp unattended.cpp winmain.cpp - include/applicationdb.h - include/applicationinfo.h + include/appdb.h + include/appinfo.h include/appview.h include/asyncinet.h include/configparser.h diff --git a/base/applications/rapps/applicationdb.cpp b/base/applications/rapps/appdb.cpp similarity index 87% rename from base/applications/rapps/applicationdb.cpp rename to base/applications/rapps/appdb.cpp index f1044c57ba8..b6fb33214ac 100644 --- a/base/applications/rapps/applicationdb.cpp +++ b/base/applications/rapps/appdb.cpp @@ -2,13 +2,13 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Classes for working with available applications - * COPYRIGHT: Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) - * Copyright 2020 He Yang (1160386205(a)qq.com) + * COPYRIGHT: Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) + * Copyright 2020 He Yang (1160386205(a)qq.com) * Copyright 2021-2023 Mark Jansen <mark.jansen(a)reactos.org> */ #include "rapps.h" -#include "applicationdb.h" +#include "appdb.h" #include "configparser.h" #include "settings.h" @@ -18,29 +18,29 @@ static REGSAM g_RegSamEnum[3] = {KEY_WOW64_32KEY, KEY_WOW64_32KEY, KEY_WOW64_64K #define UNINSTALL_SUBKEY L"Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall" static VOID -ClearList(CAtlList<CApplicationInfo *> &list) +ClearList(CAtlList<CAppInfo *> &list) { POSITION InfoListPosition = list.GetHeadPosition(); while (InfoListPosition) { - CApplicationInfo *Info = list.GetNext(InfoListPosition); + CAppInfo *Info = list.GetNext(InfoListPosition); delete Info; } list.RemoveAll(); } -CApplicationDB::CApplicationDB(const CStringW &path) : m_BasePath(path) +CAppDB::CAppDB(const CStringW &path) : m_BasePath(path) { m_BasePath.Canonicalize(); } -CApplicationInfo * -CApplicationDB::FindByPackageName(const CStringW &name) +CAppInfo * +CAppDB::FindByPackageName(const CStringW &name) { POSITION CurrentListPosition = m_Available.GetHeadPosition(); while (CurrentListPosition) { - CApplicationInfo *Info = m_Available.GetNext(CurrentListPosition); + CAppInfo *Info = m_Available.GetNext(CurrentListPosition); if (Info->szIdentifier == name) { return Info; @@ -50,16 +50,16 @@ CApplicationDB::FindByPackageName(const CStringW &name) } void -CApplicationDB::GetApps(CAtlList<CApplicationInfo *> &List, AppsCategories Type) const +CAppDB::GetApps(CAtlList<CAppInfo *> &List, AppsCategories Type) const { const BOOL UseInstalled = IsInstalledEnum(Type); - const CAtlList<CApplicationInfo *> &list = UseInstalled ? m_Installed : m_Available; + const CAtlList<CAppInfo *> &list = UseInstalled ? m_Installed : m_Available; const BOOL IncludeAll = UseInstalled ? (Type == ENUM_ALL_INSTALLED) : (Type == ENUM_ALL_AVAILABLE); POSITION CurrentListPosition = list.GetHeadPosition(); while (CurrentListPosition) { - CApplicationInfo *Info = list.GetNext(CurrentListPosition); + CAppInfo *Info = list.GetNext(CurrentListPosition); if (IncludeAll || Type == Info->iCategory) { @@ -69,7 +69,7 @@ CApplicationDB::GetApps(CAtlList<CApplicationInfo *> &List, AppsCategories Type) } BOOL -CApplicationDB::EnumerateFiles() +CAppDB::EnumerateFiles() { ClearList(m_Available); @@ -91,7 +91,7 @@ CApplicationDB::EnumerateFiles() PathRemoveExtensionW(szPkgName.GetBuffer(MAX_PATH)); szPkgName.ReleaseBuffer(); - CApplicationInfo *Info = FindByPackageName(szPkgName); + CAppInfo *Info = FindByPackageName(szPkgName); ATLASSERT(Info == NULL); if (!Info) { @@ -118,7 +118,7 @@ CApplicationDB::EnumerateFiles() } VOID -CApplicationDB::UpdateAvailable() +CAppDB::UpdateAvailable() { if (!CreateDirectoryW(m_BasePath, NULL) && GetLastError() != ERROR_ALREADY_EXISTS) return; @@ -142,7 +142,7 @@ CApplicationDB::UpdateAvailable() } VOID -CApplicationDB::UpdateInstalled() +CAppDB::UpdateInstalled() { // Remove all old entries ClearList(m_Installed); @@ -239,7 +239,7 @@ DeleteWithWildcard(const CPathW &Dir, const CStringW &Filter) } VOID -CApplicationDB::RemoveCached() +CAppDB::RemoveCached() { // Delete icons CPathW AppsPath = m_BasePath; @@ -263,7 +263,7 @@ CApplicationDB::RemoveCached() } BOOL -CApplicationDB::RemoveInstalledAppFromRegistry(const CApplicationInfo *Info) +CAppDB::RemoveInstalledAppFromRegistry(const CAppInfo *Info) { // Validate that this is actually an installed app / update ATLASSERT(Info->iCategory == ENUM_INSTALLED_APPLICATIONS || Info->iCategory == ENUM_UPDATES); diff --git a/base/applications/rapps/applicationinfo.cpp b/base/applications/rapps/appinfo.cpp similarity index 97% rename from base/applications/rapps/applicationinfo.cpp rename to base/applications/rapps/appinfo.cpp index 942668e05de..765f02016bb 100644 --- a/base/applications/rapps/applicationinfo.cpp +++ b/base/applications/rapps/appinfo.cpp @@ -2,20 +2,20 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Classes for working with available applications - * COPYRIGHT: Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) - * Copyright 2020 He Yang (1160386205(a)qq.com) + * COPYRIGHT: Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) + * Copyright 2020 He Yang (1160386205(a)qq.com) * Copyright 2021-2023 Mark Jansen <mark.jansen(a)reactos.org> */ #include "rapps.h" #include "appview.h" -CApplicationInfo::CApplicationInfo(const CStringW &Identifier, AppsCategories Category) +CAppInfo::CAppInfo(const CStringW &Identifier, AppsCategories Category) : szIdentifier(Identifier), iCategory(Category) { } -CApplicationInfo::~CApplicationInfo() +CAppInfo::~CAppInfo() { } @@ -24,7 +24,7 @@ CAvailableApplicationInfo::CAvailableApplicationInfo( const CStringW &PkgName, AppsCategories Category, const CPathW &BasePath) - : CApplicationInfo(PkgName, Category), m_Parser(Parser), m_ScrnshotRetrieved(false), m_LanguagesLoaded(false) + : CAppInfo(PkgName, Category), m_Parser(Parser), m_ScrnshotRetrieved(false), m_LanguagesLoaded(false) { m_Parser->GetString(L"Name", szDisplayName); m_Parser->GetString(L"Version", szDisplayVersion); @@ -369,7 +369,7 @@ CInstalledApplicationInfo::CInstalledApplicationInfo( const CStringW &KeyName, AppsCategories Category, int KeyIndex) - : CApplicationInfo(KeyName, Category), m_hKey(Key), iKeyIndex(KeyIndex) + : CAppInfo(KeyName, Category), m_hKey(Key), iKeyIndex(KeyIndex) { if (GetApplicationRegString(L"DisplayName", szDisplayName)) { diff --git a/base/applications/rapps/appview.cpp b/base/applications/rapps/appview.cpp index 5c1c548b6a6..3157f7634e2 100644 --- a/base/applications/rapps/appview.cpp +++ b/base/applications/rapps/appview.cpp @@ -914,7 +914,7 @@ CAppInfoDisplay::Create(HWND hwndParent) } VOID -CAppInfoDisplay::ShowAppInfo(CApplicationInfo *Info) +CAppInfoDisplay::ShowAppInfo(CAppInfo *Info) { CStringW ScrnshotLocation; if (Info->RetrieveScreenshot(ScrnshotLocation)) @@ -1280,7 +1280,7 @@ CAppsListView::SetViewMode(DWORD ViewMode) } BOOL -CAppsListView::AddApplication(CApplicationInfo *AppInfo, BOOL InitialCheckState) +CAppsListView::AddApplication(CAppInfo *AppInfo, BOOL InitialCheckState) { if (ApplicationViewType == AppViewTypeInstalledApps) { @@ -1474,7 +1474,7 @@ CApplicationView::ProcessWindowMessage( if (ApplicationViewType == AppViewTypeAvailableApps) { m_MainWindow->InstallApplication( - (CApplicationInfo *)m_ListView->GetItemData(Item->iItem)); + (CAppInfo *)m_ListView->GetItemData(Item->iItem)); } } } @@ -1851,7 +1851,7 @@ CApplicationView::SetDisplayAppType(APPLICATION_VIEW_TYPE AppType) } BOOL -CApplicationView::AddApplication(CApplicationInfo *AppInfo, BOOL InitialCheckState) +CApplicationView::AddApplication(CAppInfo *AppInfo, BOOL InitialCheckState) { return m_ListView->AddApplication(AppInfo, InitialCheckState); } @@ -1897,7 +1897,7 @@ CApplicationView::ItemGetFocus(LPVOID CallbackParam) { if (CallbackParam) { - CApplicationInfo *Info = static_cast<CApplicationInfo *>(CallbackParam); + CAppInfo *Info = static_cast<CAppInfo *>(CallbackParam); m_AppsInfo->ShowAppInfo(Info); if (ApplicationViewType == AppViewTypeInstalledApps) diff --git a/base/applications/rapps/asyncinet.cpp b/base/applications/rapps/asyncinet.cpp index 08be4839c77..f254c0ff944 100644 --- a/base/applications/rapps/asyncinet.cpp +++ b/base/applications/rapps/asyncinet.cpp @@ -2,7 +2,7 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Async Internet operation using WinINet - * COPYRIGHT: Copyright 2020 He Yang (1160386205(a)qq.com) + * COPYRIGHT: Copyright 2020 He Yang (1160386205(a)qq.com) */ #include "rapps.h" diff --git a/base/applications/rapps/cabinet.cpp b/base/applications/rapps/cabinet.cpp index 609f28faf5b..a86ad8cecc2 100644 --- a/base/applications/rapps/cabinet.cpp +++ b/base/applications/rapps/cabinet.cpp @@ -2,7 +2,7 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Cabinet extraction using FDI API - * COPYRIGHT: Copyright 2018 Alexander Shaposhnikov (sanchaez(a)reactos.org) + * COPYRIGHT: Copyright 2018 Alexander Shaposhnikov (sanchaez(a)reactos.org) */ #include "rapps.h" #include <debug.h> diff --git a/base/applications/rapps/gui.cpp b/base/applications/rapps/gui.cpp index d457cf148e9..43c641f9e0a 100644 --- a/base/applications/rapps/gui.cpp +++ b/base/applications/rapps/gui.cpp @@ -2,9 +2,9 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: GUI classes for RAPPS - * COPYRIGHT: Copyright 2015 David Quintana (gigaherz(a)gmail.com) - * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) - * Copyright 2020 He Yang (1160386205(a)qq.com) + * COPYRIGHT: Copyright 2015 David Quintana (gigaherz(a)gmail.com) + * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) + * Copyright 2020 He Yang (1160386205(a)qq.com) */ #include "rapps.h" @@ -86,7 +86,7 @@ CSideTreeView::~CSideTreeView() // **** CMainWindow **** -CMainWindow::CMainWindow(CApplicationDB *db) : m_ClientPanel(NULL), m_Db(db), SelectedEnumType(ENUM_ALL_INSTALLED) +CMainWindow::CMainWindow(CAppDB *db) : m_ClientPanel(NULL), m_Db(db), SelectedEnumType(ENUM_ALL_INSTALLED) { } @@ -276,7 +276,7 @@ CMainWindow::RemoveSelectedAppFromRegistry() if (MessageBoxW(szMsgText, szMsgTitle, MB_YESNO | MB_ICONQUESTION) == IDYES) { - CApplicationInfo *InstalledApp = (CApplicationInfo *)m_ApplicationView->GetFocusedItemData(); + CAppInfo *InstalledApp = (CAppInfo *)m_ApplicationView->GetFocusedItemData(); if (!InstalledApp) return FALSE; @@ -292,7 +292,7 @@ CMainWindow::UninstallSelectedApp(BOOL bModify) if (!IsInstalledEnum(SelectedEnumType)) return FALSE; - CApplicationInfo *InstalledApp = (CApplicationInfo *)m_ApplicationView->GetFocusedItemData(); + CAppInfo *InstalledApp = (CAppInfo *)m_ApplicationView->GetFocusedItemData(); if (!InstalledApp) return FALSE; @@ -517,7 +517,7 @@ CMainWindow::OnCommand(WPARAM wParam, LPARAM lParam) } else { - CApplicationInfo *App = (CApplicationInfo *)m_ApplicationView->GetFocusedItemData(); + CAppInfo *App = (CAppInfo *)m_ApplicationView->GetFocusedItemData(); if (App) { InstallApplication(App); @@ -578,12 +578,12 @@ CMainWindow::UpdateStatusBarText() } VOID -CMainWindow::AddApplicationsToView(CAtlList<CApplicationInfo *> &List) +CMainWindow::AddApplicationsToView(CAtlList<CAppInfo *> &List) { POSITION CurrentListPosition = List.GetHeadPosition(); while (CurrentListPosition) { - CApplicationInfo *Info = List.GetNext(CurrentListPosition); + CAppInfo *Info = List.GetNext(CurrentListPosition); if (szSearchPattern.IsEmpty() || SearchPatternMatch(Info->szDisplayName, szSearchPattern) || SearchPatternMatch(Info->szComments, szSearchPattern)) { @@ -613,7 +613,7 @@ CMainWindow::UpdateApplicationsList(AppsCategories EnumType, BOOL bReload) // set the display type of application-view. this will remove all the item in application-view too. m_ApplicationView->SetDisplayAppType(AppViewTypeInstalledApps); - CAtlList<CApplicationInfo *> List; + CAtlList<CAppInfo *> List; m_Db->GetApps(List, EnumType); AddApplicationsToView(List); } @@ -632,7 +632,7 @@ CMainWindow::UpdateApplicationsList(AppsCategories EnumType, BOOL bReload) } else { - CAtlList<CApplicationInfo *> List; + CAtlList<CAppInfo *> List; m_Db->GetApps(List, EnumType); AddApplicationsToView(List); } @@ -697,7 +697,7 @@ CMainWindow::ItemCheckStateChanged(BOOL bChecked, LPVOID CallbackParam) if (bUpdating) return; - CApplicationInfo *Info = (CApplicationInfo *)CallbackParam; + CAppInfo *Info = (CAppInfo *)CallbackParam; if (bChecked) { @@ -720,7 +720,7 @@ CMainWindow::ItemCheckStateChanged(BOOL bChecked, LPVOID CallbackParam) // this function is called when one or more application(s) should be installed // if Info is not zero, this app should be installed. otherwise those checked apps should be installed BOOL -CMainWindow::InstallApplication(CApplicationInfo *Info) +CMainWindow::InstallApplication(CAppInfo *Info) { if (Info) { @@ -785,7 +785,7 @@ CMainWindow::HandleTabOrder(int direction) // **** CMainWindow **** VOID -MainWindowLoop(CApplicationDB *db, INT nShowCmd) +MainWindowLoop(CAppDB *db, INT nShowCmd) { HACCEL KeyBrd; MSG Msg; diff --git a/base/applications/rapps/include/appdb.h b/base/applications/rapps/include/appdb.h new file mode 100644 index 00000000000..4e9c5386b3b --- /dev/null +++ b/base/applications/rapps/include/appdb.h @@ -0,0 +1,35 @@ +#pragma once + +#include <atlcoll.h> +#include <atlpath.h> + +#include "appinfo.h" + +class CAppDB +{ + private: + CPathW m_BasePath; + CAtlList<CAppInfo *> m_Available; + CAtlList<CAppInfo *> m_Installed; + + BOOL + EnumerateFiles(); + + public: + CAppDB(const CStringW &path); + + VOID + GetApps(CAtlList<CAppInfo *> &List, AppsCategories Type) const; + CAppInfo * + FindByPackageName(const CStringW &name); + + VOID + UpdateAvailable(); + VOID + UpdateInstalled(); + VOID + RemoveCached(); + + BOOL + RemoveInstalledAppFromRegistry(const CAppInfo *Info); +}; diff --git a/base/applications/rapps/include/applicationinfo.h b/base/applications/rapps/include/appinfo.h similarity index 94% rename from base/applications/rapps/include/applicationinfo.h rename to base/applications/rapps/include/appinfo.h index fb152d26bba..7ea0c65e537 100644 --- a/base/applications/rapps/include/applicationinfo.h +++ b/base/applications/rapps/include/appinfo.h @@ -65,11 +65,11 @@ IsInstalledEnum(INT x) class CAppRichEdit; -class CApplicationInfo +class CAppInfo { public: - CApplicationInfo(const CStringW &Identifier, AppsCategories Category); - virtual ~CApplicationInfo(); + CAppInfo(const CStringW &Identifier, AppsCategories Category); + virtual ~CAppInfo(); const CStringW szIdentifier; // PkgName or KeyName const AppsCategories iCategory; @@ -97,7 +97,7 @@ class CApplicationInfo UninstallApplication(BOOL bModify) = 0; }; -class CAvailableApplicationInfo : public CApplicationInfo +class CAvailableApplicationInfo : public CAppInfo { class CConfigParser *m_Parser; CSimpleArray<CStringW> m_szScrnshotLocation; @@ -143,7 +143,7 @@ class CAvailableApplicationInfo : public CApplicationInfo UninstallApplication(BOOL bModify) override; }; -class CInstalledApplicationInfo : public CApplicationInfo +class CInstalledApplicationInfo : public CAppInfo { CRegKey m_hKey; CStringW m_szInstallDate; diff --git a/base/applications/rapps/include/applicationdb.h b/base/applications/rapps/include/applicationdb.h deleted file mode 100644 index c63b1698c05..00000000000 --- a/base/applications/rapps/include/applicationdb.h +++ /dev/null @@ -1,35 +0,0 @@ -#pragma once - -#include <atlcoll.h> -#include <atlpath.h> - -#include "applicationinfo.h" - -class CApplicationDB -{ - private: - CPathW m_BasePath; - CAtlList<CApplicationInfo *> m_Available; - CAtlList<CApplicationInfo *> m_Installed; - - BOOL - EnumerateFiles(); - - public: - CApplicationDB(const CStringW &path); - - VOID - GetApps(CAtlList<CApplicationInfo *> &List, AppsCategories Type) const; - CApplicationInfo * - FindByPackageName(const CStringW &name); - - VOID - UpdateAvailable(); - VOID - UpdateInstalled(); - VOID - RemoveCached(); - - BOOL - RemoveInstalledAppFromRegistry(const CApplicationInfo *Info); -}; diff --git a/base/applications/rapps/include/appview.h b/base/applications/rapps/include/appview.h index fcf18a9d905..b563f2966e2 100644 --- a/base/applications/rapps/include/appview.h +++ b/base/applications/rapps/include/appview.h @@ -177,7 +177,7 @@ class CAppInfoDisplay : public CUiWindow<CWindowImpl<CAppInfoDisplay>> Create(HWND hwndParent); VOID - ShowAppInfo(CApplicationInfo *Info); + ShowAppInfo(CAppInfo *Info); VOID SetWelcomeText(); VOID @@ -263,7 +263,7 @@ class CAppsListView : public CUiWindow<CWindowImpl<CAppsListView, CListView>> SetViewMode(DWORD ViewMode); BOOL - AddApplication(CApplicationInfo *AppInfo, BOOL InitialCheckState); + AddApplication(CAppInfo *AppInfo, BOOL InitialCheckState); // this function is called when parent window receiving an notification about checkstate changing VOID @@ -397,7 +397,7 @@ class CApplicationView : public CUiWindow<CWindowImpl<CApplicationView>> SetDisplayAppType(APPLICATION_VIEW_TYPE AppType); BOOL - AddApplication(CApplicationInfo *InstAppInfo, BOOL InitialCheckState); + AddApplication(CAppInfo *InstAppInfo, BOOL InitialCheckState); VOID SetWatermark(const CStringW &Text); diff --git a/base/applications/rapps/include/dialogs.h b/base/applications/rapps/include/dialogs.h index 0301dbb7393..ee49f287d82 100644 --- a/base/applications/rapps/include/dialogs.h +++ b/base/applications/rapps/include/dialogs.h @@ -1,6 +1,6 @@ #pragma once -#include "applicationinfo.h" +#include "appinfo.h" #include <windef.h> #include <atlsimpcoll.h> @@ -11,12 +11,12 @@ CreateSettingsDlg(HWND hwnd); // Main window VOID -MainWindowLoop(class CApplicationDB *db, INT nShowCmd); +MainWindowLoop(class CAppDB *db, INT nShowCmd); // Download dialogs VOID DownloadApplicationsDB(LPCWSTR lpUrl, BOOL IsOfficial); BOOL -DownloadApplication(CApplicationInfo *pAppInfo); +DownloadApplication(CAppInfo *pAppInfo); BOOL -DownloadListOfApplications(const CAtlList<CApplicationInfo *> &AppsList, BOOL bIsModal); +DownloadListOfApplications(const CAtlList<CAppInfo *> &AppsList, BOOL bIsModal); diff --git a/base/applications/rapps/include/gui.h b/base/applications/rapps/include/gui.h index 48befeaf9f9..330f1118dac 100644 --- a/base/applications/rapps/include/gui.h +++ b/base/applications/rapps/include/gui.h @@ -54,8 +54,8 @@ class CMainWindow : public CWindowImpl<CMainWindow, CWindow, CFrameWinTraits> CApplicationView *m_ApplicationView = NULL; - CApplicationDB *m_Db; - CAtlList<CApplicationInfo *> m_Selected; + CAppDB *m_Db; + CAtlList<CAppInfo *> m_Selected; BOOL bUpdating = FALSE; @@ -63,7 +63,7 @@ class CMainWindow : public CWindowImpl<CMainWindow, CWindow, CFrameWinTraits> AppsCategories SelectedEnumType; public: - CMainWindow(CApplicationDB *db); + CMainWindow(CAppDB *db); ~CMainWindow(); @@ -106,7 +106,7 @@ class CMainWindow : public CWindowImpl<CMainWindow, CWindow, CFrameWinTraits> VOID UpdateApplicationsList(AppsCategories EnumType, BOOL bReload = FALSE); VOID - AddApplicationsToView(CAtlList<CApplicationInfo *> &List); + AddApplicationsToView(CAtlList<CAppInfo *> &List); public: static ATL::CWndClassInfo & @@ -123,7 +123,7 @@ class CMainWindow : public CWindowImpl<CMainWindow, CWindow, CFrameWinTraits> // this function is called when application-view is asked to install an application // if Info is not zero, this app should be installed. otherwise those checked apps should be installed BOOL - InstallApplication(CApplicationInfo *Info); + InstallApplication(CAppInfo *Info); // this function is called when search text is changed BOOL diff --git a/base/applications/rapps/include/rapps.h b/base/applications/rapps/include/rapps.h index 3a72072cadc..02cb22182ae 100644 --- a/base/applications/rapps/include/rapps.h +++ b/base/applications/rapps/include/rapps.h @@ -8,8 +8,8 @@ #include "defines.h" #include "dialogs.h" -#include "applicationinfo.h" -#include "applicationdb.h" +#include "appinfo.h" +#include "appdb.h" #include "misc.h" #include "configparser.h" diff --git a/base/applications/rapps/include/rosui.h b/base/applications/rapps/include/rosui.h index a68100a6118..48fb9a243a4 100644 --- a/base/applications/rapps/include/rosui.h +++ b/base/applications/rapps/include/rosui.h @@ -3,7 +3,7 @@ * LICENSE: GPL-2.0+ (https://spdx.org/licenses/GPL-2.0+) * FILE: base/applications/rapps/include/rosui.h * PURPOSE: ATL Layout engine for RAPPS - * COPYRIGHT: Copyright 2015 David Quintana (gigaherz(a)gmail.com) + * COPYRIGHT: Copyright 2015 David Quintana (gigaherz(a)gmail.com) */ #pragma once diff --git a/base/applications/rapps/loaddlg.cpp b/base/applications/rapps/loaddlg.cpp index 618ac035e5c..9063954ece0 100644 --- a/base/applications/rapps/loaddlg.cpp +++ b/base/applications/rapps/loaddlg.cpp @@ -2,12 +2,12 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Displaying a download dialog - * COPYRIGHT: Copyright 2001 John R. Sheets (for CodeWeavers) - * Copyright 2004 Mike McCormack (for CodeWeavers) - * Copyright 2005 Ge van Geldorp (gvg(a)reactos.org) - * Copyright 2009 Dmitry Chapyshev (dmitry(a)reactos.org) + * COPYRIGHT: Copyright 2001 John R. Sheets (for CodeWeavers) + * Copyright 2004 Mike McCormack (for CodeWeavers) + * Copyright 2005 Ge van Geldorp (gvg(a)reactos.org) + * Copyright 2009 Dmitry Chapyshev (dmitry(a)reactos.org) * Copyright 2015 Ismael Ferreras Morezuelas (swyterzone+ros(a)gmail.com) - * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) + * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) */ /* @@ -85,7 +85,7 @@ struct DownloadInfo DownloadInfo() { } - DownloadInfo(const CApplicationInfo &AppInfo) : DLType(DLTYPE_APPLICATION) + DownloadInfo(const CAppInfo &AppInfo) : DLType(DLTYPE_APPLICATION) { AppInfo.GetDownloadInfo(szUrl, szSHA1, SizeInBytes); szName = AppInfo.szDisplayName; @@ -1052,7 +1052,7 @@ CDownloadManager::LaunchDownloadDialog(BOOL bIsModal) // CDownloadManager BOOL -DownloadListOfApplications(const CAtlList<CApplicationInfo *> &AppsList, BOOL bIsModal) +DownloadListOfApplications(const CAtlList<CAppInfo *> &AppsList, BOOL bIsModal) { if (AppsList.IsEmpty()) return FALSE; @@ -1060,7 +1060,7 @@ DownloadListOfApplications(const CAtlList<CApplicationInfo *> &AppsList, BOOL bI POSITION CurrentListPosition = AppsList.GetHeadPosition(); while (CurrentListPosition) { - const CApplicationInfo *Info = AppsList.GetNext(CurrentListPosition); + const CAppInfo *Info = AppsList.GetNext(CurrentListPosition); CDownloadManager::Add(DownloadInfo(*Info)); } @@ -1071,7 +1071,7 @@ DownloadListOfApplications(const CAtlList<CApplicationInfo *> &AppsList, BOOL bI } BOOL -DownloadApplication(CApplicationInfo *pAppInfo) +DownloadApplication(CAppInfo *pAppInfo) { if (!pAppInfo) return FALSE; diff --git a/base/applications/rapps/misc.cpp b/base/applications/rapps/misc.cpp index 3d7311f8a1e..a5da21d45ce 100644 --- a/base/applications/rapps/misc.cpp +++ b/base/applications/rapps/misc.cpp @@ -2,9 +2,9 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Misc functions - * COPYRIGHT: Copyright 2009 Dmitry Chapyshev (dmitry(a)reactos.org) + * COPYRIGHT: Copyright 2009 Dmitry Chapyshev (dmitry(a)reactos.org) * Copyright 2015 Ismael Ferreras Morezuelas (swyterzone+ros(a)gmail.com) - * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) + * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) */ #include "rapps.h" diff --git a/base/applications/rapps/settings.cpp b/base/applications/rapps/settings.cpp index bac3f908349..4a00186f2ec 100644 --- a/base/applications/rapps/settings.cpp +++ b/base/applications/rapps/settings.cpp @@ -2,7 +2,7 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Functions to load / save settings from reg. - * COPYRIGHT: Copyright 2020 He Yang (1160386205(a)qq.com) + * COPYRIGHT: Copyright 2020 He Yang (1160386205(a)qq.com) */ #include "rapps.h" diff --git a/base/applications/rapps/settingsdlg.cpp b/base/applications/rapps/settingsdlg.cpp index c7db4c7c5dd..b75a136c0ff 100644 --- a/base/applications/rapps/settingsdlg.cpp +++ b/base/applications/rapps/settingsdlg.cpp @@ -2,8 +2,8 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Settings Dialog - * COPYRIGHT: Copyright 2009 Dmitry Chapyshev (dmitry(a)reactos.org) - * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) + * COPYRIGHT: Copyright 2009 Dmitry Chapyshev (dmitry(a)reactos.org) + * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) */ #include "rapps.h" diff --git a/base/applications/rapps/unattended.cpp b/base/applications/rapps/unattended.cpp index 858f24ee23b..e5fe54ed2ea 100644 --- a/base/applications/rapps/unattended.cpp +++ b/base/applications/rapps/unattended.cpp @@ -3,7 +3,7 @@ * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Functions to parse command-line flags and process them * COPYRIGHT: Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) - * Copyright 2020 He Yang (1160386205(a)qq.com) + * Copyright 2020 He Yang (1160386205(a)qq.com) */ #include "rapps.h" @@ -44,7 +44,7 @@ InitRappsConsole() } static BOOL -HandleInstallCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *argvLeft) +HandleInstallCommand(CAppDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *argvLeft) { if (argcLeft < 1) { @@ -53,11 +53,11 @@ HandleInstallCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR return FALSE; } - CAtlList<CApplicationInfo *> Applications; + CAtlList<CAppInfo *> Applications; for (int i = 0; i < argcLeft; i++) { LPCWSTR PackageName = argvLeft[i]; - CApplicationInfo *AppInfo = db->FindByPackageName(PackageName); + CAppInfo *AppInfo = db->FindByPackageName(PackageName); if (AppInfo) { Applications.AddTail(AppInfo); @@ -68,7 +68,7 @@ HandleInstallCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR } static BOOL -HandleSetupCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *argvLeft) +HandleSetupCommand(CAppDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *argvLeft) { if (argcLeft != 1) { @@ -77,7 +77,7 @@ HandleSetupCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *a return FALSE; } - CAtlList<CApplicationInfo *> Applications; + CAtlList<CAppInfo *> Applications; HINF InfHandle = SetupOpenInfFileW(argvLeft[0], NULL, INF_STYLE_WIN4, NULL); if (InfHandle == INVALID_HANDLE_VALUE) { @@ -92,7 +92,7 @@ HandleSetupCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *a { if (SetupGetStringFieldW(&Context, 1, szPkgName, _countof(szPkgName), NULL)) { - CApplicationInfo *AppInfo = db->FindByPackageName(szPkgName); + CAppInfo *AppInfo = db->FindByPackageName(szPkgName); if (AppInfo) { Applications.AddTail(AppInfo); @@ -106,7 +106,7 @@ HandleSetupCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *a } static BOOL -HandleFindCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *argvLeft) +HandleFindCommand(CAppDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *argvLeft) { if (argcLeft < 1) { @@ -114,7 +114,7 @@ HandleFindCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *ar return FALSE; } - CAtlList<CApplicationInfo *> List; + CAtlList<CAppInfo *> List; db->GetApps(List, ENUM_ALL_AVAILABLE); for (int i = 0; i < argcLeft; i++) @@ -125,7 +125,7 @@ HandleFindCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *ar POSITION CurrentListPosition = List.GetHeadPosition(); while (CurrentListPosition) { - CApplicationInfo *Info = List.GetNext(CurrentListPosition); + CAppInfo *Info = List.GetNext(CurrentListPosition); if (SearchPatternMatch(Info->szDisplayName, lpszSearch) || SearchPatternMatch(Info->szComments, lpszSearch)) { @@ -140,7 +140,7 @@ HandleFindCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *ar } static BOOL -HandleInfoCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *argvLeft) +HandleInfoCommand(CAppDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *argvLeft) { if (argcLeft < 1) { @@ -151,7 +151,7 @@ HandleInfoCommand(CApplicationDB *db, LPWSTR szCommand, int argcLeft, LPWSTR *ar for (int i = 0; i < argcLeft; i++) { LPCWSTR PackageName = argvLeft[i]; - CApplicationInfo *AppInfo = db->FindByPackageName(PackageName); + CAppInfo *AppInfo = db->FindByPackageName(PackageName); if (!AppInfo) { ConResMsgPrintf(StdOut, NULL, IDS_CMD_PACKAGE_NOT_FOUND, PackageName); @@ -231,7 +231,7 @@ ParseCmdAndExecute(LPWSTR lpCmdLine, BOOL bIsFirstLaunch, int nCmdShow) CStringW Directory; GetStorageDirectory(Directory); - CApplicationDB db(Directory); + CAppDB db(Directory); if (SettingsInfo.bUpdateAtStart || bIsFirstLaunch) { diff --git a/base/applications/rapps/winmain.cpp b/base/applications/rapps/winmain.cpp index cf3285a2783..3fb8e23d4b1 100644 --- a/base/applications/rapps/winmain.cpp +++ b/base/applications/rapps/winmain.cpp @@ -2,9 +2,9 @@ * PROJECT: ReactOS Applications Manager * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Main program - * COPYRIGHT: Copyright 2009 Dmitry Chapyshev (dmitry(a)reactos.org) + * COPYRIGHT: Copyright 2009 Dmitry Chapyshev (dmitry(a)reactos.org) * Copyright 2015 Ismael Ferreras Morezuelas (swyterzone+ros(a)gmail.com) - * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) + * Copyright 2017 Alexander Shaposhnikov (sanchaez(a)reactos.org) */ #include "rapps.h" #include "unattended.h"

1 0

[reactos] 01/01: Fix ICO_ExtractIconExW causing explorer to crash when trying to display icon for bad EXE PE header.
by Doug Lyons 27 Feb '23

27 Feb '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=60851914a84bf3f38a024… commit 60851914a84bf3f38a024de933579fa897c7e7c9 Author: Doug Lyons <douglyons(a)douglyons.com> AuthorDate: Sun Feb 26 13:03:53 2023 -0600 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Mon Feb 27 22:28:41 2023 +0100 Fix ICO_ExtractIconExW causing explorer to crash when trying to display icon for bad EXE PE header. See CORE-15879 Co-authored-by: Thomas Faber <thomas.faber(a)reactos.org> --- win32ss/user/user32/misc/exticon.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/win32ss/user/user32/misc/exticon.c b/win32ss/user/user32/misc/exticon.c index 09074c5c6a4..33f8f19b15f 100644 --- a/win32ss/user/user32/misc/exticon.c +++ b/win32ss/user/user32/misc/exticon.c @@ -616,6 +616,15 @@ static UINT ICO_ExtractIconExW( goto end; } +#ifdef __REACTOS__ + /* Check for boundary limit (and overflow) */ + if (((ULONG_PTR)(rootresdir + 1) < (ULONG_PTR)rootresdir) || + ((ULONG_PTR)(rootresdir + 1) > (ULONG_PTR)peimage + fsizel)) + { + goto end; + } +#endif + /* search for the group icon directory */ if (!(icongroupresdir = find_entry_by_id(rootresdir, LOWORD(RT_GROUP_ICON), rootresdir))) {

1 0

[reactos] 01/01: [IMM32] Fix ImmGetCompositionStringA/W (#5105)
by Katayama Hirofumi MZ 27 Feb '23

27 Feb '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=781fb4a55e4cebf52b810… commit 781fb4a55e4cebf52b810d01c510f5aa44da191f Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Mon Feb 27 12:33:45 2023 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Mon Feb 27 12:33:45 2023 +0900 [IMM32] Fix ImmGetCompositionStringA/W (#5105) Fix some typos in Imm32GetCompStrA/W. CORE-11700 --- dll/win32/imm32/compstr.c | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/dll/win32/imm32/compstr.c b/dll/win32/imm32/compstr.c index 5486397f3f4..5b1c153d0f4 100644 --- a/dll/win32/imm32/compstr.c +++ b/dll/win32/imm32/compstr.c @@ -289,15 +289,15 @@ Imm32GetCompStrA(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, case GCS_COMPREADATTR: dwBufLen = Imm32CompAttrWideToAnsi(CS_Attr(pCS, CompReadAttr), CS_Size(pCS, CompReadAttr), - CS_StrW(pCS, CompStr), - CS_SizeW(pCS, CompStr), + CS_StrW(pCS, CompReadStr), + CS_SizeW(pCS, CompReadStr), lpBuf, dwBufLen, uCodePage); break; case GCS_COMPREADCLAUSE: dwBufLen = Imm32CompClauseWideToAnsi(CS_Clause(pCS, CompReadClause), CS_Size(pCS, CompReadClause), - CS_StrW(pCS, CompStr), + CS_StrW(pCS, CompReadStr), lpBuf, dwBufLen, uCodePage); break; @@ -339,7 +339,7 @@ Imm32GetCompStrA(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, case GCS_RESULTREADCLAUSE: dwBufLen = Imm32CompClauseWideToAnsi(CS_Clause(pCS, ResultReadClause), CS_Size(pCS, ResultReadClause), - CS_StrW(pCS, CompStr), + CS_StrW(pCS, ResultReadStr), lpBuf, dwBufLen, uCodePage); break; @@ -352,7 +352,7 @@ Imm32GetCompStrA(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, case GCS_RESULTCLAUSE: dwBufLen = Imm32CompClauseWideToAnsi(CS_Clause(pCS, ResultClause), CS_Size(pCS, ResultClause), - CS_StrW(pCS, CompStr), + CS_StrW(pCS, ResultStr), lpBuf, dwBufLen, uCodePage); break; @@ -383,14 +383,15 @@ Imm32GetCompStrW(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, case GCS_COMPREADATTR: dwBufLen = Imm32CompAttrAnsiToWide(CS_Attr(pCS, CompReadAttr), CS_Size(pCS, CompReadAttr), - CS_StrA(pCS, CompStr), CS_SizeA(pCS, CompStr), + CS_StrA(pCS, CompReadStr), + CS_SizeA(pCS, CompReadStr), lpBuf, dwBufLen, uCodePage); break; case GCS_COMPREADCLAUSE: dwBufLen = Imm32CompClauseAnsiToWide(CS_Clause(pCS, CompReadClause), CS_Size(pCS, CompReadClause), - CS_StrA(pCS, CompStr), + CS_StrA(pCS, CompReadStr), lpBuf, dwBufLen, uCodePage); break; @@ -431,7 +432,7 @@ Imm32GetCompStrW(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, case GCS_RESULTREADCLAUSE: dwBufLen = Imm32CompClauseAnsiToWide(CS_Clause(pCS, ResultReadClause), CS_Size(pCS, ResultReadClause), - CS_StrA(pCS, CompStr), + CS_StrA(pCS, ResultReadStr), lpBuf, dwBufLen, uCodePage); break; @@ -444,7 +445,7 @@ Imm32GetCompStrW(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, case GCS_RESULTCLAUSE: dwBufLen = Imm32CompClauseAnsiToWide(CS_Clause(pCS, ResultClause), CS_Size(pCS, ResultClause), - CS_StrA(pCS, CompStr), + CS_StrA(pCS, ResultStr), lpBuf, dwBufLen, uCodePage); break;

1 0

[reactos] 01/01: [IMM32] Silence 'Out of boundary' log-spam
by Katayama Hirofumi MZ 26 Feb '23

26 Feb '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=88b1d89e957ea8ed59f96… commit 88b1d89e957ea8ed59f964d0379cd51d494f0be6 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Sun Feb 26 11:57:44 2023 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Sun Feb 26 11:57:44 2023 +0900 [IMM32] Silence 'Out of boundary' log-spam CORE-18852 --- dll/win32/imm32/candidate.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/dll/win32/imm32/candidate.c b/dll/win32/imm32/candidate.c index df976e407c8..9edddc82032 100644 --- a/dll/win32/imm32/candidate.c +++ b/dll/win32/imm32/candidate.c @@ -398,10 +398,6 @@ ImmGetCandidateWindow(HIMC hIMC, DWORD dwIndex, LPCANDIDATEFORM lpCandidate) *lpCandidate = *pCF; ret = TRUE; } - else - { - ERR("Out of boundary\n"); - } ImmUnlockIMC(hIMC); TRACE("ret: %d\n", ret);

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs