[EXPLORER] Cleanup after last commit CORE-15369

Improves code that was added by 0.4.15-dev-5078-g 46de3a22acc964f27a5260e144089125f83586ff
without any review yesterday.

[ISAPNP] Generate hardware ids and compatible ids in upper case

[SHELL32] Prevent use after free

'buffer' is local to the function, while m_sPath is an instance class member.
Fix that by calling the IShellLink::Resolve() function, which will allocate
and fill the m_sPath variable.

CORE-15229

[USER32][IMM32] Use wcscspn instead of wcsspn

b4575ec and db00a75 were my mistakes.
CORE-11700

[EXPLORER] Follow-up of #4718 (ba2cead)

Handle click OnNcLButtonUp instead of OnNcLButtonDown/OnNcLButtonDblClick.
CORE-15369

[EXPLORER] 'Show Desktop' button should work on button up (#4718)

Follow-up of PR #4715. CORE-15369
- The 'Show Desktop' button of taskbar works on button up instead of button down.
- Add NULL checks for m_ShowDesktopButton.
- Fix hung-up in rebooting.

[EXPLORER] Add 'Show Desktop' button at right edge of taskbar (#4715)

This PR adds a tiny button of window class "TrayShowDesktopButtonWClass" at right/bottom edge of taskbar.
This button allows the user to access "Show/Restore Desktop" feature by mouse.
You can toggle visibility of this button by registry value "TaskbarSd" in key "HKCU\SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\Advanced".
The button is themed when theme is available.
CORE-15369

[CPL][INTL] Realize automatic Japanese IME installation (#4705)

In intl.cpl, realize semi-automatic Japanese IME installation by checkbox "Install files for East Asian languages" by using rapps.exe. CORE-11700

[SHELL32] Fix property sheets that can't be closed due to failed init (#4709)

When a PropertySheet fails to init, that fact is being logged
and it shows an empty page, but it can't be closed.

Handle second failure properly by using FAILED_UNEXPECTEDLY macro.

CORE-18333

[HOTPLUG] Add French (fr-FR) translation (#4703)

[APPWIZ] Update Romanian (ro-RO) translation (#4700)

[ZIPFLDR] Set large icon correctly (#4708)

Now it's correctly displayed in Alt+Tab window selection.

[DDRAW] Handle DDLOCK_WRITEONLY in wined3dmapflags_from_ddrawmapflags() CORE-18378 (#4713)

Mutes the logging:
fixme:(dll/directx/wine/ddraw/utils.c:584) Unhandled flags 0x20.
gets logged many times per second.

It does affect several applications, e.g. the game 'Anno 1602' from 1998, and the 'Diablo 2 demo' from rapps,
For both games it can be observed with both: our VBEMP driver and the VBox4.3.12 3D-accelerated-driver.

Muting may improve performance a bit in such apps.
It gets logged although no missing features can be perceived visually in the rendering.

Fix it by importing Wine-commit b943c7910b3261c9603343369cd632f7a3b56bba
 ddraw: Handle DDLOCK_WRITEONLY in wined3dmapflags_from_ddrawmapflags().
Signed-off-by: Henri Verbeet <hverbeet@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

[GDI32] Fix ExtTextOutA with ETO_GLYPH_INDEX

CORE-18365

[NTOS] Add some sanity checks when synchronizing PDEs

[CPL] Boundary check of Control Panel applets (#4706)

- Check the boundary (i < NUM_APPLETS) of the variable i in CPlApplet functions.
- Use UINT type for the variable i.

[MEDIA][INF] The name of rapps in Japanese is too long (#4707)

Renaming: 「アプリケーション マネージャ」→「アプリ マネージャ」

[INPUT] Fix IsRebootNeeded condition

If there is any IME HKL, reboot is needed.
Follow-up to #4666 (596f04b). CORE-11700, CORE-18364

[USER32] Avoid szImeFileName buffer overrun

CORE-11700

[USER32] Don't allow invalid 'IME File' values

Improve security. CORE-11700

[IMM32] Follow-up of db00a75

Add more invalid characters. CORE-11700

[NTUSER] Use IS_CICERO_MODE() macro in co_UserActivateKeyboardLayout

CORE-11700

[IMM32] Don't allow invalid 'IME File' values

Improve security. CORE-11700

[INPUT] Follow-up of #4666 (596f04b)

Reboot is still needed for IME settings.
CORE-11700, CORE-18364

[USER32][INPUT] Support various keyboard layouts (#4666)

- Fix IntLoadKeyboardLayout function to return the correct HKL value.
- Modify LAYOUT_LIST_NODE structure to add more information.
- Fix LayoutList_GetByHkl function to choose the IME HKLs correctly.
- Ignore DELETED entries correctly.
- Improve UI/UX.
CORE-11700, CORE-13244, CORE-18364

[REACTOS] Various Russian (ru-RU) translation fixes (#4696)

Reviewed-by: Stanislav Motylkov <binarymaster@mail.ru>

[HOTPLUG] Add title bar icon (#4695)

- Add small icon to the title bar
- Set large icon correctly

CORE-15445

[HOTPLUG] Add Romanian (ro-RO) translation (#4693)

Signed-off-by: Andrei Miloiu <miloiuandrei@gmail.com>
Reviewed-by: Stanislav Motylkov <x86corez@gmail.com>
Reviewed-by: Ștefan Fulea <reactos-contributor@programmer.net>

[MMC] Implement standard "About" menu action (#4639)

Reviewed-by: Serge Gautherie <reactos-git_serge_171003@gautherie.fr>
Reviewed-by: Mark Jansen <mark.jansen@reactos.org>
Reviewed-by: Stanislav Motylkov <x86corez@gmail.com>

[SNDVOL32][MMSYS] Fix French (fr-FR) layout and translation (#4678)

Co-authored-by: Stanislav Motylkov <x86corez@gmail.com>
Co-authored-by: Hermès BÉLUSCA - MAÏTO <hermes.belusca-maito@reactos.org>
Co-authored-by: Serge Gautherie <32623169+SergeGautherie@users.noreply.github.com>

[SHELL32] Fix French message for "Overwrite Folder" confirmation message

[SHELL32] Fix French message for "Overwrite Folder" confirmation message

[INF] Update Russian (ru-RU) translation

Addendum to a03ef85b.

[HOTPLUG] Update Russian (ru-RU) translation

Addendum to 21824a25.

[INF] Add description for IDE-compatible controllers (#4690)

Add missing string. CORE-17256

[HOTPLUG] Add missing string resources for applet (#4689)

Use `IDS_CPLNAME` for the window title.
This also fixes the missing title text on the taskbar.

[MMSYS] Multimedia Control Panel diverse fixes (#4572)

- Use Unicode (WCHAR) instead of TCHAR
- Code formatting
- Use string safe functions
- Close handles after calling `CreateProcess`
- Save sound path as `REG_EXPAND_SZ` only if the path
  contains '%' character, like Windows does
- Fix `wcsdup` leaks

Reviewed-by: Mark Jansen <mark.jansen@reactos.org>
Reviewed-by: Hermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>
Reviewed-by: Katayama Hirofumi MZ <katayama.hirofumi.mz@gmail.com>
Reviewed-by: Stanislav Motylkov <x86corez@gmail.com>

[INPUT] Fix taskbar icon and remove small icon hack (#4688)

- Use int for applet index (like other CPLs).
- Revert some changes from 976d48b.

[LOGONUI] Make Logon UI compilable again (#4684)

Logon UI was excluded from CMake build, probably for not being compliant
with 'warnings are errors' standard introduced at some point.
In the future, I would probably try to make Logon UI usable
alongside with the classic msgina interface.

- Add logonui back to CMakeLists
- Replace 'wcscpy_s' Vista+ function with StringCchCopyW
- Exclude some unused functions

Signed-off-by: Vitaly Orekhov <crce2000@mail.ru>
Reviewed-by: Stanislav Motylkov <x86corez@gmail.com>
Reviewed-by: Hermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>

[REGEDIT] Fix possible buffer overflow

[CPL][INPUT] s/IDI_KEY_SHORT_ICO/IDI_CPLSYSTEM/ and small icon

The task icon of Taskbar was wrong...

[USER32] Load keyboard layouts on logon correctly (#4686)

- Add IntLoadPreloadKeyboardLayouts helper function to load the layouts on log-on.
- In UpdatePerUserSystemParameters function, call CliImmInitializeHotKeys and IntLoadPreloadKeyboardLayouts functions.
CORE-16600

[ATL] Fix window position to fit in screen area (#4512)

Clicking "Edit compatibility modes" button in the "Compatibility" tab
opens a window centered on the parent window. If we move the parent
window to one of the screen edges and then click this button again,
the new window will appear off screen.

Adjust position of created window, so now it would be completely visible.

CORE-17089

Reviewed-by: Mark Jansen <mark.jansen@reactos.org>
Reviewed-by: Hermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>

[SYSSETUP] Update German (de-DE) translation (#4679)

- Fix typos in existing translation
- Update new strings that were added during 0.4.15-dev'ing

Addendum to 3f053db3 and 6d09885c. CORE-18357

[DOC] 3rd Party Files.txt: Fix Buslogic driver version (#4685)

Correct version is in the BusLogic958.rc file.
Addendum to 5ee633f7d7e (0.4.15-dev-4556).

CORE-18180

[DOC] 3rd Party Files.txt: Add 3rdparty/ to some sdk/lib/ paths (#4642)

[BOOTDATA] Fix 'Layout Id' values of hivesys.inf

Fix them to Win2k3's settings.
CORE-11700

[NTUSER] Fix UserSetDefaultInputLang by using reordering

- Move IntReorderKeyboardLayouts function.
- Fix UserSetDefaultInputLang function.
CORE-11700

[NTFS] PrintAllVCNs(): Fix a typo in an ASSERT() (#4682)

It was introduced in 0.4.7-dev-590-g 5579428b4f1.

[SHELL32] Only hide a window when there is a window to hide

[SHELL32] Properly destroy window

[SHELL32] Don't try to remove buttons on a nonexisting window

[TASKMGR] Pixel-perfection for left border (#4680)

CORE-18061

This makes the left border 2 pixels smaller in
IDD_APPLICATION_PAGE and IDD_PROCESS_PAGE.

The goal is:
- matching XPSP3 taskmgr to 100% on left border.
- leaving more room for the content of the 2 SysListView32.
- better aligning left border of  IDD_APPLICATION_PAGE
  and IDD_PROCESS_PAGE to IDD_PERFORMANCE_PAGE
  upon tab-switching.

In addition: Update german (de-DE) translation:
Make wording of IDS_MSG_TRAYICONCPUUSAGE (used in tooltip)
match the wording used in IDS_STATUS_CPUUSAGE (used in status bar).

[NTOS:MM] Implement PeakCommitment (MmPeakCommitment, MmTotalCommittedPages) (#4650)

And return the corresponding values in SystemPerformanceInformation.
Lockless updating counters suggestion by Thomas Faber.

[SHELL32] Fill link path before initializing the dialog

This fixes the Target type, Target location and Target fields
in shortcut properties window.

CORE-15229

[SYSSETUP] Add Tango Desktop Project to the list of used projects (#4676)

[MMSYS] Add proper description to Speaker Volume dialog

Spotted by Kyle Katarn.

[CMAKE] Set KDBG define only where required (fastfat, win32k, ntoskrnl)

[CMAKE] Set _WINKD_ define only where required (kernel and setuplib)

[FLOPPY] Correctly return media changed information

If disk changed since last time, we must either return STATUS_IO_DEVICE_ERROR
or STATUS_VERIFY_REQUIRED, depending of VPB_MOUNTED flag.
This is already handled by the SignalMediaChanged() function.

CORE-18244

[MSPAINT][EXPLORER][SHELL32] Enable ATL asserts in CMake

CORE-17505

In MSVC builds they would not work reliably when enabled in the precompiled header

[SHELL32] Fix Assertion when opening "File" menu in MyComputer twice (#4675)

ReactOS explorer shell asserts when opening the "File" menu in MyComputer for the second time.
This is due to the cached copy of IContextMenu not being released before attempting to cache it again.

Fix based on the patch proposed by JIRA user I_Kill_Bugs

CORE-18353

Signed-off by: I_Kill_Bugs (original patch author)

[NETSHELL] Fix Network Tray icon activity indication (#4669)

The network activity tray icon wasn't showing activity until the Properties
dialog was opened at least once.
This was due to the fact that the tray icon was initialized once with missing
initialization parameters for pContext (default state and Adapter index).
Fix this problem by adding the missing initialization.

CORE-4497, CORE-9017, CORE-15744

[ZIPFLDR] Show icon on the taskbar (#4670)

[EXPLORER] Fix Start Menu context menu actions (#4643)

CORE-18336

The current design was not processing actions verbs correctly for some
Start Menu context menu actions (Properties, Open all users, Explore all users),
despite associated code being implemented.
This was due by incorrectly filtering command IDs, not routing to the appropriate processing.

[NTUSER] Implement IntReorderKeyboardLayouts

This function is provided for KLF_REORDER flag.
CORE-11700

[IMM32] s/Ime32LoadImeDpi/Imm32LoadImeDpi/

Typo. CORE-11700

[IMM32] Remove unnecessary NULL checks for function pointers

CORE-11700

[IMM32] s/Imm32Is16BitMode/IS_16BIT_MODE/

CORE-11700

[IMM32] s/Imm32IsCiceroMode/IS_CICERO_MODE/

CORE-11700

[DESKADP] Add some spacing between the "legend" and the data (#4668)

Follow up of PR #4665 (cb6b07ee).

Reviewed-by: Stanislav Motylkov <x86corez@gmail.com>

[BOOTDATA] Delete "Layout Id" from Japanese/Korean Keyboard Layout

CORE-11700

[STOBJECT] Add support for the mouse key tray icon

[PSDK] Add missing defines / argument names

[PSDK] Add COWAIT_DEFAULT

[STOBJECT] Keep the object alive while the thread is running,

Fix shutting down the thread.

[KERNEL32] Fix BaseDefaultPath containing extra '.'

[NOTEPAD] Follow-up of 'CJK font workaround'

- Revert some code.
- Add font name comments.
CORE-11700

[NOTEPAD] CJK font workaround

Far East Asian users may not have suitable fixed-pitch fonts.
CORE-11700

[NOTEPAD] Use DEFAULT_CHARSET instead of ANSI_CHARSET

Fix the font problem.
CORE-11700

[NOTEPAD] Add STRING_DEFAULTFONT for default font

CORE-11700

[AMSTREAM_WINETEST] Add missing dependency

[NOTEPAD] Improve UTF-8 encoding support and improve status bar (#4649)

- Add write support for UTF-8 without BOM.
- Add "UTF-8 with BOM" encoding to the "Save As" dialog.
- Show line endings and encoding on the status bar (like Notepad in Windows 10).
- Remove ignored WS_EX_STATICEDGE

- Add grip to the status window.
Co-authored-by: Hermès BÉLUSCA - MAÏTO <hermes.belusca-maito@reactos.org>

[DESKADP] Add Russian (ru-RU) translation. Addendum to cb6b07ee.

[DESKADP] Improve default display of advanced adapter properties (#4665)

+ fix caption width in french resources.

[TRANSLATION] Croatian translation of games (Solitare and Spider) (#4664)

[NEWDEV] Improve Japanese translation (ja-JP)

[SYSSETUP] Improve Japanese translation (ja-JP)

[NTUSER][USER32][IMM32] Initialize pKL->piiex by using ImmLoadLayout (#4645)

To recognize IME, we have to initialize pKL->piiex.
- Add co_ClientImmLoadLayout and User32CallImmLoadLayoutFromKernel functions to call imm32!ImmLoadLayout on user mode from kernel.
- Use co_ClientImmLoadLayout in NtUserLoadKeyboardLayoutEx.
- Improve Imm32LoadIME to sanitize the IME table.
CORE-11700

[TASKMGR] Fix number formatting to respect NLS Number format settings (#4657)

TaskMgr currently does not respect local NLS settings and forces 3 digits + comma (US convention) formatting.
Fix this with a direct reuse of SH_FormatInteger() as already implemented in other places
(dll/win32/shell32/dialogs/filedefext.cpp ...)

- Removal of CommaSeparateNumberString() and replace with SH_FormatInteger().
- PerfDataGetText(): Use a switch structure.  Return TRUE iftext could be retrieved; FALSE if not.

Co-authored-by: Hermès BÉLUSCA - MAÏTO <hermes.belusca-maito@reactos.org>
Co-authored-by: Katayama Hirofumi MZ <katayama.hirofumi.mz@gmail.com>

[CODEOWNERS] Remove automatic assignment from cmake/apitests

[NETSHELL] Fix "ghost" network activity when opening Network Status page (#4662)

Opening Network Status page creates a systematic "ghost" network activity,
activating RX+TX activity icon both on property page and tray, while no Rx/Tx.
This is due to pContext->dw[In|Out]Octets being initialized to 0 and compared
to refreshed interface data.

To circumvent this, copy refreshed interface data to the context on first update.

[THEMES] Improve text colors of Modern Dark theme (#4655)

- Update colors for all different font sizes.
- Update caption color for other fonts too.

[TRANSLATION] French translation update (#4638)

Update of various French translation files for several modules:
CALC, LABEL, DXDIAG, MPLAY32, RAPPS, SHUTDOWN, SNDVOL32, EXPLORER, WINLOGON;
CONSOLE.CPL, MMSYS.CPL; NETCFGX.DLL, SHELL32.DLL, SHLWAPI.DLL, SYSSETUP.DLL

[NTOS:EX] Fix returned number of handles for Idle System Process (#4661)

PsIdleProcess and PsInitialSystemProcess share the same handle table. This
leads ObGetProcessHandleCount() to report the same number of handles
when called on those system processes, when being enumerated by
NtQuerySystemInformation(SystemProcessInformation).

Instead, just return 0 for the handle count of the Idle process in SystemProcessInformation.
This is not done in ObGetProcessHandleCount(), since a separate
NtQueryInformationProcess(ProcessHandleCount) for the idle process should return
a non-zero value.

CORE-16577

[TASKMGR] Fix overlapping controls (French only) (#4658)

Controls are sized with 0 pixel gap, which is OK in general but show an overlap upon selection.
Add the extra 1 pixel gap.

[TASKMGR] Fix UI on SwitchTo for multiple application (#4652)

[CONSOLE.CPL] Use translated resource rather than hardcoded text (#4647)

[USRMGR] Replace old (leftover) debug MessageBox with Wine trace macros (#4640)

Remove leftover of debug code. CORE-18276

[KERNEL32] Fix kernel32.rc for UTF-8 codepage (#4648)

- Move #pragma code_page(65001) line.
CORE-18177

[NTUSER] Move TL structure definition to ntuser.h (#4644)

- Move struct _TL and TL definitions from win32.h to ntuser.h.
- Modify the type of TL.pfnFree as newly-defined TL_FN_FREE function pointer.
CORE-11700