[NTOS:PNP] Fix resource conflict detection

Previous code did not detect equal resource ranges as conflicting.
Thanks Hervé Poussineau for pointing this out!

Meanwhile, simplify the code to make it more readable.

[NOTEPAD] Fix access key for "New Window" menu item (#4024)

[IMM32] Move Imm(Get|Set)CompositionStringA/W (#4021)

- Add new file compstr.c.
- Move ImmGetCompositionStringA, ImmGetCompositionStringW, ImmSetCompositionStringA, and ImmSetCompositionStringW functions into compstr.c.
- Half-implement ImmGetCompositionStringA/W.
CORE-11700

[SETUPLIB][INPUT.CPL] Move zh-MO to ChineseTraditionalFonts group (#4010)

- The use of Traditional Chinese character is far more than Simplified one according to Wikipedia
- The code page used on zh-MO is 950, which is Traditional Chinese's one
- zh-MO is described as Traditional Chinese here: https://support.microsoft.com/en-us/windows/microsoft-traditional-chinese-ime-ef596ca5-aff7-4272-b34b-0ac7c2631a38#ID0EBBD=Microsoft_ChangJie/Quick

[CMAKE] msvc.cmake: Add arm64 CMAKE_ASM_MASM_COMPILER option support

CORE-17518

[CMAKE] CMakeLists.txt: Add arm64 to ARCH allowed values

CORE-17518

[SHELL32] Don't smartass-renaming on LNK files (#4020)

Renaming .LNK files had a bug that the selection won't include the extension-like filename part.
CORE-17807

[SDK] config.cmake: Fix generic mtune flag (#4017)

[IERNONCE] Add German translation

[INETCPL] Update German translation

[NOTEPAD] Properly fix accelerator issue in es-ES.rc CORE-17809

In 0.4.14-dev-1321-g d6e0fe78e741fa8af9ec80b7ef70853fda14b8d6
we introduced the following compiler warning for MSVC2010 release build:

reactos/base/applications/notepad/lang/es-ES.rc(12) : warning RC4204 : ASCII character not equivalent to virtual key code

That warning was muted without investing too much love by
0.4.15-dev-2469-g d0bb7757747cb6d2c500e687df78eb8afea8edc8

Today we apply the proper fix that brings us back in sync with other languages.
I do intend to port the proper fix back to 0.4.14RC as well.

[SHELL32] Add missing SS_NOPREFIX to labels in About dialog (#3969)

Signed-off-by: William Kent <wjk011@gmail.com>

[MODEMUI] Fix unrecognized escape sequence. Addendum to 05c39d8d.

[TRANSLATION] Update Turkish (tr-TR) translations (#3958)

Reviewed-by: Can Taşan <ctasan99@hotmail.com>
Signed-off-by: Süleyman Poyraz <zaryob.dev@gmail.com>

[TRANSLATION] Chinese Traditional (zh-TW) translation update (#3954)

Also sync LOCALMON/UI translation with Wine.

[RICHED20_WINETEST] Fix build about __thiscall redefinition

See https://github.com/reactos/reactos/commit/5a9535e694b6baaa4b028523263a39357d6d589b

[RICHED20_WINETEST] Restore Wine sync in txtsrv.c. ROSTESTS-355

In particular, add the missed diff in ITextHostImpl_TxGetCharFormat.

[BOOTDATA] readme.txt: Trivial correction (#4014)

Follow up of #3469 - replace "Seven" with "7" to keep Microsoft's official naming.

[ZIPFLDR] Update de-DE.rc

[IMM32] Rewrite ImmRequestMessageA/W (#4002)

- Re-implement ImmRequestMessageA and ImmRequestMessageW functions.
- Add IchWideFromAnsi, IchAnsiFromWide, Imm32RequestError, Imm32ReconvertSize, Imm32ConvertReconvert, and Imm32ProcessRequest helper functions.
CORE-11700

[ZIPFLDR] Update Polish (pl-PL) translation (#4007)

Revert "[DSOUND] Don't force status to failure by default in primarybuffer_SetFormat (#3911)"

Revert my last dsound fix, because it does not fix the actual problem with audio formats support. It only makes the situation worse. It prevents the error when creating an audio stream, but the sound is not playing correctly, both in ReactOS and Windows (with our dsound.dll). Without this fix, it works perfectly on Windows, but does not in ROS.
In particular, the sound is not playing properly in AIMP 4.71 with other audio formats besides default (44100 Hz, 16 bit, stereo). And any other apps which request these audio formats may be also affected as well.
Force the status back to failure, because otherwise the badly required code is not executed at all.
I've made a more correct fix(es) in other audio component(s) instead and will submit them if further PRs.

This reverts commit 183566ee584efbe6377c79e68bfe24b2d1f940ae.

CORE-10907

[TSCLIENT] Fix code F12 array index (#4006)

[NTOS:KE] Fix copy paste bug in exception helper routine for ARM (#4005)

[ZIPFLDR] Show error when compression method is not supported

CORE-17255

[SHELL32] Don't smartass-renaming on folders (#3993)

Selection on renaming folders shouldn't select the extension-like part of filename.
CORE-17793

[ZIPFLDR] Update de-DE.rc in context of CORE-17796

[ZIPFLDR] Improve error reporting

- Restore status text on failure
- Show errors on folder and file creation failures
- Allow to retry extraction attempt

CORE-17796

[ZIPFLDR] Refactor extraction function

Add separate function for a single file extraction.

CORE-17796

[IMM32] Implement ImmSystemHandler (#3998)

- Implement ImmSystemHandler function.
- Add Imm32UnknownProcess1 and Imm32SendChange helper functions.
CORE-11700

[IMM32] Add IMP stub functions (#3989)

- Add IMP (Input Method Profiler) stub functions (ImmIMPGetIMEA/W, ImmIMPQueryIMEA/W, ImmIMPSetIMEA/W).
- Add ImmSendIMEMessageExA/W stub functions.
- Rename nt3.c as win3.c.
The prototypes can be obtained from: https://doxygen.reactos.org/dc/d20/winnls32_8h.html .
The Japanese article of IMP is available: http://www7a.biglobe.ne.jp/~tsuneoka/win32tech/7.html
CORE-11700

[IMM32] s/4/sizeof(DWORD)/

CORE-11700

[RAPPS] Update German translation

[IMM32_APITEST] Add ImmGetImeInfoEx testcase (#3988)

CORE-11700

[IMM32] Implement ImmActivateLayout (#3987)

- Add IME_STATE and IME_SUBSTATE structures.
- Modify INPUTCONTEXTDX structure.
- Implement ImmActivateLayout function.
- Improve ImmDestroyContext function by using newly-defined Imm32FreeImeStates.
- Add Imm32FetchImeState, Imm32FetchImeSubState, Imm32LoadImeStateSentence, Imm32SaveImeStateSentence, and Imm32SelectLayout helper functions.
- Modify NtUserSetThreadLayoutHandles prototype.
CORE-11700

[UNIATA] Better split of model name between Vendor and Product

UNIATA is converting IDE identify data into SCSI identify data.
However, IDE model is described with a unique field of 20 chars,
while SCSI model is described with 2 fields of 16 + 8 chars.
When displaying SCSI model, a space is added between vendor and product
fields.

Try to split model into vendor and product on a space if possible.

CORE-17400

[RAPPS] Speed up app loading by caching the INI sections

[RAPPS] Cleanup

[SHELL32] Fix uninitialized variable usage.
This fixes log spam about NM_CUSTOMDRAW not being handled

[IMM32] GetKeyboardLayout's parameter is a thread id

s/dwLayout/dwThreadId/
CORE-11700

[IMM32] Move ImmFreeLayout from ime.c to imm.c

CORE-11700

[IMM32] Rewrite ImmGetImeMenuItemsA/W (#3983)

- Rewrite ImmGetImeMenuItemsA and ImmGetImeMenuItemsW functions.
- Add Imm32GetImeMenuItemsAW, Imm32IsImcAnsi, Imm32ImeMenuAnsiToWide and Imm32ImeMenuWideToAnsi helper functions.
- Make IME module's ImeGetImeMenuItems function optional.
CORE-11700

[IMM32] Rewrite ImmSetActiveContext (#3982)

- Re-implement ImmSetActiveContext function.
- Modify NtUserNotifyIMEStatus prototype.
- Improve ImmSetConversionStatus and ImmSetOpenStatus functions.
CORE-11700

Revert "Disable some C++ code, which doesn't compile for me"

This reverts commit d8b1ff9056d869d512da88b9c9cc9301535f5a06.

Disable some C++ code, which doesn't compile for me

[VIDEOPRT] Create and setup new registry key only for non-legacy drivers

CORE-17734

[IERNONCE] [RUNONCEEX] Add RunOnceEx functionality for ReactOS (#3926)

* [IERNONCE] Implement the registry management code.

* [EXPLORER] handle RunOnceEx by invoking RunOnceEx in iernonce.dll

* [IERNONCE] Display a dialog to show progress, and execute entries.

* [IERNONCE] Add `InitCallback` function

[NtGDI] Fix brush attribute transfer.

[IMM32] Rewrite ImmAssociateContext (#3979)

CORE-11700

[IMM32] Implement ImmFreeLayout (#3978)

- Implement ImmFreeLayout function.
- Add Imm32ReleaseIME helper function.
CORE-11700

[ApiTests|GDI] Add test for attribute changes to Bitmaps.

Only opporpate place to add this test.

[NtGDI] Set owner after changing object type.

[NtGDI] Fix leaking of bitmaps while testing INFO DC's.

Soon to be commit test for metafiles.

ret = MaskBlt(hdc, 0, 0, 3, 4, dib_hdc, 0, 0, mask_bitmap, 0, 0, SRCCOPY);

[GDI32] Fix IV msvc build.

[GDI32] Fix III msvc build.

[GDI32] Fix build errors~~

[GDI32] Fix msvc build errors.

[GDI32] Update Wine Metafile Code

Sync/Port: Metafile code from wine.

Patches by Jacek Caban, Daniel Lehman, Zhiyi Zhang. Gabriel Ivancescu, Michael Stefaniuc, Francois Gouget, Nikolay Sivov Dmitry Timoshkov, Andrew EiKum, Piotr Caban and Alexandre Julliard.

This commit is dedicated to George Bisoc!

[NTOS:SE] Define a name macro for the threshold value

Instead of having this magic value as is, describe it within a macro for readability purposes.

[NTOS:SE] Check the privilege count against the maximum threshold

In SeCaptureLuidAndAttributesArray we must ensure that we don't go onto a potential integer overflow scenario by checking against the maximum limit threshold the kernel states. In addition, write an explicit name macro for the value.

[IMM32] Trivial tiny improvements

CORE-11700

[IMM32] Rewrite ImmLockIMC (#3975)

- Add Imm32InitContext and Imm32LockIMCEx helper functions.
- Re-implement ImmLockIMC function.
- Modify CLIENTIMC and INPUTCONTEXTDX structures.
CORE-11700

[IMM32] Improve Imm32CleanupContext and ImmLockClientImc (#3974)

- Improve Imm32CleanupContext and ImmLockClientImc functions.
- Add ValidateHandleNoErr and Imm32CheckImcProcess helper functions.
CORE-11700

[USER32] GetUserObjectSecurity/SetUserObjectSecurity: Just use UserSetLastNTError() in failure path.

[IMAGESOFT] Fix build. Addendum to commit 516c8829.

[NTOS:RAWFS] LE JOKE! - Commit 7716bddd (r24564) claimed to "actually create the \\Device names so that user-mode can even talk to it", yet didn't bother to do that!

Certainly due to copy-pasta error from the original code.

A consequence of this oversight, was that the IoGetDeviceObjectPointer()
calls on these device names, in fltmgr!DriverEntry() couldn't work.
(See drivers/filters/fltmgr/Interface.c, line 1880 and below.)

[NTOS:RAWFS] Delete the previously-created devices in case the IoCreateDevice() calls fail.

[FS_REC] Fix load-lock acquisition.

[FS_REC] Register a recognizer for FAT filesystem on CD-ROMs.

This handles the \FatCdrom device, supported by MS fastfat.

[IMAGESOFT] Reorganize the bitmap resource IDs.

- Moving the IDB_* with their corresponding TBICON_* defines.
- Reordering the BITMAP resources in the same order the defines are defined.
- Fixing values for IDS_READY and IDS_TOOLBAR_TEXT.

[IMAGESOFT] Use ARRAYSIZE / un-hardcode some sizeof(TYPE) where applicable.

[IMAGESOFT] Commit db416e39 made toolbar imagelists creating failing because we overcounted the number of toolbar button bitmaps.

Indeed the count was based on the number of elements in TBBUTTON arrays;
however for some of the toolbars, extra entries corresponding to
separators are present, and thus the resulting number of elements is
always larger than the number of actual buttons for which an image is
associated. Passing this (larger) count to the InitImageList() function
therefore made image insertion fail after a certain point.

Now we pass an exact number of *images* instead.

[Win32SS|GDI] Implement seldom used API and update types for local DC's.

Two working in house application require these API. Known as FolCOL and Reach.

[NTUSER] Fix UserSendKeyboardInput() KEYEVENTF_SCANCODE use case (CORE-17144)

[NTOS:OB] Minor refactoring.

- NtQuerySymbolicLinkObject(): Use an intermediate variable for the object header.
- Simplify code in ObpLookupEntryDirectory() by calling ObpReleaseLookupContextObject() instead.
- Use TAG_OBJECT_TYPE instead of hardcoded tag values.

[NTOS:OB] Clarify and fix the usage of the Obp*DirectoryLock*() and ObpReleaseLookupContextObject() functions.

- Disentangle the usage of ObpAcquireDirectoryLockExclusive() when it's
  used only for accessing a directory structure, or as part of a lookup
  operation.

  The Obp*DirectoryLock*() -- both shared and exclusive -- functions
  are only for locking an OB directory, for reading or writing its
  structure members.

  When performing lookup operations (insertions/deletions of entries
  within a directory), use a ObpAcquireLookupContextLock() function that
  exclusively locks the directory and saves extra lock state, that can
  be used by ObpReleaseLookupContextObject() for cleanup.

- Add documentation for these functions.

[NTOS:OB] Acquire the lock before setting directory's session ID

[NTOS:SE] Do not set SE_DACL_PRESENT flag that early

The function might assign the flag yet it could possibly fail on creating a DACL and insert an "access allowed" right to the access entry within the DACL. In this case, make sure we actually succeeded on all the tasks and THEN assign the flag that the DACL is truly present.

Also, make sure that the Current buffer size variable gets its new size so that we avoid overidding the memory of the DACL if the security descriptor wants both a DACL and SACL and so that happens that the DACL memory gets overwritten by the SACL.

[NTOS:SE] Set the SACL to the World security descriptor

Implement the portion chunk of code that is responsible for setting the system access control list (SACL) to the World security descriptor, based from SeWorldSid (World security identifier).

[IMM32] Use Imm32IsImmMode and Imm32IsCiceroMode macros (#3971)

- Define Imm32IsImmMode, Imm32IsCiceroMode and Imm32Is16BitMode macros in "precomp.h".
- Improve Imm32InquireIme and Imm32CleanupContext functions.
- Add IME_SYSINFO_WINLOGON and IME_SYSINFO_WOW16 macros (from DDK) in psdk/imm.h.
- Modify CtfImeInquireExW and CtfImeSelectEx prototype.
CORE-11700

[REACTOS] Remove PowerPC Open Firmware-related code

Was forgotten during the overall PowerPC support removal.
Addendum to 6ef6fabfc5271b9fb89cd9bdc2038588d55b3ea4

[IPHLPAPI] GetAdaptersInfo(): Update Wine registry key name (#3556)

[PSDK] wincodec.idl: Add GUIDs which exist on WDK 7.1 (#3628)

[NTDLL_APITEST] Implement NtAdjustGroupsToken API tests

[NTOS:SE] Implement token groups adjusting

[IMM32_APITEST] himc: Use ok rather than ok_int (#3970)

Use ok macro rather than ok_int macro. CORE-11700

[FONTEXT] Restore folder icon

[SHELL32] CFSFolder: Add fallback for failing shell extension

CORE-17673

[RAPPS] Move the CConfigParser to a new file

[NTOS:SE] Privileges adjusting private routine must be paged

SepAdjustPrivileges is a private helper call that must be paged, alongside with its syscall -- NtAdjustPrivilegesToken.

[IMM32_APITEST] Add himc testcase (#3965)

Add himc testcase. CORE-11700

[IMM32] Define ImeDpi_IsUnicode macro and use it (#3964)

- Define ImeDpi_IsUnicode macro in precomp.h.
- Use it.
CORE-11700

[NTUSER] Scrollbar.c, Avoid potential out-of-bounds-accesses in co_IntSetScrollInfo() CORE-17777

This is an addendum to
0.4.15-dev-3174-g dda9c3979e87223b66e61f0f936f46920221e509 CORE-17769 and
0.4.15-dev-3147-g 3bf7e3ac13c5c9df373827c102b763b5b9822204 CORE-17754 CORE-17755

We have not seen this happening in real-life yet, but some code-fragments within co_IntSetScrollInfo()
e.g. line 628 if (nBar == SB_CTL) do clearly indicate that nBar can be 2 (SB_CTL).
Some lines below we definitely must not access those 4 static arrays out of bounds then via nBar as access index!

Ftr with a bit of grepping I also found some calls like NtUserSetScrollInfo(Wnd, SB_CTL, &Info, FALSE);
e.g: in win32ss/user/user32/controls/scrollbar.c so I am pretty sure nBar == 2 can happen in practice within co_IntSetScrollInfo().

I question whether any of those reads/writes to those static arrays (or the comparisons) would make any sense on index 2,
so we should aim to eliminate them altogether in the future.

[NTUSER] Scrollbar.c, improve co_IntSetScrollInfo() CORE-17769

Fixes CORE-17769 'Rapps Listview manual resize may erroneously not draw the triangles sometimes'
This could happen for both: themed and unthemed.

CORE-17769 was a regression introduced by 0.4.14-dev-1134-g 00adb1a3f967ac7f5cd56b4c39df72a7b3814603

We don't really like the added state in form of the static variables,
but the patch works good from a pure testing-perspective.

Many Thanks to the patches author: Doug Lyons

[LIBXML2] Update to version 2.9.12. CORE-17766

[WIN32SS] Disable by default the dumping function. Addendum to 69f90508.

[WIN32SS:NTUSER] Move the keyboard layout debugging code to a separate dump function.

And fix its obvious bugs, e.g. not checking for possible NULL pointers
for the optional keyboard layout sub-tables...

[BOOTDATA] Fix "DllName" casing.

[BROWSEUI] CAutoComplete::OnAutoCompStart(): Wait on the thread only when there is a handle.

Avoids spamming
(ntoskrnl\ob\obwait.c:441) Failed to reference the handle with status 0xc0000008

CAutoComplete::FinishCompletion(): Remove two useless 'if (m_pThread || !m_hThread)'
tests: No need to check these after each code instruction...

[BROWSEUI] Fix early error handling, to be taken only when ParseNow() is run (and fixes "Use of uninitialized variable hr" RTC error).
Addendum to commit d8e47d61.