[BROWSEUI] browseui.spec: Fix SHEnumClassesOfCategories() parameters. (#1559)

[WIN32SS][FREETYPE] Avoid performance regression CORE-16020

by reverting essential parts of "Font rotation at arbitrary angles"
which was once committed in several steps:

Initial commit (PR#1207)
0.4.12-dev-243-g
f757a135192c02a3c41d8da47eb9dba21d34ebd1
broke font rendering for GTK apps CORE-15623

Second commit (PR#1238)
0.4.12-dev-558-g
64987cf2731f9762d0316274bf7877a9bd3d88ba
restored font rendering in GTK apps but introduced slowdown CORE-16020

Katayama agreed to the temporal revert and will try to come up later
with a performance optimized version.
We revert in 0.4.12-RC and 0.4.13-dev.

The revert will cause failure-increases in gdi32:TextTransform
https://reactos.org/testman/compare.php?ids=67898,67901

[BOOTLIB] Check DBG with #if, not #ifdef (#1621)

[LWIP] Fix src/core/init.c a bit (#1620)

Cherry-pick
http://git.savannah.nongnu.org/cgit/lwip.git/commit/src/core/init.c?id=c0b534e5318baf870e2152c70d4d11a3a86181f3

[MBEDTLS] Use '#ifndef __REACTOS__', instead of '#if DBG & 0' (#1619)

- Don't use '|' on a '#if'
- Remove a personal comment.

[DXSDK] Improve dxerr?.h a bit (#1617)

*Use '||', instead of '|', on a '#if'.
*Synchronize some white spaces and blank lines.

[NCFTP] Check _DEBUG with #ifdef, not #if (#1616)

[NTDLL_VISTA] Check _DEBUG with #ifdef, not #if (#1615)

[ADVAPI32:REG] Remove some useless casts (#1614)

[USER32] RegisterClipboardFormatA(): Enable a SetLastError() (#1612)

[MC] Add syntax and help texts for the NET SHARE command.

[LOCALSPL] Check DBG with #if, not #ifdef

Addendum to r74433.

[NTOS:MM] PeFmtCreateSection(): Improve AMD64 image failure case

Addendum to 98638cc2dfdde460ae49941c76dd51f1b796a001.

CORE-15981

Misc addendum to CORE-14271 (#1529)

* [ADVAPI32] Simplify RtlCreateUnicodeStringFromAsciiz() return value check

RtlCreateUnicodeStringFromAsciiz() returns a BOOLEAN, not a BOOL.

No functional change.

Addendum to
CORE-14271

* [USER32] Simplify RtlCreateUnicodeStringFromAsciiz() return value check

RtlCreateUnicodeStringFromAsciiz() returns a BOOLEAN, not a UINT.

Also, add a FIXME.

No functional change.

Addendum to
CORE-14271

* [USER32] Simplify RegisterClipboardFormatA/W() a bit

No functional change.

* [UDFS] Simplify SeSinglePrivilegeCheck() return value check

No functional change.

Addendum to
CORE-14271

[RAPPS] Add Indonesian translation. (#1598)

[EXPLORER] Add Indonesian translation. (#1598)

[FORMAT] Update Indonesian translation. (#1598)

[RUNDLL32] Update Indonesian translation. (#1598)

[SUBST] Add Indonesian translation. (#1598)

[SERVICES] Add Indonesian translation. (#1598)

[RUNONCE] Add Indonesian translation. (#1598)

[FONTVIEW] Add Indonesian translation. (#1598)

[MSPAINT] Add Indonesian translation. (#1598)

[OSK] Add Indonesian translation. (#1598)

[REGEDIT] Update Indonesian translation. (#1598)

[NET] Implement adding and deleting of network shares.

[WIN32KNT_APITEST] Replace remaining TESTX() (#1609)

Addendum to b7794e8112e31a58829bac63d44de53677318c3d.

[GDI32] Meta File Fix Ups

Remove pointer hack. Safer since using wines Enhanced Meta headers with
unknown wine magic information.

Revert "[USER32] Make App Switcher use the owner window's icon (#1299)" (#1605)

This reverts commit 484943d04f0dc3f84b7c8e754d1d2505f0a41df3.

[STORPORT] Detect attached devices

[NTOSKRNL] Add support for global DOS directory in ObpLookupEntryDirectory
If any exists, we'll loop over in that directory, trying to find the object

[WIN32KNT_APITEST:NTDD] Improve failure handling

And remove comments about ReactX.

ROSTESTS-315

[FORMATTING] Add/Remove a few spaces

[USER32_APITEST:WNDPROC] Improve failure handling

ROSTESTS-316

[USER32_APITEST:WNDPROC] 0 to NULL

[FORMATTING] 3-spaces to 4-spaces

And a few additional formatting improvements.

[STORPORT] Fix resource list size calculation

[WIN32SS][USER32] Fix Task Switcher more (#1602)

This PR will fix task switcher (Alt+Tab) as Win2k3. It became more reliable. CORE-15653
c.f. Raymond Chen's article: http://blogs.msdn.com/b/oldnewthing/archive/2007/10/08/5351207.aspx

[NTOSKRNL] In case IopMountVolume call is a success, return its VPB
This fixes an extra (erroneous) reference being set on the VPB.

Based on a patch by Vadim Galyant

CORE-16080

[NTOSKRNL] On process primary token change, dereference device map

[SDK][INCLUDE] Update msgdump.h

[NDK/TESTS] The world isn't only amd64 or i386 (#1596)

Update toolchain-msvc.cmake (#1593)

Adding armasm64 as the assembler. This on its own isn't too useful right now.

[CONFIGURE] Add support for VS16.1 (#1594)

[MSHTML] Fake Success in HTMLDocument_put_bgColor()

This avoids at startup of Instant Messenger "QIP 2005 8095"
a message-box with "OLE Error 8004001".
The issue is a subtask of CORE-11537
Many Thanks to the patches author
Andreas Maier <staubim@quantentunnel.de>
JIRA-nick: andy-123

Note you still need to install Gecko 2.40 and Samba 1.3 from rapps for QIP.

[NTOSKRNL] Implement ObSetDirectoryDeviceMap

[NTOSKRNL] Implement SeGetLogonIdDeviceMap

[NTOSKRNL] Implement ObpSetCurrentProcessDeviceMap

[NTOSKRNL] Implement ObpReferenceDeviceMap()

[NTOSKRNL] Stub SeGetLogonIdDeviceMap

[NTOSKRNL] Simplify NtQueryInformationProcess:ProcessDeviceMap implementation

[NTOSKRNL] Add support for LUID mappings in ObQueryDeviceMapInformation

[NTOSKRNL] Implement support for PROCESS_DEVICEMAP_INFORMATION_EX in NtQueryInformationProcess

[NTOSKRNL] Implement ObIsLUIDDeviceMapsEnabled and call it in NtQueryInformationProcess

[NTOSKRNL] Disable LUID mapping until it's properly implemented

[NtUser] Fix Clipboard Synthesized Returns

Add supported for Meta file synthesized renderings.
See CORE-12143.

[NTOSKRNL] On device map freeing, make directory object temporary again

[NTOSKRNL] De-duplicate code between Ob(f)DereferenceDeviceMap functions

[NTOSKRNL] Add a few ASSERTs to ObpGetDosDevicesProtection for DBG builds

[NTOSKRNL] Implement ObpFreeDosDevicesProtection to free DACL

[NTOSKRNL] Modify ObpCreateDeviceMap so that it can handle any process
It is now able to set the newly created device map to any
process and will default to current process if none is provided.
It also sets system device map if no process is specified.
It also deferences existing device map in the process if needed.
Finaly, it will make the directory object permanant.

[SDK][INCLUDE] msgdump.h registered message support

[User32] Fix Clipboard Meta Test Results

Fix meta data handle returns.

[Win32SS] Support Clipboard Metafile

Major fix ups and plugging in missing code changes. Handle code is from
another project currently being developed for ReactOS.
See CORE-12143

[ROSTESTS][GLOBAL_MEM] GlobalAllocNFree might return SUCCESS even in failure case due to unintended semicolon ROSTESTS-323

[NET] Start work on the SHARE command. List shares and display their properties.

[KERNEL32_APITEST] dynamic linked RtlIsCriticalSectionLockedByThread (#1592)

Statically linked ntdll.RtlIsCriticalSectionLockedByThread had made kernel32_apitest unable to run in XP. CORE-16077

[USER32] Fix behavior of Task Switcher (Alt+Tab) (#1591)

Task Switcher didn't correctly behave on the windows with WS_EX_APPWINDOW extended style. CORE-15653

[KERNEL32_APITEST] Check CJK codepage before tests

[NtGDI] Support Clipboard Metafile

Fuctions that help transfer meta file data from one process to the next.
See CORE-12143.

[GDI32] Support Clipboard Metafile

Metafile patch from Katayama Hirofumi MZ. See CORE-12143.

[ROSAPPS][PICE] Fix a glitch at two places where code intends to replace invalid chars with spaces ROSAPPS-353

Found by some static code analysis tool reporting 'Condition always false'.
The fix is a selective back-port from the more recent version of pICE hardware.c
Revision: 1.6, Date: 2004/02/17 23:07:36

The most recent source of pICE atm can be found at
https://sourceforge.net/projects/pice/files/pICE%20source/build_20/

Thanks to DougLyons and JIRA user 'maddin200'.

[UMPNPMGR] Split the monolithic monster: service, rpc and installer.

[UMPNPMGR] PNP_QueryArbitratorFreeData and PNP_QueryArbitratorFreeSize don't do anything useful today.

[ROSAPPS][GETTYPE] Fix a copy-paste-issue when checking for Win2000 CORE-16073

[NETAPI32] Implement parts of NetGetDCName and declare some parameters optional.

[NETLOGON] Add the netlogon service.

[WIN32KNT_APITEST] Split win32knt_apitest to 3 modules (#1588)

Split win32knt_apitest to 3 modules (win32knt_apitest, win32knt_xpsp2_apitest and win32knt_2k3sp2_apitest) for usability. These 3 modules are different in linking (win32u.dll, win32u_xpsp2.dll and win32u_2k3sp2.dll). I think it's better to separate for clarity.

[NETAPI32] Implement NetpAllocWStrFromStr(), NetpAllocWStrFromAnsiStr() (not public, but useful) and NetpAllocWStrFromWStr().

[SYSSETUP] Set registry hostname in WriteComputerSettings (#1589)

Based on @Doug-Lyons's patch. CORE-16067

[NET] Fix remaining issues with logon hours.

[NET] Load the names of the days of week for the 'times' parser routine from the netmsg.dll.

[NET] Fix the timezone issue in the logon hours parser and display code of the USER command.

[TELNET] Add missing curly braces to for-statement

Fixes GCC 8 warning:
base/applications/network/telnet/src/tnmain.cpp:171:8: error: this 'for' clause does not guard... [-Werror=misleading-indentation]
        for (j = cursor; j >= 0; j--)
        ^~~
base/applications/network/telnet/src/tnmain.cpp:174:9: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'for'
         for (k = --j; k >= 0; k--)
         ^~~

[TELNET] Add missing curly braces to while-statement

Fixes GCC 8 warning:
base/applications/network/telnet/src/tmapldr.cpp:70:3: error: this 'while' clause does not guard... [-Werror=misleading-indentation]
   while (buf[len])
   ^~~~~
base/applications/network/telnet/src/tmapldr.cpp:74:4: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'while'
    if (len && (buf[len-1] == ' ')) {
    ^~

[TELNET] Add missing curly braces to if-statelent

Fixes GCC 8 warning:
base/applications/network/telnet/src/tnmain.cpp:171:8: error: this 'for' clause does not guard... [-Werror=misleading-indentation]
        for (j = cursor; j >= 0; j--)
        ^~~
base/applications/network/telnet/src/tnmain.cpp:174:9: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'for'
         for (k = --j; k >= 0; k--)
         ^~~

[TELNET] disable restrict warnings

Fixes GCC 8 warning:
base/applications/network/telnet/src/tnmain.cpp:190:13: error: 'char* strcpy(char*, const char*)' accessing 1 byte at offsets [0, 2147483647] and [0, 2147483647] may overlap 1 byte at offset 0 [-Werror=restrict]
       strcpy(&buf[cursor],&buf[cursor+1]);
       ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~

[LDR] Silence some debug logs when images overlap

[BROWSEUI] Fix indentation

Fixes GCC 8 warning:
dll/win32/browseui/explorerband.cpp:1332:17: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
                 if (!SUCCEEDED(hr) || !pParent.p)
                 ^~
dll/win32/browseui/explorerband.cpp:1335:21: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
                     hr = pParent->SetNameOf(0, pidlChild, dispInfo->item.pszText, SHGDN_INFOLDER, &pidlNew);
                     ^~

[GLU32] Fix indentation

Fixes GCC 8 warnings:
dll/opengl/glu32/src/libnurbs/internals/subdivider.cc:852:2: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
  if( jarc->pwlArc ) jarc->pwlArc->deleteMe( pwlarcpool ); jarc->pwlArc = 0;
  ^~
dll/opengl/glu32/src/libnurbs/internals/subdivider.cc:852:59: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
  if( jarc->pwlArc ) jarc->pwlArc->deleteMe( pwlarcpool ); jarc->pwlArc = 0;
                                                           ^~~~

[MSHTML] Convert tabs to spaces

Fixes GCC 8 warning:
dll/win32/mshtml/script.c:844:4: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
    if(!new_buf)
    ^~
dll/win32/mshtml/script.c:846:13: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
             This->size <<= 1;
             ^~~~

[REISERFS] Convert a tab to spaces

Fixes GCC 8 warning:
drivers/filesystems/reiserfs/src/memory.c:874:5: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
     if (Mcb->RfsdFcb)
     ^~
drivers/filesystems/reiserfs/src/memory.c:877:2: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
  if (RfsdLoadInode(Vcb, &(Mcb->Key), &RfsdIno)) {
  ^~

[TFTPD] Disable format-overflow warnings

[MINIZIP] Add curly braces to if-statement and fix indentation

Fixes GCC 8 warning:
sdk/lib/3rdparty/zlib/contrib/minizip/zip.c:521:5: error: this 'for' clause does not guard... [-Werror=misleading-indentation]
     for (i=(int)uReadSize-3; (i--)>0;)
     ^~~
sdk/lib/3rdparty/zlib/contrib/minizip/zip.c:529:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'for'
       if (uPosFound!=0)
       ^~

[LIBXSLT] Disable misleading-indentation warnings

Fixes GCC 8 warning:
dll/3rdparty/libxslt/security.c:358:5: error: this 'else' clause does not guard... [-Werror=misleading-indentation]
     else
     ^~~~
dll/3rdparty/libxslt/security.c:365:2: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'else'
  if (ret <= 0) {
  ^~

[RSAENH] Use StringCbPrintfA instead of sprintf

Fixes GCC 8 warning:
dll/win32/rsaenh/rsaenh.c:161:23: error: '%s' directive writing up to 259 bytes into a region of size 235 [-Werror=format-overflow=]
 #define RSAENH_REGKEY "Software\\Wine\\Crypto\\RSA\\%s"
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
dll/win32/rsaenh/rsaenh.c:161:23: note: in definition of macro 'RSAENH_REGKEY'
 #define RSAENH_REGKEY "Software\\Wine\\Crypto\\RSA\\%s"
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
dll/win32/rsaenh/rsaenh.c:1107:5: note: 'sprintf' output between 26 and 285 bytes into a destination of size 260
     sprintf(szRSABase, RSAENH_REGKEY, pKeyContainer->szName);
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

[USETUP] Use RtlStringCbPrintfA instead of sprintf

Fixes GCC 8 warning:
base/setup/usetup/usetup.c:3407:78: error: '%S' directive writing up to 521 bytes into a region of size 200 [-Werror=format-overflow=]
                 "Setup is currently unable to check a partition formatted in %S.\n"
                                                                              ^~
base/setup/usetup/usetup.c:3406:9: note: 'sprintf' output between 128 and 649 bytes into a destination of size 260
         sprintf(Buffer,
         ^~~~~~~~~~~~~~~
                 "Setup is currently unable to check a partition formatted in %S.\n"
                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                 "\n"
                 ~~~~
                 "  \x07  Press ENTER to continue Setup.\n"
                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                 "  \x07  Press F3 to quit Setup.",
                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                 PartEntry->FileSystem);
                 ~~~~~~~~~~~~~~~~~~~~~~

[MINGW32] Add some definitions to fix building with GLIBCXX

[CRT] Add missing posix error codes

These are required by latest mingw-w64 stl and we are mixing this with our crt headers.
Define _CRT_NO_POSIX_ERROR_CODES in all modules that define their own constants.

[CMAKE] Disable unused-const-variable warning

This prevents some errors like this one:

In file included from ../dll/win32/wldap32/ber.c:27:0:
../sdk/include/reactos/wine/debug.h:378:48: error: '__wine_dbch___default' defined but not used [-Werror=unused-const-variable=]
     static struct __wine_debug_channel * const __wine_dbch___default = &__wine_dbch_##ch
                                                ^
../dll/win32/wldap32/ber.c:29:1: note: in expansion of macro 'WINE_DEFAULT_DEBUG_CHANNEL'
 WINE_DEFAULT_DEBUG_CHANNEL(wldap32);
 ^~~~~~~~~~~~~~~~~~~~~~~~~~