[NTOS:CM] Cleanup the hive in case linking it to master fails (#4969)

Currently the failure code path doesn't do any kind of cleanup against
the hive that was being linked to master. The cleanup is pretty
straightforward as you just simply close the hive file handles and free
the registry kernel structures.

CORE-5772
CORE-17263
CORE-13559

🎊 🍾 🥳 Happy New Year 2023 to the ReactOS Community! 🎆 ⚛️ ☢️

.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:
 __,  ,__)            __,  ,__)       __, ,__)        ░▄████▄░▄███▄░▄████▄░▄█▀▀█▄░
(--|__| _ ,_ ,_      (--|\ | _       (--\ | _  _ ,_   ░▀▀░▄██░██░██░▀▀░▄██░░░░▄█▀░
  _|  |(_||_)|_)(_|    _| \|(/_(_|_)     \|(/_(_||    ░░▄██▀░░██░██░░▄██▀░░▄░▀▀█▄░
 (        |  |  ,_|   (                (__|           ░██████░▀███▀░██████░▀█▄▄█▀░
.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:

[NTOBJSHEX] Return early from GetInfoFromPidl when pcidl is null in LPCITEMIDLIST (#4946)

This fixes a crash when attempting to drag-n-drop items from NTObject Namespace and System Registry.
CORE-18480, CORE-18481

[MSPAINT] Don't allow to set image as wallpaper if it has the wrong format (#4924)

CORE-18661

Our Paint allows user to try to set a .ico file as a wallpaper, which isn't possible. Different Windows versions have different behaviour, so it was decided that the simplest fix would be to just grey out "Set as wallpaper" buttons as in 2K3 (See the Jira ticket).

[SDK][CMLIB] Properly check for failure if hive free list creation fails

HvpCreateHiveFreeCellList returns a NTSTATUS code yet the way the code path checks
checks for failure is just wrong. This was spotted whilst working on #4571 PR.

[WIN32NT_APITEST] Fix entirely broken win32knt_apitest NtGdiDdQueryDirectDrawObject

[DBGHELP_APITEST] Comment out rsym test for anything other than x86 for now

[GDI32_APITEST] Fix test results for SetWorldTransform on Windows 2003 x64

[GDI32_APITEST] Fix test results for CombineTransform on Windows 2003 x64

[XMLLITE_WINETEST] Fix stack corruption on 64 bit

[WS2_32_WINETEST] Fix use of uninitialized variable

[NTDLL_APITEST] Fix LdrEnumResources test for x64

[NDK] Fix definition of RTL_DEBUG_INFORMATION

Fixes ntdll_aiptest RtlDebugInformation on x64

[NDK] Fix definition of LDR_ENUM_RESOURCE_INFO

This fixes ntdll_apitest LdrEnumResources on x64 Windows

[LOG2LINES] Compile the tool for MSVC builds

[NTOS] Set SwapBusy properly for i386

Co-authored-by: Hermès BÉLUSCA - MAÏTO <hermes.belusca-maito@reactos.org>

[NTOS] Increment ActiveProcessors accurately

[NTOS] fix timer lock data and hardcod

[NTOS] Swap MAXIMUM_PROCESSORS with NUMBER_POOL_LOOKASIDE_LISTS

[INF] More deduplication and style harmonization (#4952)

- deduplication of manufacturer strings
- at some places harmonizes the different length of separator lines within the same file, centers the words (as requested by hbelusca during review), harmonizes those lines to 74characters length each
- moving some strings that are not to be localized into the non-localization string section
- other minor formatting preferences coauthored by StasM

[SYSSETUP] Fix *.rc combobox for the timezone is too small CORE-18587

Use the same width for COMBOBOX IDC_TIMEZONELIST that we use in timedate.cpl to
prevent strings from getting cut off for some timezones with long names.
To still make it look appealing afterwards, enlarge and move a bit to the right also IDC_TIMEPICKER.

IDC_AUTODAYLIGHT used different widths in several translations, we can harmonize
for all languages to the biggest one that was used: 230 (taken from eu-ES.rc).

And also fix a few additional whitespace-glitches in zh-CN, zh-HK, zh-TW and en-GB.

After this commit the whole dialog IDD_DATETIMEPAGE uses the exact same dimension for
all of its controls for all languages.

[TIMEDATE] Limit the expanded combobox height by using CBS_NOINTEGRALHEIGHT CORE-18665

It is good practice to not make the expanded combobox larger than the dialog itself.
CBS_NOINTEGRALHEIGHT will allow to set a fixed amount of pixels for the v6 combobox,
and therefore allows us to hide the problem for this specific dialog.

The problem was unhidden by 0.4.12-dev-882-g e3e173ffaafbccb0e9b52926d41626d5a4dc38cd

Even after this patch I think Wines comctl32 can be improved for comboboxes with very
many elements that do not use this small trick yet. I will create a follow-up-ticket.

[REG] Fix mis-converted Yes-No-All keypress shortcuts strings.

The PO \"#msgctx\" control strings were not stripped out.

[NTOS:CM] Enable verbose debug output of registry lazy flushing component

Log debug output of the lazy flusher as much information as possible so that we can examine how does the lazy flusher behave, since it didn't work for almost a decade. Verbose debugging will be disabled once we're confident enough the registry implementation in ReactOS is rock solid.

[NTOS:CM] Don't lazy flush the registry during unlocking operation

Whenever ReactOS finishes its operations onto the registry and unlocks it, a lazy flush is invoked to do an eventual flushing of the registry to the backing storage of the system. Except that... lazy flushing never comes into place.

This is because whenever CmpLazyFlush is called that sets up a timer which needs to expire in order to trigger the lazy flusher engine worker. However, registry locking/unlocking is a frequent occurrence, mainly when on desktop. Therefore as a matter of fact, CmpLazyFlush keeps removing and inserting the timer and the lazy flusher will never kick in that way.

Ironically the lazy flusher actually does the flushing when on USETUP installation phase because during text-mode setup installation in ReactOS the frequency of registry operations is actually less so the timer has the opportunity to expire and fire up the flusher.

In addition to that, we must queue a lazy flush when marking cells as dirty because such dirty data has to be flushed down to the media storage of the system. Of course, the real place where lazy flushing operation is done should be in a subset helper like HvMarkDirty that marks parts of a hive as dirty but since we do not have that, we'll be lazy flushing the registry during cells dirty marking instead for now.

CORE-18303

Translate favorites menu

Improve translation for Favorites menu

[TRANSLATION] Don't translate Documents and Settings folder

[SHELL32][USER32] Improve pt-PT translation

[TRANSLATION] Hungarian translation update for sysdm

[DXDIAG] Fix Portuguese (pt-BR) translation (#4953)

Fix minor translation error.

[NTUSER] Fix zombie window created by CTRL+ALT+DEL (#4935)

Based on patch by I_Kill_Bugs. CORE-18258
See also commit 58b0558f9.

[WIN32K:NTGDI] PATH_PathToRegion(): Update documentation (#4941)

Addendum to 215e0d9 (r64177).

[NTOBJSHEX] Fix displayed registry value type and icon (#4949)

- RegistryTypeNames[]: Remove an unwanted space

- CRegistryFolderExtractIcon::GetIconLocation():
  Add the REG_ENTRY_VALUE_WITH_CONTENT case

- CRegistryFolder::GetDetailsOf(): Fix copypasta

- CRegistryFolder::FormatValueData(): Fix magic constant

CORE-18747

[NTOBJSHEX] Fix sorting elements by different columns (#4947)

- Add ability to sort NT objects by symlinks
- Add ability to sort registry entries by type/value

CORE-18761 CORE-18762

[SHELL32] Read the label for a CD from autorun.inf (#4945)

Some CDs may set a custom label using autorun.inf.
Read the label from the file, and if it succeeds, show it to the user.

CORE-18567

[NTOBJSHEX] Formatting only

[SHELL32] Fix detailed list views in Control Panel and Registry Folder

- Add enums for Control Panel and Registry Folder columns
- Fix iColumn values in GetDetailsOf() and fix checks in
  GetDefaultColumnState()

Current CControlPanelFolder::CompareIDs was using wrong lparam column
index. Actual column index should be 1 instead of 4. Because of this
the comment column next to name in details view was not being displayed
correctly. The same fixes for CRegFolder class functions.

PR #4944. CORE-18743 CORE-18501

[RPCRT4_WINETEST] Remove obsolete todo_wine

Addendum to 00ccbd2f

[RPCRT4] RpcStringBindingParseA/W must fail, if a colon is missing in the string binding

This fixes a bug in the rpc winetest.
This patch will be sent upstream.

[RPCRT4] Call WSAStartup in rpcrt4_ip_tcp_get_top_of_tower

This fixes the crash in the rpc winetest.

[RPCRT4] RpcStringFreeA/W must set the pointer to NULL

Add a matching wine test.

This change will be sent upstream.

[DESK] Update german translation de-DE.rc (#4943)

[NTOS:IO] Replace an outdated E-Mail Address

The old Address will be gone soon.

[SYSSETUP] Fix missing comma in german translation de-DE.rc

The old string lacked a *mandatory* comma before the word "übereinstimmen"
as the part "die Sie nutzen wollen" is a relative clause,
relative to the noun "Anwendungen".
But we can fix it without having to add the comma by some shuffling.
That also improves the reading flow.

[WINLOGON] Update german translation de-DE.rc

Addendum to 0.4.15-dev-2358-g 5f033392397a101b6a9113b2d0de5c09e0a71737
to avoid regressing the german translation compared to releases/0.4.14

[WIN32K] Workaround for broken/retarded (on non-x86) FLOATOBJ_GetFloat

The official definition in winddi.h always returns the result in an ULONG. This works to assign to a FLOATL on x86, which is also an ULONG, but it doesn't work on any other architecture, where FLOATL is actually a FLOAT, so the assignment would convert the ULONG encoding of the FLOAT to an actual float, which gives broken results. Work around this by simply replacing the definition with a sane one in win32k.

[WIN32K] Fix memory leak on failure in REGION_SetPolyPolygonRgn

[WIN32K] Fix a bug in PATH_PathToRegion

[WIN32K] Fix missing NULL check in NtGdiPathToRegion

[WIN32K] Fix uninitialized variable in IntExtTextOutW

[NTOS] Fix saving of XMM registers on some trap handlers

[KMTEST] Add a NULL check

[KMTEST] Handle failure in KmtLoadDriver and KmtOpenDriver

[NTOS:IO] Bring back the NDEBUG definition

Addendum to 947f60b20716e1581b0785360d2d71363eff3657

[NTOS:IO] Allow REG_SZ type for ImagePath of a driver

Fixes the load of the Sysinternals FileMon driver.

CORE-18725

[BTRFS] btrfs.inf: Comment out mkbtrfs.exe (#4937)

It is not imported into ReactOS.

Addendum to 0.4.13-dev-788-g c8875c182ac638a9aed627ecaf39e30ab22daf40 and 0.4.14-dev-1535-g 194ea909fdff60294ce2d5434cbded63f5377754

['OLD'_WORDPAD] General resources in rsrc.rc are language-neutral.

[ROSAPPS] Exclude the templates from compilation by default.

[MDI] Template MDI app is Win32 GUI, not CUI.

[CONSOLE] Improve font preview and Update de-DE.rc (#4933)

An addendum to bc1e2dbb92340b89ed043545ff761f75ea013301 == SVN r74465
from CORE13182 (I intentionally did not want JIRA to auto-link)

[SHELL32] Improve "Empty Recycle Bin" sound code (#4927)

Split off from PR #4755. This change doesn't require WINMM fixes (#4635).

- Remove `MAX_PATH` limit
- Use ATL classes
- Play the sound asynchronously

[SHELL32] Fix My Documents properties (#4906)

Tell the caller to run default action by returning S_FALSE in else case.

CORE-17509

[NTOS:IO] Properly zero-initialize a file object created by IopParseDevice (#4931)

Fix uninitialized kernel memory leakage for a case when a file object extension is appended.

CORE-18711

[MSPAINT] Fix manifest for mspaint.

[SYSDM] System Parameters: Add close button (#4928)

Follow-up of PR #4891 (commit f125cd1c7).

[KDGDB] Fix compilation following commit de81021ba (#4930)

Use cast to be compatible with "public" structure definition.

[BOOTDATA][INF] Add Japanese (ja-JP) translation to timezone

CORE-18706

[DESK] Fix screensaver preview after calling its settings (#4921)

Fix a bug when after closing the settings window for a screensaver,
the preview for it wouldn't restart.

- Keep waiting for messages from the settings window while it's open
- Fix leaking two handles to avoid creating zombie processes

CORE-18680

[MMSYS] Prevent the same sound from being added multiple times (#4898)

Check if the list already contains the sound.

[NTGDI][FREETYPE] Follow-up of #4920 (e316d61); Rename functions

Follow the naming policy.

- ftGdiGetRealGlyph --> IntGetRealGlyph
- ftGdiGlyphCacheGet --> IntFindGlyphCache
- ftGdiGlyphSet --> IntGetBitmapGlyphNoCache
- ftGdiGlyphCacheSet --> IntGetBitmapGlyphWithCache
- FtMatrixFromMx --> IntMatrixFromMx

CORE-11848

[OSK] Delete WS_EX_NOACTIVATE workaround code

CORE-18528

[NTUSER] Fix SetActiveWindow return value

Apply set-active.patch. CORE-13257

[SENDMAIL] Fix Send To -> Desktop (create shortcut) behavior (#4913)

Several fixes and improvements to the CDeskLinkDropHandler:
- Set default working directory for shortcuts (except folders and zip files)
- Copy existing shortcut to the desktop if the source file is a shortcut
- Prevent destination file name collision

Verified on Windows XP SP3 and Windows 7 SP1.

[EXPLORER] Fix Windows XP compatibility (#4923)

CORE-18694
It regressed by 0.4.15-dev-4596-g 1aa95f389772901970369afa3abfb207df6f57f6

[NTGDI][FREETYPE] lfEscapement for TextOut (#4920)

Rotation and shearing transformation of font/text is now available. Retrial of PR #1207.
- Rename ftGdiGetTextWidth as IntGetTextDisposition and add a Y parameter.
- Apply lfEscapement values (by multiplying matrices).
- Add IntEngFillPolygon and IntEngFillBox helper functions.
CORE-11848

[NTOS] Fix double free on allocation failure in ObCreateObject

[SPOOLSS] Fix prototype of BuildOtherNamesFromMachineName

[NTOS] Improve MmDeleteProcessAddressSpace

[NTOS] Improve MmCleanProcessAddressSpace

[WIN32K] Fix missing NULL check in IntCreateDIBitmap

[NTOS:EX] Initialize ExpTimeRefreshLock

[KERNEL32] Fix a bug in GetVolumeNameForRoot

[WS2_32_APITEST] Fix uninitialized variable

[CRT_APITEST] Restore direction flag to avoid false RTC break

With set direction flag the variables will not be initialized properly.

[CRT] Add simplistic fallback implementation of sincos to make GCC 11 happy

[CRT] Replace amd64 math functions/stubs with libm

[FREELDR] Add stubs for pow and log to avoid pulling in the data from CRT (maybe fix printf functions instead?)

[CRT] Add basic version of handle_error

[CRT] Improve _matherr handling

[CRT] Implement _set_statfp

[ASMPP] Implement asm preprocessor

This converts ML style assembly to GAS compatible syntax

[LIBM] Fix ldexp.c / ldexpf.c

[LIBM] Add ldexp.c and ldexpf.c from aocl-libm-ose

[LIBM] Fix up some asm files

This allows to compile them with GAS after translation

[LIBM] Convert tables from asm to C

[LIBM] Fix build

[LIBM] Import win-libm from AMD

Source: https://github.com/amd/win-libm

[XDK] Silence some GCC x64 warnings

[CRT] Add a workaround for CORE-18255

[NTGDI][FREETYPE] lfWidth for GetTextMetrics and GetCharWidth (#4919)

- Add lfWidth member into FONTGDI structure.
- Delete IntWidthMatrix function.
- Fix IntRequestFontSize behavior for processing lfWidth value.
- Apply lfWidth in GetTextMetrics and GetCharWidth.
- Ignore the XFORM values in GetTextMetrics and GetCharWidth.
CORE-11848