[NTOS/MM] Ensure ImageBase is not used uninitialized

[KERNEL32] Addendum to 0e3a0435 (PR #803): always return TRUE in the stub-plemented SetThreadStackGuarantee() to continue satisfying programs that use it. CORE-15989

[OSK] Make the buttons themed

Our On-Screen Keyboard has a manifest and the buttons (the ones with BS_ICON styles) aren't rendered with the specific theme as it should be but instead it takes the classic theme.

The code relies on NM_CUSTOMDRAW notification, which is more intuitive and efficient than doing owner-drawn operations as NM_CUSTOMDRAW allows the controls to use styles whereas you cannot do it on owner-drawn controls.

CORE-15965

[WIN32KNT_APITEST] Forgot a extension in CMakeLists.txt

[WIN32SS][NTGDI][GDI32] Fix DPtoLP and NtGdiTransformPoints etc. (#1540)

Fix NtGdiTransformPoints, gdi32!DPtoLP, gdi32!LPtoDP and gdi32!SetMapMode functions. Fix CORE-15983. CORE-15983

[APITESTS] set_module_type(win32u module)

[APITESTS] Add NtGdiTransformPoints testcase (#1542)

Add a testcase for NtGdiTransformPoints function. set_module_type(win32u... win32dll) CORE-15983

[KERNEL32_APITEST] Rename the 'Console.c' file into 'ConsoleCP.c' and its test-function as well, in order to reflect the actual name of the corresponding test.

Add a shared "port" directory for POSIX functions needed by multiple host tools (getopt/mkstemps) and import the one and only getopt from glibc.

This finally fixes our duplicated getopt functions from different sources and gives us an up to date and the most compatible implementation.
isohybrid actually relies on a glibc-specific getopt behavior that we previously hacked into the reactos_support_code.c implementation derived from BSD/mingw-w64.
widl also needs getopt and previously used an even older BSD-derived code.

[ISOHYBRID] Fix the remaining warnings properly.

I will send these changes upstream (minus the REACTOS_ISOHYBRID_EFI_MAC_SUPPORT ifdefs).

[TOOLS] Fix/suppress all MSVC/x64 warnings (#1525)

[CONSRV] Cache the console window system menu handle; simplify its initialization code.

[REACTOS] Replace remnant legacy SPIF_SENDWININICHANGE by SPIF_SENDCHANGE (#1526)

No functional change otherwise.

[ACCESS] Reposition the "ToggleKeys" group box control (#1541)

CORE-15383

[WIN32SS:NTUSER] Use the 2nd parameter of NtUserGetThreadDesktop() as fallback. (#1065)

- The 2nd parameter is already passed in user-mode by GetThreadDesktop().
  It is then used by NtUserGetThreadDesktop() as a fallback for console
  threads.

- Lookup and validate the thread by using the IntTID2PTI() helper.
- Don't reference the desktop with too many access rights.
- Get rid of the old-school DECLARE_RETURN() & co. macros.

Co-authored-by: Hermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>

[WIN32SS][NTGDI] Fix wrong IN/OUT (#1539)

[POWERCFG] Fix minor power scheme issues

- Really fix the exception that occurs on closing the property sheet.
- Use the correct scheme ID when the current power scheme is set.

[ROSAUTOTEST] "Enforce" ending '\n' on C___Exception messages (#1533)

[FORMATTING] appveyor.yml: Remove an extra space (#1538)

Addendum to 48f98e88.

[WINSRV] Implement SrvGetThreadConsoleDesktop(). CORE-13470

It retrieves the handle to the desktop assigned to the specified
thread belonging to a console application, which is going to be
where the input thread of this console is.

[COMDLG32] Follow up of #1514 CORE-7114 Don't load asterisk extension

[NFS] Fix a typo in the precompiled header.

[POWERCFG] Fix an exception that occurs when the control panel is closed.

[KERNEL32] Add an (incomplete) pre-implementation for SetThreadStackGuarantee(). (#803)

This should remove some of the "SetThreadStackGuarantee(): stub" messages in some simple cases.

[MODERN.MSSTYLES] Remove trailing whitespace.

[THEMES] Reorganize files for future compatability.

Moved current theme assets into their own dedicated
files in preparation for .theme support.

CORE-15970

[COMDLG32] Save open/save location by extension (#1514)

Use the proper open/save initial folder for comdlg32.GetOpenFileName and comdlg32.GetSaveFileName. CORE-7114

[SDK][INCLUDE] Use FALSE for BOOL in msgdump.h

[CMAKE] Fix LINK_DEPENDS usage

Up to CMake 3.3 LINK_DEPENDS was ignored in non-Makefile generators. See also http://www.cmake.org/pipermail/cmake/2010-May/037206.html. Since CMake 3.4 it is no longer ignored, but requires CMP0005 policy to be set to NEW and it requires a full path which was already correctly used on GCC, but not on MSVC builds.

[CMAKE] Use new policies for CMP0005 and CMP0018 (#1481)

* [CMAKE] Fix for CMP0018
The new behaviour lets us explicitly disabling position independent code, avoiding the need to change undocumented variables

* LINK_DEPENDS is semicolon separated

* [CMAKE] Fix for CMP0005
it seems impossible to have cmake correct escape PROXY_CLSID_IS so move it to the header

[SDK][INCLUDE] small fix for msgdump.h

[WIN32SS][NTGDI] Use UNICODE_NULL for consistency

[WIN32SS][NTGDI] FireFox checkmark fix CORE-15303 (#1535)

The font size was a bit larger than expected. FireFox will show checkmarks correctly in View menu. CORE-15303

[POWERCFG] Show the name of the power scheme in the delete confirmation message box

[POWERCFG] Some small cleanup

- Rename IDC_HYBERNATE... to IDC_HIBERNATE...
- Rename IDS_PORCENT to IDS_PERCENT
- Replace %1 by %s in IDS_CONFIG1 and IDS_CONFIG2

[WIN32SS][NTGDI] Keep FontSubstitutes enumeration CORE-15785 (#1532)

Fix CORE-15785. When LOGFONT.lfFaceName[0] == 0, then enumeration of font substitutes had failed. CORE-15785

[ROSAUTOTEST] A timeout of 120 seconds kills the msi:action test prematurely, so increase it to 170 seconds.

This is still below the 180 seconds before sysreg2 kills the entire VM.
I've added a comment to clarify that.

[PSDK] Improve lmrepl.h

Added public domain header. This file was originally imported in r8283 from w32api where is has no copyright.

[REGTEST] Add regtest.cmd to the Run key inside regtest.cmd, so we're also started on the next reboot in case ReactOS crashed *and* the registry has been saved.

unattend.inf only allows to add RunOnce entries, which are in theory only executed on the first boot after installation.
For almost a decade, this worked anyway to start regtest.cmd again after a ReactOS crash, because the registry and thus the deletion of the RunOnce entry wasn't stored on disk in case of a crash.
Thanks to Sylvain for the hint!

As Explorer processes the Run key right after RunOnce and therefore picks up regtest.cmd a second time during the first boot, exit the script just after adding the Run key.
Otherwise, rosautotest.exe would be started twice, with catastrophic results.

[ROSAUTOTEST] Cache the result of GetLastError().

[ROSAUTOTEST] Implement a process activity timeout of 2 minutes. If there is no log output within 2 minutes, the test process is killed, and we continue with the next test.

This is a rather graceful approach compared to sysreg2's 3 minute timeout before killing and restarting the entire VM.
Since we added autochk for FAT filesystems, the filesystem is often "fixed" after a reset with the consequence that ReactOS doesn't boot up anymore.
The sysreg2 restart code still remains for handling tests causing BSODs.

[POWERCFG] Fix the initialization of the power schemes page

CORE-4995

[DPNHPAST] Import dpnhpast.dll from Wine (Staging) 4.5

[BROWSEUI] Enable the Disconnect Network Drive menu item. (#1522)

CORE-13516, CORE-13518

[MPR] Implement the function WNetDisconnectDialog(). (#1522)

CORE-13516, CORE-13518

[NETPLWIZ] Implement the Disconnect Network Drive dialog. (#1522)

CORE-13516, CORE-13518

[APPWIZ.CPL] Set a default shortcut-link description (#1520)

CORE-15511

[SHELL32] Make the new-item menu work (aka. make it dynamic) when the .lnk handler doesn't have any ShellNew entry.

[SHELL32] Code formatting/style/whitespace fix + fix two memory leaks in failure paths of CNewMenu::LoadItem().

[SHELL32] Implement 'New Link' (Retrial of #1510), CORE-15511 (#1518)

Correctly create cache about ".lnk" and handling in the member functions.

Co-authored-by: Hermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>

[MSPAINT] Set default extension .bmp (#1515)

Set the default extension of mspaint to .bmp. CORE-7114

[STOBJECT] Store the "Show x icon in the taskbar" setting for the hotplug, power and volume icons.

CORE-12365
CORE-12972
CORE-15234

[POWERCFG] Fix indentation and coding style. No code changes!

[STOBJECT][MMSYS][POWERCFG] Store the "Show x icon in the taskbar" setting for the hotplug, power and volume icons.

CORE-12365
CORE-12972
CORE-15234

[EXPLORER] Minor cleanup/comment fixes.

[NTOS:CM] In CmpParseKey(), do not assert but instead correctly return failure if CmpHandleExitNode() doesn't return a valid node, or CmpCreateKeyControlBlock() fails.

[NTOS] Fix typos.

[FREELDR] Add some missing UiDrawBackdrop() calls.

[OPENGL32] Fix the order of the specified callbacks for the DrvSetCallbackProcs() call.

References:
- https://github.com/OpenSWR/openswr/blob/master/ogldriver/wgl.cpp
- https://github.com/Igalia/mesa/blob/master/src/gallium/state_trackers/wgl/stw_icd.h
- "OpenGL Installable Client Driver, Version 1.101 -- Reference Guide" by Microsoft Corporation.

[OPENGL32] Fix usage of TEB members.

[NTOS][USERSRV] Silence noisy debug output.

[SYSSETUP] Fix a broken DPRINT
Thanks to Thomas for spotting it.

[STOBJECT] Mark hotplug, power and volume context menu default items

[GDI32_APITEST] Fix build s/IGNORE/NO_CHECK/

[GDI32_APITEST] Rewrite DPtoLP testcase (#1509)

Sorry, I had to rewrite gdi32!DPtoLP testcase. ROSTESTS-319

[GDI32_APITEST] Rewrite LPtoDP testcase (#1508)

Sorry, I had to rewrite gdi32!LPtoDP testcase. ROSTESTS-319

[POWRPROF] EnumPwrSchemes: Fix callback string lengths

The callback routine should get the real lengths of the name and description strings in bytes instead of the buffer sizes in characters. This fixes buffer overflows in powercfg.cpl.

[ACGENRAL] Fix MSVC warning C4090 "different 'const' qualifiers" about a ShimLib_ShimFree() call (#1506)

Addendum to 007cc5cd.
CORE-7538

[BOOTDATA] Add registry keys for ComDlg32 (#1511)

Add registry keys for ComDlg32. CORE-7114

[NTOSKRNL] Revert f71588ff as it seems to break the tests

[POWERCFG] Use a list of power schemes (power policy, id, name and description) rather than a bare array of power policies. This will simplify adding and deleting schemes quite a lot.

[POWERCFG] Fix the undue change of monitor switch off time

CORE-12495

[POWERCFG] Fix coding style and indentation. No code changes!

[NTOSKRNL] Drop the useless Timestamp field

[GDI32] Reduce verbosity on debug builds

[NTOSKRNL] Don't set VACB dirty on release if already dirty

CORE-15954

[ATL][ATLTEST] Adapt to RATL and improvements (#1501)

- Modify atldef.h for non-ReactOS environments.
- Update ATL testcase project files (from v120_xp to v140_xp).
- #define HAVE_APITEST and use it.
- s/NULL/0/

[HIDCLASS] Implement IOCTL_HID_GET_FEATURE/IOCTL_HID_SET_FEATURE

[KERNEL32] GetSystemPowerStatus(): Set the BATTERY_FLAG_CRITICAL flag if the battery life gets lower than 5 percent.

[KERNEL32] GetSystemPowerStatus(): Fix coding style. No code changes!

[STOBJECT] Improvements to the power notification icon

- Use GetSystemPowerStatus() instead of querying each battery.
- Add an ugly AC power icon. Should be fixed or replaced.
- Display AC power, battery charging, battery depleting and error cases. The AC power case was missing.
- Update some strings because we are no longer using float for the charge percentage.

@Turkish translators: Please check and update the translation of IDS_PWR_CHARGING!

[DBGHELP] SymFromAddr: Handle 2 or more symbols at the same address correct.

There could be more then 2 symbols at the same addr.
Adjust function to handle that case correct.

[DBGHELP] Set LastError for SymGetModuleBase-calls to ERROR_MOD_NOT_FOUND.

Correct last error in module_find_by_addr.

[DBGHELP] set correct values in symt_fill_sym_info (SYMBOL_INFO).

* symtype: set to SymPdb not SymCv (may be for old pdb-files pre VS 6.0)
* flags: return SYMFLAG_PUBLIC_CODE for SymTagPublicSymbol if its a function.
* flags: do not return SYMFLAG_FUNCTION for SymTagFuncion.

[DBGHELP] Add SYMFLAG_- and SYMTYPE_-defines.

[DBGHELP] use correct record in codeview_snarf_public.

Use public_vx iststead of data_vx for public symbols.

[DBGHELP] search for debugfiles in modulepath too.

Debug symbols (like PDB, ...) could not be found if the direcory of
the module and application differs. Applying this patch they will
be found.
Dbghelp searches the module path too. (See MS docs).

[CMAKE] msvc.cmake: Remove now redundant '/we4028' special case

Addendum to 545a2c79367e1457e4bc7ac14d3c0f840c7e063c.

CORE-7538

[APPVEYOR] Enable fast_finish (#1494)

It stops the remaining jobs if any error happened. It becomes less time for waiting for building.

[BOOTDATA] AHKAppTests.cmd: Fix 'media' typo (#1502)

Addendum to df2a8c4. s/media/medium/

[NTOBJSHEX] Reorder include files in the precompiled header

[APPHELP] Fix warnings

[RSHELL] Disable exceptions in atl when compiling without exceptions

[SHELL32_APITEST] Disable exceptions in atl when compiling without exceptions

[MSGINA_APITEST] Disable exceptions in atl when compiling without exceptions

[NETREG] Use exceptions with stl

[MSGINA] Disable exceptions in atl when compiling without exceptions

[NTOBJSHEX] Disable exceptions in atl when compiling without exceptions

[APPHELP_APITEST] Disable exceptions in atl when compiling without exceptions

[SHELL32] Use the same definitions for shell32 and it's static libs