[0.4.10][AUTOCHK] Don't check any volume on boot CORE-14638

This will avoid triggering a FAT repair on
unclean FAT volumes.

If dosfstools.fsck works fine in Linux, its
usage on ReactOS triggers worse corruption
than unclean shutdown.

'autochk during OS startup' has been added during 0.4.9-dev-xxxx.
Given I've no time for debugging this, I
kill it off. This is just a workaround of course.

For now the best chances to repair a FAT FS corruption after non-clean-shutdown
is to mount that FS within 2k3sp2 and give the MS chkdsk an attempt to repair it.
Success is not guaranteed, but it works surprisingly often.
The earlier you do that after non-clean-shutdown,
the better your chances for success will be.

The workaround was picked from
0.4.13-dev-169-g ab69736a1d9c76260c500b11491328244d027893

[0.4.10][BOOTDATA] Fix formatting glitches in hivesys.inf, e.g. PORTUGESE_BRAZILIAN CORE-17828

PORTUGESE_BRAZILIAN regressed in 2017 with SVN r74654 (commit hash 8e52193 ).

Fix picked from
0.4.15-dev-3317-g 6ce259e694fb4adc3980bbacc5f66ce6f7a5ce09 PORTUGESE_BRAZILIAN
0.4.15-dev-3321-g 4b88a8709c8af9a085756e999bdd81d82412870d ICELANDIC

