[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

[ATL] Retire atlnew

[ATL][ATL_APITEST] Add test + implementation for CAtlFileMapping

[NTOSKRNL] Drop the no longer needed Dirty field

CORE-15954

[NTOSKRNL] Don't mark VACB dirty on unpin

This could happen if BCB was marked dirty previously.
Marking VACB dirty on unpin could lead to a double write of
the VACB, even if clean.
Indeed, now that setting BCB dirty leads to marking VACB
dirty, the VACB can be flushed in between by the lazy-writer.
The BCB state is not reset on VACB flush, contrary to the VACB state.
Thus, on unpin even if the VACB was already flushed, we were
setting back the dirty state, leading the VACB to be flushed again.

This could bring a small performance downgrade. Though it remains
limited since this is mostly used for FS metadata.
Possibly it could lead to metadata corruption, but this is likely
less possible.

CORE-15954

[NtGDI] Update Region code

Wine Sync/Port and update for Regions. See CORE-15906.

[GDI] Fixup attribute support for Regions

Start to fix CORE-13817 when GDI batch is turn on for SelectClipRgn.

[NTOSKRNL] NtPowerInformation: Report an AC powered machine by default.

[KMTESTS:CC] Add tests for CcCopyWrite

CORE-15384

[KERNEL32] Implement InvalidateNLSCache

[KERNEL32] Fix handle leak and caching (follow-up of 7e38267)

[KERNEL32] Era first year is GANNEN

[BOOTDATA] Improve AHKAppTests.cmd (follow-up of #1476)

[USBHUB_NEW] Support NULL HubExtension in USBH_PdoRemoveDevice.

Device removal can happen multiple times for PDOs.

[USBHUB_NEW] Fix inverted check in USBH_RestoreDevice.

This allows re-enabling a disabled device.

[USBCCGP] Allow unconfiguring the device in USBCCGP_PDOSelectConfiguration.

Fixes device removal for USB audio devices.