[TCPIP] Remove duplicate declarations from ticonsts.h

[SDK][IP] Pass IP options to ReceiveDatagramHandler

[TCPIP] Implement IOCTL_ICMP_ECHO_REQUEST in tcpip.sys
Also clean up ICMP handling code in sdk/lib/drivers/ip
CORE-10760

[IPHLPAPI] Make icmp functions use IOCTL_ICMP_ECHO_REQUEST from tcpip.sys
This adds missing features like using events and APCs within IcmpSendEcho2
functions and others.
CORE-10742 CORE-14411

Co-authored-by: Tim Crawford <crawfxrd@gmail.com>

[SDK][IP] Fix formatting

[TCPIP][FORMATTING] Format TiDispatch function. No functional changes

[USETUP] de-DE Whitespace nitpicking CORE-16792

To harmonize it to 3 separator-spaces how it is done
historically in other dlgs.

[USETUP] de-DE Fix typo CORE-16792

Fix typo in my last changeset
0.4.14-dev-1323-g
d079342e0a0684b4fb6530ee7ea4728f2a0ae985

[USETUP] de-DE Overhaul german translation CORE-16792

[USETUP] de-DE Split a line of text into two parts CORE-16791

[TRANSLATION] Updating Spanish translation (#2503)

Updating translation for:
notepad, reactos, usetup, explorer, netshell, msgina, setupapi, shell32.

- Complete and Fix the SPANISH TRANSLATION on Setup
- Correct the random names, fixing the denominations, translate of the English words, and fixing random and incorrect denominations like "Cabinet" or "distribuciones".
- Fixed the notepad shortcuts of "Save" (Ctrl+g) and "Replace".

[NTOS:MM] Since the _MMPTE_SUBSECTION structure for x86 (without PAE) uses the SubsectionAddressLow truncated by the least significant three bits, the _CONTROL_AREA, _LARGE_CONTROL_AREA, _SUBSECTION, _MSUBSECTION structures must be 8-byte aligned.
Corresponding checks in MmArmInitSystem() have been removed.

[NTOS:MM] In MmArmInitSystem, perform self-tests after per-arch initialization.

Mm(Non)PagedPoolStart won't be initialized otherwise, making the tests
pretty pointless.

[NTOS:MM] Initialize MmSubsectionBase on x86 builds. CORE-16731

Spotted by Vadim Galyant.

[NTOS:MM] Get rid of unnecessary MmZeroingPageThreadActive.

[NTOS:MM] Fix MmZeroingPageThreadActive race condition.

If a thread calls MiInsertPageInFreeList after the zero page thread has
woken up from its wait but before it acquired the PFN lock,
MmZeroingPageEvent might get set again even though there is no additional
work to be done.
By setting MmZeroingPageThreadActive together with the event, the variable
correctly indicates for future callers that the zero thread will process
newly added free pages.

This partially reverts r72990.

[NTOS:PO] Call power IRP handlers at PASSIVE_LEVEL when needed. CORE-11648 CORE-16704

This means we now correctly handle the DO_POWER_PAGABLE flag.
In particular, Windows's usbhub.sys calls KeDelayExecutionThread from a
power IRP dispatch routine. We now handle this correctly.

[Win32SS] Fix Mask for Draw Frame States

Fixes drawing of caption bar buttons.

See CORE-8808.

[DEVMGR] CDeviceView::RecurseChildDevices: Skip invalid device nodes

CORE-16780

[MBEDTLS] Update to version 2.7.14. CORE-16787

[ACPICA] Update to version 20200326. CORE-16783

[HOTPLUG] Add the context menu to the device tree view

[ROSAUTOTEST] Submit results to https://reactos.org:8443 instead of https://tlsv1.reactos.org

It is much easier server-side to maintain a different TLS configuration on another port rather than another subdomain.

[RAPPS] Ensure the download dialog is always destroyed
Ideally, this entire download thing should be rewritten, but that is
something for another day.

[ATL] Add some hacks for gcc,
since parsing templates is not something it can do

[RAPPS] Use the provided SizeBytes for the progress,
when the host does not give a file size

[RAPPS] Enable ATL assertions

[ATL] Fix an ATLASSERT expression

[INCLUDE/CRT] Add some debug macro implementations for _DEBUG
CORE-11834

[CRT] Implement _CrtDbgReport and _CrtDbgReportW
Most functionality is working, except output to file
CORE-11835

[HOTPLUG] Display devices that are attached to a removable device

[NTOS:PNP] Add set functions for debugging PNP and IO managers. (#2457)

Useful functions for debugging IO and PNP managers:

PipDumpDeviceNodes() - displays information about a node(s) in the device tree;
PipDumpResourceRequirementsList() - displays information about a Io List;
PipDumpCmResourceList() - displays information about a Cm List

The tree list of devices (DEVICE_NODE structures) is perhaps the main one in the PnP manager. They also store information about the hardware resources required and assigned to devices.
These functions can help with debugging. For example, you can call PipDumpDeviceNodes() before and after device enumeration and compare the resulting information.

For PipDumpDeviceNodes() it is possible to optionally output:
- allocated resources and boot configuration resources
- resources required
- translated resources
It is possible to displays both a single node and the entire tree.
Optionally, you can display child nodes.

The information output format for resource lists is maximally compressed, since often the only debugging port is a monitor.

The DebugLevel parameter allows dumping in two modes:
0 - unconditional;
1 - if NDEBUG is not defined in "debug.c".

[NTOS:MM] Make the definitions and macros for x86 more human-readable. (#2487)

- Move also PDE_MAPPED_VA definition from common miarm.h to ..arch/mm.h.
- Add assert "PAE not yet implemented".

[TRANSLATION][USETUP] Update Spanish translation of USetup. (#2497)

Correct the random names, fixing the denominations, translate of the English words, and fixing random and incorrect denominations like "Cabinet" or "distribuciones".

[ROSAUTOTEST] Use tlsv1.reactos.org for submitting test results now that reactos.org is TLS 1.2+

We still need to be able to submit test results from Windows Server 2003, which supports TLS 1.0 only.

[QCAP] Import qcap.dll from Wine Staging 3.3. CORE-16350 (#2421)

Fix also MSVC build.

[DOC] Add qcap entry in README.WINE

Co-Authored-By: Thomas Faber <18138474+ThFabba@users.noreply.github.com>

[DXSDK] Sync axextend.idl with Wine Staging 3.3 (#2421)
Required by Wine's qcap.dll CORE-16350

[DXSDK] Restore all deleted code in axextend.idl to propely fix compilation
Also regularize some parts of added code and replace IAMStreamControl interface
with Wine's one, because otherwise compilation fails.

[TASKKILL] Merge our updated terminate_processes() function with the one
from Wine, using __REACTOS__ defines to separate the changes.

We note that in effect it completely contains the functionality of
Wine's send_close_messages(), so we keep that latter disabled.

[UTILMAN] Implement the Registry management code skeleton (#2456)

The following SaveAppSettings(), QueryAppSettings() and InitAppRegKey() are helper functions.
This is merely a base Registry skeleton for Utility Manager as more work on it has to come later.

[TASKKILL] Improve taskkill utility (#2459)

Make the argument process of command utility "taskkill" behave same as what Windows does.

Now options are no longer detected as parameters when placed after options that accept one.
For example, `taskkill /im /f` will no longer regard `/f` as a process name.

If `/?` and `/f` options appear more than once, an error will be reported, telling they are not allowed more than once.
(e.g. `taskkill /f /pid 1000 /f`)

If only one option `/f` is given without `/pid` or `/im`, an eerror will be reported, telling that one must specify `/pid` or `/im`.
(e.g. `taskkill /f`)

Additional changes:
- Combine SendCloseMessages() and TerminateProcesses() functions.
- Protect new written code with `#ifdef __REACTOS__`

[WIN32K:NTUSER] ntuser.h: Rename NtUserWaitForInputIdle() 3rd parameter (#2499)

Addendum to 568b6d05.

[WINETESTS] Remove forced _WIN32_WINNT defines (#2463)

Only comctl32 is affected.

Import
https://source.winehq.org/git/wine.git/commit/be3cda6ec0856d143d4737ee9306844569b76e15

[PSDK] guiddef.h: Remove unwanted '&& !defined(CINTERFACE)' (#2492)

Cherry-pick
https://source.winehq.org/git/wine.git/commit/4626db2bb2a61cdceb00e0e346628062ee51aade
Import
https://source.winehq.org/git/wine.git/commit/54f58769fee73a55675d503e0281e8f381a8e8b1
https://source.winehq.org/git/wine.git/commit/34f3e7793b40d0eac60c373f66f262be2e0d5cdb

[RPCRT4] Handle ERROR_BAD_NETPATH failure if an RPC pipe connection is done on an non-existing UNC path.

Addendum to 5274857d (see originally commit 13f41d74). CORE-6561

[NTUser] Fix Strings and Format to Hooks

Allocate heap instead of data segment to be used for callbacks on user side.
Move and correct initial hook call out setup. Use it in more than one hook call.

This fixes issues with strings out of alignment and use of kernel pointers.

See CORE-13907 and CORE-16769. KsStudio still needs retested.

Small wow update.

[SYSDM] Cumulative update 2020-04-01 for SMBIOS parser (#2495)

[RPCRT4] Addendum to support for remote pipes names in ncacn_pipe_name() (24cd7bbe).
CORE-6561 CORE-13442

- Call GetComputerNameA() only when a non-empty server name has been
  provided, thus slightly improving speed for the most common case when
  local calls (with an empty server name) are done.

- When a server name is passed, trim any leading UNC server prefix since
  the latter will be restored when building the pipe name string.

[README] Add "open in Gitpod" button

[GITPOD]Add gitpod config

this commit adds support for Gitpod.io, a free automated
dev environment that makes contributing and generally working on GitHub
projects much easier. It allows anyone to start a ready-to-code dev
environment for any branch, issue and pull request with a single click.

[RDBSS] Avoid CORE-13938

Pierre recommended this workaround for 0.4.8rls before.
Avoids "GetVolumeInformation now fails on NFS volume"

This workaround was recurrently applied for all releases
0.4.8, 0.4.9, 0.4.10, 0.4.11, 0.4.12, 0.4.13.

I never got any reply in the regression-ticket and recurrently
applying this over and over again is a waste of time.
So I decided to commit to master today, but will leave
the ticket unresolved, so when a proper fix will arrive in the future,
the existing ticket will remind us to undo this workaround.

Please note that I replaced #if 0 with #if 1
as discussed with Pierre. That's different to the patch in ticket.

[NTOS:MM] Using the macro MiIsPteOnPdeBoundary(). (#2496)

The SYSTEM_PD_SIZE constant should not be used to determine the page boundary for page tables. It is better to use the portable MiIsPteOnPdeBoundary() macro for this.

[STOBJECT] Change eject hardware dialog buttons from OK/Cancel to just OK. (#2493)

It's just an information dialog that needs one button, it's not actually doing different actions based on user input.

[SPEC2DEF] Omit skipped exports from ordinal numbering.
CORE-16769

[HOTPLUG] Display class icons for removable devices

[CONFIGURE] Simplify version detection for modern Visual Studio

[HOTPLUG] Add 32 pixel icon. (#2486)

Add a 32 pixel icon to the main icon for hotplug.cpl so the Safely Remove Hardware dialog looks normal.

[HOTPLUG] Show removable devices and update the list on arrival and removal.

[ROSAPPS][PICE] Fix ROSAPPS-355 Unreachable code

Just to satisfy static code analysis. No change in behavior expected.

The same way as the external applications maintainers
fixed it in their latest version
https://sourceforge.net/projects/pice/files/pICE%20source/build_20/

By killing the whole function ScrollUp() with the disabled code.
Like the original authors I left the functions unused declaration
existing within hardware.h

[NTOS:KD] Move dump of stack trace on crash to debugger

Release builds now show again the bugcheck screen, instead of freezing.

[NTOS:KE] Remove dead code

[NTOS:KD] Fix compilation with CMAKE_BUILD_TYPE=Release

This was broken by commit a890fc64d17e7c4a09a3cbe24e0dfc972d743dda

[NTOS:KD] Fix compilation with CMAKE_BUILD_TYPE=Release

This was broken by commit f7ec84eea0790899ca34b79f2ec5a308204254ac

[ACLUI] Properly enumerate cache+queue entries (#1937)

[PORTCLS] Stub missing IoControlCode switches (#1756)

Stub missing switches: IOCTL_KS_HANDSHAKE and IOCTL_KS_METHOD.

[KERNEL32_APITEST][WS2_32_APITEST] Call SetLastError() before checking LastError (#2437)

Follow-up to cb5aa7be.

[APITEST][NTDLL] Initial implementation of NtSetInformationProcess() testcase

The following testcase only performs argument checks for ProcessForegroundInformation class for the moment. The testcase will be expanded with further tests when needed.

[REACTOS] Remove forced _WIN32_IE defines

No impact, as already deactivated or superseded or superfluous.

Import
https://source.winehq.org/git/wine.git/commit/7770e26f2d773dcade6f073585b6fac2661e1eb3

[NTDLL_WINETEST] Remove forced 0x500 API versions (#2483)

No impact, as conditional.

Import
https://source.winehq.org/git/wine.git/commit/71d810319059d171df0e40d85752519e50e58056

[SHELL32] HCR_GetIconA(): Properly use icon_idx (#2482)

CORE-11713

Import
https://source.winehq.org/git/wine.git/commit/7cc2806e0e0d97c0df93e2695351a4d3a3c72254

[WINMM] Add a WARN() when winejoystick.drv is missing (#2480)

Follow-up to 97150ce9.

Import
https://source.winehq.org/git/wine.git/commit/175a2a027637719d1f3c644d97b6df4aacabb296

[GDI32] Allow BI_BITFIELDS compression in CreateDIBitmap (#2448)

CORE-16750, CORE-15960

[RPCRT4] Fix 'Routeines' typos (#2481)

Addendum to b6fdb56b9adf780902cc9cf65af1f76365a96ce4.

Import
https://source.winehq.org/git/wine.git/commit/c73352047d20a503c67bdd583f6ef8d98711a678

CORE-15682

[CMAKE] Fix symbol directory for non-MSVC builds

[KDGDB] Various fixes and improvements

[NTOSKRNL] Fix build

[HOTPLUG] Enable the hotplug.cpl and add the 'Safely Remove Hardware' dialog

[RTL] Introduce RtlpImageNtHeader,
which implements the required functionality.
ntdll and ntoskrnl now have a wrapper for this, with SEH.
This protects the function against malformed / bad images,
whilst still being able to use the code in freeldr et al.
Idea from Thomas.
CORE-14857

[NDK] Ensure RTL_IMAGE_NT_HEADER_EX_FLAG_NO_RANGE_CHECK is available regardless of NTOS_MODE_USER being defined.
CORE-14857

[LDR] Include source dll in static snap failure dprint

[MSV1_0] LsaApLogonUserEx2: use RtlRunDecodeUnicodeString to decode password

[CMAKE] kernel32_winetest: Enable 'generated.c', at last

Follow-up to 20f403966652c15911d9614bbf804113d6e9d10c.

ROSTESTS-251

[KERNEL32_WINETEST] Uncomment out 'LPEXCEPTION_RECORD' checks

Revert part of r70802 (dd21fa115a596cbf0c8565e8e9db919acfaa8c6b).
Follow-up to 2e82175ba48412bd29ac43d9d616cb8f8a1068e4.

ROSTESTS-251

[WINETESTS] */generated.c: Remove forced 0x0501 API versions

This actually affects 2 modules only:
* kernel32: this file is still commented out, these defines were going to be deactivated anyway.
* rpcrt4: I don't know why it had these defines (still) activated. Anyway.

Import
https://source.winehq.org/git/wine.git/commit/aa384d36429110970aa72acbc5043158de6aff03

[SHELL32_WINETEST] shlexec.c: Remove forced 0x0501 API versions

No impact, as already deactivated.

Import
https://source.winehq.org/git/wine.git/commit/8fc0b7d525ae5e85b5a796eaf6572d22591e02c6

[RPCRT4_WINETEST] ndr_marshall.c: Remove forced 0x0500 API versions

Import
https://source.winehq.org/git/wine.git/commit/57f08ba5259e56ae681bab1e61f86d6e8ab96f37

[ADVAPI32] wine/crypt.*: Misc fixes

Import
https://source.winehq.org/git/wine.git/commit/62df03af96822bda36aef398d00bf5998f490e90
https://source.winehq.org/git/wine.git/commit/8b9e3dae4fb875d6a445bd5182bb6ae32f5680c7
https://source.winehq.org/git/wine.git/commit/2df16753f0008099e119f39469badd10e7450bad

And use explicit '#ifndef __REACTOS__'.

[ADVAPI32][PSDK] CryptReleaseContext(): dwFlags parameter is DWORD, only

Import
https://source.winehq.org/git/wine.git/commit/9cc8b501eea8dc3bd589ab3e5df3b750e43699d9

[ADVAPI32] wine: Remove remnant ros_diff.patch

Added on 4234345b03677c2b01d0f313108e24d684e4d2ed (r48090),
merged on d4a89f6ccaf19953acf0d218e9e0f985c5d6004b.

CORE-14434

[WORDPAD] Restore code commas in no-NO.rc

".../no-NO.rc:81: syntax error"

[KMTEST] Delete special-purpose driver services on unload. ROSTESTS-263

Based on a patch by Serge Gautherie.

[Win32SS] Form Sanity to Hook Callbacks

Fix WH_CALLWNDPROC/RET data to user hook calls. See CORE-13019 and CORE-13907.

[User32] Fix null return.

See CORE-16769.

[SDK][NDK][PSTYPES] Align the PROCESS_PRIORITY_CLASS structure during compile time (#2478)

CORE-16757

After doing investigations on the priority class structure alignment, it's been revealed that in Windows XP and Server 2003 this PROCESS_PRIORITY_CLASS structure is aligned as a 4-bytes of size hence NtQueryInformationProcess() probes the alignment of user mode arguments buffer output and buffer length with requirement of a ULONG.

As PROCESS_PRIORITY_CLASS was initially aligned as a 1-byte size because both BOOLEAN and UCHAR are just unsigned characters, the compiler may not align such structure and gracefully let the default alignment of such structure as is, 1-byte because an unsigned char has a size of 1 byte. Setting an align attribute to this structure fixes the problem of a potential datatype misalignment which caused GetPriorityClass() to not retrieve the process' priority class properly.

[MSV1_0] use precompiled headers (#2475)

[NTOS:MM] Make _X86PAE_ the only define for PAE-enabled kernel

[SDK][NDK] MMPTE for PAE (#2453)

This not only makes PAE possible, but also allows to make the definitions and macros in the memory manager header files more human-readable.

CORE-16702

* [SDK][NDK] Make the _MMPTE_HARDWARE structure more compact and more human-readable.

* [SDK][NDK] Add definitions a struct _MMPTE for PAE mode.

[WINEFILE] Fix 'LANGUAGE_SI_SI' typo (#2474)

[NTDLL_APITEST] Add testcase for process priority class. CORE-16757

Currently the testcase function only addresses the behaviour of getting an
unaligned buffer (either of correct length size or not) for now.

[STORAHCI] Replace some __inline by FORCEINLINE

Functions were declared as __inline, then used, and then defined.
This leads to some linking errors with some compilers.

[TCPIP] Add some static specifiers, to prevent exporting FORCEINLINE function outside compilation unit