[0.4.10][UNIATA] Workaround random crashes on Virtual PC (#3377) CORE-12441 CORE-17371

I am not sure why this line in id_queue.cpp was added.
I don't know how I could improve that case.
Then, disable it, for the time being.

It regressed by 4b9cf2e33949a107fe6567632f1ac066c44a9d2c (r71252). CORE-12441 CORE-17371

fix picked from 0.4.15-dev-2740-g ba77a09c04a02841a6b88428697bd725402c1b9d

While touching uniata do also PARTIALLY port back some bm_devs.h changes:
0.4.15-dev-1522-g 88d36c93a27f5d764027e2475c84bdde1a84c4a6 (Properly mark ReactOS diff from upstream)

[0.4.10][NTGDI] Fix potential BSOD 0x1E CORE-17626

in CreateDIBPalette() when passing invalid arguments to CreateDIBSection.
This could be triggered by using the broken test-application "GDIProg".

After this patch not only the BSOD is fixed but also the app does
properly start up, like it is the case on 2k3sp2.

The problem was unhidden by the innocent and unrelated 0.4.12-dev-266-g 8ab3652c0112d152ba87ea10db15a6837dc89e94
I was not able to trigger the bug in releases older than 0.4.12,
but it is rather obvious that the mentioned revision is not really 'guilty', which
is why I will port the fix back further into releases even older than 0.4.12 as well.

Thanks to the patches author Doug Lyons.

a squashed port of:
0.4.15-dev-2734-g 514147776a7e70636911033ab6c89779b2c8ee1e (fixes the BSOD)
0.4.15-dev-2775-g c596fd3ef6933c9eec95a47c807d01d3776bbcc0 (improvement #3758)
0.4.15-dev-2776-g 4130f0b1c5f869614c71818bb7c30447d894afc8 (compilation fix)
and some white-space tweaking that was committed after those.

[0.4.10][NTOS/MM] Properly handle page faults in regions marked with PAGE_NOACCESS or PAGE_GUARD CORE-14694

This was part of the commits for ROSTESTS-110, but much more important:
It fixes CORE-14694 "reproducible BSOD 0x1A MEMORY_MANAGEMENT in OllyDbg v2.01"

Fix picked from 0.4.11-dev-650-g 47ac7a2b28df852a30e047426408bc368d37e5db

which also requires me to "Move up MmAlterViewAttributes() for later use in MmNotPresentFaultSectionView()"
like it was done in: 0.4.11-dev-649-g 47ac7a2b28df852a30e047426408bc368d37e5db
Ofc I moved the function up individually in each branch I do port this back to, without modifying the func.

[0.4.10][UDFS] Fix crash on boot in release builds with UDFS removable media inserted (#4061) CORE-17598

Changes to specific files and their effects are as follows:
create.cpp - Allows booting past second stage with UDFS media inserted without BSOD
close.cpp - Allows shutdown without hang
dircntrl.cpp - Allows New Hardware Wizard not to hang on initial third phase install

It began to crash with the introduction of the UDFS driver in SVN r74901 == git 3a104c8f20b540cb057609c5035200409dc9d83a

Fix picked from 0.4.15-dev-3326-g a91f5e8e4db2fc9af3d09175e537f16307db17f3

[0.4.10][NETSHELL] Fix MSVC compiler warning in de-DE.rc

MSVC2010 compiler warning shellext\netshell\lang/de-DE.rc(118) : warning RC4206 : title string too long; truncated at 256
introduced by SVN r71434 == git 6ce8ee357a964114dad69bfe7a3b6002ddf47f48

The string was displayed properly and was not truncated even before this patch.

Fix picked from 0.4.15-dev-2885-g 7962db4cc1fbd27770549ddb970013cc86fc8613

[0.4.10][DESK] Fix GetDC/ReleaseDC error management

Fix missing ReleaseDC related to the spectrum (color depth)

We leaked one device context for each bpp change within the
desk.cpl-session, which gave the following logging when closing desk.cpl:
(win32ss/user/ntuser/windc.c:749) err: [00060138] GetDC() without ReleaseDC()!
1 time for each leak.

Partial backport of (#2707)
Fix picked from 0.4.15-dev-2867-g d635ce0cc0336a9b77d243ddce946876f48ec4f6

[0.4.10][WORDPAD] Fix icons for toolbar and formatbar (#3792) CORE-5823

* CORE-5823 Fix icons for toolbar and formatbar in Wordpad.
which regressed by SVN r47776 == git 5dbfbb5201a915bc65d88f5818afc81cc6f54231

* Add wordpad_ros.diff noting toolbar.bmp and formatbar.bmp are not wine-synced

Thanks to the patches author Doug Lyons.

Fix picked from 0.4.15-dev-2862-g e068c684522ededfb0ffd70e8af736bb14c7b392

[0.4.10][CDFS] Fix a macro-copy-paste and shrink the binary size CORE-17405

picked from 0.4.15-dev-1456-g 889eab7

[0.4.10][EXT2] Fix filesystem corruption regressions CORE-17572 CORE-17195

It regressed when we upgraded Ext2Fsd to version 0.69 from version 0.68
via CORE-13980 in 0.4.8-dev-117-g a1d7e9936d8e58bc07ff2cc73a937ce845c7d542

The fix is a partial revert of that.
Thanks to the patches author Doug Lyons.

VBox https://reactos.org/testman/compare.php?ids=77904,77908 LGTM
KVM https://reactos.org/testman/compare.php?ids=77903,77907 LGTM

fix picked from 0.4.15-dev-2792-g cb408102cc3e3ed853a009962482251e45861699

[0.4.10][WINLOGON] Display the shutdown message popup dialog on the current input desktop. (#3259) CORE-17050

Display the shutdown message popup dialog on the current input desktop,
and periodically monitor for any change of the input desktop. When the
latter changes, close the dialog and recreate it on the new input desktop.

In addition, retain the current dialog position and restore it when the
dialog is recreated on the new desktop.

A patch authored by HBelusca.

The regression was introduced by 0.4.10-dev-579-g
c697f191cf72ce02d987aeac7cfff87174461622

This commit is merging back the changes of 0.4.15-dev-1096-g
0ff9b0ff7ec95f3b8cb833b450815a08af4d5c1d
(the minimized patch from JIRA avoiding most surrounding style changes)

[0.4.10][LSASRV] LsapLogonUser(): Fix 'AuthenticatingAuthority' copypasta (#2839)

Reported by contributor 'qarmin', found by static code analysis.
CORE-17074

The glitch once was introduced 7 years ago by SVN r58933 == git 7934e35cdf55b07334f04d86d9c7717b0cb41c0b
There is no real-world-issue known to date caused by this bug.

fix cherry picked from commit 0.4.15-dev-303-g 99d0a348aa4417672ad07720404765b4b2dbe9f5

[0.4.10][COMCTL32] Fix regression on showing current mouse pointer in main.cpl (#2829)

GetIconInfo() fill info.hbmColor with NULL HBITMAP handle on black/white bitmaps.

CORE-17062

It broke by 0.4.9-dev-719-g b3fb8555bf9abcc04eeac31c7153d33e2f027fc0

fix is squashed port of 2 commits:
0.4.15-dev-534-g 887764e6075fdde7fb81d2a54d5255164c046511 (fix)
0.4.15-dev-2705-g a3cadf630a9b3a61f4ae8525aef7af07e4b53c0d (nitpick)

[0.4.10][COMCTL32] Button.c Fix CORE-17260

Press'n'Hold of a button, then leaving the button rect
should redraw the button in unpressed state

Thanks to JIRA user 'I_kill_Bugs' for perfecting
my initial experiments to fix that.

It regressed by 0.4.9-dev-719-g b3fb8555bf9abcc04eeac31c7153d33e2f027fc0

No need to sync anything with Wine, their code is
correct on Wine head. The bug was in ros specific diff.

cherry picked from commit 0.4.15-dev-1085-g c947eb4d176603e0ff29217c079d8f5e1c2119e1

[0.4.10][SHELL32] Update all languages RCs to fix possible text truncation CORE-15765

IDD_BROWSE_FOR_FOLDER displays text controlled by the calling applications in
IDC_BROWSE_FOR_FOLDER_TITLE. Its old height could lead to text truncation
for existing 3rd party apps.
The new height value for IDC_BROWSE_FOR_FOLDER_TITLE
was inspired by IDC with the same from dialog IDD_BROWSE_FOR_FOLDER_NEW.
I could change all languages to the same height,
as dimensions of the dialog and surrounding controls were same.

picked from 0.4.12-dev-781-g 1d7287288de6073ae5a5db16727b169a57ff6ec1

[0.4.10][SHELL32] Fix version string in Shell About dlg CORE-16571, & some de-DE

It "regressed" and was displayed cut-off since the switch to git
(our string got longer back then).

Fixed by porting back 2 commits:

0.4.14-dev-976-g 6ce72ba172e69d25e6844244051e467d9d4b1e05
0.4.14-dev-884-g a57f7b50c01efadc50336b5b6664edbf645aff72

and also port back some german translation updates and
trivial whitespace fixes at the same time in *.rc.

[0.4.10][SYSDM] Restore the general tabs layout and soft localization update

Although the new layout that I committed in the previous commit
was favored by community ratings votes 8:0,
some devs still preferred the old layout with the bigger logo
because they liked the easteregg. So this commit restores
that.

But it syncs to that features latest&greatest implementation
we have in 0.4.15-dev-1685-gd0c237a instead of reverting the
previous commit to not unnecessarily have many bugs with it
that were still affecting the older releases beforehand.

Aside from the general tabs dialogs layout this commit does
also fix some minor issues in the translations for the oldest
branches in pl-PL, jp-JA, zh-CN and zh-TW.

And it fixes an x64 issue in general.c for 0.4.7 and 0.4.8.

[0.4.10][SYSDM] Fix version string cutoff CORE-17429

The version string cutoff started to happen when we switched from SVN
with its short revision number to git with the longer hashes.
0.4.7-dev-502-gc2c66af was the first git-only rev (2017-Oct-03)

and brings the dialogs layout closer to XP.
Also fixes some other truncations for specific languages.
And unifies the touched text controls sizes for all languages.

The credits have been moved to readme.txt where they are more present and
we can avoid having to groom all languages files each time we want to add
a new dev (and sysdm.cpl to grow each time). Less maintenance.

sysdm.cpl binary size I18N=all
master before 925.696bytes  after 705.024bytes  gcc 8.4.0 dbg RosBE2.2.1
0.4.14 before 923.136bytes  after 702.464bytes  gcc 8.4.0 dbg RosBE2.2.1
0.4.13 before 904.192bytes  after 684.544bytes  gcc 4.7.2 dbg RosBE2.1.6
0.4.12 before 881.664bytes  after 665.600bytes  gcc 4.7.2 dbg RosBE2.1.6
0.4.11 before 836.096bytes  after 618.496bytes  gcc 4.7.2 dbg RosBE2.1.6
0.4.10 before 806.912bytes  after 588.800bytes  gcc 4.7.2 dbg RosBE2.1.6
0.4. 9 before 788.480bytes  after 585.728bytes  gcc 4.7.2 dbg RosBE2.1.6
0.4. 8 before 788.480bytes  after 585.728bytes  gcc 4.7.2 dbg RosBE2.1.6
0.4. 7 before 788.992bytes  after 585.728bytes  gcc 4.7.2 dbg RosBE2.1.6

readme.txt still fits into a single NTFS cluster afterwards with 3702bytes

Also a very! soft update of some translations ported back.

[0.4.10][APISETS] api-ms-win-core-winrt-l1-1-0: Ro*() are NT6.2+ (Win8). CORE-16707 #2739

I double-checked and although the crash in Motorbike does *not* happen by default in all of those builds:
0.4.14-RC-54-g6a6672d (hidden by opt-in apisets-default in release branch)
0.4.13-release-1-g2ac9d98 (hidden by opt-in apisets-default in release branch)
0.4.12-release-17-g79780cd (hidden also, I don't understand why)
0.4.11-release-28-gc4d930d (hidden also, I don't understand why)
0.4.10-release-35-g94b4a3e (hidden also, I don't understand why)
0.4.9-release-41-gd3a79fe (hidden also, I don't understand why)
0.4.8-release-46-gd4d58d7 (hidden, does not have apisets yet)
0.4.7-release-56-g835b508 (hidden, does not have apisets yet)

It was really only crashing in master by default, not on the release branches.

I decided to port the proper fix back nevertheless, because:
- the exports are wrong for our target
- I can not guarantee otherwise that it would not pop up all of a sudden later
- avoids the crash in rls-branches in case the app was intentionally started in Vista-compat-mode
- and the fix also decreases the binary size of that apiset.

Fix picked from 0.4.15-dev-179-g 02825c20e4f94d7b9f0e7a1e5941ab415aeadb22

[0.4.10][SHELL32] Fix regression CORE-12364 explorer-sorting

The sorting in explorer broke by
SVN r73128 == git 24fcf531e78423b314a2d6a4ac750df69e478e3a

Thanks to Doug Lyons for providing this patch.

[0.4.10][COMCTL32] Treeview undo handling of WM_SYSKEYDOWN (#2731) CORE-12203 CORE-17020

Remove WM_SYSKEYDOWN handling at component level, in consistency with other components (ListView,...)
Early embodiement of the fix proposed to WineHQ:
https://bugs.winehq.org/show_bug.cgi?id=49097 in order to remove functional limitation in ReactOS.

Fixes regressions CORE-17020 and CORE-12203 that were once introduced
by SVN r72320 == git 297e33f2286ef0ec210108562cd0e87ad02d036b
during COMCTL32 WineSync 1.9.16 CORE-11866

Fix picked from 0.4.15-dev-191-g 295ba62820a1cdad86ebadc4d8a2d20c799a08d4

[0.4.10][APPWIZ] Don't hung up in Gecko download cancellation #3049 CORE-14538

Do not call EndDialog outside the dialog procedure.

fix picked from commit 0.4.15-dev-511-g 90456e54a42604909bf2d93cb74abbbf6574268f

This hang/crash in 2nd stage setup did *not* affect our official carrier-releaser isos that
we host on sourceforge, because those come bundled with Gecko already and therefore always
skipped the download.
Most users were safe in the past therefore.
Still everyone that would like to build an iso by himself could potentially run
into those crashes, and it did not only happen upon user-driven cancellation,
but also often crashed upon network failure / timeout.
A very important fix therefore.
For some unknown reason the likelihood of this crash gradually increased from
0.4.7 where it happened almost never, until 0.4.14 where it crashed much more likely.
Not well understood, maybe depends on memory layout or something similar.

[0.4.10][MMSYS] 100% sync of the module to 0.4.11-release-46-g12d8367

All of those are super safe commits.

excerpt of 0.4.11-dev-909-g 07b120bd5605350356578129112801bb40a60798 zh-CN [TRANSLATION] Update Simplified Chinese translation. (#1094)
excerpt of 0.4.11-dev-149-g 31019e8c0cbee14b8a6f3e30980bf0d95c175309 ro-RO [TRANSLATION] Update Romanian translation (#851)

mmsys.cpl binary size in bytes RosBE2.1.6 GCC4.7.2 dbg I18N=all:
0.4.11    which is the source 741.376
0.4.10  old 741.376   patched 741.376
0.4. 9  old 741.376   patched 741.376
0.4. 8  old 741.376   patched 741.376
0.4. 7  old 741.888   patched 741.376

Syncing the module will simplify future backmerges.

[0.4.10][MMSYS] Fix combo box and play icon on "Sound" properties page (#2690)

CORE-16935

I merged that back, because at least the play-icon was a regression.
It regressed by SVN r68073 == git 888c2fb9675536c1132222860378b85d1129d53e

Ftr we did not always use an icon here, but instead earlier we just had some
font glyph displaying the play-symbol. It was once changed to icon to improve
display for some asian localization (japanese? iirc).

fix picked from commit 0.4.15-dev-86-g bdb4da009acc13136a07f98a3c710687b75a7848

[0.4.10][DESK] Update zh-CN.rc & zh-TW.rc

Improves some strings, but actually I ported that back to reduce binary size.
The translated strings are shorter even than the english ones. Win-win!
Source was state of 0.4.12-release-31-g62822df.

desk.cpl built with RosBE2.1.6 on Windows with GCC4.7.2 debug building with I18N=all:
                                   539.136bytes for 0.4.12-release-31-g62822df (was source)
0.4.11 before 539.648bytes   after 539.136bytes
0.4.10 before 539.648bytes   after 539.136bytes
0.4. 9 before 539.648bytes   after 539.136bytes
0.4. 8 before 539.648bytes   after 539.136bytes
0.4. 7 before 528.896bytes   after 528.384bytes

[0.4.10][DESK] Dialog is too big still for et-EE.rc Estonian CORE-17428

et-EE.rc was added by 0.4.8-dev-930-g d80ef2c90a375db1e6aad689f4957962a5235a10
while in parallel the dialogs have been shrinked by 0.4.8-dev-939-g 2595538558f59671b6b68536c4e23d040e7bfc5c

Since both were under git PR review in parallel, that specific
language was left at the old dialog sizes accidently.

We can fix that glitch by syncing et-EE.rc to 100% to the improved state of that file in
0.4.11-release-44-g958556c because the glitch has been fixed by several commits before that rev.

[0.4.10][DESK.CPL] Remove monitor resource scaling (#876)

This improves both: the visual quality of the dialog and also
decreases desk.cpl binary size.

Fix picked from 0.4.11-dev-244-g 6f138e2811210a748ebec151dceceb89713080df

------------------------------
But it also inherently includes
0.4.11-dev-94-g 25f4ed75a01844f0e991bc22c821e5153aa952b3
"[CPL][DESK] Update monitor resource (#816)"
which replaces the CRT bmp with a TFT bitmap

[0.4.10][WIN32SS] Fix BSOD 0x1E nullptr deref via Git-bash CORE-16586

Thanks to the patches author Doug Lyons!

fix picked from 0.4.15-dev-1457-g 870aa1254f07f6ec7d8c1ad93d53fa6c6b18264b
and the whitespace addendum 0.4.15-dev-1458-g 449a3417862fe3b2b2847c5789f6e4d3bd0feb96

[0.4.10][APISETS] Fix last build issue for MSVC Rls-config CORE-17402

dll\apisets\api-ms-win-crt-utility-l1-1-0_stubs.c(41) : error C2169: 'llabs' : intrinsic function, cannot be defined

Allows to complete the whole 'ninja bootcd' after 'configure -DCMAKE_BUILD_TYPE=Release'
for compiler MSVC 2010SP1 16.0.40219.1 with RosBE 2.1.6

Similar to the fix committed in 0.4.15-dev-1453-g 4ad7b6d634bf2096eda1b97b45e9c5b15bbe80c7

[0.4.10][RAPPS] "Let's encrypt" Certificate-Issuer-Update hamster-wheel

"Let's encrypt" certificate issuer string changed within last 24hours remotely.
Update its name to make RAPPS stop complaining on DB update.

And also fix a bug in rapps mixing A()/W() in the
messagebox that informs about that cert check failure.

Fix is port of
0.4.15-dev-1120-g 12caaece255b216b452072d82faecfa98430d4b6
0.4.15-dev-1617-g f77f5a30cfd841bb94ae3c88627e1ae374807115

Next time this happens I will relentlessly undefine CMakeLists.txt USE_CERT_PINNING

[0.4.10][WIN32SS] Fix regression CORE-15165 LMarbles Installer

Fixes CORE-15165 "'LMarbles 1.0.6 installer' from rapps can not be restored after it has been minimized once"
for LMarbles-installer and many similar installers.

It regressed once by 0.4.10-dev-340-g 75b09f3f88b214149305faedd13b832ec5b99f16

Many Thanks to the fixes author Katayama Hirofumi MZ.

I decided to port back together with its user32-apitest 'SwitchToThisWindow' and that ran with
37 tests executed (0 marked as todo, 0 failures), 0 skipped.
in the back-port.

The fix is a squashed port of the following master-commits:
0.4.11-dev-632-g e39863bdf672cb3c3e6d83264e1212c4d281ea8b (#999)
0.4.11-dev-628-g 41b5f295142de30df8cff0e333b9a9cdc28b3d79 (#995)
0.4.11-dev-561-g 5467cc949322e942a9dfecd33167d3e15e6c5dc8 (#989)
0.4.11-dev-560-g 1f3353ac3be2d9204c44d7251c31586fcb07b114 (#988)
0.4.11-dev-559-g 4bd0166e633194df337b577488860d0a178e9daa (#987)
0.4.11-dev-508-g 4ccad609e0f07285dcf36c9a7bdad5b7c8dd8bad (#976)

test was added to master by:
0.4.11-dev-631-g 665269be9cac5d0045fa6440e1a390152347b9ff (#998)
0.4.11-dev-515-g 3e00e7fb22fe98e32f6d93c34e1d1cf058146758 (#980)

releases/0.4.10 was the only affected branch, so starting with today we have
no affected branch left.

[0.4.10][WIN32SS] Fix some regressions of font rendering / font placement

This commit deals with the following tickets:
CORE-14994 "Spotify 1.0.29.92 has no icons on the buttons"
which regressed by 0.4.10-dev-466-g 35f62fc5ba0b69e7335ff41400cb3b45660f4557

CORE-15166 "Media Player Classic HomeCinema MPC HC 1.7.13 'elapsed time' in status bar uses incredibly tiny font"
which regressed by 0.4.10-dev-466-g 35f62fc5ba0b69e7335ff41400cb3b45660f4557

CORE-15303 "Firefox 28 does not longer show the checkmarks in the menubar"
which regressed by 0.4.10-dev-466-g 35f62fc5ba0b69e7335ff41400cb3b45660f4557

CORE-16125 "Marlett, the 'top-right-closing-x' drawing regressed"
which regressed in master by 0.4.13-dev-110-g 19fc9415bfc811c6c0eef197f72d8bdffcfedba2 (in older releases by different commits)

---------------
The fix that manages to fix them all is a backport of the code-base that evolved up to 0.4.12-release-24-g4572c75. This was the source of porting.

*Before* the fix I found the following state for 0.4.11-release-39-gdeae917 by testing:
CORE-14994 ok
CORE-15166 ok
CORE-15303 ok
CORE-16125 affected

*Before* the fix I found the following state for 0.4.10-release-48-g72a4ecb by testing:
CORE-14994 affected
CORE-15166 affected
CORE-15303 affected
CORE-16125 ok

So the before-state was kind of exclusive-OR.
*After* the fix all of the mentioned tickets are fixed for releases/0.4.10 and releases/0.4.11

Ftr:
0.4.12-release-24-g4572c75 had all 4 tickets fixed, because it did receive the needed patches beforehand.
0.4.9-release-56-g9be0765 had all 4 tickets fixed, because it was never hit yet by the guilty commits in master branch. It does not heavily need those fixes yet therefore.

And last but not least, the backport to releases/0.4.10 will also inherently contain the fix for
CORE-15331 "[WIN32SS][FONT] Fix NEWTEXTMETRIC.ntmCellHeight value" (which was already fixed in initial releases/0.4.11)

In sum this backport contains the following commits (at least):
0.4.13-dev-483-g 641a0ea99be50975e79498e48ae73b2edfc5c616 (#1670) CORE-14994 == 0.4.12-RC-38-g d5c85cd
0.4.13-dev-110-g 19fc9415bfc811c6c0eef197f72d8bdffcfedba2 (#1535) CORE-15303 == 0.4.12-RC-16-g e1ac9dc
0.4.12-RC-43-g 6f651b6ec5d25072cda94dcc3eca12fdcede6a38 CORE-16125 (please note, that this is an interim solution and NOT yet the final master-fix of 0.4.13-dev-699-g b8383b4 which would have fixed that as well)
0.4.11-dev-914-g 88ddb37acba00219eda08593b1c9773bb4c86f39 (#1093) CORE-15166
0.4.11-dev-656-g 7f679a116b92f55e6f4364c689958a984595027f (#1010) CORE-15331

Porting any of those fixes on its own is dangerous, only their combination leads to tolerable results.

[0.4.10][WIN32SS] Fix regression CORE-15558 Font enumeration

Fixes CORE-15558 "AbiWord font listbox should display a specific font only once, but displays each of them many times instead"
which regressed by SVN r75491 == 0.4.7-dev-32-g d10c0ec1123111f6d31f9447c88234dd7255e07c

by porting back the commits:
0.4.12-dev-320-g 6e4e5a004c9da6276695d90187fb577812e8892f
and
0.4.13-dev-107-g ae8417fd9068dd32ec180636117bba1dc0603325 to prevent introducing another regression CORE_15785 with the initial fix.
and some more addendums to prevent introducing regression CORE_15755

I verified afterwards that all 3 issues: CORE-15558, CORE_15785 and CORE_15755 are in fixed state.

Actually I don't really like the resulting code due to the qsort() that it introduces into
win32ss/gdi/gdi32/objects/font.c
We did not need that at all before 0.4.7-dev-32-g d10c0ec1123111f6d31f9447c88234dd7255e07c
In case that qsort() would ever reveal any new trouble like reduced performance or stack overflow,
we should structurally revert all that context to 0.4.7-dev-31 instead.
IntEnumFontFamilies() is called immediately upon process initialization for many applications and the qsorted-list is *huge*,
so this indeed could be relevant for performance.

[0.4.10][CDFS_NEW][SDK] Replace intermediate workarounds with proper fix CORE-14067

Historically we did hit assertions and exceptions in MS CDFS driver
when special characters 'äöüßÄÖÜ' were used in a valid filesystem.

We did already apply varying workarounds for those issues
into CDFS_NEW for 0.4.8, 0.4.9, 0.4.10
while we were in the learning curve still.

Later we found the root cause to be our faulty XDK
FsRtlIsAnsiCharacter* macros.

By fixing the macros we can replace the varying historic workarounds
and sync to the final 0.4.11-dev master state.

We did not ever observe new issues with any other filesystems that do
also use those macros during the last years. So we can sync now.

[0.4.10][OLE32][SHELL32] Apply the proper Wine fix for regression CORE-14696

CORE-14696 "OLE32 regression, can't delete file or folder via context menu"
It regressed by 0.4.10-dev-161-g 85037eb74503634e8c16f489f70faa926f7c32a6

[SHELL32] We initially had an interim shell32 workaround from JIRA-user "sraboy"
committed to mitigate CORE-14696 into 0.4.10-RC-11-g 6f6f20f5dfb0f88e29c8ea218dc3be3470c94cb4
which allowed us at least to delete via context menu again, but copying
within the same explorer instance via context-menu was not handled by that yet.
We can revert that workaround today. The new OLE32-patch handles both cases properly:

[OLE32] Don't free handles that are just copied in the inproc case.
This imports wine commit https://source.winehq.org/git/wine.git/commit/25b136d40080a80c006b89e02708d2e248504a42,
by Huw Davies and Zebediah Figura
proper fix picked from commit 0.4.12-dev-96-g 959994b67cefbbfae862a7461c8ac25d486aab68

[0.4.10][WIN32K][NTUSER] Tiny codestyle fix to NtUserLockWindowUpdate (#1066)

Fix indentation (inconsistent with the rest of the file) and return actual BOOL.
picked from 0.4.11-dev-838-g 62ec69278d4f569736a053f7ebc53f9fd1becc2e

[0.4.10][NTUSER] Silence a noisy trace

picked just that part of 0.4.12-dev-121-g3ec7b16

[0.4.10][NTOS][USERSRV] Silence noisy debug output.

picked from 0.4.13-dev-82-g f5471b7b4b43c877ead8e2e6b845c0379d6dcc02

[0.4.10][FFS] Silence some only-informative KdPrints.

picked from 0.4.13-dev-924-g 40162252693a3a59b6d28d6f8d19a52c92a7cb22

[0.4.10][NTOSKRNL] Demote some noisy dprints

picked from 0.4.13-dev-988-g 1539fb1414d099096475ee7a859e8da54379ad79

[0.4.10][SETUPAPI] Demote some noisy dprints

picked from 0.4.13-dev-987-g 4872912eb26b7f4a5567dddface4aa96d7ee658a

[0.4.10][SETUP] Silence some noisy DPRINT1

picked from 0.4.12-dev-260-g 0150bb964a411474d6cbd60ee45421bb6281deae

[0.4.10][COMCTL32][WIN32SS] Fix multiple scrollbar redraw issues

This fixes:
- CORE-15911 "Scrollbars do not disappear when Maximizing and not needed"
- CORE-10617 "ListView corrupt scrollbar upon resizing the column-header"

and fixes regressions:
- CORE-15429 "Uninitialized scrollbars in 'My Computer' permanently drawn"
- CORE-16466 "Uninitialized scrollbars in 'My Computer' do still flash up for a fraction of a second before getting overpainted"
both unhidden by SVN r75735 == git 0.4.7-dev-168-g 6af37fd54e53658bb4d832d01e4ee2546af98835

by porting back the commits:
0.4.14-dev-312-g b931f643e35e6a23dbef99e785804039ea579b6a
0.4.13-dev-535-g 1158c241941a13d8374f3f1b3016c672c824e7f0

[0.4.10][WIN32SS] Improve many apps not starting with active taskbar pane (#2396)

Many thanks to the patches author JIRA-user "blueghost"
tibor.fuzi@gmail.com

The patch fixes regression CORE-16703, which was unhidden by explorer commit
8ad0eae6879029219ce3b62b554869a9d33aefa9 == SVN r74968
although the root-cause was missing functionality in WIN32SS.

The patch also fixes CORE-16705 and CORE-16706.
fix picked from 0.4.14-dev-1137-g 848d7ec63356d00ff965ed71bb4385ba68771d39
----------
And while touching winpos.c I decided to also port back the
fix for the format-string-glitch from #1893 authored by Stanislav Motylkov.
fix picked from 0.4.13-dev-1079-g b16ca9cd65c0279ef35dff5e0f0d733d7f1c77e4

[0.4.10][COMCTL32] Updown Suppress the NM_RELEASEDCAPTURE notification CORE-14365

This fixes a strange toggle-back-effect in 'Zeckensacks Glide Wrapper 0.84c'
by importing Wine commit
https://source.winehq.org/git/wine.git/commit/9dbe886e3ffd1c38a89f9cf256b7d172e408ec58
Thanks to patches author Akihiro Sagawa <sagawa.aki@gmail.com>

Fix picked from 0.4.13-dev-1169-g 841732e45c5abfc2572b88b392c09decdc119bf7

This was no regression, but I could not resist porting it back, because it's just
too tempting to fix a bug by deleting code. Win-win!

[0.4.10][COMCTL32] Fix regression CORE-14619 "Checkbox buttons theming"

It regressed by Wine sync in 0.4.9-dev-719-g b3fb8555bf9abcc04eeac31c7153d33e2f027fc0

Thanks to the patches author Andreas Maier <staubim@quantentunnel.de>
JIRA username 'andy-123'

Fix picked from 0.4.13-dev-531-g 49165276f27f79bfab5c48664865ef44cd40142e

[0.4.10][MSI] Fix regression CORE-14646

Pressing Next-button in setup of "1C Enterprise (training version) 8.3.10.2252"
did not have any effect anymore. Setup could not be completed anymore.

The regression was introduced to ros by 0.4.8-dev-739-g f8b992f2d31aaea06dd6a360ad19165d56c331dc
and was confirmed being a Wine-regression in
https://bugs.winehq.org/show_bug.cgi?id=45445

Thanks to Fabian Maurer for communicating to Wine and thanks to
Thomas Faber for regenerating cond.tab.c from the fixed source file cond.y
using BISON.

fix picked from 0.4.13-dev-443-g 10f00f8c1c844480146a251bff9c19f8ed3279a7

[0.4.10][WIN32SS] Fix assert with Winamp 5.666 CORE-10774

Fixes CORE-10774 "Winamp asserts upon startup when Bentoo skin is selected"

Reference:
Alexandre Julliard : Ignore empty rectangles in ExtCreateRegion.
https://source.winehq.org/git/wine.git/commitdiff/8ac61c5a1e56d0933de8af69dbb2a1640d13e05d

fix picked from 0.4.12-dev-979-g 2f5c0ee0a9ffc6619ba35ed120e023283fdb1a88

[0.4.10][WIN32SS] Fix BSOD CORE-15433 Opera 12.18 browsing duckduckgo.com

CORE-15433 "Fix assertless BSOD when Opera 12.18 browses duckduckgo.com"
fix ported from 0.4.12-dev-20-g 43e2089 (I ported it back earlier into 0.4.11-RC-7-g 7d0eadd already)

style_name == NULL is ok,
but some of the code was just not handling it properly and dereferenced it.

[0.4.10][WIN32SS] State of the art for positioning popups, menus and submenus

This is the hard work of Mark Jansen, and in sum reflects the state of 0.4.13-rls-0
when leaving out all work of Jim Tabor and Katayama in this file.
I picked only one unrelated single-line-commit of Timo Kreuzer additionally.

In sum it fixes the following tickets: CORE-16306, CORE-16297, CORE-15863, CORE-15001, CORE-16298.

And is a backport of the following master-commits:
0.4.14-dev-40-g e1984d0 CORE-16306 "Regression, UltraISO shows Languages popup menu at wrong position" (ported back earlier into 0.4.13-RC-12-g 609f2ae)
0.4.13-dev-815-g 6948764 CORE-16297 "Regression, The popup menus position is not perfect for systray icons context menu of Abyss webserver"
0.4.13-dev-814-g 84e263c CORE-16297 "Regression, The popup menus position is not perfect for systray icons context menu of Abyss webserver"
0.4.13-dev-813-g 1c8cdaa CORE-16297 "Regression, The popup menus position is not perfect for systray icons context menu of Abyss webserver"
0.4.13-dev-793-g b5c6af4 CORE-15863 "Regression, popup-menus that expand a sub-popup-menu may erroneously overlay itself"
0.4.13-dev-792-g 7c45a64 CORE-15863 "Regression, popup-menus that expand a sub-popup-menu may erroneously overlay itself"
0.4.13-dev-791-g a59df38 CORE-15863 "Regression, popup-menus that expand a sub-popup-menu may erroneously overlay itself"
0.4.13-dev-713-g 08c6d21 just fixes a compiler warning, unrelated one-liner, only commit of Timo in here
0.4.12-dev-369-g 007ec03 CORE-15001 "Wrong placement of the context menu in systray leads to autoselection, e.g. Abyss WebServer"
0.4.12-dev-346-g d2626f0 CORE-15001 "Wrong placement of the context menu in systray leads to autoselection, e.g. Abyss WebServer"
CORE-16298 "Window system menu does not appear where expected" which is a dupe of one of the tickets above, don't ask me which exactly

There is only one ticket in this context that will NOT get fixed in the final state, but we do accept that,
as it is minor and also MS Windows is not really deterministic in all cases for that:
CORE-15733 "[Win32SS?] dropdown-menu position may be wrong, when the window overflows the screen-borders"

This patch supersedes / is a better replacement for the safe-intermediate-state-fixes that I committed earlier into:
0.4.12-RC-49-g d52cac4 & 0.4.12-RC-48-g 3113d13
0.4.11-RC-35-g a9be77c & 0.4.11-RC-30-g 83e4a61 & 0.4.11-RC-27-g 1d47cfd

[0.4.10][WIN32K:NTUSER] Acquire the user lock exclusively in NtUserGetSystemMenu CORE-15512

It is rare, but I have seen this crash happening as well
when interacting with the taskbar quickly after the boot.

fix picked from 0.4.12-dev-203-g c526f94

[0.4.10][WIN32SS] Fix RIGHT-expansion-arrow in submenus being drawn too large CORE-13211

A squashed port of
0.4.13-dev-702-g 1e91a1690d554c771d703f12416321bbe4c5b53f
0.4.13-dev-809-g 65e4efd475e98b970fa5add48a13fd6360997d62
0.4.14-dev-186-g b3033b81a4256cd3482b36acbd60870ef1ddb23d
which in sum brings us to the nice state of 0.4.13-release-0-g4ee3778a4ed2d80c96e7a0f7a1e52e78d3a6568e
where I used the same trio successfully already.

CORE-13211 "RIGHT-expansion-arrow in submenus being drawn too large"
did affect many apps, e.g: UltraISO and Abiword
It was fixed by 0.4.13-dev-702-g 1e91a1690d554c771d703f12416321bbe4c5b53f (#1752)
but on its own it did introduce 2 regressions:

The first was that the
"upwards scrolling arrow in language selection submenu of UltraISO was now drawn much too large"
(discussion and pics also in CORE-13211)
That was fixed by the addendum 0.4.13-dev-809-g 65e4efd475e98b970fa5add48a13fd6360997d62 (#1753)

But we had still another regression from the first commit to solve:
CORE-16470 "The checkmarks in Abiwords menubar became black boxes"
That last regression was fixed by 0.4.14-dev-186-g b3033b81a4256cd3482b36acbd60870ef1ddb23d (#2009)

[0.4.10][WIN32SS] Fix scrollbar bug CORE-15557 & CORE-13986

Fixes CORE-13986 "Wise Installer draws scrollbars enabled while they should be drawn disabled"
by porting back 0.4.11-dev-586-g 929a2c6 (the mentioned CORE-ID in that commit was wrong due to typo)

which requires to pick also 0.4.14-dev-1167-g 8bb9434 to prevent the first fix introducing regression
CORE-16721 "Scrolling by pressing the scrollbar arrows does not draw the arrows as pressed"

and finally port back 0.4.14-dev-243-g 19fbdda to fix
CORE-15557 "disabled scrollbars being clickable"

In sum this brings the scrollbar code to the state of 0.4.13-release-0-g 4ee3778

[0.4.10][DESK][SHELL32] Allow Desk.cpl to run standalone on WinXP/2003 again

CORE-15412

addendum to 0.4.10-dev-502-g d559ca9

That copy-paste was done once intentionally to keep our imports similar to 2003.
It's worth reading the conversation in PR #748: Getequ seems to be ok with this.

The ideal solution most likely would be to instead *statically*
link against RegLoadMUIStringW() from advapi32_vista when WINNT=0x502.
For now the stuff runs again at least.
I left the func existing in advapi32_vista.dll.

fix picked from commit 0.4.12-dev-99-g 6f8cb05

[0.4.10][USERENV] sq-AL: Fix '%SistemDrive%' typo (#2469)

Not to translate!

Fixes CORE-16762 "inability to pass 2nd stage setup in Albanian"

The regression was introduced by SVN r62040 == git 258cfa34a0f1961cb45f4cae91f1ef6d4398b439

fix picked from commit 0.4.14-dev-1224-g
1edfddf07a7ef360eb3d5409a67217bd2d36bb0b

[0.4.10][JSCRIPT] Fix regression CORE-13810 "Vypress Chat 2.1.9 MSI setup fatal error"

The regression was unhidden by MSI-winesync 1.5.10 almost 7years ago:
8939969e21ddd1472e64596e11dee2ab58c9025e == SVN r57070

Finally we can fix it by adding MS enumerator extension to Jscript.
Many thanks to the patches author
Andreas Maier <staubim@quantentunnel.de>
JIRA-nick: andy-123

fix picked from commit 0.4.12-RC-18-g 9ebb42a4c01db05bb50be72fcfa49bd7a5a6afe4

It went afterwards into master in slightly modified form as 0.4.13-dev-890-g 83dcae1
But I decided to take the older state from 0.4.12RC, so that I do not have to risk
touching releases/0.4.12 again to sync to "latest&greatest".

[0.4.10][COMCTL32] Fix regression CORE-16280

The visible symptom for us was, that the Apply button in our
desk.cpl was always visible, even in case no user input was done in the dlg yet.

A very old regression. It regressed somewhere around
SVN r42990 – Good (2009-09-03)
SVN r44397 – Fail (2009-12-04)

The fix is a backport of 0.4.13-dev-908-g
344d5599351c5abae618dae9693a655cdc34dab0

Unchange the property sheet page after WM_INITDIALOG (#1842)
In my testing on Win2k3, EN_CHANGE is generated in OnInitDialog, that is correct.
The target is property sheet.
The property sheet does unchange the page after WM_INITDIALOG generation.

[0.4.10][USP10] Slightly tweak our intermediate fix

CORE-14226 was already effectively fixed by
0.4.11-RC-3-g 2fc4683
0.4.10-RC-3-g 71ea3f9
0.4.9-RC-4-g c19ce95
0.4.8-RC-5-g 937c261
and therefore no release ever was affected by it,
but today I undo the intermediate and sync to the final state of this fix
like we did commit it into master after Wine review
by master commit 0.4.12-dev-151-g c086edf36678e072afc58efcde12608c2c3c55eb
The final fixes commit message:

[USP10] Don't mix character and glyph index caches. CORE-14226

Import Wine commits by Nikolay Sivov:
* d1e618d31da usp10: Use glyph index consistently in ScriptGetGlyphABCWidth().
* 8d018d8d1da usp10: Make sure metrics cache is used with glyph indices only.
* eda3a3e37da usp10: Remove extra parentheses.

Fixes font display in UltraISO and GOG and many other installers.

See also https://bugs.winehq.org/show_bug.cgi?id=44410

[0.4.10][COMCTL32] Fix regression with disabled menubar buttons in IDA5free CORE-14209

PR #1599 "[COMCTL32] Saturated images: Take mask-images in account."
Many Thanks to the patches author
Andreas Maier <staubim@quantentunnel.de>
JIRA-nick: andy-123

CORE-14209 IDA5free issue with disabled menubar buttons
regressed by SVN r66418 == git
8370f1581307de727f906dd8fc899f9aa233b368

The patch also improves the rendering of disabled menubar buttons for SumatraPDF CORE-10601.

Even after the patch we are not perfect at deciding when to draw masked
and when to draw desaturated in 32bpp, but it definitely improves end users experience.

Fix picked from 0.4.13-dev-881-g
4093681ad0f66fda0a9a6d65a6a6af8fed1941b4
and as preparation 0.4.13-dev-880-g
49a797543f1201726999f0d67fabed327b58744f

[0.4.10][COMCTL32] Address regression CORE-14622

"Avira Antivir Personal 6.35.00.243" had issues painting its
setup wizard dialog with correct background color.
That part was a regression of SVN r74136 == git
b1b42795a94d8e3cb69e2d9e5bd0af4929610361

and also did not draw its umbrella bmp, which did never work
in ros up to now afaik.

https://reactos.org/testman/compare.php?ids=69394,69478 VBOX
https://reactos.org/testman/compare.php?ids=69395,69479 KVM

Thanks to patches author Doug Lyons and HBelusca.

Wine has been reported to be affected by that bug as well:
https://bugs.winehq.org/show_bug.cgi?id=46636
Not sure whether our patch will be accepted by Wine,
to my naive eye this looks like an interim solution.

fix picked from commit 0.4.14-dev-107-g
f2f833c53c27c58cf9b762fcc967aaaf41d2668f

[0.4.10][WIN32SS] Fix last user visible parts of regression CORE-10855

When Themesvc was disabled, some Window titlebar
frames were drawn too thin until the window
was actively resized in y by the user.

That and similar symptoms started to show up after SVN r68904 ==
6dfa71c487dbb193ed7fb1a249a8c964ec3aef0d

Many thanks to patches author Doug Lyons.

fix picked from commit 0.4.14-dev-121-g
9e14a762d5fa9a0a46ca5b0eb89b5932a8be52dd

[0.4.10][WIN32SS] Fix maximized window positioning and 2 addendum commits

Fix CORE-5177 "TaskMgr and ProcExp positioning when maximized."

which regressed by SVN r47264 == git
0ac861114eb408c03c8c110704244c719c4eb3db

fix cherry picked from commit 0.4.12-dev-954-g
1f619b27814f3bbf9673754a69bcae50c89fed19
-----------
but since above commit did introduce a new regression CORE-15893
symptom "Most of maximized windows have extra row of pixels around"
I had to port back those 2 follow-up-master-commits as well:
0.4.12-dev-956-g
197c4f875d3332dbc4656a6360e3c7e9def2ed5
and 0.4.13-dev-397-g
3cc50cedaf5e74041608d403054d1fe6394596e6

Side-note: This results in the same final state as 0.4.12rls0, because
I ported 0.4.13-dev-397-g3cc50ce into 0.4.12-RC-29-gab7f77a earlier.

[0.4.10][SHELL32] Fix SHFileOperation Move operation (#1487)

Reduce failures of Move operation of shell32!SHFileOperation function. CORE-13450

And also fixes regression CORE-13176 "Drag and drop with multiple files is broken"
which regressed by SVN r74380 == git
fd91fe3337ca9e758cc9f07aea8503fdd24da206

fix picked from commit 0.4.12-dev-1080-g
c65156002c184222e8307e0299ece359809db4dc

[0.4.10][WIN32SS:NTGDI] Fix missing text in labels and buttons for .NET apps.

Fixes CORE-15000 "Totally Broken/messed up text in many .NET applications"
and its duplicates
CORE-14679 "Missing fonts in Reg Organizer 8.16"
CORE-15398 "Messed fonts in Microsoft SQL Server 2005 Express Edition"
CORE-15611 "Word 2003: Organization Chart has corrupted display"

There was a missing break statement in ftGdiGetGlyphOutline().
Found by JIRA contributor 'I_Kill_Bugs' :-)

Add also brackets around 'case's to make code navigation easier.

Fix picked from 0.4.12-dev-865-g
0e56ec8892a737823b4b008f02ae0261cbf0b64a

[0.4.10][WIN32K:NTUSER] Fix CORE-12243 OS-freezing when closing nLite

This fixes CORE-12243 and allows .NET applications like nLite 1.4.9.3
to exit properly without freezing the whole OS.

The actual fix was picked from 0.4.11-dev-78-g
4d48b88bfbfcb6845e8742f78154156e984f6363
"[WIN32K:NTUSER] co_UserDestroyWindow(): Simplify the destruction of the owned windows."

but it made sense to also pick the related 0.4.11-dev-77-g
e286c4c5202ac2d6e7909ee05cdc95f76b8f4897
"[WIN32K:NTUSER] Optimize IntWinListOwnedPopups() a little bit. Improve a trace."

and 0.4.11-dev-76-g
f644a50cd77492dc06615a632ca500a1687aa63b
"[WIN32K:NTUSER] Code formatting only."

[0.4.10][WIN32SS] Require at least the amount of extra memory, not exact amount

Fixes 3 application crashes:
CORE-15576 "Civilization II Multiplayer Gold Edition crashes when trying to open a map"
CORE-13924 "Click-n-type Virtual Keyboard 3.03.4012 from rapps crash when opening program preferences"
CORE-8790 "IceChat crashes with OutofMemory"

It regressed around SVN r56947, r56948, r56949.

cherry picked from commit 0.4.12-dev-417-g
e022d8d7cb72bc611d6990062ad0f2962c0cab1d

[0.4.10][COMCTL32] Workaround v6 button redraw regressions CORE-13278 & CORE-16093

Most likely not correct, as it breaks comctl32:buttonv6 tests
https://reactos.org/testman/compare.php?ids=68146,68147

But greatly eliminates perceived flickering when hovering
with the mouse over buttons in unthemed mode.
We suffered from these regressions for years and nobody did
find a proper solution, therefore it's time to at least workaround them.
The patch did not expose any new real-world-issues after I retested
the whole rapps suite against it in 0.4.12RCs.

CORE-13278 was once introduced/unhidden by SVN 73806 ==
c607de7

CORE-16093 was once introduced/unhidden by SVN 74149 ==
301c476

The comctl32 part of the fix was picked from 0.4.13-dev-1194-g
a40331b0f4c0a423fae101dfc976376d7c7db647

[0.4.10][NTOSKRNL] Restore section layout for NTOSKRNL to fix BSOD regression CORE-14683

Thomas Faber thinks that it's a bit more
safe to just move "after .rsrc" instead of the proper "after .reloc".
A temporary tribute until ROSBE-154 will be fixed properly.

For now we use this just for NTOSKRNL, but not for drivers yet, see
MODULE TYPE sdk/cmake/gcc.cmake
As using also for drivers would immediately lead to BSODs like CORE-16183.
To allow installing DVDWritenow without BSOD,
we need the script at least for ntoskrnl!

The regression was introduced by SVN r55835 == git
6ebf8c253770af0c029f1d7937a33235108d59e3

This patch (after .rsrc) was already committed into 0.4.15-dev-220-g
d28677795e98cdd4fdf0f4aff2773197a6b39718

[0.4.10][EXPLORER] systray notification area - icon crosstalk on NIS_SHARED or NIS_HIDDEN CORE-14262

Fixes a regression introduced by SVN r68893 == git 332652615adb97c3b328d558f5f3eb21a902afa1
Symptom "TightVNC Server may show a doubled icon and/or corrupts its neighbor icon in the systray"
Fix picked from 0.4.13-dev-481-g8d6c9e8
------------------
I allowed myself to also port the fix for some TRACE-loggings missing ending "\n".
That second regression was introduced during 0.4.8-dev'ing when substantial parts of this code
were moved from trayntfy.cpp into syspager.cpp.
releases/0.4.7 was not affected by that yet.
That 2nd fix was picked from 0.4.13-dev-480-g7796df3

[0.4.10][WINLOGON] Start the screensaver on the user desktop CORE-16548

since support for secure screensaver desktop is not
completely implemented in Winlogon.

This fixes a regression introduced between
0.4.10-dev-579-gc697f19 and 0.4.10-dev-582-g3270979

fix cherry picked from commit 0.4.14-dev-581-g
40db38571689131bb8692f7bde7e9c37215edbce

[0.4.10][WIN32K:NTUSER] Reference menus owned by a popup menu. CORE-15504

Fixes a BSOD mentioned in CORE-15504

cherry picked from commit 0.4.12-dev-218-g
e3c8002dfc79e478a6d156c52abd4e243d36d754

[0.4.10][NTOS] Fix a regression in the PE loader CORE-11717

which caused images to be loaded up to MmHighestUserAddress,
thus overwriting the shared user page.

This is now fixed by using MM_HIGHEST_VAD_ADDRESS as the margin instead.
This commit allows again to load WIN32K.sys of Win2K3SP2
which regressed by SVN 67793 == git
3565260212c7b6f12a0fc74dc86aa84ab5590002

fix cherry picked from commit 0.4.14-dev-143-g
56da95671ce87fc3c9fffbf33f411759ce330e49

[0.4.10][WIN32K:NTUSER] Avoid dereferencing a null pointer in NC_DoNCPaint.

Fixes BSODs CORE-15569, CORE-12963, CORE-13410

For non-technicians:
Author Thomas Faber just imprisons the BSODs into an artificial cage
made of curly-braces. The BSODs can't h(a)unt any users anymore
and inevitably starve therefore.

cherry picked from commit 0.4.12-dev-338-g
b2dcb8e0d8925fc1aed2adeaf74ad0291e29f8f6

[0.4.10][NTUSER] Fix regression CORE-11828

Fixes symptom "mouse pointer vanishes when closing
applications via x in title-bar"

The bug in WIN32SS was unhidden by SVN r65496 == git
5c1a8109c84ed0aefdac2488ff666070a7ae52f6
, when we introduced explorer_new. The old explorer
had a workaround built within.

Many thanks to patches author James Tabor.
cherry picked from commit 0.4.14-dev-1012-g
820dbfccd015d8890d5948b6abd5089e8ab8cf86

[0.4.10][WIN32K:ENG] Fix fullscreen switch regressions CORE-15325 & CORE-14684 PR#1659

In PDEVOBJ_vRefreshModeList, find a proper match for the current
display mode instead of assuming a constant index.

Both tickets CORE-15325 & CORE-14684 regressed by SVN r74404
384affe098a91ce38623e4703f25ab139eff68de

fix cherry picked from commit 0.4.13-dev-456-g
00e882c2b1ba302d01c2c63a803d78e4fbac7168
Many thanks to the patches author Thomas Faber.

With this commit we sync PDEVOBJ_vRefreshModeList() to the state I used in 0.4.12rls and 0.4.13rls.

This patch does fix:
- regression CORE-15325 "Pocket Tanks 1.6 not filling the whole screen because it does not switch its resolution when coming from desktop res 1024x768 32bpp"
- regression CORE-14684 "Fall 0.1.3 opens just a white window instead of content when started with desktop resolution 1024x768 32bpp"
- regression "Globulation 2 0.9.4.4 not filling the whole screen when configured to use fullscreen and running it from desktop resolution 1024x768 32bpp"

but also helps to fix the last app that was still affected by CORE-14363 "systray icons do flash through now when an app is running in fullscreen".
CORE-14363 actually was fixed for >95% of the apps already by my last commit into the releases, but there was an outlier that does require Thomas patch additionally, that was:
- "Pengupop 2.2.3 when being run from desktop with 1024x768 32 bpp"

I committed it beforehand:
- for master:    0.4.13-dev-456-g00e882c
- for 0.4.12rls: 0.4.12-RC-34-gec3cc13
- for 0.4.11rls: 0.4.11-release-4-g11e0b0b

[0.4.10][WIN32SS] Fix many regressions of fullscreen switching, all caused by SVN r75407

CORE-16393 'switching to fullscreen via Alt+Enter in cmd now leaves the taskbar visible'
CORE-15477 'TileWorld 1.3.2 when started in fullscreen mode it now leaves the taskbar visible'
CORE-14979 'It depends on the desktop color-depth bpp whether taskbar is hidden in fullscreen of Anno 1602'
CORE-15599 'Rocks&Diamonds 3.3.1.2 from rapps leaves its taskbar visible when switching into fullscreen via alt+enter'
CORE-15600 'LGeneral from rapps leaves the taskbar visible when selecting fullscreen from context menu'
CORE-15654 'Exiting "Bound Around Demo" from fullscreen leaves black borders around the desktop'

All those cases regressed by SVN r75407 == git
09ab5ea7ed2ab0e709ed2e8f8f9f6c3711b2ac7f

To date I am not aware of more that broke by that historic commit, but we still have some cases that still
do not work as intended, they are linked as blocker against the EPIC CORE-11242.
But at least no regressions known anymore to date by that specific commit into display.c.

In sum this brings us in the context of this regression in this file
for the functions UserChangeDisplaySettings() & NtUserChangeDisplaySettings()
to the state of
- for master:    0.4.14-dev-38-g96ef605
- for 0.4.13rls: 0.4.13-RC-11-g0a03b36
- for 0.4.12rls: 0.4.12-release-4-g8c5e349
- for 0.4.11rls: 0.4.11-release-3-g29e0b55

[0.4.10][MSPAINT] Revert LoadDIBFromFile for workaround (#2186) CORE-16566

This fixes the regression of mspaint not being able any longer to
overwrite an existing file.
The regression was once introduced by
0.4.7-dev-118-g
44fc547ffa910f1bce117f739c19c714317db172
when trying to determine the resolution on load.

Our new fix does cover both issues and even reduces imports: win-win.

fix cherry picked from commit 0.4.14-dev-691-g
be40a0e29611fb45e4b314c81be373774808b1c4

[0.4.10][WINED3D] Workaround regression CORE-15408

Many 3D applications crashed with 'WineD3D fake window' msgbox
if they were used in VirtualBox with graphics driver
'VBoxDisp.sys from VM guest additions + installed 3D acceleration'

That was a regression of SVN r75847 == git 0.4.7-dev-319-g
3d13ed394efdd0ce0aef4e10965b1d08334394ec

Contrary we had no such issues with the SW emulation.
The reason for the bug is that
VBoxDisp only supports OpenGL 2.1 (GLSL 120) and WineD3D
tried to use shaders for OpenGL 3.2 (GLSL 150).

This workaround disables the usage in a hard-coded way for
all graphics adapters. Should be ok for now, as ros does not
really support any modern GPU drivers yet.
Proper working runtime detection would be better ofc.

Thx to Doug Lyons & Fabian Maurer

Fix cherry picked from 0.4.14-dev-113-g
082f230ac76087915355c962a307f21ade9dae96

[0.4.10][GDIPLUS] Fix a regression painting gradient CORE-15479

patch is import of Wine commit 0937186f7d15fed60f77fa2014d650f4d0b6b20b
by Nikolay Sivov

regression was introduced by SVN r75872 == git 0.4.7-dev-344-g
c6f32e4a5f8b998a1efb7cd4698cf8247bea1adc

fix cherry picked from 0.4.12-dev-360-g
daadcc61414330c909625c08fe2b909c0bf5621f

[0.4.10][CDFS] Make windows gcc release build compile CORE-15659

This fixes a regression introduced by 0.4.8-dev-164-g
ec6b3ecbe48ad51c366563196cc93a6ae74e8b69
when the new driver was activated for compiling.

Many thanks to Thomas Faber, Pierre Schweitzer, Serge Gautherie

The fix is squashed backport of commit 0.4.12-dev-431-g
bccad87f3ccf266be2179d1322e7a422aeb2f3b7
and commit 0.4.12-dev-432-g
3463b2db9fc08e5ebe2f5b146d3fe12248f632f3

[0.4.10] [SDK] Remove 'RC' from version.cmake

[0.4.10] [RICHED20] Revert ocr_reverse.cur to fix CORE-15190

Old cursor looks better.
We revert to SVN state r75865 and note the diff for Amine
so that the wrong version won't come back in the future.

Old file is just 326bytes while the one that we do not want is 33.134bytes
which means we will save some RAM as well.

cherry picked from commit 0.4.11-dev-634-g
ad34c91d1c003a9e78b7b9795c3b5a08a633a5e9

[0.4.10] [FONT][WIN32SS] Include the style name in the registry key.
Thanks to Doug Lyons for pinpointing the issue.

CORE-14044

cherry picked from commit 0.4.11-dev-311-g
3d44f79e3e6fb72b36f555b3fcb281790c13dcd6

[0.4.10] [FONTS] Fix underline/underscore position of Tahoma (#892)

Because of FontForge's bug, the underline position value of Tahoma and
Tahoma Bold was wrongly set (#789, #809). This commit fixes those values.

CORE-15079

cherry picked from commit 0.4.11-dev-309-g
b560491ccf86d226a4720f784ddaa7dd03ee2504

[0.4.10] [SHELL32] In _SHExpandEnvironmentStrings(), fall back to SHExpandEnvironmentStringsForUserW() if the simplified code paths failed.

This allows the code to give a last chance for expanding the string.
And if it still fails, we bail out as expected and the caller will
take appropriate actions.

As a by-product, this commit (together with the previous one e2960914)
fixes the regression CORE-14981 (Quick-Launch bar couldn't be opened
in LiveCD).

cherry picked from commit 0.4.11-dev-208-g
7ee298bbfc3838e7fbf92d775206ca9f4d0c38dc

[0.4.10] [SHELL32] In _SHGetUserShellFolderPath(), check the return value of _SHExpandEnvironmentStrings()
and bail out if it failed. That way we won't use and cache in the registry an invalid-expanded string.

CORE-14981

cherry picked from commit 0.4.11-dev-207-g
e29609147b6da48f4a0166b39be8664a996b06b8

[0.4.10] [WIN32K:NTUSER] Fix WIN32K BSOD mentioned in CORE-15027

Fixes accessing nullptr / invalid handle in Window->SystemMenu
I could trigger this occasionally when moving Opera 12.18 window
in front of Spotify 1.0.29.92 window.

Fix was:
authored by Thomas Faber (Thank you!)
tested by me
reviewed by James Tabor

cherry-picked from 0.4.11-dev-195-g
ef016bfe0a9dc3cc7924671277b44884e39d89d5

[0.4.10] [WLNOTIFY] Disable the Logoff-Handler routine for the scheduler service.

The Logoff-Handler might try to notify the scheduler service which might already
have been shut down. We need to fix the shutdown sequence,
so that the service manager stops all running services.

Might fix CORE-14521 according to patches author ekohl.

I tested myself, it properly works around regression CORE-14999 aka
"Long time to shutdown after installing .NET Framework"

cherry picked from commit 0.4.11-dev-189-g
5e2fe08918fc5a3c52d586a8dc8b8df851473c20

[0.4.10] [FREETYPE] Fix Regression "Monospaced font in LibreOffice" CORE-15018

LibreOffice started to show GUI in a mono-spaced font after 0.4.10-dev-466-g
35f62fc5ba0b69e7335ff41400cb3b45660f4557

We can fix this by partial revert
as proposed by Katayama Hirofumi MZ, who was author of those changes as well.
We revert the changes in penalty system here.

cherry-pick from master 0.4.11-dev-126-g
31c981c02a8c1aa2d686e2efc0a1debecd16ffa8

[0.4.10] [WIN32SS][FONT] Fix GetTextFace function and related (#829) CORE-14926

Google Chrome with -no-sandbox parameter in ReactOS wouldn't display
the web page anymore because first-chance exception raised.

- Remove FullName, Style, and FaceName members from TEXTOBJ structure.
- Add TextFace member into TEXTOBJ structure.
- Add MatchFontName() and MatchFontNames() helper functions.
- Fix GetTextFace() and related.

The symptom started showing with
0.4.10-dev-466-g
35f62fc5ba0b69e7335ff41400cb3b45660f4557

The fix was cherry-picked from commit
0.4.11-dev-48-g
c8749d379beeb3a078c296feca1f4cdb3e9d11f1

Many Thx to Katayama, Mark & Doug.

[0.4.10] [SHELL32] Avoid regression CORE-14974

Adjust the IDC_PICKICON_LIST control's height in the IDD_PICK_ICON dialog resource. (#824)

It changed the number of rows from 4 to 3 unintentionally by
0.4.10-dev-466-g
35f62fc5ba0b69e7335ff41400cb3b45660f4557

The fix was cherry-picked from
0.4.11-dev-30-g
39dc3cdaf6dbd5bbb9187f5b407300be1a9803a2

[0.4.10] [USER32/COMCTL32] Avoid regression CORE-14906

This patch has not been committed to master yet.
It fixes the vertical position of button text,
which was shifted one pixel to the bottom since guilty commit
0.4.10-dev-466-g
35f62fc5ba0b69e7335ff41400cb3b45660f4557

[0.4.10] [CONUTILS] Fix regression with file redirection of 'more' CORE-14592

In ConWrite(), emit an \r when a \n is encountered but not already preceded by \r.

And, don't emit \r\n when an \r alone is encountered.
This fixes the problem of extra newlines appearing when redirecting
"more" output to a file.

CORE-14592

cherry picked from commit 0.4.11-dev-1-g
b277cbdf222a67f7bf4e7acc246189be91474aa0

[0.4.10] [ADVAPI32] Avoid regression CORE-14874 "Java 7u80 fails to install"

by reverting "[ADVAPI32] Additions for CreateProcessAsUserA/W()."
This reverts commit 0.4.10-dev-260-g
d4b4da5385b76a7c1c7a1b0e2d52e6cf4a448c11

Afterwards I could install Java 7u80 again
and the Java-dependent app "pdfSam 2.2.4" did install and run.

[0.4.10] Avoid regression CORE-14955 "Ddraw fullscreen crashes"

This reverts commit 0.4.10-dev-55-g
7af3969e9fe68b5f55a088f93a6b69570822c733

and therefore downgrades D3D8,D3D9,DDRAW,WINED3D from 3.9 to 3.3 for this rls.

This is still newer than last release where we had to go back to 3.0 even.

Purpose of this revert is to fix crashes when Ddraw apps switch
into fullscreen with VBEMP and inbuilt Mesa.
I tested, before:
 DxDiag crashed when switching to fullscreen,
 Diablo II crashed immediately,
 Monster-Truck-Madness 2 demo crashed after main menu

Afterwards all of these apps do run.
DXTN does still work after that, even with VBEMP and inbuilt Mesa.

This will be the interim fix, which I will keep for rls at least
until someone manages to fix CORE-14955 properly in master with Wine 3.9.

[0.4.10] [CDFS] ease further observation of CORE-14067

re-enable assert (requested by Pierre)
but then at least log the ambiguous file-name (in case it'll assert again)

[0.4.10] [CDFS] state-of-the-art-fix for CORE-14067

Prevents CDFS asserting on more characters, tested for "äöüßÄÖÜ" at least.
Thanks to Pierre Schweitzer for that generalization.

This patch supersedes 0.4.10-RC-14-g
15c578fe0d02464d0fc8583db46bb596999510a4

[0.4.10] [CDFS] explicitly allow 'ü' character in filenames CORE-14067

A patch from Doug Lyons to appease regression CORE-14067 that works well for me.
Eventually the real bug is in func FsRtlIsAnsiCharacterLegalHpfs() ?

[SHELL32] Fix CShellLink::GetIconLocation (#810)

Fixes a regression from PR #784.
CORE-14961

cherry-picked from commit 0.4.11-dev-11-g
6eb6aa289518b530f40b065837d2bcf7506595e5