reactos.git
8 days ago[0.4.10][COMCTL32] IP Address ctrl CORE-9853, LVIS-CUT CORE-16722, RC, compiler-warning releases/0.4.10
Joachim Henze [Wed, 17 Apr 2024 23:57:37 +0000 (01:57 +0200)]
[0.4.10][COMCTL32] IP Address ctrl CORE-9853, LVIS-CUT CORE-16722, RC, compiler-warning

This ports back:
0.4.15-dev-7156-g 414f3da7a62f5ce6d066ef135fc15681361a7495 [COMCTL32] IP Address control: Fix background drawing (#6133) CORE-9853
0.4.15-dev-6773-g 855008d97b2c7e1a1b72ad268b68d83c739ebc6d [COMCTL32] Invalidate LVIS_CUT selected items too (#4218) CORE-16722

And port back a few selected RC-improvements.

To compensate binary increase, I stripped a few TRACE's and debug-functions in the older branches within listview.c.
That fixed the following compiler warning for (GCC) *rls*-configuration:
warning: variable 'prev' set but not used [-Wunused-but-set-variable] within the function ranges_assert()

comctl32.dll binary therefore shrinks on all branches:
comctl32.dll master    RosBEWin2.2.2 GCC8.4.0dbg               2.001.408 (0.4.15-dev-7906-g1b85a5f)
comctl32.dll 0.4.14rls RosBEWin2.2.2 GCC8.4.0dbg  1.997.824 -> 1.982.976
comctl32.dll 0.4.14rls RosBEWin2.1.6 GCC4.7.2dbg  1.807.360 -> 1.794.560
comctl32.dll 0.4.13rls RosBEWin2.1.6 GCC4.7.2dbg  1.803.264 -> 1.790.464
comctl32.dll 0.4.12rls RosBEWin2.1.6 GCC4.7.2dbg  1.817.088 -> 1.803.264
comctl32.dll 0.4.11rls RosBEWin2.1.6 GCC4.7.2dbg  1.809.408 -> 1.796.096
comctl32.dll 0.4.10rls RosBEWin2.1.6 GCC4.7.2dbg  1.806.848 -> 1.793.536
comctl32.dll 0.4. 9rls RosBEWin2.1.6 GCC4.7.2dbg  1.806.848 -> 1.793.536
comctl32.dll 0.4. 8rls RosBEWin2.1.6 GCC4.7.2dbg  1.632.768 -> 1.618.944
comctl32.dll 0.4. 7rls RosBEWin2.1.6 GCC4.7.2dbg  1.602.048 -> 1.588.224

11 days ago[0.4.10][COMCTL32] status.c sizegrip-backports 2024-04-15
Joachim Henze [Mon, 15 Apr 2024 16:23:26 +0000 (18:23 +0200)]
[0.4.10][COMCTL32] status.c sizegrip-backports 2024-04-15

This ports back 3 commits:

0.4.15-dev-7861-g a771729e696827e3696699522e63548fecee9550 [COMCTL32] Fix Status Bar Updating (#6696)
which fixes CORE-19497 'Regression, part of the statusbar with the sizegrip is not redrawn properly'
which regressed on this branch by 0.4.10-release-210-g 2466b5d6d95b9306a8ac61b02a7a1c9c27fa178f

also port back:
0.4.13-dev-888-g 76dfa2b284b03e276b91782eff69fcecdd72210c [COMCTL32] Checking size grip bounds in x-axis only (#991)
0.4.13-dev-887-g b9ddad0bbb2cf7f9501b401d84b6847f695b1ddf [COMCTL32] Hide size grip when the parent window is maximized CORE-15240 (#991)

To compensate binary increase, I stripped a few TRACE's and the debugstr_t() function in the older branches.
comctl32.dll binary therefore has a tendency to shrink a tiny bit on most branches:
comctl32.dll master    RosBEWin2.2.2 GCC8.4.0dbg               2.001.408 (0.4.15-dev-7906-g1b85a5f)
comctl32.dll 0.4.14rls RosBEWin2.2.1 GCC8.4.0dbg  1.998.336 -> 1.997.824
comctl32.dll 0.4.14rls RosBEWin2.1.6 GCC4.7.2dbg  1.807.872 -> 1.807.360
comctl32.dll 0.4.13rls RosBEWin2.1.6 GCC4.7.2dbg  1.804.800 -> 1.803.264
comctl32.dll 0.4.12rls RosBEWin2.1.6 GCC4.7.2dbg  1.817.600 -> 1.817.088
comctl32.dll 0.4.11rls RosBEWin2.1.6 GCC4.7.2dbg  1.809.920 -> 1.809.408
comctl32.dll 0.4.10rls RosBEWin2.1.6 GCC4.7.2dbg  1.806.848 -> 1.806.848
comctl32.dll 0.4. 9rls RosBEWin2.1.6 GCC4.7.2dbg  1.806.848 -> 1.806.848
comctl32.dll 0.4. 8rls RosBEWin2.1.6 GCC4.7.2dbg  1.633.792 -> 1.632.768
comctl32.dll 0.4. 7rls RosBEWin2.1.6 GCC4.7.2dbg  1.602.048 -> 1.602.048

2 weeks ago[0.4.10][REGEDIT] Fix condition of enabling load/unload hive menu item (#6251), ...
Joachim Henze [Fri, 12 Apr 2024 06:56:18 +0000 (08:56 +0200)]
[0.4.10][REGEDIT] Fix condition of enabling load/unload hive menu item (#6251), + RC-stuff

Port back:

0.4.15-dev-7362-g 1aeac0bf24e716460d10702c846859600a9754e6 [REGEDIT] Fix condition of enabling load/unload hive menu item (#6251) CORE-19298
which was an addendum to 0.4.15-dev-243-g 4d1cd723175c25a76c17f58cb93f4e83dfd8881c (where I ported back parts earlier from)

and ports back various unrelated RC-tweaks, e.g.
partially 0.4.15-dev-7887-g 64a59a17b5e2828adb1a81e24647e5ec1266e78e [REGEDIT] RC-tweak, only the it-IT-addendum
0.4.15-dev-6940-g 03dc2ef724806614983664509214dcb87c68a6c2 [REGEDIT] Update Romanian (ro-RO) translation (#5961)
and a few others

regedit.exe binary does shrink a tiny bit on all branches:

regedit.exe master          RosBEWin2.2.2 GCC8.4.0 dbg x86            1.049.600 (0.4.15-dev-7886-g938a39a)
regedit.exe releases/0.4.14 RosBEWin2.2.2 GCC8.4.0 dbg x86  755.712 ->  755.200
regedit.exe releases/0.4.14 RosBEWin2.1.6 GCC4.7.2 dbg x86  750.080 ->  749.056
regedit.exe releases/0.4.13 RosBEWin2.1.6 GCC4.7.2 dbg x86  742.400 ->  741.888
regedit.exe releases/0.4.12 RosBEWin2.1.6 GCC4.7.2 dbg x86  744.448 ->  743.424
regedit.exe releases/0.4.11 RosBEWin2.1.6 GCC4.7.2 dbg x86  744.448 ->  743.424
regedit.exe releases/0.4.10 RosBEWin2.1.6 GCC4.7.2 dbg x86  744.448 ->  743.424
regedit.exe releases/0.4. 9 RosBEWin2.1.6 GCC4.7.2 dbg x86  744.448 ->  743.424
regedit.exe releases/0.4. 8 RosBEWin2.1.6 GCC4.7.2 dbg x86  743.936 ->  742.912
regedit.exe releases/0.4. 7 RosBEWin2.1.6 GCC4.7.2 dbg x86  743.936 ->  742.912

2 weeks ago[0.4.10][COMDLG32] RC Backports 2024-04-11
Joachim Henze [Wed, 10 Apr 2024 23:53:15 +0000 (01:53 +0200)]
[0.4.10][COMDLG32] RC Backports 2024-04-11

This ports back:

*PARTIALLY* 0.4.15-dev-7884-g d72d61fe475bc1419d1bd98a4cfacc15bc8b67d1 [COMDLG32] *.rc Improve translations (#6729)

0.4.15-dev-7351-g 449a08a6a8d5c70527d2cd7d07db3ae39ee642eb [COMDLG32] Fix IDC_TOOLBARPLACES size (#6228) CORE-16925
This part is relevant only for releases/0.4.10 to releases/0.4.14 because that dlg was only introduced during 0.4.10-dev'ing

0.4.15-dev-3493-g a36cc8054d266b75ad3bb53cf0a5ab8d72a412c9 [COMDLG32] Fix ico3 not shown in Print dialog (#2831)

0.4.15-dev-95-g 78957049524c01f29f115969558ec38118521d08 [COMDLG32] Enlarge list area in CHOOSE_FONT dialog (#2670)

As always pick some selected/improved translations here and there from newer branches.
Furthermore strip the accelerators from the pshhelp-buttons *in the older branches*, not in master. They do collide in so many languages,
and since the old releases will never get any help-functionality ported to, they are actually counterproductive.
Stripping them is not so easy though, because some dlgs, especially the file-open-dlgs get dynamically resized at runtime
by the height of that button, and by the height of the read-only-checkboxes. Ugly stuff.
Maybe they can be hidden later, and de-stringified to save a few bytes and missing translations.
Not important for today, maybe something to consider in 10years from now.
The stripped accelerators can then serve as a reminder for that.

Binary size of comdlg32.dll shrinks slightly for all branches:

comdlg32.dll master          RosBEWin2.2.2 GCC8.4.0 dbg x86               1.011.712 (0.4.15-dev-7874-g0cfc9ab)
comdlg32.dll releases/0.4.14 RosBEWin2.2.2 GCC8.4.0 dbg x86  1.008.128 -> 1.006.080
comdlg32.dll releases/0.4.14 RosBEWin2.1.6 GCC4.7.2 dbg x86    974.336 ->   972.288
comdlg32.dll releases/0.4.13 RosBEWin2.1.6 GCC4.7.2 dbg x86    974.336 ->   972.288
comdlg32.dll releases/0.4.12 RosBEWin2.1.6 GCC4.7.2 dbg x86    958.464 ->   956.416
comdlg32.dll releases/0.4.11 RosBEWin2.1.6 GCC4.7.2 dbg x86    944.128 ->   941.568
comdlg32.dll releases/0.4.10 RosBEWin2.1.6 GCC4.7.2 dbg x86    941.568 ->   939.520
comdlg32.dll releases/0.4. 9 RosBEWin2.1.6 GCC4.7.2 dbg x86    914.432 ->   912.384
comdlg32.dll releases/0.4. 8 RosBEWin2.1.6 GCC4.7.2 dbg x86    914.944 ->   912.384
comdlg32.dll releases/0.4. 7 RosBEWin2.1.6 GCC4.7.2 dbg x86    899.584 ->   896.512

4 weeks ago[0.4.10][I8042PRT] Add service registry entry for the driver CORE-17819
Joachim Henze [Fri, 29 Mar 2024 14:02:02 +0000 (15:02 +0100)]
[0.4.10][I8042PRT] Add service registry entry for the driver CORE-17819

CORE-17819 This allows Microsoft VirtualPC 2004/2007 to boot LiveCD SAFEMODE to desktop,
the regular mode booted also without that addition already.

Fix picked from 0.4.15-dev-1537-g b1c460d34e2f04257d69d62ea32458b11ce6ba11
This commit was part of a much bigger PR3396, which I don't pick entirely today.
For CORE-17819 only this part is actually needed.

4 weeks ago[0.4.10][SHELL32] RC-backports 2024-03-27
Joachim Henze [Wed, 27 Mar 2024 20:37:11 +0000 (21:37 +0100)]
[0.4.10][SHELL32] RC-backports 2024-03-27

This backport covers tons of improvements within the Shell32-RCs.
E.g. It improves the IDD_FILE_PROPERTIES and IDD_FOLDER_PROPERTIES significantly in the older releases for all langs, but also the Run-Dlg and IDD_OPEN_WITH.

The following master commits are part of it:

0.4.15-dev-7838-g e3bfcdf9e2a35aeb8571781c3128d1ffcb479a4a [SHELL32] pl-PL.rc: Fix IDD_FILE_PROPERTIES & IDD_FOLDER_PROPERTIES (#6678)

Some 100%-safe parts of the yet uncommitted PR6657 [SHELL32] Update Romanian (ro-RO) translation. I didn't pick all of that yet, as most parts are still WIP.

0.4.15-dev-7833-g ee0329e0e21d30851b78a8d509c32b831cf89ff8 [SHELL32] *.rc: accelerator fixes, drive-properties bg-BG and fr-FR (#6591).
   I addressed many of the FIXMEs already in the backport so this state is superior to master even.

0.4.15-dev-7216-g d387c799ed1669052483dc67affeaca5c139ff74 [SHELL32] 640x480-ify four dialogs (#6004) IDD_DRIVE_PROPERTIES, IDD_DRIVE_TOOLS, IDD_DRIVE_HARDWARE, IDD_FOLDER_CUSTOMIZE

0.4.15-dev-6333-g fa429c8e2ffbba5338307166479553faa9cb23c7 [SHELL32] *.rc: Use 1996 as birth year in IDD_ABOUT for consistency (this leaves now only the cmd-part left to be done in this context)

0.4.15-dev-5993-g a5bff2457a1014c266ce9c808d22a9967a4baca2 [SHELL32] Improve disk properties pie chart (#5239) CORE-18906
(I picked only the rc changes for that, but left the code changes out of the backport, the result is still appealing to my eyes)

0.4.14-dev-1492-g 4af7888b29d072157fed46e0d221da238d5c5542 [SHELL32] CORE-16895 RC IDD_OPEN_WITH All langs

In releases/0.4.14 and lower I stripped the en-GB.rc,
because that was (aside from 3-4 lines of forgotten sync) the same content to en-US.rc still. Real intended changes reached en-GB.rc not before 0.4.15-dev.
Furthermore this translation does not exist in 2k3sp2 afaik.
Also strip GUI-controls for some other features that are yet unimplemented in older ros versions, and that I won't port back, namely:
- drive-indexing/compression-checkboxes and "Disk Cleanup"-button in IDD_DRIVE_PROPERTIES
- controls for GUI-versions of "Backup" and "Defragmentation" in IDD_DRIVE_TOOLS DIALOGEX
- and the related icons for the above

Binary size of shell32.dll shrinks for all branches:

shell32.dll master          RosBEWin2.2.2 GCC8.4.0 dbg x86               10.579.968 (0.4.15-dev-7836-g72a27e6)
shell32.dll releases/0.4.14 RosBEWin2.2.2 GCC8.4.0 dbg x86  9.316.684 ->  9.201.664
shell32.dll releases/0.4.14 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.345.024 ->  9.230.336
shell32.dll releases/0.4.13 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.199.616 ->  9.086.976
shell32.dll releases/0.4.12 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.146.368 ->  9.031.680
shell32.dll releases/0.4.11 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.077.248 ->  8.954.880
shell32.dll releases/0.4.10 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.069.568 ->  8.945.152
shell32.dll releases/0.4. 9 RosBEWin2.1.6 GCC4.7.2 dbg x86  8.821.760 ->  8.618.496
shell32.dll releases/0.4. 8 RosBEWin2.1.6 GCC4.7.2 dbg x86  8.736.256 ->  8.536.576
shell32.dll releases/0.4. 7 RosBEWin2.1.6 GCC4.7.2 dbg x86  8.663.040 ->  8.466.944

7 weeks ago[0.4.10][SHELL32] Fix 2 compiler warnings for the rls-cfg 'unused variable'
Joachim Henze [Mon, 4 Mar 2024 19:32:53 +0000 (20:32 +0100)]
[0.4.10][SHELL32] Fix 2 compiler warnings for the rls-cfg 'unused variable'

This fixes the same 2 warnings that we fixed on master by 0.4.15-dev-7676-g a08b83a90ba494dfe62b8bf618e548c4926666ea (#6539)
But I ported back only the first commit of (#6539) and left out the 2nd.
The 2 warnings were introduced by 0.4.10-dev-202-g 698cbc6184722f29389ccd75ca96d88d6cd4b010

binary size of shell32.dll tends to shrink slightly (e.g. in releases/0.4.13),
but is not enough to reach the next lower quantum for most branches:

master          RosBEWin2.2.2 GCC8.4.0 dbg x86               10.550.272 (0.4.15-dev-7715-gbb297bc)
releases/0.4.14 RosBEWin2.2.2 GCC8.4.0 dbg x86  9.316.684 ->  9.316.684
releases/0.4.14 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.345.024 ->  9.345.024
releases/0.4.13 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.200.640 ->  9.199.616
releases/0.4.12 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.146.880 ->  9.146.368
releases/0.4.11 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.077.248 ->  9.077.248
releases/0.4.10 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.069.568 ->  9.069.568

No older branch than releases/0.4.10 was affected.

7 weeks ago[0.4.10][SHELL32] COpenWithMenu: Use ShellExecuteExW to open the file
Joachim Henze [Mon, 4 Mar 2024 13:12:56 +0000 (14:12 +0100)]
[0.4.10][SHELL32] COpenWithMenu: Use ShellExecuteExW to open the file

This picks:
0.4.11-dev-728-g 9baf05f164bd12b4351a101a17cf7db9f5debb7b
which fixes regression CORE-15353 'NOTEPAD does not open text file via openWith-dialog if the filepath contains spaces'
which was unhidden by 0.4.10-dev-599-g 932df378bf35146a56c796bec36c5c689109fbd6
and affected only releases/0.4.10. No older release branches.
I ported it back further anyway, because I do perceive the newer code as superior.

binary size of shell32.dll shrinks slightly:

releases/0.4.10 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.072.128 -> 9.069.568
releases/0.4. 9 RosBEWin2.1.6 GCC4.7.2 dbg x86  8.824.832 -> 8.821.760
releases/0.4. 8 RosBEWin2.1.6 GCC4.7.2 dbg x86  8.738.816 -> 8.736.256
releases/0.4. 7 RosBEWin2.1.6 GCC4.7.2 dbg x86  8.666.624 -> 8.663.040

2 months ago[0.4.10][SHELL32] Backport format+logging-trivia, ARRAYSIZE/_countof
Joachim Henze [Sun, 25 Feb 2024 00:50:03 +0000 (01:50 +0100)]
[0.4.10][SHELL32] Backport format+logging-trivia, ARRAYSIZE/_countof

Picks various trivial improvements from newer branches, does not fix specific JIRA-tickets and also doesn't
*entirely* cover a specific PR, therefore none of those are mentioned here.
Main motivation is to reduce large scale white-noise diff between the branches, to make
future commits diffs (with some fixes that actually matter) smaller in size and therefore easier to review.

Specifically:
- large-scale formatting & logging improvements, including but not limited to 0.4.15-dev-7653-g 6f91b6c0fe61728cdd9d97e97e5bfbee03ed881e
- strip trailing whitespace
- also typos like 'unkown', missing \n as it was
  part of 0.4.15-dev-6323-g 3854a5d8e93d2c50153bda26ea378e65cc4241fa and
  0.4.15-dev-4492-g 03422451b3b4a7df96a9e9af5ce7533fd6cb55d9
  (I left out the MOUSE_MOVE-one from that, because I consider the missing \n there intentional)
- ports back many usages of ARRAYSIZE/_countof as it was added as part of some commits from Hermes, e.g.:
  *partial* pick of 0.4.13-dev-622-g 8a3080151668efc99be9e7a2fb22a24392c85ea3 and
  *partial* pick of 0.4.15-dev-3642-g 83be315abffe14f0b391ff9cd543010aa0d57b9e (I left out the literals here for now, ...
    I *might* pick them later on their own, but not now. I first want to verify their isolated impact on all compilers generated file-sizes.
    Some of those changes would even increase the binary size, especially those where the same literal is used multiple times,
    because string-pooling is currently not enabled for all toolchains and configurations.)
- COpenWithMenu.cpp: strip double-semicolon
- COpenWithMenu.cpp && folder_options.cpp: add \n to some dbg-prints without increasing their length
- CFSDropTarget.cpp: add \n to some dbg-prints, strip some good-path-traces that are no longer present in master in return to avoid size increase, fix indentation
- CShellLink.cpp: add \n to some dbg-prints without increasing their length
- iconcache.cpp: fix wrong indentation
- shlfileop.cpp: partially pick 0.4.15-dev-7388-g d689cfe1582db3e554784cc29917cf3637817c64 just the stripping of #ifdef __REACTOS__ but NOT the actual fix! (the ifdefs were only present in 0.4.14)
- 0.4.10 and lower: shellpath.c Strip some TRACEs with wrong format-strings, strip unused string my_DocumentsW

binary size of shell32.dll has tendency to shrink slightly, which is caused solely by the logging changes:

master          RosBEWin2.2.2 GCC8.4.0 dbg x86               10.538.496 (0.4.15-dev-7640-gbdcfc6b)
releases/0.4.14 RosBEWin2.2.2 GCC8.4.0 dbg x86  9.317.888 ->  9.316.684
releases/0.4.14 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.345.024 ->  9.345.024
releases/0.4.13 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.201.152 ->  9.200.640
releases/0.4.12 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.146.880 ->  9.146.880
releases/0.4.11 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.077.760 ->  9.077.248
releases/0.4.10 RosBEWin2.1.6 GCC4.7.2 dbg x86  9.074.176 ->  9.072.128
releases/0.4. 9 RosBEWin2.1.6 GCC4.7.2 dbg x86  8.825.856 ->  8.824.832
releases/0.4. 8 RosBEWin2.1.6 GCC4.7.2 dbg x86  8.740.864 ->  8.738.816
releases/0.4. 7 RosBEWin2.1.6 GCC4.7.2 dbg x86  8.668.160 ->  8.666.624

2 months ago[0.4.10][NETSHELL] Double-click and mostly RC-improvements
Joachim Henze [Sun, 11 Feb 2024 22:34:37 +0000 (23:34 +0100)]
[0.4.10][NETSHELL] Double-click and mostly RC-improvements

0.4.15-dev-7606-g 874b5a4e00872bb6ba0c9d4ce695290dfe7e552a [NETSHELL] Trivial tweaks (#6484)
0.4.15-dev-7126-g a6341b49d866b037ad5bc1f6e6eb7bc641dec474 [NETSHELL] 640x480-ify complete that module (#6109)
0.4.15-dev-7018-g cbc60aa07a024f77f543f813b0c648688050c3a6 [NETSHELL] Double click on component should open properties (#6008) CORE-19330
PARTIALLY 0.4.15-dev-6857-g 55563fa891475a70026b97b48258175907d797f2 [NETSHELL] IDD_NETPROPERTIES must fit 640x480 CORE-19284 (#5899) (I left out the 3 accelerators)

Also port back some other rc tweaks and strip EOL-whitespaces and the IDS_NET_REPAIR from the context menu which is still unimplemented in the older branches
and for which I do never intend to port back the implementation.

binary size of netshell.dll shrinks slightly on all branches:

master          RosBEWin2.2.2 GCC8.4.0 dbg x86             659.968 (0.4.15-dev-7598-geb55d98)
releases/0.4.14 RosBEWin2.2.2 GCC8.4.0 dbg x86  644.608 -> 640.512 (netshell.dll is one of the rare modules where GCC8.4.0 produces more efficient output than GCC4.7.2)
releases/0.4.14 RosBEWin2.1.6 GCC4.7.2 dbg x86  687.104 -> 682.496
releases/0.4.13 RosBEWin2.1.6 GCC4.7.2 dbg x86  685.568 -> 681.472
releases/0.4.12 RosBEWin2.1.6 GCC4.7.2 dbg x86  666.624 -> 662.016
releases/0.4.11 RosBEWin2.1.6 GCC4.7.2 dbg x86  657.408 -> 653.312
releases/0.4.10 RosBEWin2.1.6 GCC4.7.2 dbg x86  576.512 -> 572.928
releases/0.4. 9 RosBEWin2.1.6 GCC4.7.2 dbg x86  576.512 -> 572.416
releases/0.4. 8 RosBEWin2.1.6 GCC4.7.2 dbg x86  576.512 -> 572.416
releases/0.4. 7 RosBEWin2.1.6 GCC4.7.2 dbg x86  576.000 -> 571.904

2 months ago[0.4.10][JOY] Add IDCANCEL action to MainPageProc (#6346) CORE-19424 + RC tweaks
Joachim Henze [Wed, 7 Feb 2024 21:49:01 +0000 (22:49 +0100)]
[0.4.10][JOY] Add IDCANCEL action to MainPageProc (#6346) CORE-19424 + RC tweaks

This ports back:
0.4.15-dev-7456-g cbc3ee2287c1972ac8257da60388a5de9ea6c0e5 CORE-19424 "Game Controller" control panel applet : ESC does not close
and a few trivial rc improvements.

Tendency for binary size is to shrink slightly, but manages to only reach the next lower quantum in releases/0.4.14:

master          RosBEWin2.2.2 GCC8.4.0 dbg x86                144.384 (0.4.15-dev-7557-gc858429)
releases/0.4.14 RosBEWin2.1.6 GCC4.7.2 dbg x86     133.632 -> 133.120
releases/0.4.13 RosBEWin2.1.6 GCC4.7.2 dbg x86     125.440 -> 125.440 (interesting: 0.4.12 & 0.4.13 do have the exact same code *within* this module)
releases/0.4.12 RosBEWin2.1.6 GCC4.7.2 dbg x86     127.488 -> 127.488 (interesting: 0.4.12 & 0.4.13 do have the exact same code *within* this module)
releases/0.4.11 RosBEWin2.1.6 GCC4.7.2 dbg x86     125.952 -> 125.952
releases/0.4.10 RosBEWin2.1.6 GCC4.7.2 dbg x86     125.952 -> 125.952
releases/0.4. 9 RosBEWin2.1.6 GCC4.7.2 dbg x86     125.952 -> 125.952
releases/0.4. 8 RosBEWin2.1.6 GCC4.7.2 dbg x86     125.952 -> 125.952
releases/0.4. 7 RosBEWin2.1.6 GCC4.7.2 dbg x86     125.952 -> 125.952

I am not aware yet which exact commit in the toolchain or dependent libs made it shrink from releases/0.4.12 to releases/0.4.13

2 months ago[0.4.10][NETCFGX] RC-fixes, Radiobuttons, optimization
Joachim Henze [Sun, 4 Feb 2024 19:43:25 +0000 (20:43 +0100)]
[0.4.10][NETCFGX] RC-fixes, Radiobuttons, optimization

This backports the following commits:
0.4.15-dev-7247-g 8f349ab3c28a6e13c671b0393fee4784d1ef218e [NETCFGX] Create only one "Alternative Configuration" tab CORE-19328
0.4.15-dev-7124-g 6a7315228cca84e85619c54d7b2a755697fa55a6 [NETCFGX] IDD_TCPIP_ALTCF_DLG Fix radiobutton (#6083) CORE-19327
0.4.15-dev-7099-g 5012c6a2b02930397856f2e7271d96fd6b790205 [NETCFGX] IDD_TCPIP_BASIC_DLG Radiobuttons need 2 arrow-down-presses instead of one (#6049) CORE-19327
*PARTIALLY* 0.4.15-dev-7023-g a95863b1320b3735ab52669e7c9d615a1b084ec3 [NETCFGX] Half-implement TCP/IP alternate configuration (#6009) (I picked only some formatting, and the RC changes from that, not the initialization implementation yet) CORE-19331
0.4.15-dev-7015-g 9e42809fc1ea712c77dec62d0939700a12da8a8c [NETCFGX] Use CheckDlgButton/IsDlgButtonChecked instead of BM_GETCHECK/BM_SETCHECK CORE-19331
0.4.15-dev-6990-g 49399361652f291a3190e5599dc92df0b81183cb [NETCFGX] Strip 2x wrong WS_DISABLED style on dlgs (#5987)
0.4.15-dev-6904-g 1997c12252e3ae3d034550e99b0afd944587c120 [NETCFGX] 640x480-ify (#5946)
0.4.15-dev-6858-g 0177b1af45ee8b2161cab9bed18984eae110a551 [NETCFGX] *.rc 640x480-ify IDD_TCPIP_BASIC_DLG and IDD_TCPIP_ALTCF_DLG (#5902)

The binary size shrinks slightly on all branches:

master          RosBEWin2.2.2 GCC8.4.0 dbg x86                602.112 (0.4.15-dev-7264-ga328886)
releases/0.4.14 RosBEWin2.1.6 GCC4.7.2 dbg x86     570.880 -> 568.832
releases/0.4.13 RosBEWin2.1.6 GCC4.7.2 dbg x86     569.344 -> 567.808
releases/0.4.12 RosBEWin2.1.6 GCC4.7.2 dbg x86     548.864 -> 547.328
releases/0.4.11 RosBEWin2.1.6 GCC4.7.2 dbg x86     502.272 -> 500.736
releases/0.4.10 RosBEWin2.1.6 GCC4.7.2 dbg x86     452.096 -> 450.560
releases/0.4. 9 RosBEWin2.1.6 GCC4.7.2 dbg x86     452.096 -> 450.560
releases/0.4. 8 RosBEWin2.1.6 GCC4.7.2 dbg x86     452.096 -> 450.560
releases/0.4. 7 RosBEWin2.1.6 GCC4.7.2 dbg x86     452.096 -> 450.560

4 months ago[0.4.10][TASKMGR] Some RC backports (2023-12-03) (#5949) (#5978)
Joachim Henze [Sat, 2 Dec 2023 23:53:01 +0000 (00:53 +0100)]
[0.4.10][TASKMGR] Some RC backports (2023-12-03) (#5949) (#5978)

ports back:
0.4.15-dev-7082-g 628d4a92e978eda0da873b4234bbded65dcde8b0 [TASKMGR] Fix text cutoff statusbar2 CPU%, improve statusbar3 Mem (#5978)
0.4.15-dev-6928-g b693eddd74f689c58f232a8e92849f705444b0d3 [TASKMGR] Harmonize and beautify the borders in performance tab (#5949)

Note:
Regarding (#5978) I intentionally did not port-back the chinese colons for the CPU% in the statusbar,
because the older release-branches do structurally use the english : still
here and in other controls.

Binary size shrinks slightly on all branches:

master taskmgr.exe RosBEWin2.2.2 GCC8.4.0dbg             695.808 (0.4.15-dev-7112-gd795021)
0.4.14 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  548.864 -> 547.840
0.4.13 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  542.208 -> 541.184
0.4.12 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  543.232 -> 542.720
0.4.11 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  543.232 -> 542.720
0.4.10 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  530.432 -> 529.920
0.4. 9 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  530.432 -> 529.920
0.4. 8 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  530.432 -> 529.920
0.4. 7 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  529.408 -> 528.896

       taskmgr.exe MS XPSP3 german                                           140.800 Bytes
0.4.14 taskmgr.exe RosBEWin2.1.6 MSVC2010SP1rls I18N=de-DE  109.056 Bytes -> 109.056 Bytes (my current taskmgr of choice)
0.4. 8 taskmgr.exe RosBEWin2.1.6 MSVC2010SP1rls I18N=en-US  105.984 Bytes -> 105.984 Bytes
0.4. 7 taskmgr.exe RosBEWin2.1.6 MSVC2010SP1rls I18N=en-US  105.472 Bytes -> 105.472 Bytes

5 months ago[0.4.10][PARPORT] Fix I/O port length check, [REACTOS] Logging & formatting, e.g...
Joachim Henze [Sun, 5 Nov 2023 12:34:18 +0000 (13:34 +0100)]
[0.4.10][PARPORT] Fix I/O port length check, [REACTOS] Logging & formatting, e.g. CORE-14388 CORE-19105

Ports back a real fix:
0.4.15-dev-1173-g ba09834c5ebdaf8d3e2a518c7fb78eaa7d89bb50 [PARPORT] Fix I/O port length check

Ports back logging and formatting:
0.4.15-dev-6794-g 4eace8d762426e9ca7de0b357eb92d17299d9a9b [IPHLPAPI] Silence obsolete FIXME in GetAdaptersAddresses (#5834) CORE-14388
0.4.15-dev-6438-g b12ab486d83905275b906a800f3f8d5394df23be [MUP] Mute DPRINT1's that slow down shared folder accesses (#5545) CORE-19105

And mutes some other loggings as well for stuff, that I either never want to port back, e.g. the IMM-implementations,
or which is not really helpful in the older branches:
fixme:(win32ss/user/user32/misc/imm.c:446) WINNLSEnableIME is UNIMPLEMENTED!
fixme:(../dll/win32/iphlpapi/address.c:290) GetAdaptersAddresses - Semi Stub: Family 2, Flags 0x0000002e, Reserved 00000000, pAdapterAddress 00000000, pOutBufLen 0143EBD4.

when browsing the startmenu:
fixme:(dll/win32/comctl32/toolbar.c:394) [00080086] TBSTYLE_REGISTERDROP not implemented
fixme:(dll/win32/comctl32/toolbar.c:5636) [00080086] response 2045774661 not handled to NM_CUSTOMDRAW (CDDS_PREERASE)
fixme:(dll/win32/comctl32/toolbar.c:5673) [00080086] response 3225142 not handled to NM_CUSTOMDRAW (CDDS_POSTERASE)
fixme:(dll/win32/comctl32/toolbar.c:5636) [000200BA] response 9 not handled to NM_CUSTOMDRAW (CDDS_PREERASE)
fixme:(dll/win32/comctl32/toolbar.c:5636) [00080086] response 2288892 not handled to NM_CUSTOMDRAW (CDDS_PREERASE)
fixme:(dll/win32/comctl32/toolbar.c:5673) [00080086] response 3291448 not handled to NM_CUSTOMDRAW (CDDS_POSTERASE)

and a bigger block regarding the parallel port during booting:
(drivers/parallel/parport/parport.c:127) Parport DriverEntry
(drivers/parallel/parport/fdo.c:391) AddDevice(B0B50548 B0FED980)
(drivers/parallel/parport/fdo.c:25) AddDeviceInternal()
(drivers/parallel/parport/fdo.c:513) FdoPnp()
(drivers/parallel/parport/fdo.c:582) IRP_MJ_PNP / IRP_MN_FILTER_RESOURCE_REQUIREMENTS
(drivers/parallel/parport/fdo.c:513) FdoPnp()
(drivers/parallel/parport/fdo.c:546) IRP_MJ_PNP / IRP_MN_START_DEVICE
(drivers/parallel/parport/misc.c:42) Calling lower device B0FED980
(drivers/parallel/parport/fdo.c:116) FdoStartDevice ()
(drivers/parallel/parport/fdo.c:160) Port: BaseAddress 0x378  Length 8
(drivers/parallel/parport/fdo.c:160) Port: BaseAddress 0x778  Length 8
(drivers/parallel/parport/fdo.c:174) Interrupt: Level 20  Vector 55
(drivers/parallel/parport/fdo.c:195) New LPT port: Base 0x378
(drivers/parallel/parport/fdo.c:513) FdoPnp()
(drivers/parallel/parport/fdo.c:586) Unknown minor function 0x9
(drivers/parallel/parport/fdo.c:513) FdoPnp()
(drivers/parallel/parport/fdo.c:586) Unknown minor function 0x14
(drivers/parallel/parport/fdo.c:513) FdoPnp()
(drivers/parallel/parport/fdo.c:564) IRP_MJ_PNP / IRP_MN_QUERY_DEVICE_RELATIONS / BusRelations
(drivers/parallel/parport/fdo.c:351) FdoQueryBusRelations()
(drivers/parallel/parport/fdo.c:233) FdoCreateRawParallelPdo()
(drivers/parallel/parport/fdo.c:378) Done
(drivers/parallel/parport/pdo.c:165) PdoPnp()

5 months ago[0.4.10][TASKMGR] Mainly Graph-stuff backports 2023-10-31
Joachim Henze [Tue, 31 Oct 2023 20:01:57 +0000 (21:01 +0100)]
[0.4.10][TASKMGR] Mainly Graph-stuff backports 2023-10-31

This backports the following commits:
0.4.15-dev-6770-g ddd1d19b3fbf6731eb5339d5834f7f519caeae39 [TASKMGR] Get rid of cplusplus extern c (#5808)
partially 0.4.15-dev-6120-g 6e77747b305add73133faa2049b8bd34e3e2534e [TASKMGR] Simplify tray icon code
0.4.15-dev-6114-g 9a6c85f68aa0e2ec4595d4673e59f5ade2521a6f [TASKMGR] Fix PerfDataDeallocCommandLineCache, part of CORE-18014 (unresolved still)
0.4.15-dev-6113-g 7989e3f26cafd74c9d8fdeef64deece0da591634 [TASKMGR] GraphCtrl_Dispose: Delete hdcGraph first, part of CORE-18014 (unresolved still)
0.4.15-dev-6112-g dc14a9f6e0d9b6ce3ccc84daba923edb1cc0167f [TASKMGR] GraphCtrl: Use DeleteDC instead of DeleteObject to delete inst->hdcGraph, part of CORE-18014 (unresolved still)
partially 0.4.15-dev-4994-g 15a0f7adb0694107b981bbf6c5292036f1e55d79 picked a tiny part from PR4657 (I picked only the refactoring to switch-statement, no functional changes. Especially NOT the formatting changes which would require the additional winnls.h-include)
0.4.15-dev-3737-g f8faa0b660da189ce00c614e19aa0a47a05d2520 [TASKMGR] Fix a heap corruption bug (#4311), just an addendum to PR4141 which is picked with this backport as well. Older branches were never affected.
partially 0.4.15-dev-3514-g 1c82bf032498d3d107aae7174c5fb179680f0eb5 [TASKMGR] Avoid freezing in getting icons, from (PR4180) CORE17894. I picked only the 1000ms->100ms part and the stripping of WM_QUERYDRAGICON call. Therefore I don't consider CORE17894 as fully covered.
0.4.15-dev-3486-g 545e1190f2caf714ff897d909645adc04178e3a6 [TASKMGR] Avoid hangs as much as possible (#4166) CORE17894
partially 0.4.15-dev-3483-g 403222dd4f73836b38c11a42fa6c3fc1614681d7 [TASKMGR] Preserve graphs history on resizes (#4141). I left aside the structs type renaming and OOM-Handling upon graph creation. Picked all the logical changes though.
0.4.15-dev-3269-g 0ed04e36406fc24b3d08e795a89962c253226f1f [TASKMGR] Make performance graph grid scroll (#3581)
0.4.15-dev-3268-g a4ab9a1e196a21633349496c211756f1e5cdbaf2 [TASKMGR] Formatting only (#3581). Covers the last bits of that PR.

Main motivation was getting the toggling of ShowKernelTimes in the Performance tab
switch on and off in realtime without introducing gaps in the graph.
It also makes the grid scroll together with the data, like on Windows.

Most other parts I picked solely for their binary-shrinking effect.
I decided to strip the ID_HELP_TOPICS from the rc files, as this was not implemented,
and I would never port that back later. So it is one less non-functional-button in the older branches.
I favored memset() over Zeromemory() in this usermode-app everywhere, and favored for (;;) over while(1).

Binary size shrinks slightly on all branches:

master taskmgr.exe RosBEWin2.2.2 GCC8.4.0dbg             696.832 (0.4.15-dev-6820-gb3194e3)
0.4.14 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  549.888 -> 548.864
0.4.13 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  542.720 -> 542.208
0.4.12 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  545.792 -> 543.232
0.4.11 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  545.792 -> 543.232
0.4.10 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  532.480 -> 530.432
0.4. 9 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  532.480 -> 530.432
0.4. 8 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  532.480 -> 530.432
0.4. 7 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  531.456 -> 529.408

       taskmgr.exe MS XPSP3 german                                           140.800 Bytes
0.4.14 taskmgr.exe RosBEWin2.1.6 MSVC2010SP1rls I18N=de-DE  110.080 Bytes -> 109.056 Bytes (my current taskmgr of choice)
0.4. 8 taskmgr.exe RosBEWin2.1.6 MSVC2010SP1rls I18N=en-US  108.032 Bytes -> 105.984 Bytes
0.4. 7 taskmgr.exe RosBEWin2.1.6 MSVC2010SP1rls I18N=en-US  107.520 Bytes -> 105.472 Bytes

6 months ago[0.4.10][SDK:RTL] RtlpCallQueryRegistryRoutine(): Correctly set SpareData and SpareLe...
Joachim Henze [Sat, 21 Oct 2023 20:23:28 +0000 (22:23 +0200)]
[0.4.10][SDK:RTL] RtlpCallQueryRegistryRoutine(): Correctly set SpareData and SpareLength (#5466)

ports back:
0.4.15-dev-6640-g 02883d1c163c3e11c95ffb873f071be384b13bda

SpareData and SpareLength need to be calculated correctly, as they are used
later in that function as well.
This allows to not overwrite Source UString when writing to Destination UString.

Fixes the problem described in the following JIRA issue, where services could
not start in 2nd-stage-bootcd-setup when installing ReactOS in a very-long-named directory.

CORE-18988

6 months ago[0.4.10][TASKMGR] Backport many fixes and improvements
Joachim Henze [Thu, 19 Oct 2023 13:08:33 +0000 (15:08 +0200)]
[0.4.10][TASKMGR] Backport many fixes and improvements

Some commits I do port back all the way down till releases/0.4.7:
partially 0.4.15-dev-6768-g 63eaf2bd529f5bf94cde65afc26944e98b47850b [TASKMGR] *.rc Deduplicate dynamic menu strings (#5772) (I left out the FIXME-comments)
0.4.15-dev-6712-g 46b4b5581d1fc4c1b50cfc0ee8b82b77be182dc6 [TASKMGR] Favor _countof(), no functional change intended (#5757)
0.4.15-dev-6703-g 6528ab8fcb41eb47ee17cf738760126e6758af94 [TASKMGR] de-DE.rc Update part 3
0.4.15-dev-6702-g 65ce20896afa19032612871506dd5d1e8eb73119 [TASKMGR] *.rc Strip unused IDD_DEBUG_CHANNELS_DIALOG
0.4.15-dev-6700-g e15d68ba5d9906674b363cf8362ed08494e53a76 [TASKMGR] de-DE German Update & bug-fixes
partially 0.4.15-dev-6447-g 016acd170dd75a7a24af235339a5321e2c24bf5a [TASKMGR] Do not free shared icon for ShellAbout() calls (I picked just the taskmgr part) (was part of larger PR5519)
0.4.15-dev-6297-g 831288fe691ec311c5b910f05d9b833b4b952261 [TASKMGR] Fix text truncation for fr-FR.rc (#5380) CORE-18523
0.4.15-dev-6127-g c706222f3f1b2a63e8cc969ee756c8be92527853 [TASKMGR] Fix the license. Turns out it's LGPL-2.1-or-later from the texts.
0.4.15-dev-6124-g 9927163455b83fea29c6420a688535b8e233bff6 [TASKMGR] Reorder page controls to have natural TAB-ordering.
0.4.15-dev-6122-g d5707eefe2f08d230c07ab4693ac0bd59e809f85 [TASKMGR] High speed update is twice per second, and not every second.
0.4.15-dev-6121-g 2da0506f2a9fc02acede7d7ff479399013830fad [TASKMGR] Remove more dead code.
partially 0.4.15-dev-6120-g 6e77747b305add73133faa2049b8bd34e3e2534e [TASKMGR] Simplify tray icon code. (I left out using the strsafe.h-dependent stuff, as this is safe also in old-style)
0.4.15-dev-6119-g 0b4c8bdd1cb4cf1a50c1076c7ded19ee7e9f9198 [TASKMGR] Clamp the values returned from PerfDataGetProcessorUsage() and PerfDataGetProcessorSystemUsage() inside these functions.
0.4.15-dev-6118-g 7b53126375ee8889cf24615fa2ea646feabee0ae [TASKMGR] Completely reformat perfpage.c, perfpage.h, procpage.h and trayicon.c
partially 0.4.15-dev-6117-g da7dcec6a06c052339120ed4baa3caf24594f163 [TASKMGR] Reformat the files' headers
0.4.15-dev-6115-g 36aea8e56eb6eb9f1cab5fd6ce040b80ef6a302f [TASKMGR] affinity.c: Use _countof instead of sizeof/sizeof (was part of the larger CORE18991)
partially 0.4.15-dev-6109-g 99fb812be4ba86139438bf7961f2030ac9e016be [TASKMGR] Refactor trayicon.c
0.4.15-dev-5404-g c093d4f8038e7ddec1ca193edc573c48e12f12ff [TASKMGR] Prevent context menu on idle process (#4889) CORE-18640
0.4.15-dev-5031-g 46ea5171bdfaba2606486e27821fdd9ee88e551a [TASKMGR] Pixel-perfection for left border (#4680) CORE-18061
0.4.15-dev-4988-g de5af76811bbd318cdef46e193f306833e3fa740 [TASKMGR] Fix overlapping controls (French only) (#4658)
0.4.15-dev-4987-g f8c28b977363444ad31124a6794347c53532b43f [TASKMGR] Fix UI on SwitchTo for multiple application (#4652)
0.4.15-dev-3497-g e8a7e0da2bcbd842f95e4224e72bfb5d748dbd05 [TASKMGR] Make 'Hide When Minimized' non-default (#4169)
0.4.15-dev-3085-g 515e0cf38f8d07e59bb003f930eb08d33dc5f85d [TASKMGR] Handles: Get number only, skip data (#3910)
and a few *.rc improvements that are not explicitly mentioned here,
especially I stripped the menuLoop-statusbar-help-texts within the older branches.

I also port back the following feature into all releases down to releases/0.4.8, but I left out releases/0.4.7 for now:
0.4.13-dev-875-g 3276824586fefd531f4473046618fa2724442a53 [TASKMGR] Implement proper process tree ending (#1738)

A few improvements are still left to be done in a future commit (mostly around the plotgraphs and the trayicon-handling),
but I need to have a closer look at them and their impact on binary size,
so I do not want to squash them into this bigger backport,
where I concentrated on the safe/obvious and easy things.

Binary size decreases for all branches:
master taskmgr.exe RosBEWin2.2.2 GCC8.4.0dbg  708.608 -> 696.832 (0.4.15-dev-6768-g63eaf2b)
0.4.14 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  660.992 -> 549.888
0.4.13 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  659.968 -> 542.720
0.4.12 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  662.016 -> 545.792
0.4.11 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  662.016 -> 545.792
0.4.10 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  645.120 -> 532.480
0.4. 9 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  645.120 -> 532.480
0.4. 8 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  645.120 -> 532.480
0.4. 7 taskmgr.exe RosBEWin2.1.6 GCC4.7.2dbg  645.120 -> 531.456

       taskmgr.exe MS XPSP3 german                             140.800 Bytes
0.4.14 taskmgr.exe RosBEWin2.1.6 MSVC2010SP1rls I18N=de-DE  -> 110.080 Bytes (my current taskmgr of choice)
0.4. 8 taskmgr.exe RosBEWin2.1.6 MSVC2010SP1rls I18N=en-US  -> 108.032 Bytes
0.4. 7 taskmgr.exe RosBEWin2.1.6 MSVC2010SP1rls I18N=en-US  -> 107.520 Bytes

6 months ago[0.4.10][DOC] Correct README.WINE regarding winhttp.dll
Joachim Henze [Sun, 1 Oct 2023 03:48:41 +0000 (05:48 +0200)]
[0.4.10][DOC] Correct README.WINE regarding winhttp.dll

Because we reverted [WINHTTP] in releases/0.4.9 and releases/0.4.10 and releases/0.4.11
to the state before 9048599788ef615f61233faf73b97533abfbbd4e (==SVN r75944) which was WineStaging-2.9

Ftr: Back then the winhttp_winetest was not reverted as well, which makes a few of the tests hang. (which is one but not the only reason)

Current state of those tests:
0.4.14-release-93-g10d0e9b  RosBELin GCC8.4.0 does not hang 'most of the times' but sometimes it does, later during 0.4.15-dev even more tests were disabled here
0.4.14-release-93-g10d0e9b  RosBEWin2.1.6dbg  winhhtp.dll 301.568 Winestaging-4.18 kernel32_vista-dep    winhttp_apitest 21executed, 1failure   winhttp_winetest notification 643executed, 26fail, 1skip  winhttp_winetest url 243executed, 1failure   winhttp_winetest winhttp 4964executed, 14failures, 1skip
0.4.13-release-159-ge70a565 RosBEWin2.1.6dbg  winhhtp.dll 301.568 Winestaging-4.18 kernel32_vista-dep    winhttp_apitest 21executed, 1failure   winhttp_winetest notification 409executed, 26fail, 1skip  winhttp_winetest url 223executed, 1failure   winhttp_winetest winhttp (hangs more than 5minutes)
0.4.12-release-188-gd73c71e RosBEWin2.1.6dbg  winhttp.dll 303.616 Winestaging-3.9  kernel32_vista-dep    winhttp_apitest 21executed, 1failure   winhttp_winetest notification 409executed, 26fail, 1skip  winhttp_winetest url 223executed, 1failure   winhhtp_winetest winhhtp 5037executed, 15failures, 0skip
0.4.11-release-197-gdbcd8aa RosBEWin2.1.6dbg  winhhtp.dll 297.984 Winestaging-2.9                        winhttp_apitest 21executed, 1failure   winhttp_winetest notification (hangs more than 5minutes)  winhttp_winetest url 223executed, 1failure   winhttp_winetest winhttp (hangs more than 5minutes)
0.4.10-release-214-g711e70c RosBEWin2.1.6dbg  winhhtp.dll 295.424 Winestaging-2.9
0.4.9-release-225-g9ff478e  RosBEWin2.1.6dbg  winhttp.dll 295.424 Winestaging-2.9                        winhttp_apitest 21executed, 1failure   winhttp_winetest notification (hangs more than 5minutes)  winhttp_winetest url 223executed, 1failure   winhttp_winetest winhttp (hangs more than 5minutes)
0.4.8-release-234-g862997f  RosBEWin2.1.6dbg  winhttp.dll 302.592 Wine-3.0         kernel32_vista-dep    winhttp_apitest 21executed, 1failure   winhttp_winetest notification 409executed, 26fail, 1skip  winhttp_winetest url 223executed, 1failure   winhttp_winetest winhttp 5028executed, 12failures
0.4.7-release-263-g0c09cd7  RosBEWin2.1.6dbg  winhttp.dll 301.056 WineStaging-2.16 kernel32_vista-dep    winhttp_apitest 21executed, 1failure   winhttp_winetest notification (hangs more than 5minutes)  winhttp_winetest url 223executed, 1failure   winhttp_winetest winhttp 5015executed, 12failures

We toggled some of the hanging tests in:
0.4.14-dev-787-g 0685e02d9ee69ba5ff4d024d41ec40801d701161 workaround more tests in winhttp:winhttp hang
0.4.14-dev-717-g 68490c16133faded90c9bcbe064c2a0e2e72bad1 Restore winhttp:notification test_persistent_connection execution
0.4.8-dev-861-g 89670a48aba0939f5ca2b2cc2e0ed29918a337ae workaround winhttp:notification test_persistent_connection hang
0.4.8-dev-237-g a22031dba643c9706fc4760c8cbf4a480c4ed848 workaround winhttp:notification test_persistent_connection hang

Since the testbots do no longer run with the GCC4.7.2 builds, it does not really make a lot of sense to port any of those workarounds back atm.

I do consider none of the above states really desireable, therefore I don't touch the mess today, just document it.
I dislike: the kernel32_vista-dep.
I dislike: the behavior in the synthetic tests.
I appreciate: In practice all of the above do behave rather sane in most practical (non-synthetic) tests.
Remember: The Flash-ONLINE-setups are broken also on XPSP3. We should not accept that as a valid testcase. That does not work even in 0.4.9RosBEWin2.1.6 / 0.4.13RosBEWin2.1.6 / 0.4.14RosBEWin2.1.6! Despite what the ticket CORE-13952 says. It lies!
Flash OFFLINE setup from rapps works. The app is discontinued.

I could imagine in the future:
-downgrade releases/0.4.8 and 0.4.7 also back to Winestaging 2.9 to kill the kernel32_vista-import. (Could be done with or without suppressing the hangs in the [too new] synthetic tests)
-observe whether releases/0.4.13 has any other issues in winhhtp-context, and if so we might want to step back to Winestaging-3.9 (the natural fit).

But not today.
The reverts were questionable from the standpoint of the synthetic tests and also
from the standpoint of the broken "Adobe Flash Online Updater". They might work by chance on the linux builds, but that does not transfer to the Windows builds. And most likely points to some (memory) corruption.
Reverts could only be justified by the kernel_vista removal.

7 months ago[0.4.10][BUTTERFLIES] Reduce count of (DLGPROC) casts, [CMIDRIVER] Remove useless...
Joachim Henze [Mon, 18 Sep 2023 14:11:00 +0000 (16:11 +0200)]
[0.4.10][BUTTERFLIES] Reduce count of (DLGPROC) casts, [CMIDRIVER] Remove useless aps file

ports back the following commits:
0.4.15-dev-3822-g 78a7d7dc324ab4597de81ec470c70058d9b6c083 [CMIDRIVER] Remove useless Visual Studio resource editor aps-file
0.4.12-dev-399-g 32faa247331dc744d307aa9fd60146aa59cde8ba [BUTTERFLIES] Fix GCC build, addendum to 0.4.12-dev-397-g 2890a22179528006e637cd5efdfc2abb7e2ceb5b
a tiny fraction of 0.4.12-dev-397-g 2890a22179528006e637cd5efdfc2abb7e2ceb5b just the [BUTTERFLIES] Avoid DLGPROC cast

Ftr, there are still a few DLGPROC casts left after this commit in ros source tree,
but I don't want to port back the commits yet, which addressed them.
Too fresh still, no need to rush that.

7 months ago[0.4.10][USER32][COMCTL32] Redraw children when the combo box is dropped down (#4138...
Joachim Henze [Sat, 16 Sep 2023 13:58:42 +0000 (15:58 +0200)]
[0.4.10][USER32][COMCTL32] Redraw children when the combo box is dropped down (#4138) (#5679)

port back the following commit:
0.4.15-dev-6613-g e13ebd44c60053cf58a5aed104a1950f06152f62 [USER32] Pure Whitespace sync from comctl32/combo.c, no functional change
0.4.15-dev-6612-g d97313181e2ed5112d530cc4ae3f3bc85451059a [USER32] Sync comctl32 combo.c code in context of CORE-17883
0.4.15-dev-3453-g ff89651ed018be182ed9568d8644f56e046ee7dd [COMCTL32] Redraw children when the combo box is dropped down (#4138) CORE-17883

and tweak [USER32] combo.c a bit more even than on master, e.g. strip a few TRACEs. Possible because
unlike master we don't need to be as much in sync as possible to Wine in the older branches.
But we need to have binary size under control for the backports due to base-addresses.

Binary size:
user32.dll master    RosBEWin2.2.2 GCC8.4.0dbg               1.579.008
user32.dll 0.4.14rls RosBEWin2.1.6 GCC4.7.2dbg  1.448.448 -> 1.448.448
user32.dll 0.4.13rls RosBEWin2.1.6 GCC4.7.2dbg  1.445.376 -> 1.444.864
user32.dll 0.4.12rls RosBEWin2.1.6 GCC4.7.2dbg  1.455.616 -> 1.455.104
user32.dll 0.4.11rls RosBEWin2.1.6 GCC4.7.2dbg  1.453.056 -> 1.451.008
user32.dll 0.4.10rls RosBEWin2.1.6 GCC4.7.2dbg  1.434.624 -> 1.434.112
user32.dll 0.4. 9rls RosBEWin2.1.6 GCC4.7.2dbg  1.422.336 -> 1.421.824
user32.dll 0.4. 8rls RosBEWin2.1.6 GCC4.7.2dbg  1.421.824 -> 1.421.824
user32.dll 0.4. 7rls RosBEWin2.1.6 GCC4.7.2dbg  1.418.752 -> 1.417.216

7 months ago[0.4.10][COMCTL32] Listview: marqueeRect before starting highlight (#5555), RC: VB6...
Joachim Henze [Mon, 11 Sep 2023 23:12:59 +0000 (01:12 +0200)]
[0.4.10][COMCTL32] Listview: marqueeRect before starting highlight (#5555), RC: VB6-InstallWizard (#2270)

port back:
0.4.15-dev-6456-g 918a8180718f0901e231b12fe2274b82baa926b6 [WINESYNC][COMCTL32] Listview: Initialize marqueeRect from left-click coordinates before starting a marquee highlight (#5555)

The infoPtr->marqueeRect structure is not currently initialized before the
marquee highlight sequence starts, resulting in the RECT having initial
coordinates of (0,0)-(0,0). These coordinates cause the first item in
the listview control to be identified as being within the range of the
marqueeRect's coordinates. This can be best observed when our filebrowser is in details-mode.

That item is then set to LVIS_SELECTED even though it is not part of the
marquee selection.

Wine commit: daf95aaadf3a59f0ccc129a853327417b5e4f07c

author: Hugh McMaster <hugh.mcmaster@outlook.com>
Mon, 26 Apr 2021 10:59:51 +0000 (20:59 +1000)

---------------

Also fix CORE-16650 'VB6 installation does not show the wizard correctly'
by porting back:
0.4.14-dev-844-g ea35843c300f381845b69c634e51ba849a0ea5a7 [COMCTL] Use DIALOG instead of DIALOGEX resources (#2270)

This is needed, because property sheets can modify the dialog resource,
and example code (and some existing applications) expect DIALOG.
One such application is the VB6 setup.
CORE-16650

---------------

also port back a few other trivial improvements from never branches, e.g.:
a tiny fraction of 0.4.14-dev-150-g 00f14ee1f22a1c285462459f859fa17c90ccc19f the type/cast-changes in LISTVIEW_MouseWheel()

---------------

0.4.13-dev-26-g 6122f3f436a4551e2a85318b7e6452a6fbada2c5 [COMCTL32] Remove an unnecessary include

---------------

and port back also a few trivial RC improvements, mainly to keep binary size under control.

Binary size:
comctl32.dll master    RosBEWin2.2.2 GCC8.4.0dbg               1.999.872
comctl32.dll 0.4.14rls RosBEWin2.1.6 GCC4.7.2dbg  1.808.896 -> 1.808.896
comctl32.dll 0.4.13rls RosBEWin2.1.6 GCC4.7.2dbg  1.810.944 -> 1.805.312
comctl32.dll 0.4.12rls RosBEWin2.1.6 GCC4.7.2dbg  1.823.232 -> 1.818.112
comctl32.dll 0.4.11rls RosBEWin2.1.6 GCC4.7.2dbg  1.816.064 -> 1.810.944
comctl32.dll 0.4.10rls RosBEWin2.1.6 GCC4.7.2dbg  1.813.504 -> 1.808.384
comctl32.dll 0.4. 9rls RosBEWin2.1.6 GCC4.7.2dbg  1.813.504 -> 1.808.384
comctl32.dll 0.4. 8rls RosBEWin2.1.6 GCC4.7.2dbg  1.639.424 -> 1.634.304
comctl32.dll 0.4. 7rls RosBEWin2.1.6 GCC4.7.2dbg  1.634.304 -> 1.629.184

7 months ago[0.4.10][REACTOS] Optimize existing PNGs (#5486) (#5492) (#5506)
Joachim Henze [Fri, 8 Sep 2023 17:26:35 +0000 (19:26 +0200)]
[0.4.10][REACTOS] Optimize existing PNGs (#5486) (#5492) (#5506)

This ports back the following commits about lossless PNG-shrinking:
0.4.15-dev-6410-g cf4bb8fced0e685662ed2fe061c251f5720a054d [WALLPAPERS] Optimize PNG images (#5506)
0.4.15-dev-6398-g e5993f13f0223eb1a068ff7cf19f6449f6f574d3 [REACTOS] Optimize PNG images (#5492)
0.4.15-dev-6376-g 8f3a09722ecd98c16664527cca611d7cbeb25ee1 [MSPAINT] Optimize images for help docs (#5486)

Most of those PNGs have not been touched ever since we switched from SVN to git, so they can safely be ported back by copying over.
But did not port them back any further than where they existed already.

Tweak a few related txt/html files also with trivial stuff like EOL-whitespace-removal.

8 months ago[0.4.10] Anti-flicker backport winpos.c
Joachim Henze [Sun, 20 Aug 2023 00:53:56 +0000 (02:53 +0200)]
[0.4.10] Anti-flicker backport winpos.c

This effective one-liner-backport fixes flickering in some apps like:
- CORE-7652 Hyperlink Blinking text while installing vc2008 redist package
and many duplicate-tickets of that same symptom, like:
- CORE-18553 CPUID CPU-Z 1.84 "about" flickers
- CORE-7753 Caesar 3: Autorun.exe flickers endlessly
- CORE-10741 API monitor high CPU usage
- CORE-15300 Worksoft Navigator 0.985 installer is locked in an endless redrawing loop
- CORE-16325 Some of MWSnap's installation windows fail to display content
- CORE-17925 MS Visual C++ 2008 Redistributable Setup takes long time
- CORE-18153 BitComet 1.86 forces the shell to a endless redrawing
- CORE-18434 Ubisoft installers (SetupUbi) - continously refresh of window
- CORE-15546 Shed 1.0 from rapps - setup does not draw some dlgs initially, but redraw can be enforced
- CORE-18582 Emule 0.50a - Interface frozen / flickering until window is moved

And since I do also port back the follow-up-commit of that, we can achieve that without introducing
the following regressions which the initial fixes caused:
- CORE-18817 [NTUSER] Winpos.c modification generates bad first drawing of some elements
- CORE-18830 Explorer Drawing glitch: Missing "X" on "Folder" pane when Right Click > Explore from Desktop icons
Those 2 things did not happen in the older releases beforehand.

In sum this is a squashed backport of:
0.4.15-dev-6507-g 49dbc8fbbb090812d21ac10e79d42c787638e2ec [NTUSER] Whitespace-fixes winpos.c
0.4.15-dev-5684-g 57eb2a6bf469383848a559feedc1eed7fc13dd5c [NTUSER] Missing "X" on "Folder" pane when Right Click > Explore from Desktop icons & Rapps toolbar errors (#5068) CORE-18830, CORE-18817
the superseded 0.4.15-dev-5618-g 8866b9d7b03e62b3ed3041acb0b3791fbc1733aa Update win32ss/user/ntuser/winpos.c
the superseded 0.4.15-dev-5617-g 7866eb28862b31f2ee90045293c662c8ac834a70 [NTUSER] Fix winpos bug hyperlink&images loop redrawing CORE-7652 (#4995)

To prevent increased binary size for the added conditions in the code in older branches, strip a single good-path-TRACE()
which had no trailing \n in the older branches,
but where that \n was added in master head meanwhile. So that TRACE was a diff to older branches anyway.

8 months ago[0.4.10][MSAFD] Fix non-blocking sockets support for recv() (#5575)
Joachim Henze [Fri, 18 Aug 2023 19:44:27 +0000 (21:44 +0200)]
[0.4.10][MSAFD] Fix non-blocking sockets support for recv() (#5575)

by porting back:
0.4.15-dev-6491-g 442f5dfab50da48700a20d4bdc6917e48105ad86 [MSAFD] Fix non-blocking sockets support for recv() (#5575) CORE-14486 for Nginx web server; Firefox 52, Mypal 29.3.0, New Moon 28 web browser
partially 0.4.15-dev-4492-g 03422451b3b4a7df96a9e9af5ce7533fd6cb55d9 [REACTOS] Add '\n' to debug logs. Only the [MSAFD] parts

Also strip some EOL-whitespace and sacrifice some good-path-TRACEs to not increase the binary sizes:

msafd.dll GCC8.4.0dbg RosBE2.2.2 master             155.648
msafd.dll GCC4.7.2dbg RosBE2.1.6 0.4.14  144.384 -> 142.336
msafd.dll GCC4.7.2dbg RosBE2.1.6 0.4.13  144.384 -> 142.336
msafd.dll GCC4.7.2dbg RosBE2.1.6 0.4.12  145.920 -> 144.896
msafd.dll GCC4.7.2dbg RosBE2.1.6 0.4.11  143.872 -> 141.824
msafd.dll GCC4.7.2dbg RosBE2.1.6 0.4.10  142.336 -> 140.800
msafd.dll GCC4.7.2dbg RosBE2.1.6 0.4. 9  142.336 -> 140.800
msafd.dll GCC4.7.2dbg RosBE2.1.6 0.4. 8  142.336 -> 140.800
msafd.dll GCC4.7.2dbg RosBE2.1.6 0.4. 7  142.336 -> 140.800

8 months ago[0.4.10][BROWSEUI] Fix pos of taskbar toolbar right-click menu (#5421)
Joachim Henze [Wed, 16 Aug 2023 22:09:07 +0000 (00:09 +0200)]
[0.4.10][BROWSEUI] Fix pos of taskbar toolbar right-click menu (#5421)

Don't call ClientToScreen in handling NM_RCLICK message because NMMOUSE.pt is already in screen coordinates.
CORE-18966

A backport of:
0.4.15-dev-6298-g b4e946f5b0147480d0df34654b55f9a7705787ff

and strip EOL whitespace

8 months ago[0.4.10][IP][TCPIP][LWIP] Backports 2023-08-13 (also fixes a BSOD)
Joachim Henze [Sun, 13 Aug 2023 14:31:13 +0000 (16:31 +0200)]
[0.4.10][IP][TCPIP][LWIP] Backports 2023-08-13 (also fixes a BSOD)

0.4.15-dev-6399-g f8a6542b15efa996d672609e651485f50310a6c0 [IP] Don't reference uninitialized PCB to avoid BSOD. CORE-18982
0.4.15-dev-6395-g 2010a5b8d915beebd64c68cb568b03c29ab64d75 [IP] Unlock Connection when TCPAllocatePort() fails. CORE-18371
0.4.15-dev-5707-g 874d317a7104e01571ea39b56d3d603246937fda [IP] Allow 0xFFFF as Valid Network Port Number (#5074) CORE-18371 CORE-18764
PARTIALLY pick 0.4.14-dev-1326-g 792b64ea468179886a42126165d59c79bc8e1bbf [TCPIP][FORMATTING] Format TiDispatch function. No functional changes (#2112)
0.4.13-dev-368-g 3a98d96eac606d3fc258ef298b80a71f00d96188 [LWIP] Fix src/core/init.c a bit (#1620)

and strip some EOL-whitespace.

Just to make sure that the binary sizes did not increase:
tcpip.sys master GCC8.4.0dbg RosBEWin2.2.2              462.848
tcpip.sys 0.4.14 GCC4.7.2dbg RosBEWin2.1.6   425.984 -> 425.984
tcpip.sys 0.4.13 GCC4.7.2dbg RosBEWin2.1.6   417.792 -> 417.792
tcpip.sys 0.4.12 GCC4.7.2dbg RosBEWin2.1.6   421.888 -> 421.888
tcpip.sys 0.4.11 GCC4.7.2dbg RosBEWin2.1.6   421.888 -> 421.888
tcpip.sys 0.4.10 GCC4.7.2dbg RosBEWin2.1.6   413.696 -> 413.696
tcpip.sys 0.4. 9 GCC4.7.2dbg RosBEWin2.1.6   413.696 -> 413.696
tcpip.sys 0.4. 8 GCC4.7.2dbg RosBEWin2.1.6   413.696 -> 413.696
tcpip.sys 0.4. 7 GCC4.7.2dbg RosBEWin2.1.6   413.696 -> 413.696

8 months ago[0.4.10][NTOS][WIN32SS][UDFS] Fix double free in ObCreateObject, fix retvals CORE...
Joachim Henze [Tue, 1 Aug 2023 22:51:41 +0000 (00:51 +0200)]
[0.4.10][NTOS][WIN32SS][UDFS] Fix double free in ObCreateObject, fix retvals CORE-14271

Backport the following commits:
0.4.15-dev-6401-g 53b30e3f3bf454356dad537c782543cba35e0f1a [NTOSKRNL][NTGDI] Formatting addendum, no functional change
0.4.15-dev-5487-g e7bbbf049e3c53f7477fb3c53e1910ac417066b5 [NTOS] Fix double free on allocation failure in ObCreateObject
partially 0.4.13-dev-358-g 38db074491768faf5a8dd1e83139fd16a7681d92 [WIN32SS][UDFS] Misc addendum to CORE-14271 (#1529)
0.4.12-dev-648-g 17af7f0c2741f2a8173a0bd091e97b9bd14d5777 [UDFS] Correctly check SeSinglePrivilegeCheck() return value (#1324) CORE-14271
0.4.12-dev-552-g 0483a5a38089217868401fb9fe5e9885cddebcc6 [NTOS:OB] Correctly check SeSinglePrivilegeCheck() return value (#1323) CORE-14271

Just for verifying that the size didn't increase anywhere:

ntoskrnl.exe GCC8.4.0dbg RosBEWin2.2.2 master               5.124.096
ntoskrnl.exe GCC4.7.2dbg RosBEWin2.1.6 0.4.14  4.413.440 -> 4.413.440
ntoskrnl.exe GCC4.7.2dbg RosBEWin2.1.6 0.4.13  4.367.360 -> 4.367.360
ntoskrnl.exe GCC4.7.2dbg RosBEWin2.1.6 0.4.12  4.377.600 -> 4.377.600
ntoskrnl.exe GCC4.7.2dbg RosBEWin2.1.6 0.4.11  4.377.088 -> 4.377.088
ntoskrnl.exe GCC4.7.2dbg RosBEWin2.1.6 0.4.10  4.344.320 -> 4.344.320
ntoskrnl.exe GCC4.7.2dbg RosBEWin2.1.6 0.4. 9  4.311.552 -> 4.311.552
ntoskrnl.exe GCC4.7.2dbg RosBEWin2.1.6 0.4. 8  4.296.704 -> 4.296.704
ntoskrnl.exe GCC4.7.2dbg RosBEWin2.1.6 0.4. 7  4.231.168 -> 4.231.168

udfs.sys     GCC8.4.0dbg RosBEWin2.2.2 master             835.584
udfs.sys     GCC4.7.2dbg RosBEWin2.1.6 0.4.14  745.472 -> 745.472
udfs.sys     GCC4.7.2dbg RosBEWin2.1.6 0.4.13  745.472 -> 745.472
udfs.sys     GCC4.7.2dbg RosBEWin2.1.6 0.4.12  749.568 -> 749.568
udfs.sys     GCC4.7.2dbg RosBEWin2.1.6 0.4.11  749.568 -> 749.568
udfs.sys     GCC4.7.2dbg RosBEWin2.1.6 0.4.10  749.568 -> 749.568
udfs.sys     GCC4.7.2dbg RosBEWin2.1.6 0.4. 9  749.568 -> 749.568
udfs.sys     GCC4.7.2dbg RosBEWin2.1.6 0.4. 8  749.568 -> 749.568
udfs.sys     GCC4.7.2dbg RosBEWin2.1.6 0.4. 7  749.568 -> 749.568

user32.dll   GCC8.4.0dbg RosBEWin2.2.2 master               1.585.152
user32.dll   GCC4.7.2dbg RosBEWin2.1.6 0.4.14  1.448.448 -> 1.448.448
user32.dll   GCC4.7.2dbg RosBEWin2.1.6 0.4.13  1.445.376 -> 1.445.376
user32.dll   GCC4.7.2dbg RosBEWin2.1.6 0.4.12  1.455.616 -> 1.455.616
user32.dll   GCC4.7.2dbg RosBEWin2.1.6 0.4.11  1.453.056 -> 1.453.056
user32.dll   GCC4.7.2dbg RosBEWin2.1.6 0.4.10  1.434.624 -> 1.434.624
user32.dll   GCC4.7.2dbg RosBEWin2.1.6 0.4. 9  1.422.336 -> 1.422.336
user32.dll   GCC4.7.2dbg RosBEWin2.1.6 0.4. 8  1.421.824 -> 1.421.824
user32.dll   GCC4.7.2dbg RosBEWin2.1.6 0.4. 7  1.418.752 -> 1.418.752

win32k.sys   GCC8.4.0dbg RosBEWin2.2.2 master               3.477.504
win32k.sys   GCC4.7.2dbg RosBEWin2.1.6 0.4.14  2.904.064 -> 2.904.064
win32k.sys   GCC4.7.2dbg RosBEWin2.1.6 0.4.13  2.895.872 -> 2.895.872
win32k.sys   GCC4.7.2dbg RosBEWin2.1.6 0.4.12  2.887.680 -> 2.887.680
win32k.sys   GCC4.7.2dbg RosBEWin2.1.6 0.4.11  2.867.200 -> 2.867.200
win32k.sys   GCC4.7.2dbg RosBEWin2.1.6 0.4.10  2.863.104 -> 2.863.104
win32k.sys   GCC4.7.2dbg RosBEWin2.1.6 0.4. 9  2.834.432 -> 2.834.432
win32k.sys   GCC4.7.2dbg RosBEWin2.1.6 0.4. 8  2.830.336 -> 2.830.336
win32k.sys   GCC4.7.2dbg RosBEWin2.1.6 0.4. 7  2.830.336 -> 2.830.336

8 months ago[0.4.10][NTUSER] NtUserWaitForInputIdle: Fix unhandled exception BSOD 0x1E, and tweaks
Joachim Henze [Sun, 30 Jul 2023 19:53:44 +0000 (21:53 +0200)]
[0.4.10][NTUSER] NtUserWaitForInputIdle: Fix unhandled exception BSOD 0x1E, and tweaks

Backport the following commits:

0.4.15-dev-6397-g 6b7efb331c0cd20c8610cd24d5e7221cff59969a [NTUSER] co_IntSendMessageWithCallBack: Formatting only, no functional changes
0.4.15-dev-6227-g c7e4c3a8e9d921233c1731ef64c388ed40b15fcb [NTUSER] NtUserWaitForInputIdle: Fix unhandled exception BSOD 0x1E CORE-18728 CORE-19014 (#5391)
0.4.15-dev-697-g 9f428f5522c9a6a4ec8cf19f7f0c543c944b9483 [NTUSER] Demote ERR to TRACE, also some minor corrections (#3132)

win32k.sys master GCC8.4.0dbg RosBEWin2.2.0              3.477.504
win32k.sys 0.4.14 GCC4.7.2dbg RosBEWin2.1.6 2.904.064 -> 2.904.064
win32k.sys 0.4.13 GCC4.7.2dbg RosBEWin2.1.6 2.895.872 -> 2.895.872
win32k.sys 0.4.12 GCC4.7.2dbg RosBEWin2.1.6 2.887.680 -> 2.887.680
win32k.sys 0.4.11 GCC4.7.2dbg RosBEWin2.1.6 2.867.200 -> 2.867.200
win32k.sys 0.4.10 GCC4.7.2dbg RosBEWin2.1.6 2.863.104 -> 2.863.104
win32k.sys 0.4. 9 GCC4.7.2dbg RosBEWin2.1.6 2.834.432 -> 2.834.432
win32k.sys 0.4. 8 GCC4.7.2dbg RosBEWin2.1.6 2.830.336 -> 2.830.336
win32k.sys 0.4. 7 GCC4.7.2dbg RosBEWin2.1.6 2.830.336 -> 2.830.336

9 months ago[0.4.10][DMILIB][UDMIHELP][DXDIAG][SYSTEMINFO] Backport "Include only one user-mode...
Joachim Henze [Sun, 23 Jul 2023 20:50:27 +0000 (22:50 +0200)]
[0.4.10][DMILIB][UDMIHELP][DXDIAG][SYSTEMINFO] Backport "Include only one user-mode DMI library (#1054)"

partially 0.4.11-dev-797-g c1195f5300b3116594bb27b23fb16e321775f36b [DXDIAG][SYSTEMINFO] Include only one user-mode DMI library (#1054)
partially 0.4.11-dev-796-g baf2c0cc2d849fffa68a1634ed87e7da08805682 [DMILIB][UDMIHELP] Allow functions to be linked with C++ code.

The [SYSDM]- and [DRWTSN32]-parts are not needed to be ported back, because those did not have the dependency yet in 0.4.10, but only in 0.4.11

And the commit is also not needed to be ported back further than releases/0.4.10, because both: dmilib and udmihelp have only been introduced during 0.4.10-dev'ing.
They do not exist in 0.4.9 yet. Also no such users therefore.

9 months ago[0.4.10][DXDIAG] Backport trivial translation fixes 2023-07-23
Joachim Henze [Sun, 23 Jul 2023 16:53:11 +0000 (18:53 +0200)]
[0.4.10][DXDIAG] Backport trivial translation fixes 2023-07-23

0.4.15-dev-5770-g 793e9f20ef84873705a782165e5c446b133e19e7 [DXDIAG] Fix text truncation on button for french fr-FR.rc CORE-18853
0.4.15-dev-5768-g 29d0deef1384e7050098ee2f04900009550ec5d0 [DXDIAG] Fix text truncation german de-DE.rc CORE-18853 (#5137)
0.4.15-dev-5533-g 5b8886de2647ee7a5e96d59449c3924341f3d61c [DXDIAG] Fix Portuguese (pt-BR) translation (#4953)

and some other translation improvements + EOL-whitespace.

master dxdiag.exe GCC8.4.0dbg            406.016
0.4.14 dxdiag.exe GCC4.7.2dbg 375.296 -> 375.296
0.4.13 dxdiag.exe GCC4.7.2dbg 375.296 -> 374.784
0.4.12 dxdiag.exe GCC4.7.2dbg 376.320 -> 375.808
0.4.11 dxdiag.exe GCC4.7.2dbg 376.320 -> 375.808
0.4.10 dxdiag.exe GCC4.7.2dbg 376.320 -> 375.808
0.4.9  dxdiag.exe GCC4.7.2dbg 371.712 -> 371.200
0.4.8  dxdiag.exe GCC4.7.2dbg 371.712 -> 371.200
0.4.7  dxdiag.exe GCC4.7.2dbg 371.200 -> 371.200

9 months ago[0.4.10][ADVAPI32][PSDK] Selected Backports 2023-07-22
Joachim Henze [Sat, 22 Jul 2023 17:56:14 +0000 (19:56 +0200)]
[0.4.10][ADVAPI32][PSDK] Selected Backports 2023-07-22

0.4.15-dev-5927-g bfc6a119c57c367f8cb6a8e782ec2c81901719a0 [ADVAPI32] Do not fail on creating a registry key if the security attributes have an invalid length. (#5230) fixes regression CORE-15471
partially 0.4.15-dev-999-g 889b077b23490f5eeeb5558501a8a25c637396d1 [REACTOS] *.spec: Fix some function/parameter types (only the [ADVAPI32] part)
0.4.14-dev-1245-g 8ae8083378546fc7d907e489aeebddbe3c8d9399 [ADVAPI32][PSDK] CryptReleaseContext(): dwFlags parameter is DWORD, only
0.4.13-dev-666-g f3a9b524f102eb4c9f8759d0c51990e354f488a2 [ADVAPI32] Fix copying the TokenSource string
0.4.13-dev-363-g 46259092f97446e455b92a1517a3f90f4d934fa9 [ADVAPI32:REG] Remove some useless casts (#1614)
0.4.12-dev-237-g 446e13745c72d9874091b9a808f371cc737dc2bd [ADVAPI32] Stop pretending our advapi32 is NT6+, inconsistent TEB definitions CORE-15526

also fix a bit of whitespace, wrong format-strings and missing \n in debugprints

---------
in 0.4.9 + 0.4.10: scm.c: fix wrong arguments count in 2 TRACEs, which were introduced by the guilty 0.4.9-dev-417-g 32bb592e126c7f147e482522cff44aecaf7fb45e
Contrary to that delete all those TRACEs in 0.4.7 and 0.4.8 because they have been touched a thousand times after their introduction to fix all their format-string-glitches.

---------

advapi32.dll master GCC8.4.0dbg            671.232
advapi32.dll 0.4.14 GCC4.7.2dbg 602.624 -> 602.624
advapi32.dll 0.4.13 GCC4.7.2dbg 602.112 -> 602.112
advapi32.dll 0.4.12 GCC4.7.2dbg 615.424 -> 615.424
advapi32.dll 0.4.11 GCC4.7.2dbg 613.888 -> 613.888
advapi32.dll 0.4.10 GCC4.7.2dbg 604.672 -> 604.672
advapi32.dll 0.4.9  GCC4.7.2dbg 607.744 -> 604.672
advapi32.dll 0.4.8  GCC4.7.2dbg 602.624 -> 589.824
advapi32.dll 0.4.7  GCC4.7.2dbg 601.600 -> 589.312

9 months ago[0.4.10][SDK] Shorten autogenerated Copyright strings (#5210)
Joachim Henze [Sat, 15 Jul 2023 21:53:59 +0000 (23:53 +0200)]
[0.4.10][SDK] Shorten autogenerated Copyright strings (#5210)

Ports back:
0.4.15-dev-5910-g 99e5f51d5b0398d2a90005dfa3f838cca644d40b
in a slightly adapted form, which respects that we were still
a "Team" in 0.4.7 - 0.4.14 and no "Project" yet.
The idea with the "Project" was born during 0.4.15-dev'ing.
That existing difference does also manifest in media/inf/*.inf and I don't intend to
ever change that for the older branches.
That means also that in the older branches we do not need to scroll at all to
see all the text.
Also adopt ReactOS birth year from 1998 to 1996 which we do also use in the msgina startup-dlgs-pics
all the way from releases/0.4.7 at least to todays master head.

9 months ago[0.4.10][TRANSLATION] Fix font glitches in zh-CN.rc & zh-TW.rc
Joachim Henze [Thu, 13 Jul 2023 01:07:48 +0000 (03:07 +0200)]
[0.4.10][TRANSLATION] Fix font glitches in zh-CN.rc & zh-TW.rc

Actually I was working on a much bigger commit in the same context for releases/0.4.7
when I realized that related glitches do also affect master head
and also other release-branches in some areas.

this ports back (also) the following commit:
0.4.15-dev-6308-g 2d9877470d54952d904a6ad18e7f4998d4292a4c

While touching those files, port back some minor translation improvements also.

9 months ago[0.4.10][RAPPS] Selected backports
Joachim Henze [Sun, 9 Jul 2023 23:34:59 +0000 (01:34 +0200)]
[0.4.10][RAPPS] Selected backports

partially 0.4.15-dev-6292-g 5f533bd0198d733ea77ef1b3a75bda512e9ea66d [RAPPS] Update *.rc
partially 0.4.15-dev-6280-g ec974d8a817528616286c91ef839dedaf18b2478 [RAPPS] Main toolbar: Tooltip strings refactoring (#5007)

and pick some other translation improvements from master, strip outdated email-addresses

releases/0.4.7 - releases/0.4.11: Had a bug regarding category "Selected for installation",
it was populated with garbage after starting up the app. The bug in ReactOS is outside of the rapps source-folder.
So I decided to delay introduction of that feature to releases/0.4.12

9 months ago[0.4.10][NETSTAT] Picky Backports up to master-state 2023-07-06
Joachim Henze [Wed, 5 Jul 2023 23:05:39 +0000 (01:05 +0200)]
[0.4.10][NETSTAT] Picky Backports up to master-state 2023-07-06

0.4.15-dev-6256-g 8ef47d2e5e53f62fb48073a34e4467c743a18d6c [NETSTAT] Optimize a bit (#5405)
partially 0.4.15-dev-6245-g 5ee97b95370cc1ad05d944206b51ad597f111ac8 [NETSTAT] -b flag implies -o flag on Windows XP/2003. CORE-19006 (#5377)
partially 0.4.15-dev-6211-g 40864bc15cb77ceb07e73f8625486c120192bb18 [NETSTAT] Fix crash when parsing the protocol CORE-19005 (#5363)
0.4.15-dev-3338-g 0e75fc924045d4bcc8a2d859a905f251711cb1ce [NETSTAT] Fix coverity #1477187 "Double free" (#4069) CORE-17831 [I ported this double-free-fix earlier already into releases/0.4.13 & releases/0.4.14 and it didn't affect any older releases than that]
partially 0.4.14-dev-479-g 1fa2780796671e23a98882d011fa1324b29ad211 [NETSTAT] Fix output formats
0.4.13-dev-584-g 5e10c4ed32b4caeab9ec09b6d705973e7caaaad4 [NETSTAT] ShowUdpTable(): Fix "tcp" copypasta (#1699)
partially 0.4.13-dev-579-g b695971c7f48c6e658a8671a2b23799dd79d98f2 [NETSTAT] Simplify some code
partially 0.4.13-dev-578-g ab7dc56d6cc0f93b02247943100459f227dbdd3d [NETSTAT] Formatting
partially 0.4.13-dev-519-g dda5ec44b057406e5bef7e93322161465e1f5fb6 [NETSTAT] Simplify DoFormatMessage() (just the changed retval)
0.4.11-dev-820-g 29c883e2a5595228a7773912df1006ddd2504de6 [NETSTAT] Add support for displaying UDP connections owning process
0.4.11-dev-814-g 2b5507336017cdea726f146a225823a932df54c2 [NETSTAT] Add support for displaying TCP connections owning process

9 months ago[0.4.10][IPHLPAPI][TCPIP][IP][LWIP][SDK][PSDK] Port back groundworks for netstat
Joachim Henze [Sun, 2 Jul 2023 23:47:10 +0000 (01:47 +0200)]
[0.4.10][IPHLPAPI][TCPIP][IP][LWIP][SDK][PSDK] Port back groundworks for netstat

This fixes:
CORE-15363 'netstat TCP foreign address port is wrong'
CORE-5401 'Netstat returns ERROR_NO_DATA Pipe being closed'
CORE-5126 'UNIMPLEMENTED: iphlpapi: GetExtendedUdpTable'
even without touching the netstat binary yet.
And is mostly based on work done by Pierre Schweitzer during 0.4.11-dev'ing.

by porting back:
0.4.15-dev-6231-g a5360f542f5b26616673a193c906f808160f29e9 [IPHLPAPI] Formatting only, no functional change
partially 0.4.14-dev-1425-g f540b2b20ca5f35ce2678bbfa05c120a21fd5f4b [IPHLPAPI] GetAdaptersAddresses(): Add 1 free() in ERROR_NO_DATA case (#2526) (but leave out adding the comments in .h)
0.4.12-dev-686-g 22f60faf439c8d6b195085acfd94912de6d214af [IPHLPAPI] Remove dead code to fix CID 513306, 14426731442697
0.4.11-dev-880-g bf052e120b3ad451b1b450cc7c1ebf12b1f57ffa [IPHLPAPI] Reduce code duplication and use a single function for TCP and UDP enumerations + fix corruption
0.4.11-dev-867-g 159f5b676ab749064dfafe60d712e979537d9a1a [IPHLPAPI] Fix broken size computation leading to buffer overrun
0.4.11-dev-861-g 05657bfcfd6c8b490ab404a25222edeb9fc4712c [SDK] Add a non documenter TOIID used to query modules information
0.4.11-dev-827-g ce7f9211a0620300515f62cc359b3c55909951d8 [TCPIP] In InfoTdiQueryGetConnectionTcpTable(), only return if there's a connection
0.4.11-dev-823-g 0ec92100bcaaf01fe3d824eeae20835c75d16e59 [TCPIP] Properly display local information for established connections
0.4.11-dev-819-g fbdfe7bd4067b2ea83033a0080cecd214c51437a [SDK] Add GetExtendedUdpTable()
0.4.11-dev-818-g 370b03b5235aa6e087d1de7aa8f3c2ea6ad280f5 [IPHLPAPI] Reimplement GetUdpTable() using GetExtendedUdpTable()
0.4.11-dev-817-g 58cef01422060fc69d790d49c60df69b737e69c9 [IPHLPAPI] Implement GetExtendedUdpTable() CORE-5126
0.4.11-dev-816-g 6fb6c7b50b7f7b65737b373553ca0d400d924f36 [IPHLPAPI] Implement getOwnerUdoTable(), to get UDP connections with owner PID
0.4.11-dev-815-g 13de7f08af1fa37c6f0aeb8bd58d8971aab8d069 [TCPIP] Implement enumerating UDP connections with owner PID
0.4.11-dev-813-g 73c87d5c1431d09b42ad60094dee1ce9eb4107b8 [IPHLPAPI] Reimplement GetTcpTable() using GetExtendedTcpTable()
0.4.11-dev-812-g d76ac645e8ca4d6deb0c673400da74069505ce7b [IPHLPAPI] Don't leak memory
0.4.11-dev-811-g c949a12506046b0aa9bc74e6a8fa1dabefcecba7 [IPHLPAPI] Implement the IPv4 TCP_TABLE_OWNER_PID_* cases in GetExtendedTcpTable()
0.4.11-dev-810-g dca7e5689ff1f885a67bf98058136c1d3c15d7b9 [IPHLPAPI] Implement getOwnerTcpTable(), to get TCP connections with owner PID
0.4.11-dev-809-g 411504b5f4a56443233c8acc9247851448dcf544 [TCPIP] Implement enumerating TCP connections with owner PID
0.4.11-dev-808-g 935978ee57da95bcfcc6aadaaf248a7176083897 [TCPIP] Store creator PID in ADDRESS_FILE
0.4.11-dev-807-g 702d44c683c1289a0e023022c8f63aaafb939c1b [IPHLPAPI] Implement the IPv4 TCP_TABLE_BASIC_* cases in GetExtendedTcpTable()
0.4.11-dev-804-g 91b013d3877d9f3a08c21ce3cdec1950a34220f9 [TCPIP] When enumerating sockets, retrieve their state and return it
0.4.11-dev-803-g 738f3276683a782eb74cabc0bac0f63dfc410613 [IP] Add a TCPGetSocketStatus() wrapper around LibTCPGetSocketStatus()
0.4.11-dev-802-g 29c15104230e95467229a3515e092c5e744bdeff [LWIP] Implement a LibTCPGetSocketStatus() function in our LwIP glue
0.4.11-dev-792-g 8a98c85c1f7686a27db2c095061f5224047c816f [IPHLPAPI] Fix checking entity when enumerating TCP/UDP connections
0.4.11-dev-784-g f29016dabe924c5077ba507330210e2c2b093e2b [TCPIP] Properly handle listening sockets
0.4.11-dev-782-g 007f43dd56647c43cbf70ca2f5be24fe7f57e6f3 [TCPIP] Implement returning UDP connections CORE-5401
0.4.11-dev-781-g 77d5f04f22fce0f4ba5108e21f8070d90b891177 [TCPIP] Implement returning TCP connections CORE-5401
0.4.11-dev-799-g 500a5151ea1ef6d35ea4f77ecd16679ce7bdf852 [TCPIP] Fix returned IP address when querying TCP connections CORE-15363
0.4.11-dev-783-g d18b1fe24b933c3d214cbcd9c116d0bf094329fb [IPHLPAPI] Properly count UDP entries CORE-5401
0.4.11-dev-780-g e3cb9697a4e6614d4efa156ee5d0c2383fa016a6 [IPHLPAPI] Implement getNumUdpEntries() and getUdpTable() CORE-5401
0.4.11-dev-779-g 678204790cc57ebf74d60791c869b3957dd003a9 [IPHLPAPI] Implement getNumTcpEntries() and getTcpTable() CORE-5401
----------------
Strip some DPRINTs entirely in IPHLPAPI/TCPIP in releases/0.4.7 - 0.4.10 (instead of fixing them like we did on master). Many of them had x64 issues:
This was partially *inspired* by watching the following commits:
0.4.13-dev-14-g f9a967b2ae48fbe1a6bfde76935d2bba67e2330b [IPHLPAPI] Fix misc 64 bit issues (only the parts in iphlpapi_main.c with the format strings)
0.4.10-dev-409-g 0f8439aa71c3349282eea09e001965744292eaaa [IPHLPAPI] Fix 64 bit issue, just the one within [IPHLPAPI] ifenum_reactos.c
0.4.9-dev-856-g 6c9359221c9bcf5549fb8693834de51ab509cdee [IPHLPAPI] ipstats_reactos: Review all TRACE/WARN calls
This does not only fix the x64 incompatibilities by superseding those 3 commits but also will keep the file-sizes under control for the backports.

10 months ago[0.4.10][AFD] ReceiveActivity: Don't return STATUS_FILE_CLOSED in case of FCB overrea...
Joachim Henze [Wed, 14 Jun 2023 21:42:23 +0000 (23:42 +0200)]
[0.4.10][AFD] ReceiveActivity: Don't return STATUS_FILE_CLOSED in case of FCB overread (#4972)

fix CORE-18328 'FileZilla 3.8 unable to list content of a remote directory due to undue ECONNRESET'
I tested to list the contents of ftp.heise.de and downloaded some files from its pub.

by porting back:
0.4.15-dev-5870-g 9f9b81e3961462698294c0858f113757a23d58a4 [AFD] ReceiveActivity: Don't return STATUS_FILE_CLOSED in case of FCB overread CORE-18328 (#4972)

10 months ago[0.4.10][NTUSER] co_UserDestroyWindow: Validate window before destroying it (#5054)
Joachim Henze [Sat, 10 Jun 2023 10:39:51 +0000 (12:39 +0200)]
[0.4.10][NTUSER] co_UserDestroyWindow: Validate window before destroying it (#5054)

Prevent the window from being destroyed twice
fixes CORE-18821 'WIN32K BSOD 0x1E when opening Build menu in AVR Studio 4.19'
port of 0.4.15-dev-5938-g 77d46531648c244b6eec2cdb75a710d4e8ffd8cb

And port back also some unrelated formatting improvements,
and kill a block of code that was commented out in all branches,
and was not x64 compatible in 0.4.7/0.4.8, and therefore was a diff already anyway.

10 months ago[0.4.10][BLUE] Optimize ScrIoControl for speed (#5185)
Joachim Henze [Tue, 6 Jun 2023 21:42:56 +0000 (23:42 +0200)]
[0.4.10][BLUE] Optimize ScrIoControl for speed (#5185)

Optimize IOCTL_CONSOLE_FILL_OUTPUT_ATTRIBUTE and IOCTL_CONSOLE_FILL_OUTPUT_CHARACTER for speed by using cache.
a part of CORE-18838
It is faster because it minimizes the amount of pointer dereferences within the for-loops.

fix picked from 0.4.15-dev-5881-g 3526ffc094277e03d1a01435036bfb089806b88d
---------
also partially pick unrelated cleanup from:
0.4.14-dev-439-g 75f188c20028fe5493ca827e8758e01eecfa36f6 (only the change in blue.c)
---------
and take back with me some whitespace-tweaks to bring the file closer to todays releases/0.4.14

10 months ago[0.4.10][BOOTDATA] hivesys.inf Play safe with ServiceGroupOrder
Joachim Henze [Mon, 5 Jun 2023 21:43:36 +0000 (23:43 +0200)]
[0.4.10][BOOTDATA] hivesys.inf Play safe with ServiceGroupOrder

Restores *inferior* code for older branches.

I recently ported back the *proper* 0.4.15-dev-4352-g cfe387a9b09751241060a6c0f24c94bc46df98b5
as part of:
0.4.10-release-186-g e5d8019d24f5e093ef785467d77a8ba0a115aded
0.4.9-release-197-g 557db9a1acb1b03bb865aa3029d387ca2d66c881
0.4.8-release-203-g 78daf4169a67904682e8066f82f30f2695c630fe
0.4.7-release-229-g e0a3b0c8f5d6a1164b247c1ea97f1263aec460fd

That backport worked very well in releases/0.4.11 to releases/0.4.14

and it also caused no severe issues for releases/0.4.7 to releases/0.4.10
e.g. the mouse and keyboard works well, tested with VBox4.3.12 & MS VirtualPC 2007 SP1 6.0.192.0
but I noticed some logspam that happens for releases/0.4.7 to releases/0.4.10 and only during 2nd stage:

bad-path (with the commas):
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\system32\drivers\kbdclass.sys' with status 0xc000003a
(../ntoskrnl/io/iomgr/driver.c:696) Failed to load class upper filters: C000003A
(../ntoskrnl/io/iomgr/device.c:434) No DriverUnload function on PnP driver! '\Driver\I8042PRT' will not be unloaded!
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\system32\drivers\mouclass.sys' with status 0xc000003a
(../ntoskrnl/io/iomgr/driver.c:696) Failed to load class upper filters: C000003A
(../ntoskrnl/io/iomgr/device.c:434) No DriverUnload function on PnP driver! '\Driver\I8042PRT' will not be unloaded!
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\system32\drivers\vbemp.sys' with status 0xc000003a
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\system32\drivers\kbdclass.sys' with status 0xc000003a
(../ntoskrnl/io/iomgr/driver.c:696) Failed to load class upper filters: C000003A
(../ntoskrnl/io/iomgr/device.c:434) No DriverUnload function on PnP driver! '\Driver\I8042PRT' will not be unloaded!
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\system32\drivers\mouclass.sys' with status 0xc000003a
(../ntoskrnl/io/iomgr/driver.c:696) Failed to load class upper filters: C000003A
(../ntoskrnl/io/iomgr/device.c:434) No DriverUnload function on PnP driver! '\Driver\I8042PRT' will not be unloaded!
...
(../base/services/umpnpmgr/umpnpmgr.c:3653) Removal pending: ACPI\PNP0303\2&ccb86d17&0000
(../base/services/umpnpmgr/umpnpmgr.c:3623) Safe removal: ACPI\PNP0303\2&ccb86d17&0000
(../base/services/umpnpmgr/umpnpmgr.c:3653) Removal pending: ACPI\PNP0F03\2&ccb86d17&0000
(../base/services/umpnpmgr/umpnpmgr.c:3623) Safe removal: ACPI\PNP0F03\2&ccb86d17&0000
(../base/services/umpnpmgr/umpnpmgr.c:3653) Removal pending: ACPI\PNP0303\2&ccb86d17&0000
(../base/services/umpnpmgr/umpnpmgr.c:3623) Safe removal: ACPI\PNP0303\2&ccb86d17&0000
(../base/services/umpnpmgr/umpnpmgr.c:3653) Removal pending: ACPI\PNP0F03\2&ccb86d17&0000
(../base/services/umpnpmgr/umpnpmgr.c:3623) Safe removal: ACPI\PNP0F03\2&ccb86d17&0000

good-path (without the commas):
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\System32\DRIVERS\i8042prt.sys' with status 0xc000003a
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\System32\DRIVERS\i8042prt.sys' with status 0xc000003a
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\system32\drivers\vbemp.sys' with status 0xc000003a
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\System32\DRIVERS\i8042prt.sys' with status 0xc000003a
(../ntoskrnl/mm/ARM3/sysldr.c:3025) ZwOpenFile failed for '\SystemRoot\System32\DRIVERS\i8042prt.sys' with status 0xc000003a
...
(../../ntoskrnl/mm/ARM3/sysldr.c:176) Loading: \SystemRoot\System32\DRIVERS\i8042prt.sys at F771B000 with 1b pages
...
and no such 'Removal pending'

So we do still miss some unknown commit from 0.4.11-dev'ing that will avoid that additional logspam with the corrected hivesys.inf.
I have not identified that commit yet, and until I will, I do play safe by restoring the wrong List-multi-string in hivesys.inf.

11 months ago[0.4.10][REGEDIT] Port back several improvements and crash-fixes
Joachim Henze [Mon, 1 May 2023 17:55:28 +0000 (19:55 +0200)]
[0.4.10][REGEDIT] Port back several improvements and crash-fixes

0.4.15-dev-5985-g 31fdaca8c014ef916be824685e3af09712aef4dc [REGEDIT] Strip redundant include, minor formatting (#5264)
0.4.15-dev-5970-g 9d7d3314b337d6ab5589a66c4c69a563dd8b2c78 [REGEDIT] Check whether pszSelectKey is NULL on CDN_FILEOK (#5255)
0.4.15-dev-5898-g 84e580b67e5a407afdd669c3204ef97d115d8cb0 [REGEDIT] Fix ListView selection and finding (#5150)
(superseded) 0.4.15-dev-5802-g cce3eb93931163db4e463650e6f44e451e77a239 Fix buffer read past end problem. (#5146) CORE-15896 and CORE-18230
partially 0.4.15-dev-5753-g 1ee9ea451861f4059937ddb78070700950b95d43 [REGEDIT] Avoid buffer overflow in SelectNode. CORE-18602 (I left the wscpy-replacement at the end for the fixed size strings, as they would increase binary size, and are totally safe)
partially 0.4.15-dev-5308-g 5daf5cd057d1b0499653196e73a1b2c410a7313a [REGEDIT] Use shell icons for the tree view (#4841) CORE-10764
0.4.15-dev-4850-g 568383c9b91c2e529ae4a3609e416ec8044cfc23 [REGEDIT] Fix issue at Find registry key (#4341)
partially 0.4.15-dev-2265-g bebdfda8b76c342aabec28cb25fb787c7762a310 [REGEDIT]... Use newer file open dialog (#3571)
0.4.15-dev-1470-g 9de87871877451e6fe30a8c2e98015c9ead7b55a [REGEDIT] Display search finished messagebox when using Find Next (F3) menu item (#3371) CORE-17368
0.4.15-dev-1302-g 6aae5f4fdd87efe805f7bd44c601044b0cbe47c2 [REGEDIT] Make "finished find" messagebox owned by Regedit window (#3369) CORE-17367
0.4.15-dev-169-g 0461de33c5de3fee70a004b2f9c9e03ca70cfc67 [REGEDIT] Fix HeapFree() on the wrong variable (#2736)
0.4.14-dev-1484-g c83211277131a19e6c9a5ceb81f855a86c3be159 [REGEDIT] Fix tree view popup menu issues CORE-16887
0.4.14-dev-1468-g 763de230b2a630bf534b442677f4752880da7dc4 [REGEDIT] Fix inconsistent menu state for 'Permissions' CORE-16889
a more efficient solution than 0.4.14-dev-1467-g b280615b85d25bc31a03ab975fcafdf2da8075ba [REGEDIT] Crash on selecting 'New' on root item 'My computer' CORE-16888
0.4.14-dev-1466-g 13dd917589a9869585f10f38a05f90a52e9f9ddf [REGEDIT] CORE-16892 fr-FR Translation update
0.4.13-dev-464-g 6b9572728219f93c1109e83132b6dd6723151f84 [REGEDIT] Adjust ListView column widths (#1663) CORE-15187
0.4.12-dev-47-g 63a0ac04612a1a3f51543d4a2c35b71bfb1ebc0c [REGEDIT] Set background brush to child window (#1122) CORE-15442
0.4.11-dev-481-g efbebf983cc1708eca69194fa6461d6d54622707 [REGEDIT] Improve the layout of controls (#967) CORE-15187
0.4.11-dev-480-g e4c2cff6c8ba1bb6a61e89e0dd3527e255c81eb3 [REGEDIT] Don't paint background in WM_PAINT (#970) CORE-15187
0.4.11-dev-458-g 125db5ead912a90f288e3f75fa713e27430bf4f9 [REGEDIT] Trivial resizing code fix (#950)
0.4.11-dev-457-g b3231e0bf5dbf57bb9d657486db7ce3059cf2e31 [REGEDIT] Code formatting (#949)

a few fragments only of 0.4.15-dev-5304-g 36a7f0dc7c10a9cf7c137ed617a0d419999a02f9 [REGEDIT] framewnd.c resize_frame_rect(), main.h strip externs before func declarations and reordering, treeview.c InitTreeViewImageLists()

also port back a few selected translation improvements, and strip the suggestions-feature in older branches <= 0.4.13 which does not exist in 2k3sp2.

12 months ago[0.4.10][CONSOLE] Harmonize IDC_LBOX_FONTTYPE height to 40 for *.rc (#5211), ...
Joachim Henze [Tue, 11 Apr 2023 16:11:58 +0000 (18:11 +0200)]
[0.4.10][CONSOLE] Harmonize IDC_LBOX_FONTTYPE height to 40 for *.rc (#5211), ...

port back:
0.4.15-dev-5901-g c37addcbf5feb05219bbe8e030931f0dd54d6adc [CONSOLE] Harmonize IDC_LBOX_FONTTYPE height to 40 for *.rc (#5211), tweak some headers
and partially 0.4.15-dev-5503-g 5123edd731094465b901c0d69813628b7533da8c [CONSOLE] Improve font preview and Update de-DE.rc (#4933)

and also pick a few other rc improvements here and there, mainly dupe-accelerator-fixes, and whitespace+logmuting in some .c

12 months ago[0.4.10][BROWSEUI] Port back several fixes and tweaks
Joachim Henze [Sun, 2 Apr 2023 00:34:50 +0000 (02:34 +0200)]
[0.4.10][BROWSEUI] Port back several fixes and tweaks

0.4.15-dev-5688-g 460a01b167066f755e5ae06ac9fbd913f73065d2 [BROWSEUI] CORE-18646 Fix crash while editing a label in the folder tree, add missing break
0.4.15-dev-5686-g a777cc2cc4768bd25190863d4aeef7cbd356841c [BROWSEUI] CORE-18646 Fix crash on backspace while editing a label in the folder panel (#5059)
0.4.15-dev-4497-g beefb07d183d289397baa1360e3fd44636db15bc [BROWSEUI] CORE-12804 Remove useless variable and unreachable code (#4483) and strip EOL whitespace
0.4.13-dev-1186-g d41c5be3bd961f69b3ea40b27565870972968b4e [BROWSEUI] Fix a PIDL leak.
0.4.13-dev-283-g 801ec51a91c3af778d88d8f9446eee0cb1d8e943 [BROWSEUI] Fix indentation

12 months ago[0.4.10][SYSSETUP] Backport selected RC improvements
Joachim Henze [Thu, 30 Mar 2023 19:36:47 +0000 (21:36 +0200)]
[0.4.10][SYSSETUP] Backport selected RC improvements

0.4.15-dev-5890-g 56bf3969cd88aa9028f9961355c56f72ece13207 [SYSSETUP] Tweak IDD_COMPUTERPAGE positioning (#5158)
0.4.15-dev-5875-g 7c755d9b849a75382cc3579fe146c05c3ac08f14 [SYSSETUP] Delete obsolete lt-LT.rc
0.4.15-dev-5874-g c7622345462d62d841c299c6fae478362b25cea9 [SYSSETUP] Fix font glitch in ja-JP.rc (#5192)
0.4.15-dev-5871-g 3963d7d5f27e2d8f60294f8d463109efb599fbfd [SYSSETUP] Delete obsolete fi-FI.rc
0.4.15-dev-5772-g 784946db8c743bf5c66b14223d3b756df457874e [SYSSETUP] Tweak resources CORE-18870
0.4.15-dev-5732-g 909b165c4ab993c99edf33b0beacfea758054d8f [SYSSETUP] Removed qemu credit entirely (#4971)
0.4.15-dev-5678-g cc54ecd779eeb178ceeed1777cb1776f50c43593 [SYSSETUP] Fix a space-typo in IDS_WZD_PASSWORDEMPTY for *.rc (#5049)
0.4.15-dev-1102-g ad4c174eb646abd4b8654289a1ec966f5a0f2b81 [SYSSETUP] fr-FR: Change keyboard accelerator for Owner Organization field (#3282)

0.4.14-dev-1445-g 173a2af48c337c1d1cbb12d32fd271c8ff868d00 CORE-16874 [SYSSETUP] Update authors page in second stage setup
 (that part got unhidden by 0.4.10-dev-466-g 35f62fc5ba0b69e7335ff41400cb3b45660f4557 . Picked as one of the needed preparations in case I would ever port back the guilty rev later)

and a few other gems from other partially picked master-commits that are not explicitly mentioned here.
--------
Also strip the IDC_LOCALETEXT and IDC_LAYOUTTEXT from the resources as that was only implemented in releases/0.4.15,
is even today still not properly localized there, they changed their dimensions again with that
and I don't intend to port that back anytime soon. And if I ever will, then all the resources would need to be touched again anyway.

13 months ago[0.4.10][NETSHELL] Port back some RC improvements
Joachim Henze [Fri, 24 Mar 2023 20:06:53 +0000 (21:06 +0100)]
[0.4.10][NETSHELL] Port back some RC improvements

port back:
0.4.15-dev-5785-g c766ee9ebba49df62ac995b2bb276862d1cc622c CORE-18874 [NETSHELL] Shrink radio buttons vertically (#5157)
0.4.14-dev-1481-g fff2fc9327b73486fbdb4c91e7b68d2eaace06f4 CORE-16903 [NETSHELL] RC Overhaul all languages IDD_LAN_NETSTATUS & IDD_LAN_NETSTATUSADVANCED
0.4.14-dev-293-g 2b39b42a0aeff2b503849882dfc67e14c219f933 CORE-15248 [NETSHELL] Fix checkboxes in the network adapter properties
partially 0.4.12-dev-315-g 01e5cb0c0423dfef5c6e34ceb0c4bf9f29a3f5f8 [NETSHELL] Fix incorrect resource IDs.
and port back also a few other small RC-improvements from master here and there

13 months ago[0.4.10][REACTOS] Backport "http://www.reactos" -> "https://reactos" and readme.txt
Joachim Henze [Sun, 19 Mar 2023 11:32:59 +0000 (12:32 +0100)]
[0.4.10][REACTOS] Backport "reactos" -> "https://reactos" and readme.txt

by porting back:
0.4.15-dev-5669-g 09dde2cff938513e81ac5451b84e69e11b6acfa1 [REACTOS] "http://www.reactos" -> "https://reactos" (#5043)
0.4.15-dev-3250-g da4d0545ebf9610331ec186a1275fb44ed22245f [BOOTDATA] readme.txt: Trivial correction Seven->7 (#4014)
0.4.15-dev-2940-g a559d830b546c0f76c37bf0a2e9b1de0128ee58f [BOOTDATA] Update readme.txt
0.4.14-dev-1550-g b7503df6edf7552d20f3232db027baa987569b28 [BOOTDATA] readme.txt: Update now-redirected 'www.reactos.org' URLs (#2633)

13 months ago[0.4.10][BOOTDATA] Port back some hivesys.inf fixes / cleanup
Joachim Henze [Tue, 14 Mar 2023 22:15:41 +0000 (23:15 +0100)]
[0.4.10][BOOTDATA] Port back some hivesys.inf fixes / cleanup

0.4.15-dev-5102-g 18d40ebe81cbc4646f48247d831bc1d53752dd1b [BOOTDATA] hivesys.inf: Remove redundant fastfat "Start" entry. The one in the [AddReg] section has the very same value.
-------
0.4.15-dev-5012-g e3f8f52f3d719d9893d17f879c9ec2d66dcebb5b [BOOTDATA] Delete "Layout Id" from Japanese/Korean Keyboard Layout
Katayama explained: These registry values are for pre-installed IMEs we don't have. It is safe to remove them also in the older branches.
-------
0.4.15-dev-4352-g cfe387a9b09751241060a6c0f24c94bc46df98b5 [BOOTDATA] Add two missing separating commas in the ServiceGroupOrder List multi-string.
-------
PARTIALLY PICK: 0.4.15-dev-4412-g 94d144f8a577123e36bf611383eaa46b1e151b32 [BOOTDATA] Fix some services groups
- AudioSrv belongs to "AudioGroup" (and not "Audio" that doesn't exist).
- lanmanworkstation belongs to "NetworkProvider" (without a space).
I dared only to pick the "NetworkProvider" and "AudioGroup" part for now. The MUP part I might do later when actually being able to retest CORE12107 "VMWare tools" myself.
-------
0.4.15-dev-1097-g 0689455bb3e66f89b64aecefc2367efd454b9c16 [BOOTDATA] Comment out biosinfo.inf (#3266)
This allows ros to boot with its LiveCD files using MS NTLDR!
I stripped the line here, instead of commenting it as the master commit did. The file does not exist.
-------
Additionally to those backports I cleaned up some of the stuff that was previously commented out and/or broken:

- instead of following 0.4.15-dev-3322-g c6e002bfb3069e0c1dc2b313b1a5bc11821eac07 to fix the Latvian kbdlv1.dll QUERTY->QWERTY typo, I just stripped those lines, as
they are commented out anyway. And were added in the broken and already commented way by 0.4.9-dev-825-g c2089d5319ea1a12d5788d2fa798e5d08b1e3ef4
We never had them active. No such file even.

- Also strip some 'translated' service strings that were just copy-pasted from the english strings section
without actually getting translated at all. The english ones will be used anyway.
It is enough to have that redundant stuff in master to allow translators to see it.

- Also strip the disabled kbdfo.dll Faeroese keyboard layout. Enough to have those commented lines in master as a TODO reminder. Was never active ever. No such file even.

- Also strip the disabled BENGALI_INSCRIPT kbdinben.dll. Where the existing lines even had bugs in the older branches as proven by 0.4.15-dev-3322-g c6e002bfb3069e0c1dc2b313b1a5bc11821eac07
It was always disabled and was neither localizable, nor did it have the correct layout code even. No such file even.

None of those changes will introduce any additional diff or break any existing features.
They just slightly move the already existing diffs position in time towards newer release-branches.

13 months ago[0.4.10][AUDIOSRV] Update to master state 2023-02-18
Joachim Henze [Mon, 13 Mar 2023 20:14:16 +0000 (21:14 +0100)]
[0.4.10][AUDIOSRV] Update to master state 2023-02-18

by porting back:
0.4.15-dev-5710-g 9672dc50479f42c6da51b426618f05a9394a9ef2 CORE-16912 [AUDIOSRV] Overhaul logging interfaces (#5067)
0.4.15-dev-445-g 353b5440474f3bf2c9dd16eee11bd42f11ea5d6d CORE-16814 [AUDIOSRV] Disable Logging to file (#2599)
0.4.13-dev-894-g 5f7575909f7730aca9112e7b736951c9cc6e9fea CORE-16307 [AUDIOSRV][BOOTDATA][INCLUDE][MMSYS] Rename RosAudioSrv to AudioSrv (#1826)
0.4.13-dev-715-g 353e05a4f363db3a9779d0ceea16230eb26a3e22 [AUDIOSRV] Fix the coding style. No code changes!

13 months ago[0.4.10] Update Scrollbar code to state of master 2023-March-12
Joachim Henze [Sun, 12 Mar 2023 10:03:05 +0000 (11:03 +0100)]
[0.4.10] Update Scrollbar code to state of master 2023-March-12

by porting back:
0.4.15-dev-5784-g 2aea4ab94529a0a6b1620217b272e5b6f06733b8 [NTUSER] Scrollbar.c pure whitespace fixes
0.4.15-dev-5680-g 5ad57373701686876c604f3bd58fbb2e449d5d81 CORE-18050 [NTUSER] Delete temporary workarounds in co_IntSetScrollInfo() (#4985)
0.4.15-dev-5355-g 0484beb04b8c35599e9a32fa033645d6cc57ca40 CORE-18593 [NTUSER] Don't redraw scrollbar if it is hidden command prompt (#4849)
0.4.15-dev-4492-g 03422451b3b4a7df96a9e9af5ce7533fd6cb55d9 Add '\n' to debug logs (partially, only the scrollbar.c part)
0.4.15-dev-3875-g 977c129f331cd5b92e0c7a4925e85fc14e147ec9 CORE-18048 [USER32] SetScrollPos should not bypass UserApiHook (#4372)
0.4.15-dev-3849-g fd28a69de63f1a801384e3d6c5675189aa82455e CORE-17780 [WIN32SS] Store the scrollbar theming enabled flag in the scrollbar (#4367)
0.4.15-dev-3175-g 222acf5a3ed11f74adbe0d1bdee527447fd81bbf CORE-17777 [NTUSER] Scrollbar.c, Avoid potential out-of-bounds-accesses in co_IntSetScrollInfo()
0.4.15-dev-3174-g dda9c3979e87223b66e61f0f936f46920221e509 CORE-17769 'Rapps Listview manual resize may erroneously not draw the triangles sometimes'
0.4.15-dev-3147-g 3bf7e3ac13c5c9df373827c102b763b5b9822204 CORE-17754 and CORE-17755 [NTUSER] Improve Themed Scrollbars by Minimizing Updates (#3953)
0.4.15-dev-3086-g 236649c626f97c68a9d0a6709a1bb342681b2aa0 CORE-16375 [UXTHEME][NTUSER] Fix flashing of scrollbar when scrolling (#3868)
0.4.15-dev-2375-g ffea5152e607512f09efdb0bd02499e481877f59 [WIN32SS][NTUSER] Fix an integer underflow within scrollbar info setting
0.4.14-dev-1134-g 00adb1a3f967ac7f5cd56b4c39df72a7b3814603 [WIN32SS] Improve Drawing Scrollbars
- CORE-14755 fixed, flashing scrollbar triangles (we know 131 affected apps just from rapps!)
- CORE-13931 fixed, FamiTracker invisible about-dlg
- CORE-14685 improves a bit, but is not entirely fixed
- CORE-11561 'Notepad scrollbars problem' fixed

This will also speed up NSIS installers that display their file-copy-progress by
a self-scrolling listview.

13 months ago[0.4.10][WMISVC][SRVSVC][WKSSVC][DHCPSVC][WUAUSERV] Start/Stop behavior
Joachim Henze [Mon, 6 Mar 2023 10:47:27 +0000 (11:47 +0100)]
[0.4.10][WMISVC][SRVSVC][WKSSVC][DHCPSVC][WUAUSERV] Start/Stop behavior

by partially porting back:
0.4.15-dev-xxx [WMISVC][WUAUSERV] Fix 2 DPRINTs that lack arguments
0.4.13-dev-684-g eab73ad1a4613c79af7f8eed45d3b4497ec08dfe [SRVSVC] Make the server service stoppable
0.4.13-dev-683-g 4073a8c58db8441489f142b65f088dfd28c9c2f7 [WKSSVC] Make the workstation service stoppable
0.4.13-dev-680-g eb532bc64159f62ca766efa1bb8a2d199dcf95e9 [WMISVC] Accept stop and shutdown commands when the service is running
0.4.12-dev-1063-g 15a828c86d1d68a9dabaaa77bb6bb9d5e7104f42 [DHCPCSVC] Eliminate unused code and associated parameters

and revert 0.4.10-dev-558-g 55368bacc8cba3f9b729d1e06e0184f11907e294 (#355) [WUAUSERV][WUSA]
on this branch as that Service stub does not even start in releases/0.4.10-0.4.12.
And it doesn't allow any application to run that wouldn't have run before.
Even with those changes the .NET4.5 and Python 3.5 mentioned in that PR don't work yet.
We postpone introduction of WUAUSERV to releases/0.4.13 therefore.

13 months ago[0.4.10][NTOS:MM] Fix BSOD0x1E, Ignore sections either PointerToRawData or SizeOfRawD...
Joachim Henze [Tue, 28 Feb 2023 02:56:59 +0000 (03:56 +0100)]
[0.4.10][NTOS:MM] Fix BSOD0x1E, Ignore sections either PointerToRawData or SizeOfRawData is zero

VC1 Linker fills BSS.SizeOfRawData with data in IMAGE_OPTIONAL_HEADER.SizeOfUninitializedData but keeps PointerToRawData to zero.

Fixes CORE-18797 and some duplicates of that. (Apps, e.g. Doom95, compiled with Watcom-C-compilers wouldn't run otherwise.)
which regressed by
0.4.10-dev-136-g c7eb46d9fdda30a3b286e244eba023d95f0a4a9b
fix picked from
0.4.15-dev-5589-g 0d4a8d0ea6fba6da0c13f597f614ebd6f20086fd
----------
Port back also some unrelated formatting-improvements within section.c, e.g.: return(xyz) -> return xyz
A *partial* port of
0.4.15-dev-1793-g b7eb0fddf3b675ab495c41e44861a924eded4730
----------
Port back also
0.4.12-dev-973-g d29e216f994f8aa321c32ec77b9ad507357b37b3
[NTOSKRNL] In MmCreateImageSection, properly check whether cache was initialized
This fixes a crash that occurs if nul is typed in cmd.exe (BSOD0x1E)

13 months ago[0.4.10][ROSAPPS] Exclude the templates from compilation by default.
Joachim Henze [Sun, 26 Feb 2023 12:18:07 +0000 (13:18 +0100)]
[0.4.10][ROSAPPS] Exclude the templates from compilation by default.

As end-users don't need those programming examples in the iso.

picked from
0.4.15-dev-5505-g 11031042270e348fae94cf5f782c09b6f4b2e815

14 months ago[0.4.10][NTUSER] Port back minor improvements
Joachim Henze [Sat, 25 Feb 2023 02:42:32 +0000 (03:42 +0100)]
[0.4.10][NTUSER] Port back minor improvements

0.4.15-dev-3341-g 4bc95812fd70ea382e234b3c8e2e043f51fcae3a defwnd.c Mark default as unreachable.
0.4.15-dev-316-g 938df97b54aa0d61d6c2134b5e50d0a2873c0bd6 nonclient.c NC_IconForWindow(): Remove redundant if() (#2842)
and:
-defwnd.c: delete a malformed TRACE() which had a format string, but no argument.
-sysparams.c: A single line from 0.4.14-dev-676-g 9ce12f0b4cd571e2121027511900ff0953120bc6 avoiding a wcslen() call in SpiLoadTimeOut()

14 months ago[0.4.10][NETSHELL] Repair "Network Connection 'State' does nothing" CORE-18844
Joachim Henze [Sun, 19 Feb 2023 21:57:16 +0000 (22:57 +0100)]
[0.4.10][NETSHELL] Repair "Network Connection 'State' does nothing" CORE-18844

It regressed in this branch by 0.4.10-release-176-g dbe175ef7961c2aa6b856fe7754d8022a9ded955
 ( in master it was by 0.4.15-dev-5613-g 7a17c7d9adadf88eafc382d792771bea5fc412f8 )

It was the only place within the ros sources where message WM_SHOWSTATUSDLG
was sent also from code. And therefore the only one that needed to be updated as well.

Fix picked from:
0.4.15-dev-5712-g a5cd42c1ea8c2106a1a15dde66530d6f3b4ae02e

14 months ago[0.4.10][COMDLG32][SHELL32] Support accelerator keys F2/del, Ignore unused flag,...
Joachim Henze [Tue, 14 Feb 2023 20:06:25 +0000 (21:06 +0100)]
[0.4.10][COMDLG32][SHELL32] Support accelerator keys F2/del, Ignore unused flag, de-DE update

by porting back:

[SHELL32] Ignore unused flag SEE_MASK_UNICODE (#5033)

0.4.15-dev-5676-g 5c22ce374280c05b4d583e4a16192327746f1926
to mute logspam of the kind
fixme:(dll/win32/shell32/shlexec.cpp:1934) flags ignored: 0x00004000
e.g. when starting Rapps.
-------------------------------------------------------------
[SHELL32] brsfolder.c Delete unused #ifndef __REACTOS__ block

It is a partial pick of:
0.4.15-dev-3642-g 83be315abffe14f0b391ff9cd543010aa0d57b9e
and strip all EOL-whitespace in that file.
This part is done as the file is part of a backport into even older branches.
brsfolder.c will be 100% in sync between releases/0.4.7 - releases/0.4.14 afterwards.
-------------------------------------------------------------
[SHELL32] de-DE.rc Update regarding IDC_BROWSE_FOR_FOLDER_NEW_FOLDER

0.4.15-dev-2957-g c30b1fe893e8c7cfb9c8031b3c8cc83a311a294f
and pick other small tweaks in that file from master
-------------------------------------------------------------
[COMDLG32] Support shortcut keys on Open/Save Dialog (#3238)

Enable key accelerators on File Open/Save Dialog. CORE-14332
0.4.15-dev-1050-g c8e1460ac57e7862ccef0b5602764b6d72167a70
--------------------------------------------------------------
[SHELL32] Implement VK_DELETE on SHBrowseForFolder (#2661)

Enable Del/Delete key to delete the selected folder in "Browse for Folder" dialog.
You can use https://jira.reactos.org/secure/attachment/40118/getfolder.zip as a test program.
0.4.15-dev-39-g 771457f37f0673c8fa1f6e11268b7b5885780813
--------------------------------------------------------------
[SHELL32] Sync brsfolder.c with keyboard shortcut code from wine (#2290)

0.4.14-dev-1187-g fc4c4d4911bfcec348f58b9e59c4f215ed6f58c5

14 months ago[0.4.10][SYSSETUP] Fix *.rc combobox for the timezone is too small CORE-18587
Joachim Henze [Sat, 4 Feb 2023 12:58:45 +0000 (13:58 +0100)]
[0.4.10][SYSSETUP] Fix *.rc combobox for the timezone is too small CORE-18587

by porting back:
0.4.15-dev-5542-g f9dfc4537571ce33f01a685567afea7a2b57346d CORE-18587
0.4.14-dev-980-g b1c7ca3024fdded47e3708b70b5811bb1c0428ca Attention comment
0.4.14-dev-979-g ba17d87636aae02273abfd784a22b5c0aa2fecf9 Attention comment
and pick a few places where translators decided to shorten their strings

14 months ago[0.4.10][NETSHELL] Lan Status Dlg should show on LBUTTONUP in systray,
Joachim Henze [Thu, 2 Feb 2023 11:21:59 +0000 (12:21 +0100)]
[0.4.10][NETSHELL] Lan Status Dlg should show on LBUTTONUP in systray,

not on LBUTTONDOWN

by porting back:
0.4.15-dev-5613-g 7a17c7d9adadf88eafc382d792771bea5fc412f8 CORE-18813 LBUTTONUP
and strip EOL whitespace

15 months ago[0.4.10][NTGDI] Check if face->charmap is not zero before accessing its encoding...
Joachim Henze [Sat, 21 Jan 2023 10:15:19 +0000 (11:15 +0100)]
[0.4.10][NTGDI] Check if face->charmap is not zero before accessing its encoding (#4390)

Prevents a BSOD 0x1E upon dereference.
CORE-18091 CORE-18558

By partially porting back:
0.4.15-dev-5587-ga999c43 a999c4374607861cdc5576e8a9255958e15273f6
and take also some cosmetic changes from master head, like EOL-whitespace-removal.

15 months ago[0.4.10][SHELL32] Improve Right-click copy/move/link menu
Joachim Henze [Wed, 18 Jan 2023 21:56:00 +0000 (22:56 +0100)]
[0.4.10][SHELL32] Improve Right-click copy/move/link menu

by porting back:
0.4.15-dev-5588-g e89675768be212fca2e59be68a4276cd308abd82 [SHELL32] Add 3 accelerators for IDM_DRAGFILE MENU de-DE.rc (#5001)
pick a single line only of 0.4.15-dev-984-g a5a30fc249e24108317a37fd430b119e49e38f10 [SHELL32] CFSDropTarget.cpp ERR->TRACE to mute logspam
0.4.11-dev-858-g 1d55f459d6402eb5f97e5ebf65a32e24cdd7a13b [SHELL32] CFSDropTarget: Fix right click menu when shown over the tree view CORE-11240

and strip some EOL-whitespaces in the 3 related source-files.

CORE-11240

15 months ago[0.4.10][SETUP][BOOTDATA][FONTS][INF][MEDIA][ROSTESTS] Monospaced fonts and substitution
Joachim Henze [Fri, 6 Jan 2023 08:49:11 +0000 (09:49 +0100)]
[0.4.10][SETUP][BOOTDATA][FONTS][INF][MEDIA][ROSTESTS] Monospaced fonts and substitution

Switches from "FreeMono" and "Dejavu Sans Mono" to "Courier New" and "Lucida Console"
and therefore brings us closer to 2k3sp2 stock fonts. Saves some RAM also.
Deletes some "Ubuntu" font left-overs in some Themes.

CORE-18605 (and partially CORE-16264)

By porting back the following commits:
0.4.15-dev-5362-g c47758b102e1866f8d4bed71071f91c50b7e3e89 [SETUP][INF][MEDIA] Get rid of the last Ubuntu references CORE-18607 (#4871)
0.4.15-dev-5336-g bd40c13f6f33687ea56c103c95877db426f3864e [SETUP][BOOTDATA][FONTS][INF] Delete DejaVu Sans Mono and re-map (#4856)
0.4.15-dev-5335-g a4193ade03d5093a27deab99dae7c3183a538931 [SETUP][FONTS][INF][GDI32_APITEST] Delete FreeMono and re-map (#4852)
0.4.15-dev-5324-g d03545214831c3c603fa2f3056c8a26d30de3b69 [SETUP][INF][ROSTESTS] Delete font substitutions and tests for "DejaVu Sans" (#4829)
0.4.13-dev-768-g dc1aa8010a1f5c92edc5fa531611950807edf370 [FONTS][SETUP][BOOTDATA] Add Lucida Console font (#1793) CORE-16264
0.4.13-dev-750-g faa3237d469640031d3846f94a490edd33dee100 [FONTS][BOOTDATA][SETUP] Add Courier New fonts (#1783)

15 months ago[0.4.10][CMAKE] Update baseaddress_msvc.cmake
Joachim Henze [Thu, 29 Dec 2022 17:46:43 +0000 (18:46 +0100)]
[0.4.10][CMAKE] Update baseaddress_msvc.cmake

by hand!

Motivation is to fix the following warnings at configure-time for *MSVC x86 dbg* :
-- wuauserv has no base address
-- fusion has no base address
-- fusion_1_1 has no base address
-- fusion_2_0 has no base address
-- wlnotify has no base address
-- localspl_apitest.dll has no base address
-- load_notifications has no base address

This fixes CORE-14831 for *MSVC dbg* for this release-branch at least.
Meaning: It hides the effect, without fixing the cause.
It is not a sustainable solution for master yet.
We'd need to improve the python script for that.

I tested afterwards the bootup of a build configured via:
-DENABLE_ROSTESTS=1 -DENABLE_ROSAPPS=1
and built with:
MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuration
and verified that we had no relocation happening until 3rd stage desktop VBox4.3.12.

MSVC 2010SP1 (16.0.40219.1) x86 target in *rls* configuration was not
tested yet, as that up to now did never reach the desktop yet for unrelated reasons.

I will not upload an iso to sourceforge after this commit,
because it will not have any impact for our primary GCC 4.7.2 toolchain.

16 months ago[0.4.10][NTUSER] Fix zombie window created by CTRL+ALT+DEL (#4935) CORE-18258
Joachim Henze [Mon, 26 Dec 2022 00:46:11 +0000 (01:46 +0100)]
[0.4.10][NTUSER] Fix zombie window created by CTRL+ALT+DEL (#4935) CORE-18258

Based on patch by I_Kill_Bugs.
Fixes the regression which was introduced to master by 0.4.15-dev-1126-g 58b0558f94abee52ce1c261063d4f4b884e96310

fix picked from 0.4.15-dev-5532-g ca9ded7af8b21ec0b5b4e600f05a6a8b13e555a2

16 months ago[0.4.10][NTUSER][USER32][OSK] Initial support of WS_EX_NOACTIVATE flag (#4731)
Joachim Henze [Sat, 24 Dec 2022 01:44:44 +0000 (02:44 +0100)]
[0.4.10][NTUSER][USER32][OSK] Initial support of WS_EX_NOACTIVATE flag (#4731)

WS_EX_NOACTIVATE flag forbids the window to be activated
fix picked from 0.4.15-dev-5137-g 826bd41d88df8b83494ce921f7f4a3ec55544c7f (1 of 3 commits for CORE-18417)
My reason for picking it is that it fixes CORE-13257 'Reboot leaving Evernote 5.9.8 in tray causes WIN32K BSOD 0x50'
I also had to pick together with it the
0.4.15-dev-5491-g 1fbed1710e0d24ea670557636e10cd776b67f000 to prevent it from introducing (+2 failures in user32:msg_focus) unittests.
----------------
The above patch allows/encourages us to pick then also:
0.4.15-dev-5492-g 47f3a4e144b897da0e0e8cb08c2909645061dec9 [OSK] Delete WS_EX_NOACTIVATE workaround code
(which referenced CORE-18528, but is not fixing this minor GUI glitch yet, it just removes workaround-code that is no longer needed,
I just mention the ticket here, because the commit on master did that as well)
----------------
----------------
The following 2 things were small *unrelated* improvements nearby, and I decided to take them with me:
0.4.15-dev-4323-g 9f5cde9cbeab658b6530b018cd166a63ed3e321f [WIN32K:NTUSER] window.c Remove broken assert in NtUserCreateWindowEx. CORE-18123 fixed
and
a tiny part of 0.4.15-dev-3476-g 55a1c29341bf452377fa9bab9b2773d813261c39 [WIN32K] appswitch.c: delete unused func DWORD wtodw(const WCHAR *psz)
----------------
and also backport a few whitespace improvements in the files touched.

16 months ago[0.4.10][KEYBOARD] Fix problems in our keyboard layouts
Joachim Henze [Wed, 21 Dec 2022 17:25:08 +0000 (18:25 +0100)]
[0.4.10][KEYBOARD] Fix problems in our keyboard layouts

- This fixes problems with "F17" or "F14" shortcut key in the menu items of some programs,
  e.g. SQLiteAdmin 0.8.3.2

CORE-17906 CORE-3903

fix picked from 0.4.15-dev-5145-g cdf907074d607a2025d4e099bc823286e24c8be4
--------------
[KEYBOARD] Switch to QWERTZ for Croatian keyboard layout (#4177)
fix picked from 0.4.15-dev-3503-g 2196a06fc1d11243899e36124c965ee0399bd31b
--------------
[KBDHU] Fix Hungarian keyboard layout. (#1765) CORE-15567

partial pick from 0.4.13-dev-732-g 48200015f7223e8cc98c8a6e03cbce9db23aa57c
(I left out the header comment to keep the file pure ASCI)
--------------
Also ported back some EOL whitespace trimming

16 months ago[0.4.10][SYSSETUP] Trivial rc update
Joachim Henze [Wed, 21 Dec 2022 03:01:47 +0000 (04:01 +0100)]
[0.4.10][SYSSETUP] Trivial rc update

picked from 0.4.15-dev-5519-g e8e38b83012e768982f2fa370930b62a4753f49b
picked from 0.4.15-dev-2937-g 1078340f5c611f15755f80b546002a8992692fa1
and ported back some trivial changes in some other languages as well.

16 months ago[0.4.10][INF] Backport String deduplication and minor fixes
Joachim Henze [Sun, 18 Dec 2022 14:26:19 +0000 (15:26 +0100)]
[0.4.10][INF] Backport String deduplication and minor fixes

Fixes a bug in msmouse.inf missing " at begin of last string in section [Strings.0416].
partial pick of 0.4.15-dev-1105-g d50cbb6eed9afaeb11db590d4aabe05122a772a4 (only the changes in msmouse.inf were picked) (#3173)
------------
in releases/0.4.7 to releases/0.4.11 the whole section [Strings.0415] existed twice in NET_NIC.inf,
deleting one of them entirely is not an oopsie, and does just reflect the state of the newer branches!
------------
partial pick of 0.4.15-dev-4867-g bbccad0ed60d619e49eb6e5d4e85cf9e0d932e2e Deduplicate the "ReactOS =" strings,
they are non-localizable now CORE-18191. I intentionally left out the "ReactOS team"->"ReactOS project" part here.
------------
full pick of 0.4.15-dev-309-g 7c8d5635aa3075774318adc93873d2d7645bfb2e [PROCESSR][INF] Fix "Intel Intel" duplication typos
which was an addendum to 35093185219d8e5bd1f19105e97445918b6143c7 == SVN r46465
------------
also ported back deduplication of some manufacturer strings.
------------
all changes to shortcuts.inf and syssetup.inf.tpl are exclusively stripping EOL-whitespace, no other changes in those files.
------------
the only part that was not picked from master is the stripped 's' in disk.inf, which is possible,
because the file has been replaced altogether in master by an MS file.

16 months ago[0.4.10][DESK] Fix handle leaks in screensaver preview and other small fixes
Joachim Henze [Sun, 11 Dec 2022 03:30:01 +0000 (04:30 +0100)]
[0.4.10][DESK] Fix handle leaks in screensaver preview and other small fixes

- Fix leaking two handles to avoid creating zombie processes when closing the
screensaver preview

fix is *partial* pick of 0.4.15-dev-5495-g 71123b72fa6ecb06ce22520747e562369e96dfec which *partially* fixes CORE-18680 (#4921)
The other part of that fix is not safe, neither on master, nor when porting back due to yet unfixed Win32K bugs.
-------------------------
*partial* pick of 0.4.15-dev-3154-g 158a479a4e996b84615f70602e88862f6656a98f "\r\n"->"\n" in desk.cpl
-------------------------
furthermore port back some EOL-whitespace cleanup and some minor translation improvements
-------------------------
also *partial* pick of 0.4.15-dev-5521-g 5ecb9e8cb5ae16a47446c0abdb2ed2b0666628f3 (#4943)
only the parts that are relevant for older releases
-------------------------
0.4.14-dev-111-g 72e5c2e77c447825b82d19977ea3180067483908 hShell32 is always non-NULL in epilogue

16 months ago[0.4.10][DXDIAG] A bit closer to current releases/0.4.14 for translations and dlg...
Joachim Henze [Sat, 10 Dec 2022 17:43:50 +0000 (18:43 +0100)]
[0.4.10][DXDIAG] A bit closer to current releases/0.4.14 for translations and dlg layout

ports back some minor tweaks, mostly in translations:
0.4.14-dev-1409-g 951e5f285167be60e87cbdebdf676127310554cb CORE-12477 part3
0.4.14-dev-1408-g f03c4c4378de97fa93f41228fe63594f635917c9 CORE-12477 part2
0.4.14-dev-1405-g dcc6dcb0bd6cb076fabe6fe0a45f9556eed1b379 CORE-12477 part1
- Less text truncation for all languages
- Manually Tweaked fr-FR the most, see CORE-12477
- Manually Tweaked de-DE enlarged a buttons size
- For All Languages harmonized the maximized available space for dynamic strings in 2nd tab right side from 100->120

16 months ago[0.4.10][NTUSER] Fix Toolbar re-painting when Rebar parent has been altered (#4380...
Joachim Henze [Wed, 7 Dec 2022 01:13:23 +0000 (02:13 +0100)]
[0.4.10][NTUSER] Fix Toolbar re-painting when Rebar parent has been altered (#4380) CORE-12342

Fixes:
CORE-12342 '7zip is not drawing some of its controls until forcing a repaint'
CORE-8734 '[WIN32SS] dxdiag exposes a tabsheet redraw problem'
...
and many other duplicates in JIRA.

If a Rebar window is altered by calling either ShowWindow() or UpdateWindow()
then we were only updating the parent and omitting their children windows.

This is closer to the behavior of the very very old SVN r27394 == git f1a7f30887ac02e9a6c6605d0409af2618771154 .
And therefore intentionally excludes parts of SVN r27403 == git e2651a0dbc4d98dbda0d87746b87bd9575fb4f4c .

The bots have no complaints about it:
KVM https://reactos.org/testman/compare.php?ids=81266,81290
VBox https://reactos.org/testman/compare.php?ids=81267,81291

fix picked from 0.4.15-dev-3885-g ea7c14919ea83e2c4b44472b729c42a7bdee6d67

16 months ago[0.4.10][SHELL32][USER32] Improve MSVC2010SP1 bootcd icon rendering CORE-14982 and...
Joachim Henze [Sun, 4 Dec 2022 20:27:39 +0000 (21:27 +0100)]
[0.4.10][SHELL32][USER32] Improve MSVC2010SP1 bootcd icon rendering CORE-14982 and CORE-15150

During 0.4.10-dev' Katayama worked heavily on the icon selection dialog and when
we branched the releases/0.4.10 that was WIP, so we had a pretty inconsistent state unfortunately
regarding extracting the desktop icons. The required fixing took a very long time
into 0.4.11-dev'ing back then.
So my only option for releases/0.4.10 back then was to appease the problems via
0.4.10-RC-13-g 3a1fe9d04434953dc8d44c406e9253a2e9e32247
which worked very well and gave a consistent state for the GCC build.

Unfortunately it lead to the MSVC2010SP1 dbg build bootcd to not show some icons on the desktop
and within the whole startmenu. As said: GCC was not affected by that. And LiveCD was neither.

So today I do port back the remaining improvements in this context from 0.4.11-dev'ing to
improve again also for the MSVC2010SP1 builds bootcd of releases/0.4.10:
0.4.11-dev-361-g 2cbac3d049c7b3b1e6833ab092a8a2cff9a7771f [SHELL32] CORE-14982
0.4.11-dev-357-g 40737bdddb384bd2842c0cc3f6cacf3f4295731f [USER32] Fix the ICO_ExtractIconExW() helper CORE-15150 <- this part is essential
0.4.11-dev-29-g 19e9a441183d907540bb22f8711db281334467b9 [SHELL32] Check the result from PathUnExpandEnvStringsW

And while touching the file CShellLink.cpp anyway, I took also some other small unrelated improvements with me:
0.4.11-dev-360-g 1e20a3e5b58f3f30e31b0360ce318c7b5196e770 [SHELL32] 'Fix a potential buffer overrun' (which is a tiny part of CORE-8545)
0.4.11-dev-256-g ae9ec94240962893ce88e9e85d29e141afa1f4ae [SHELL32] Code formatting only (I picked only the CShellLink.cpp changes here)
0.4.11-dev-28-g 608cb60e446e3cfe4d8c1989468cfb2fe02d7d0a [SHELL32] Remove some mostly dead code from CShellLink

17 months ago[0.4.10][NTGDI] Commit minimized Flip_Fix_9.patch
Joachim Henze [Tue, 22 Nov 2022 15:19:36 +0000 (16:19 +0100)]
[0.4.10][NTGDI] Commit minimized Flip_Fix_9.patch

flip_fix_9.patch fixes:
CORE-16984 " 'SPINA Thrulg' / 'SPINA Thyr' / 'Sim Thyr' have images flipped"
CORE-17194 "StretchDIBits test" isn't work correctly"
           "Output of 'Project 3 Test'"
CORE-14701 "DVDStyler 3.0.4 transparent toolbars"
CORE-14701 "DVDStyler 3.0.4 erroneously black around icons of welcome-dlg"
CORE-14671 "'Peazip' shows icons in buttons and menubar vertically flipped"
           "Double Commander shoes icons flipped in buttons, menubar, listview and the treeview"
CORE-13273 "Welcome to Lazarus" icon shows flipped
CORE-13026 "'CudaText app' icon shows flipped"
CORE-18653 The dibobj.c part also fixes a BSOD 0x1E nullptr deref in Win32K when running the setup of 'Auslogics Disk Defrag Pro 4.9.4.0' in releases/0.4.7 to releases/0.4.13

Not all of those are duplicates, although they appear to be at first glance.
It affects different controls and some of those tickets do have different 'guilty revs' than others.

The patch does consist of 2 parts:

1.) win32ss/gdi/ntgdi/dibobj.c
This one is the most clean part of it, that addresses most of the flipping issues now, and the BSOD 0x1E.

2.) The hack in comctl32.h redefining the version:
We used that in the past to appease some, but not all of the issues listed above.
But it does hide additional issues, e.g. in DvDStyler, therefore we seem to still need that appeasement even today.
Most likely it would make sense to aim to avoid this part in the future.
part 2.) was committed as first appeasement on its own already into
0.4.14-RC-24-g 198b61e
0.4.13-RC-7-g 67211fa
0.4.12-RC-5-g 8449527
0.4.11-RC-16-g b906163
0.4.10-RC-7-g f1e80fe
0.4.9-RC-34-g 9d758ae
Interestingly this part is not needed for releases/0.4.7.
It became necessary for all newer releases with the COMCTL32 Winesyncs during 0.4.8-dev' and 0.4.9-dev'ing and their massive button.c-changes.

We also created a summary of manual test-results:
https://jira.reactos.org/browse/CORE-17415?focusedCommentId=126668&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-126668

If you read this comment via git blame and your goal is to get rid of the hack in comctl32.h, I would highly recommend
to redo the tests of that testing table, and if that is asked too much, then please test at least the following cases:
CORE-14701 "DVDStyler 3.0.4 transparent toolbars"
CORE-14701 "DVDStyler 3.0.4 erroneously black around icons of welcome-dlg"
           "DVDStyler 3.0.4 erroneously black/transparent within comboboxes of properties of VMGM menu"
and try to add what might be needed to fix them, and double-check again:
           "DoubleCommander optionsDlg the most complex testcase, contains flip-prone icons in treeview, listview, menubar, buttons, statics". Only some of its flipped icons were impacted by the comctl32.h change before.

fix is a squashed pick from
0.4.15-dev-1603-g 232c45fcd71b12a0d726b47f4a72e9a16f87432a
0.4.15-dev-5396-g 9c5bb096bc21cee00a8b8968214d24d5c5500d88 [COMCTL32] partial revert of the upper commit

17 months ago[0.4.10][DEVMGR] Fix wrong menu item hint (#4854) CORE-18608
Joachim Henze [Wed, 16 Nov 2022 12:31:35 +0000 (13:31 +0100)]
[0.4.10][DEVMGR] Fix wrong menu item hint (#4854) CORE-18608

Item hint for "Resources by connection" was incorrectly set to the hint for "Resources by type".

fix picked from 0.4.15-dev-5334-g 075ec50298e80df79ff654a08ee7ae0f4e055084
and also port back stripping of some EOL-whitespace

17 months ago[0.4.10][WDMAUD.DRV][MMIXER] Improve AC97 driver from rapps by defining USE_MMIXER_LIB
Joachim Henze [Mon, 14 Nov 2022 17:20:43 +0000 (18:20 +0100)]
[0.4.10][WDMAUD.DRV][MMIXER] Improve AC97 driver from rapps by defining USE_MMIXER_LIB

By taking alternative code-paths in WdmAud and bypassing Sysaudio.

This is a squashed backport of:
0.4.15-dev-791-g 6d7ebc20481a587bd19e8a8049317113071b0817 the USE_MMIXER_LIB which gives the best results when paired with those 2 previous patches:
0.4.15-dev-765-g b8e936a57b4770e133772cf2dd66f30671a1524b CORE-17214 (#3148) wdmaud-racecondition-fix and
0.4.15-dev-796-g a27f0debca4c0fe8cd916d2406124f9d30ae5dbe CORE-17276 winmm:mixer-testbot-crash-fix

Defining USE_MMIXER_LIB will fix/improve:
- the test execution times of "GCCLin_x86 on Test VBox" will be dramatically improve (iirc by ~10-15min)
- CORE-8726/CORE-9986/CORE-16564 AC97 driver from rapps will work in the same session that the driver is installed, not a single reboot is needed anymore
- CORE-13202 Unhandled exception from wdmaud.drv when recording sound in Scratch 1.4 leads to app-crash (gets fixed even for older builds that did not receive 0.4.15-dev-2794-g 81f8bce yet)
- CORE-13488 A deadlock in "DiabloII" character selection screen and "The Lion King II"
- CORE-9981 "DosBox + Commander Keen6" almost 100% fixed, DosBox + Commander Keen6 properly plays music instead of garbled output,
            same improvement for "ScummVM 2.0 with Monkey Island 2"

The playback is not yet *entirely* perfect, still a few hiccups now and then, but by orders of magnitude better than before.

Most of the above mentioned issues were a regression of SVN r44721 == 4b9731d0db1a4868c0759c8f43cde24f4be7ae8a

Defining USE_MMIXER_LIB will also have some negative aspects:
- CORE-17277 crash of dsound:duplex on "GCCLin_x86 on Test VBox" gets unhidden on the bot, but was proven to be broken even beforehand already. The driver beforehand was just not found and the tests were skipped therefore.
- CORE-17278 crash of dsound:capture on "GCCLin_x86 on Test VBox" gets unhidden on the bot, but was proven to be broken even beforehand already. The driver beforehand was just not found and the tests were skipped therefore.
- CORE-17744 Fox Audio Player 0.10.2 can not longer play an mp3 file with 22kHz, WMM audio output
- It may also have a negative impact for CORE-17285 "Realtek HD Audio" but Oleg Dubinsky accepted to tolerate that and aims to approach it differently. We resolved that as WontFix therefore.

17 months ago[0.4.10][SETUP][INF][MEDIA][FONTS] Massive font and font substitutions upgrade
Joachim Henze [Sun, 6 Nov 2022 02:15:43 +0000 (03:15 +0100)]
[0.4.10][SETUP][INF][MEDIA][FONTS] Massive font and font substitutions upgrade

This fixes tons of font rendering issues for many localizations, brings the set
of fonts closer to the 2k3sp2 stock fonts, and adapts some font substitutions.
Best of all is that it shrinks the iso size by ~13MB on average and stabilizes ros by
consuming significantly less memory.

It is a squashed backport of the following commits:

0.4.15-dev-5287-g 4615c824db153724da055ec3479c5c0452ac1460 [SETUP][INF] Delete obsolete font substitutions "DejaVu Serif" (#4827)
0.4.14-dev-1515-g 30fae1a2df24939da1073b3af411cab00fe0653e [MEDIA][FONTS] Fix French glyphs of Tahoma font CORE-16899 (#2607)
0.4.14-dev-586-g 9525593660b0cab5e88a3ea709e9caf05868cbe9 [MEDIA][FONTS] Improve tahomabd.ttf (Latin) No.35 CORE-8408
0.4.14-dev-584-g 94f92b0b2e8415c4f203a7c82d3f885ce57cd33f [MEDIA][FONTS] Improve tahomabd.ttf (Latin) No.34 CORE-8408
0.4.14-dev-469-g 0b938a35bdf5a68773cf425ba1f002426358667d [MEDIA][FONTS] Improve tahomabd.ttf (Latin and Punctuation) No.33 CORE-8408
0.4.14-dev-467-g c3da02701239ecc5e0c829c151e4034e6e7f533c [MEDIA][FONTS] Improve tahomabd.ttf (Punctuations) No.32 CORE-8408
0.4.14-dev-462-g 591b2b807ccf0243313b1787675a656ea6df8568 [MEDIA][FONTS] Improve tahomabd.ttf (Punctuations) No.31 CORE-8408
0.4.14-dev-456-g d8187d33be9b47c10edc3fb3178b61a461909284 [MEDIA][FONTS] Improve tahomabd.ttf (Punctuations) No.30 CORE-8408
0.4.14-dev-430-g f41366daade22615419c250bb7e3f8609bb1d16e [MEDIA][FONTS] Improve tahoma.ttf (Punctuations) No.29 CORE-8408
0.4.14-dev-426-g 55251a0a8e24bad72def1389351b00e349a917a5 [MEDIA][FONTS] Improve tahomabd.ttf (Latin, Greek and Cyrillic) No.28 CORE-8408
0.4.14-dev-400-g b7076dc2bce3e87f20feb11e8d8da95d0d7c3378 [MEDIA][FONTS] Improve tahoma.ttf (Latin) No.27 CORE-8408
0.4.14-dev-397-g 551b101720033b1b66193d161e88fe83668bc298 [MEDIA][FONTS] Improve tahomabd.ttf (Latin) No.26 CORE-8408
0.4.14-dev-396-g 8cbcb172f602e9b3816297197b5c29885b696a85 [MEDIA][FONTS] Improve tahoma.ttf (Latin and Cyrillic) No.25 CORE-8408
0.4.14-dev-395-g 65cbadf2d608f2aeeca6efc372482ab6dceae346 [MEDIA][FONTS] Improve tahoma.ttf (Cyrillic, Greek and Latin) No.24 CORE-8408
0.4.14-dev-391-g 454ea3d85b612f760a657d660c9bccc56b6e92fb [MEDIA][FONTS] Improve tahomabd.ttf (Cyrillic and Latin and Greek) No.23 CORE-8408
0.4.14-dev-389-g 9ef61bb887e107ec7161fb4d6e8945023350c310 [MEDIA][FONTS] Improve tahomabd.ttf (Cyrillic and Latin and Greek) No.22 CORE-8408
0.4.14-dev-385-g 6f2662688b4dcb44d8da7fb5f99edf4817070b0e [MEDIA][FONTS] Improve UniVGA16.ttf (VGA) font CORE-8408
0.4.14-dev-378-g 871d322f922c6ecc2cdd11f68ddb85b923a7fe6c [MEDIA][FONTS] Improve tahoma.ttf (Cyrillic and Latin) No.21 CORE-8408
0.4.14-dev-377-g 21f4e69ee8e3809000c0a3d36acccbc568177356 [MEDIA][FONTS] Improve tahomabd.ttf (Cyrillic and Latin) No.20 CORE-8408
0.4.14-dev-374-g e4de09812736b9ff6ba36cad36df0365f4e49c2f [MEDIA][FONTS] Improve tahomabd.ttf (Cyrillic and Latin) No.19 CORE-8408
0.4.14-dev-370-g dda81852976fb529ea9a10b81557ae1419e66920 [MEDIA][FONTS] Improve tahoma.ttf (Cyrillic and Latin) No.18 CORE-8408
0.4.14-dev-367-g 36163a235b32433e31f2e4ed7530ef19452ad386 [MEDIA][FONTS] Improve tahomabd.ttf (Cyrillic and Latin) No.17 CORE-8408
0.4.14-dev-357-g 203a6babdffffe21c0ae6081e460fff5798bb436 [MEDIA][FONTS] Improve tahomabd.ttf (Cyrillic and Latin) No.16 CORE-8408
0.4.14-dev-356-g 7d7f134f9430ea5ec9e59ff012eb996ae08b1ef9 [MEDIA][FONTS] Improve tahoma.ttf (Cyrillic and Latin) No.15 CORE-8408
0.4.14-dev-339-g 54c8cc66aba5cd32966e9d2721fecfbe99c9b32d [MEDIA][FONTS] Improve tahoma.ttf (Cyrillic) No.14 CORE-8408
0.4.14-dev-338-g 9f803ab091c707f5958402958250f5afba8b56b3 [MEDIA][FONTS] Improve tahomabd.ttf (Cyrillic and Latin) No.13 CORE-8408
0.4.14-dev-333-g 67c78d88c885ca92c72e2bcf59ddcf1d429096b1 [MEDIA][FONTS] Improve tahomabd.ttf (Cyrillic and Latin) No.12 CORE-8408
0.4.14-dev-330-g 15748cf03ad02dbf75a26b4c406386815c32223f [MEDIA][FONTS] Improve tahomabd.ttf (Cyrillic) No.11 CORE-8408
0.4.14-dev-329-g 463a3f8c06725742121bedff900cf1adabd67d9f [MEDIA][FONTS] Improve tahoma.ttf (Cyrillic) No.10
0.4.14-dev-328-g c58eced19bbca429ff8eb8e6f8a060d609fe5d61 [MEDIA][FONTS] Improve Cyrillic glyphs of tahoma.ttf a bit
0.4.14-dev-327-g ff715073cac5767f618c4679480a3cbac1b6beab [MEDIA][FONTS] More improve Cyrillic glyphs of tahomabd.ttf
0.4.14-dev-325-g 6b1ca2895b148958841d4780e466fda0d2da5e88 [MEDIA][FONTS] Improve Cyrillic glyphs of tahomabd.ttf
0.4.14-dev-324-g ff77ddff5e3389e8bd446862c33f41aed0b1cdb2 [MEDIA][FONTS] Improve Cylliric glyphs of tahoma.ttf more
0.4.14-dev-323-g 6fe6a88b04e30fb3cb21bc4c78b1e5715f19da1a [MEDIA][FONTS] Improve Cyrillic bitmap glyphs of tahoma.ttf
0.4.14-dev-231-g 2627b52d9588963e1d52244e89ead85d93f9e191 [MEDIA][FONTS] Improve Tahoma font (tahoma.ttf) for Hungarian (#2018) CORE-16480
0.4.14-dev-20-g 2f4fb903b46207ab380365908401ded63752a108 (partially) [MEDIA][FONTS] Delete many fonts CORE-16407 (#1955)
0.4.14-dev-6-g f45dd6577321355752e29ccd5f49039cfdaa761e [MEDIA][FONTS] Replace 'Verdana' fonts CORE-16400 (#1948)

0.4.13-dev-799-g 10c08e1d62705a3ac59d6c8050280bfbd96540ed [SETUP][BOOTDATA][INF] Helvetica-to-Arial substitutes (#1809) CORE-16264
0.4.13-dev-798-g 257aed0e690b4cbf443c3c26a458079f9bbcfd36 [SETUP][BOOTDATA][INF] Delete Tahoma-to-Tahoma substitutes (#1808) CORE-16264
0.4.13-dev-785-g 3f1ab92d3aca8b7b0965a1004e4a5f25b4d64025 [SETUP][INF] Helvetica must be substitute to Arial font (#1803)
0.4.13-dev-784-g 01a134832588e222d987347268ecbedd7fd9482c [MEDIA][FONTS] Replace Georgia fonts (#1802) CORE-16293
0.4.13-dev-783-g 6f796ab04360e7d462fb1778bf1c3e9a6a04c9f2 [MEDIA][FONTS] Replace 'Palatino Linotype' fonts (#1800) CORE-16292
0.4.13-dev-769-g 5bbc2c6308a2802ed84c281e1dde5e2384ddaee0 (partially) [SETUP][INF] Fix font substitutes (#1794) CORE-16264
0.4.13-dev-765-g 04a361d091095a96bb3af376e46daf4c5fe81c63 [FONTS][SETUP] Add Trebuchet MS fonts (#1791) CORE-16272
0.4.13-dev-764-g dbb4432b25dc65b9d65f02bfc09d47d63b8cd5e5 [FONTS][SETUP][BOOTDATA] Add Times New Roman fonts (#1790) CORE-16272
0.4.13-dev-753-g bb1878157bbe4ca825764a4ef933f3f086b369dd [FONTS][SETUP] Add Palatino Linotype fonts (#1785) CORE-16272
0.4.13-dev-752-g 5ce98c85194214111f9ecff6f81fbaa015735fd6 [FONTS] s/Geogia/Georgia/
0.4.13-dev-751-g e282f99ce5dbc7ff4cdfdf9226a618da1fb42c1a [FONTS][SETUP] Add Geogia fonts (#1784) CORE-16272
0.4.13-dev-749-g 1463f4b02ac7d8514baa808614a18b280cae1972 [FONTS][SETUP][BOOTDATA] Add Arial fonts (#1781) CORE-16272
0.4.13-dev-742-g 4cfe7f84bbc85cd303a72d7aecd302d46014128f (partially) [SETUP] Add codepage-specific font substitutes (#1772) CORE-16266
0.4.13-dev-737-g f054844f33a9d8927ea716a2a17d90c2c7e7c521 [SETUP] Sort font substitutes (#1770)

18 months ago[0.4.10][DDRAW] Handle DDLOCK_WRITEONLY in wined3dmapflags_from_ddrawmapflags() CORE...
Joachim Henze [Wed, 26 Oct 2022 22:05:21 +0000 (00:05 +0200)]
[0.4.10][DDRAW] Handle DDLOCK_WRITEONLY in wined3dmapflags_from_ddrawmapflags() CORE-18378 (#4713)

Mutes the logging:
fixme:(dll/directx/wine/ddraw/utils.c:584) Unhandled flags 0x20.
gets logged many times per second.

It does affect several applications, e.g. the game 'Anno 1602' from 1998, and the 'Diablo 2 demo' from rapps,
For both games it can be observed with both: our VBEMP driver and the VBox4.3.12 3D-accelerated-driver.

Muting may improve performance a bit in such apps.
It gets logged although no missing features can be perceived visually in the rendering.

The regression was introduced to master by 0.4.9-dev-155-g 9987f02917539e05268ba2ad11e62d24a3636972

Fix it by importing Wine-commit
https://source.winehq.org/git/wine.git/commit/b943c7910b3261c9603343369cd632f7a3b56bba
 ddraw: Handle DDLOCK_WRITEONLY in wined3dmapflags_from_ddrawmapflags().
Signed-off-by: Henri Verbeet <hverbeet@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
fix picked from 0.4.15-dev-5070-g 305aae75f59a9e0e43b9213d86dbafe9e750c525

18 months ago[0.4.10][WIN32SS] Fix RLE4 bitmap decoding (#1188) CORE-10553
Joachim Henze [Wed, 26 Oct 2022 09:43:20 +0000 (11:43 +0200)]
[0.4.10][WIN32SS] Fix RLE4 bitmap decoding (#1188) CORE-10553

fix picked from 0.4.12-dev-189-g 968c8f37f14cf90fba994082274671b85f14764e

It fixes:
CORE-10553 'WinRar SFX have corrupted image'
CORE-11399 'Microsoft paint (from XPSP3) does not show bitmaps on the tools'

Please note, that MS mspaint.exe does not start in all older rls-branches yet for unrelated reasons,
it did start and render properly in releases/0.4.11, but does not start in releases/0.4.7 yet.
Still I marked the ticket CORE-11399 as resolved in all of them as that has nothing to do with the RLE stuff.
The WinRAR SFX worked properly in all of them.

18 months ago[0.4.10][BCRYPT] Add ECDSA P384 to known algorithms (#4236)
Joachim Henze [Tue, 25 Oct 2022 00:19:43 +0000 (02:19 +0200)]
[0.4.10][BCRYPT] Add ECDSA P384 to known algorithms (#4236)

Today 2022-10-25 was the day, when I spotted the first URL in rapps that is not longer happy with just having
ECDSA_P256 but now needs even ECDSA_P384 to make the rapps download succeed. *eye-rolling*
It was https://bitbucket.org/Swyter/tld-downloads/downloads/diabloiidemo.exe
that failed the rapps download with

fixme:(dll/win32/bcrypt/bcrypt_main.c:359) algorithm L"ECDSA_P384" not supported
err:(win32ss/user/user32/windows/messagebox.c:1048) MessageBox: L"Paket konnte nicht geladen werden! Pr\00fcfen Sie Ihre Internetverbindung!"

So I have no choice but to follow the totally stupid crypto-rat-race and
add the missing structs and handlers in BCryptOpenAlgorithmProvider().

When will the world ever learn that the new cryptos do outdate faster than the new shit even can be implemented in time everywhere?
Each additional crypto algorithm effectively enlarges the attack surface of the end-points instead of making stuff more secure.
The world is rushing towards totally breaking backward compatibility to older systems. This is built-in obsolescence. For which gain?
This race can never be won. Each crypto-iteration just intensifies the existing chaos. Enough ranting...

fix picked from 0.4.15-dev-3662-g c3e0df06113b67a4c3967cd018220c4f2bd7fa2b

It is a cherry-pick a part of Michael Müller's commit from Wine
https://source.winehq.org/git/wine.git/commit/76b6c360fa7f3d6a0a14ed935075f5eb10f2f719

The commit doesn't fix CORE-14198 at all, same like on master, but since the original commit was linked
to that JIRA ticket as well, I decided to do the same with the backport.

18 months ago[0.4.10][REACTOS] Remove references to ReactOS Foundation (#4557)
Joachim Henze [Sun, 23 Oct 2022 22:26:08 +0000 (00:26 +0200)]
[0.4.10][REACTOS] Remove references to ReactOS Foundation (#4557)

See https://reactos.org/wiki/ReactOS_Foundation for more information.

the first part of CORE-18191

picked from 0.4.15-dev-4866-g 6881d8c6247c8d27e57c3a6112bf386204f089be

Respect:
-Colins wish to leave out for now all the matches in kbdtool for legal uncertainty
-boot_images.cmake on master vs. boot/CMakeLists.txt in older branches
-treating cpu.inf special in this first commit for the sake of keeping sync to all other infs
-unlike master I decided to groom also cdmake, because I don't see any reason yet to dump that
 in the older branches
-all releases down to releases/0.4.7 were released for the first time after the "ReactOS foundation"
 dissolved already in 2015. So all are definitely fine to receive this commit.
-port back some EOL whitespace fixes as well in common.ver to sync the file to 100% to master head

18 months ago[0.4.10][NTUSER] Fix taskbar panes activation (#2046) (#3294) (#3697) (#3700), [SHELL...
Joachim Henze [Sat, 22 Oct 2022 20:31:23 +0000 (22:31 +0200)]
[0.4.10][NTUSER] Fix taskbar panes activation (#2046) (#3294) (#3697) (#3700), [SHELL32] (#4800)

This is a squashed backport of the following commits:
0.4.13-RC-53-g 4f628f6b167c0e1936eaa501efac2eeef69533b8 (partial pick of shell32-changes for Miranda CORE-14439 & Winamp CORE-13584, explorer left out)
0.4.14-dev-369-g 141cf04239f3049c1c180fb2c6c3aa84b13c8590 (#2046) NTUSER
0.4.15-dev-1126-g 58b0558f94abee52ce1c261063d4f4b884e96310 (#3294) NTUSER
0.4.15-dev-2618-g 32b0cf6fc6b8f5fab33c10c782f40d36feebbedb (#3697) NTUSER
0.4.15-dev-2621-g 59d4c1120310b7c08641cc3aab82072251900d98 (#3700) NTUSER
0.4.15-dev-5242-g e6fb0e0c25d4509bf30a6ad602c9ba7d143cd77a (#4800) Cleanup

brings us closer to releases/0.4.14

In the older branches it will fix the following bugs (or prevents introducing them):
CORE-13584 'Winamp 5.666 erroneously has a secret tab shown in taskbar'
CORE-14380 'Effective File Search 6.8.1 when you open up the app via desktop link there is no pane in taskbar'
CORE-14439 'Miranda IM 0.10.21 snap-to-monitor-border-feature brings ros to halt'
CORE-15655 'Click-N-Type Virtual Keyboard 3.03.0412 has no taskbar pane'
CORE-15669 'DVD Write Now 1.5.12 SP2 setup has no taskbar pane'
CORE-15716 'Photofiltr 7.2.1 has no taskbar pane'
CORE-15731 'Java JRE 6u45 setup, last page of setup has no taskbar pane'
CORE-16030 'Moo0 Audio Recorder - the taskbar icon does not appear'
(CORE-17330 Regression, many installers that should have only 1 taskbar pane, now have 2 of them erroneously)

18 months ago[0.4.10][NTFS] PrintAllVCNs(): Fix a typo in an ASSERT() (#4682), DPRINT1->DPRINT...
Joachim Henze [Sun, 16 Oct 2022 21:32:17 +0000 (23:32 +0200)]
[0.4.10][NTFS] PrintAllVCNs(): Fix a typo in an ASSERT() (#4682), DPRINT1->DPRINT, EOL-WS

First part with the typo in the assert was introduced by 5579428b4f1361481dae3ea0b901c0a915401148
which was merged into ros by a merge-task during 0.4.8-dev'ing.
releases/0.4.7 didn't have that code yet.
fix picked from 0.4.15-dev-5035-g f308c6a2bc9cc8678385a55fab6956867595d71d
----------
[NTFS] Do not print debug message for every function call by default
That 2nd part was picked from 0.4.13-dev-450-g 155243494059037a5cccd3b1c57f0b77ca897c95
----------
And for all the files that I had to touch for the 2nd part in the older branches,
I ported back also some EOL-whitespace-fixes within like they were done during 0.4.15-dev'ing.

18 months ago[0.4.10][RPCRT4] Add I_RpcSNCHOption() as 'stub' CORE-12534, and fix 'Routeines'...
Joachim Henze [Sat, 15 Oct 2022 20:23:05 +0000 (22:23 +0200)]
[0.4.10][RPCRT4] Add I_RpcSNCHOption() as 'stub' CORE-12534, and fix 'Routeines' typos (#2481)

fixes BSOD 0xc0000139 STATUS_ENTRYPOINT_NOT_FOUND upon boot when
replacing just the file advapi32.dll with 2k3SP2 version 5.2.3790.3959

We can not reach the desktop yet, but at least the GUI,
and we do no longer BSOD and can instead properly ACPI-shutdown at this point.

picked from 0.4.15-dev-71-g 0b530fe4964b2cf29a75ba540bd9e956dbfc7e42
----------
also pick unrelated 0.4.14-dev-1260-g 6e79e60883aed64ab18201ac24e945d2933faa2b [RPCRT4] Fix 'Routeines' typos (#2481)
which contrary to its commits comment in master is very little only 'related' to CORE15682
but it imports https://source.winehq.org/git/wine.git/commit/c73352047d20a503c67bdd583f6ef8d98711a678
and is a very safe fix, so I picked that as well.

18 months ago[0.4.10][NTGDI] 'otm' is always non-NULL in epilogue CORE-16993
Joachim Henze [Mon, 10 Oct 2022 00:33:14 +0000 (02:33 +0200)]
[0.4.10][NTGDI] 'otm' is always non-NULL in epilogue CORE-16993

is a small optimization picked from 0.4.14-dev-108-g 1bf982ff88b1c0400c1a66ed9aaa8faeba74a75c
otm is checked for NULL already a few lines before in that function, and is not touched
anymore in between the two locations. Tiny bit shorter+smaller now,
and might satisfy also some static code-analysis.
The origin of checking it twice for NULL dates back into the pre-0.4.7-era of ros.

18 months ago[0.4.10][CMD] Quick fix for the REM command parser. CORE-17030
Joachim Henze [Sun, 9 Oct 2022 17:53:35 +0000 (19:53 +0200)]
[0.4.10][CMD] Quick fix for the REM command parser. CORE-17030

This avoids spamming "Syntax Errors" to the cmd-prompt when starting the VS2010 command prompt.
Or when typing something like "@Rem Hello" at the prompt.

Fix picked from 0.4.15-dev-202-g 9c11be5a3a95a35604f73062719431cf6df1ffb6

18 months ago[0.4.10][DBGHELP] Accept NULL symbol path as reset-to-default CORE-17073 (#3257)
Joachim Henze [Sat, 8 Oct 2022 21:27:38 +0000 (23:27 +0200)]
[0.4.10][DBGHELP] Accept NULL symbol path as reset-to-default CORE-17073 (#3257)

It fixes Heap Corruption when starting KernRate, which I could confirm also in
all the older ros branches by using:

gflags /p /enable Kernrate_i386_XP.exe /full
Kernrate_i386_XP.exe

The older builds did not expose the user-visible symptom of "the app crashing" yet
(due to different memory layout),
but the heap corruption was already happening in all of them.

This is a squashed backport of:
0.4.14-RC-92-g d3e301e28b16a2a2d9f7617a6e431d57bcc1879d (the actual fix)
0.4.14-RC-93-g 651a0115480848b51ad0c20c1e0226c6bd8925f0 (fix for compiling on GCC RosBE *Lin* 2.1.2 carrier-releaser)
0.4.14-release-24-g 57efac32b102eace2f0ec71f7fc7f396baf40e7a (just a whitespace addendum)

18 months ago[0.4.10][OPENGL32] Remove Err Log Spam from wglMakeCurrent in nominal cases (#3096...
Joachim Henze [Fri, 7 Oct 2022 16:18:17 +0000 (18:18 +0200)]
[0.4.10][OPENGL32] Remove Err Log Spam from wglMakeCurrent in nominal cases (#3096) CORE-17224

The log-spam was triggered at end of 2nd stage and within some games.

fix picked from commit 0.4.15-dev-655-g ba006b3d497cade418ceae0d3fb4686d9eab3102

18 months ago[0.4.10][SHELL32] Bypass OpenWith for Delete and F2 (Rename) Keys (#4032) CORE-17810
Joachim Henze [Wed, 5 Oct 2022 09:01:04 +0000 (11:01 +0200)]
[0.4.10][SHELL32] Bypass OpenWith for Delete and F2 (Rename) Keys (#4032) CORE-17810

Logspam started during 0.4.9-dev'ing when zipfldr got committed,
logspam became much worse by 0.4.10-dev-599-g 932df378bf35146a56c796bec36c5c689109fbd6
But the actual bug of 'calling those functions when not needed' is much older than that.

Fix picked from 0.4.15-dev-3275-g cee171f5d20c0d3c268372085ca4f32bc44d6740

18 months ago[0.4.10][CMAKE] Update GCC dbg baseaddress.cmake
Joachim Henze [Mon, 3 Oct 2022 03:40:27 +0000 (05:40 +0200)]
[0.4.10][CMAKE] Update GCC dbg baseaddress.cmake

by hand!

Motivation is to fix the following warnings at configure-time for *GCC dbg* :
-- wuauserv has no base address
-- fusion has no base address
-- fusion_1_1 has no base address
-- fusion_2_0 has no base address
-- wlnotify has no base address
-- localspl_apitest.dll has no base address
-- load_notifications has no base address

This fixes CORE-14831 for *GCC dbg* at least for this branch.
Meaning: It hides the effect, without fixing the cause.
It is neither a sufficient fix for other toolchains/configs,
nor a sustainable solution for master yet.
We'd need to improve the python script for that.

19 months ago[0.4.10][SHELL32] Copy without FILE_ATTRIBUTE_READONLY from a CDROM CORE-18089 (...
Joachim Henze [Thu, 22 Sep 2022 14:17:07 +0000 (16:17 +0200)]
[0.4.10][SHELL32] Copy without FILE_ATTRIBUTE_READONLY from a CDROM CORE-18089 (#4386)

That problem in the shell was unhidden when we switched from our own to the MS-PL CDFS-driver
in 0.4.8-dev-164-g ec6b3ecbe48ad51c366563196cc93a6ae74e8b69
because our own old CDFS driver erroneously never reported any CD file being read-only.

Fix picked from 0.4.15-dev-4348-g 513d0a6d2d4652671c9b7c92c495edce6d9491a4

While backporting the fix, I decided to port back also some EOL-whitespace-stripping in that file.

19 months ago[0.4.10][SHELL32] Fix uninitialized variable usage. CORE-15251 (#3986)
Joachim Henze [Tue, 20 Sep 2022 16:24:35 +0000 (18:24 +0200)]
[0.4.10][SHELL32] Fix uninitialized variable usage. CORE-15251 (#3986)

This fixes log spam about NM_CUSTOMDRAW not being handled,
e.g. when interacting with the start-menu or filebrowsers menubar.
Less logging implies faster painting here with dbg-builds.

fix picked from commit 0.4.15-dev-3229-g 22e58e68aa35a1ba49d280b12f99f7f03f5f8633

The bug was a regression of 0.4.9-dev-151-g 0aed0fd167863d1674e73a7fdb22fd1fb7a319f0 very likely,
The symptom with the logging can neither be observed in unpatched releases/0.4.8 nor in releases/0.4.7

19 months ago[0.4.10][WINHTTP] Fix 1 MSVC2010SP1 x86 dbg warning LNK4199 (#4414) CORE-18104
Joachim Henze [Mon, 19 Sep 2022 22:24:20 +0000 (00:24 +0200)]
[0.4.10][WINHTTP] Fix 1 MSVC2010SP1 x86 dbg warning LNK4199 (#4414) CORE-18104

which I could observe all the way from releases/0.4.7 up to releases/0.4.14
with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuration

Creating library dll\win32\winhttp\winhttp.lib and object dll\win32\winhttp\winhttp.exp
LINK : warning LNK4199: /DELAYLOAD:ole32.dll ignored; no imports found from ole32.dll

Fix picked from 0.4.15-dev-4326-g 2f7dc168f29c0693203cc10901cb6dc4bb23ea75

And just for the record: Wine did the same with their:
https://source.winehq.org/git/wine.git/commitdiff/f400f25c2516654f85e0246a4815a2bf62a73cfd

19 months ago[0.4.10][FREELDR] Fix 14 MSVC2010SP1 x86 dbg warnings C4146 CORE-18104 (#863)
Joachim Henze [Sat, 17 Sep 2022 23:10:18 +0000 (01:10 +0200)]
[0.4.10][FREELDR] Fix 14 MSVC2010SP1 x86 dbg warnings C4146 CORE-18104 (#863)

the warnings could be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuration:

C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(682) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(689) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(692) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(707) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(715) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(722) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(731) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(779) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(782) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(797) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(817) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(1026) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(1153) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
C:\0410rls\reactos\boot\freeldr\freeldr\lib\fs\btrfs.c(1194) : warning C4146: unary minus operator applied to unsigned type, result still unsigned

Fix picked from 0.4.11-dev-170-g d0e29db1cb61c057d0d674ad06337d9582ae89c6

All those bugs were introduced by 0.4.10-dev-595-g 3b69eee7a65fc7b768b5a92cbf5955f1e6f79068

19 months ago[0.4.10][HOST-TOOLS] Fix all remaining (>78) MSVC2010SP1 x86 dbg warnings C4018,C4996...
Joachim Henze [Mon, 15 Apr 2019 11:29:33 +0000 (13:29 +0200)]
[0.4.10][HOST-TOOLS] Fix all remaining (>78) MSVC2010SP1 x86 dbg warnings C4018,C4996,... CORE-18104 (#1525)

From releases/0.4.9 to releases/0.4.10 we introduced tons of compiler warnings
within the whole host-tools component although the 'offending' code did not change in between both.
I am not aware which commit unhid them exactly during 0.4.10-dev',
but I will follow the footsteps of master with the fixes.
releases/0.4.9 still had 0 warnings within host-tools, and releases/0.4.13 managed to achieve the same again!

fix is a pick from:
0.4.11-dev-436-g da019b0fc57ac4c6b9a161332a576693c6e8ccd9 (the mkhive\registry.c part)
0.4.13-dev-132-g f47f45dbdd68e21ec7ab283877cbd139f418c804 (picked the remaining parts of that)
0.4.13-dev-133-g c02257ef57858bfe91f106e8c91c669e4a141293
0.4.13-dev-134-g 2c11c411153eea31d10025dddf7332ce641be629

the warnings could be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuration:

where the
0.4.13-dev-132-g f47f45dbdd68e21ec7ab283877cbd139f418c804 part fixed the:

[6/12736] Performing build step for 'host-tools'
[13/250] Building CXX object sdk\tools\hhpcomp\CMakeFiles\hhpcomp.dir\hhpcomp.cpp.obj
C:\0410rls\reactos\sdk\tools\hhpcomp\hhpcomp.cpp(59) : warning C4996: 'chdir': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _chdir. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\direct.h(123) : see declaration of 'chdir'
C:\0410rls\reactos\sdk\tools\hhpcomp\chmc\chmc.c(160) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\hhpcomp\chmc\chmc.c(447) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'
C:\0410rls\reactos\sdk\tools\hhpcomp\chmc\chmc.c(476) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\hhpcomp\chmc\chmc.c(511) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
C:\0410rls\reactos\sdk\tools\hpp\hpp.c(44) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'
C:\0410rls\reactos\sdk\tools\isohybrid\isohybrid.c(1201) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\isohybrid\isohybrid.c(1204) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\kbdtool\parser.c(448) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'
C:\0410rls\reactos\sdk\tools\kbdtool\parser.c(523) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'
C:\0410rls\reactos\sdk\tools\kbdtool\parser.c(598) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\stdio\ffileread.c(30) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\stdio\fcons.c(62) : warning C4996: 'fdopen': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fdopen. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(721) : see declaration of 'fdopen'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\stdio\flag.c(44) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\stdio\flag.c(73) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\stdio\flag.c(98) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\stdio\filewrite.c(82) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\fconv.c(231) : warning C4996: 'ecvt': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _ecvt. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdlib.h(864) : see declaration of 'ecvt'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\fconv.c(320) : warning C4996: 'fcvt': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fcvt. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdlib.h(865) : see declaration of 'fcvt'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\format.c(745) : warning C4996: 'gcvt': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _gcvt. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdlib.h(866) : see declaration of 'gcvt'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libschily\searchinpath.c(221) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\libsiconv\sic_nls.c(450) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(169) : warning C4996: 'access': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _access. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(299) : see declaration of 'access'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(225) : warning C4996: 'access': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _access. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(299) : see declaration of 'access'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(414) : warning C4996: 'open': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _open. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(312) : see declaration of 'open'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(420) : warning C4996: 'read': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _read. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(313) : see declaration of 'read'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(425) : warning C4996: 'close': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _close. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(302) : see declaration of 'close'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(464) : warning C4996: 'open': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _open. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(312) : see declaration of 'open'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(467) : warning C4996: 'read': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _read. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(313) : see declaration of 'read'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(469) : warning C4996: 'close': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _close. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(302) : see declaration of 'close'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(536) : warning C4996: 'open': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _open. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(312) : see declaration of 'open'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(541) : warning C4996: 'read': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _read. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(313) : see declaration of 'read'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\boot.c(549) : warning C4996: 'close': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _close. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(302) : see declaration of 'close'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\eltorito.c(447) : warning C4996: 'open': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _open. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(312) : see declaration of 'open'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\eltorito.c(453) : warning C4996: 'read': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _read. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(313) : see declaration of 'read'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\eltorito.c(457) : warning C4996: 'close': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _close. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(302) : see declaration of 'close'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\eltorito.c(585) : warning C4996: 'open': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _open. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(312) : see declaration of 'open'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\eltorito.c(594) : warning C4996: 'read': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _read. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(313) : see declaration of 'read'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\eltorito.c(615) : warning C4996: 'lseek': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _lseek. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(310) : see declaration of 'lseek'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\eltorito.c(623) : warning C4996: 'write': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _write. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(318) : see declaration of 'write'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\eltorito.c(624) : warning C4996: 'close': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _close. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(302) : see declaration of 'close'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\match.c(74) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\match.c(201) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\multi.c(124) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\multi.c(126) : warning C4996: 'lseek': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _lseek. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(310) : see declaration of 'lseek'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\multi.c(130) : warning C4996: 'read': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _read. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(313) : see declaration of 'read'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\multi.c(124) : warning C4996: 'fileno': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fileno. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(723) : see declaration of 'fileno'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\multi.c(126) : warning C4996: 'lseek': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _lseek. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(310) : see declaration of 'lseek'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\multi.c(130) : warning C4996: 'read': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _read. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(313) : see declaration of 'read'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\tree.c(1628) : warning C4996: 'access': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _access. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\io.h(299) : see declaration of 'access'
C:\0410rls\reactos\sdk\tools\mkisofs\schilytools\mkisofs\write.c(204) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'

C:\0410rls\reactos\sdk\tools\wpp\preproc.c(162) : warning C4013: 'open' undefined; assuming extern returning int
C:\0410rls\reactos\sdk\tools\wpp\preproc.c(165) : warning C4013: 'close' undefined; assuming extern returning int

C:\0410rls\reactos\sdk\tools\widl\widl.c(863) : warning C4996: 'fdopen': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fdopen. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(721) : see declaration of 'fdopen'
C:\0410rls\reactos\sdk\tools\widl\widl.c(910) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
C:\0410rls\reactos\sdk\tools\widl\widl.c(912) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
C:\0410rls\reactos\sdk\tools\widl\widl.c(914) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
C:\0410rls\reactos\sdk\tools\widl\widl.c(916) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
C:\0410rls\reactos\sdk\tools\widl\widl.c(918) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
C:\0410rls\reactos\sdk\tools\widl\widl.c(920) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
C:\0410rls\reactos\sdk\tools\widl\widl.c(922) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
C:\0410rls\reactos\sdk\tools\widl\widl.c(924) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
C:\0410rls\reactos\sdk\tools\widl\widl.c(926) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'

C:\0410rls\reactos\sdk\tools\widl\typelib.c(264) : warning C4013: 'read' undefined; assuming extern returning int
C:\0410rls\reactos\sdk\tools\widl\typelib.c(270) : warning C4013: 'lseek' undefined; assuming extern returning int
C:\0410rls\reactos\sdk\tools\widl\typelib.c(343) : warning C4013: 'open' undefined; assuming extern returning int
C:\0410rls\reactos\sdk\tools\widl\typelib.c(383) : warning C4013: 'close' undefined; assuming extern returning int

C:\0410rls\reactos\sdk\tools\widl\utils.c(281) : warning C4013: 'open' undefined; assuming extern returning int
C:\0410rls\reactos\sdk\tools\widl\utils.c(283) : warning C4013: 'write' undefined; assuming extern returning int
C:\0410rls\reactos\sdk\tools\widl\utils.c(285) : warning C4013: 'close' undefined; assuming extern returning int

[194/250] Building C object sdk\tools\widl\CMakeFiles\widl.dir\parser.yy.c.obj
parser.l(488) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'
parser.l(542) : warning C4996: 'fdopen': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _fdopen. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(721) : see declaration of 'fdopen'
parser.l(562) : warning C4996: 'unlink': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _unlink. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\stdio.h(299) : see declaration of 'unlink'

[197/250] Building C object sdk\tools\widl\CMakeFiles\widl.dir\parser.tab.c.obj
parser.y(454) : warning C4996: 'strdup': The POSIX name for this item is deprecated. Instead, use the ISO C++ conformant name: _strdup. See online help for details.
        C:\VS2010_SP1\VC\INCLUDE\string.h(238) : see declaration of 'strdup'

... and so on ... too many to name them all
----------
0.4.13-dev-134-g 2c11c411153eea31d10025dddf7332ce641be629 part fixed the:
C:\0410rls\reactos\sdk\tools\widl\getopt.c(233) : warning C4013: '_getopt_internal' undefined; assuming extern returning int
----------
0.4.13-dev-133-g c02257ef57858bfe91f106e8c91c669e4a141293 part fixed the:
C:\0410rls\reactos\sdk\tools\isohybrid\isohybrid.c(978) : warning C4101: 'orig_gpt_size' : unreferenced local variable
C:\0410rls\reactos\sdk\tools\isohybrid\isohybrid.c(978) : warning C4101: 'gpt_size' : unreferenced local variable
----------
partial port (registry.c) of 0.4.11-dev-436-g da019b0fc57ac4c6b9a161332a576693c6e8ccd9 fixed the:
C:\0410rls\reactos\sdk\tools\mkhive\registry.c(369) : warning C4101: 'SubKeyCell' : unreferenced local variable

2 years ago[0.4.10][HOST-TOOLS:MKSHELLLINK] Fix 8 MSVC2010SP1 x86 dbg warnings C4018 CORE-18104
Joachim Henze [Sun, 10 Apr 2022 20:45:40 +0000 (22:45 +0200)]
[0.4.10][HOST-TOOLS:MKSHELLLINK] Fix 8 MSVC2010SP1 x86 dbg warnings C4018 CORE-18104

From releases/0.4.9 to releases/0.4.10 we introduced tons of compiler warnings
within the whole host-tools component although the 'offending' code did not change in between both.
I am not aware which commit unhid them exactly during 0.4.10-dev',
but I will follow the footsteps of master with the fixes.
releases/0.4.9 still had 0 warnings within host-tools!

warnings can be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuration:
[6/12736] Performing build step for 'host-tools'
C:\0410rls\reactos\sdk\tools\mkshelllink\mkshelllink.c(147) : warning C4018: '<' : signed/unsigned mismatch
C:\0410rls\reactos\sdk\tools\mkshelllink\mkshelllink.c(153) : warning C4018: '<' : signed/unsigned mismatch
C:\0410rls\reactos\sdk\tools\mkshelllink\mkshelllink.c(155) : warning C4018: '<' : signed/unsigned mismatch
C:\0410rls\reactos\sdk\tools\mkshelllink\mkshelllink.c(157) : warning C4018: '<' : signed/unsigned mismatch
C:\0410rls\reactos\sdk\tools\mkshelllink\mkshelllink.c(159) : warning C4018: '<' : signed/unsigned mismatch
C:\0410rls\reactos\sdk\tools\mkshelllink\mkshelllink.c(161) : warning C4018: '<' : signed/unsigned mismatch
C:\0410rls\reactos\sdk\tools\mkshelllink\mkshelllink.c(164) : warning C4018: '<' : signed/unsigned mismatch
C:\0410rls\reactos\sdk\tools\mkshelllink\mkshelllink.c(169) : warning C4018: '<' : signed/unsigned mismatch

partial pick from: 0.4.13-dev-132-g f47f45dbdd68e21ec7ab283877cbd139f418c804

2 years ago[0.4.10][WPP] Mute 3 MSVC2010SP1 x86 dbg warnings C4146 CORE-18104
Joachim Henze [Thu, 7 Apr 2022 22:54:01 +0000 (00:54 +0200)]
[0.4.10][WPP] Mute 3 MSVC2010SP1 x86 dbg warnings C4146 CORE-18104

warnings can be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuration:
[7441/11751] Building C object sdk\tools\wpp\CMakeFiles\wpp.dir\ppl.yy.c.obj
[7442/11751] Building C object sdk\tools\wpp\CMakeFiles\wpp.dir\ppy.tab.c.obj
ppy.y(402) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
ppy.y(402) : warning C4146: unary minus operator applied to unsigned type, result still unsigned
ppy.y(402) : warning C4146: unary minus operator applied to unsigned type, result still unsigned

partial pick from: 0.4.13-dev-132-g f47f45dbdd68e21ec7ab283877cbd139f418c804

Yes the /wd4244 wasn't really necessary for x86, but only for x64,
but taking it with me as well synced the file to 100%.

2 years ago[0.4.10][CMAKE] Fix 9 MSVC2010SP1 x86 dbg warnings LNK4221 CORE-18104 CORE-14373
Joachim Henze [Thu, 31 Mar 2022 16:28:22 +0000 (18:28 +0200)]
[0.4.10][CMAKE] Fix 9 MSVC2010SP1 x86 dbg warnings LNK4221 CORE-18104 CORE-14373

by always unconditionally enabling "String Pooling" /GF

Pros:
- consistency between the toolchains, as GCC always has this enabled
- shrinks the binary sizes for MSVC dbg builds
- smaller binaries will also make relocations less probable with MSVC dbg and a given set of (often outdated) precalculated base-addresses
- interestingly it also fixes/hides some linker warnings LNK4221

which can be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuration, here example from releases/0.4.7:
[819/11751] Linking C static library dll\opengl\mesa\main\mesa_main.lib
precomp.h.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

[830/11751] Linking C static library dll\opengl\mesa\math\mesa_math.lib
precomp.h.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

[858/11751] Linking C static library dll\opengl\mesa\swrast\mesa_swrast.lib
precomp.h.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

[879/11751] Linking C static library dll\opengl\mesa\tnl\mesa_tnl.lib
precomp.h.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

[897/11751] Linking C static library dll\opengl\mesa\vbo\mesa_vbo.lib
precomp.h.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

[1602/11751] Linking C static library sdk\lib\3rdparty\adns\adns.lib
internal.h.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

[4327/11751] Linking C static library sdk\lib\tdilib\tdilib.lib
precomp.h.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

[11714/11751] Linking C static library win32ss\user\winsrv\consrv.lib
consrv.h.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

[11726/11751] Linking C static library win32ss\user\winsrv\concfg\concfg.lib
precomp.h.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library

It manages to make all the mentioned 9 LNK4221 disappear without introducing any other warnings into those modules, or into any other module.
Ftr I do not understand why String Pooling hides those LNK4221. But its effectiveness is evident.
But some other LNK4221 in other modules do still remain after it. Their count is not 0 afterwards, just significantly reduced.

Cons:
- String Pooling may slightly increase the build time, but the impact is negligible, even on my 12 year-old PC.
- String Pooling may make lousy code crash, that illegally wants to write into string constants, like we saw it for ROSTESTS298 for example.
  But that is actually not a bad thing. That test was also broken beforehand and crashed even on real Windows.
  We didn't find any similar problems within the 'productive code' yet.

fix is a partial pick of:
0.4.12-dev-354-g 5e673f311854a125c3347ae00c924d347972003c
and its addendum
0.4.12-dev-655-g 237110604b52fa45e10058b074770471081843be [CMAKE] msvc.cmake: Remove 2 now redundant '/GF' (#1239)

2 years ago[0.4.10][CYLFRAC] Fix 3 MSVC2010SP1 x86 dbg warnings C4305 CORE-18104
Joachim Henze [Thu, 31 Mar 2022 04:14:33 +0000 (06:14 +0200)]
[0.4.10][CYLFRAC] Fix 3 MSVC2010SP1 x86 dbg warnings C4305 CORE-18104

can be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuration:
C:\047rls\reactos\modules\rosapps\applications\screensavers\cylfrac\cylfrac.c(109) : warning C4305: 'function' : truncation from 'double' to 'float'
C:\047rls\reactos\modules\rosapps\applications\screensavers\cylfrac\cylfrac.c(110) : warning C4305: 'function' : truncation from 'double' to 'float'
C:\047rls\reactos\modules\rosapps\applications\screensavers\cylfrac\cylfrac.c(137) : warning C4305: 'initializing' : truncation from 'double' to 'GLfloat'

partial pick of 0.4.15-dev-2951-g 5d8e834897781944dcaf80de6c3df9c2c51212c7

2 years ago[0.4.10][ROSHTTPD] Fix 3 MSVC2010SP1 x86 dbg warnings C4373 CORE-18104
Joachim Henze [Tue, 29 Mar 2022 17:37:34 +0000 (19:37 +0200)]
[0.4.10][ROSHTTPD] Fix 3 MSVC2010SP1 x86 dbg warnings C4373 CORE-18104

can be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuation:
[11438/11751] Building CXX object modules\rosapps\applications\net\roshttpd\CMakeFiles\roshttpd.dir\config.cpp.obj
C:\047rls\reactos\modules\rosapps\applications\net\roshttpd\include\httpd.h(70) : warning C4373: 'CHttpDaemon::OnAccept': virtual function overrides 'CServerSocket::OnAccept', previous versions of the compiler did not override when parameters only differed by const/volatile qualifiers
        C:\047rls\reactos\modules\rosapps\applications\net\roshttpd\include\socket.h(132) : see declaration of 'CServerSocket::OnAccept'
[11441/11751] Building CXX object modules\rosapps\applications\net\roshttpd\CMakeFiles\roshttpd.dir\httpd.cpp.obj
C:\047rls\reactos\modules\rosapps\applications\net\roshttpd\include\httpd.h(70) : warning C4373: 'CHttpDaemon::OnAccept': virtual function overrides 'CServerSocket::OnAccept', previous versions of the compiler did not override when parameters only differed by const/volatile qualifiers
        C:\047rls\reactos\modules\rosapps\applications\net\roshttpd\include\socket.h(132) : see declaration of 'CServerSocket::OnAccept'
[11442/11751] Building CXX object modules\rosapps\applications\net\roshttpd\CMakeFiles\roshttpd.dir\roshttpd.cpp.obj
C:\047rls\reactos\modules\rosapps\applications\net\roshttpd\include\httpd.h(70) : warning C4373: 'CHttpDaemon::OnAccept': virtual function overrides 'CServerSocket::OnAccept', previous versions of the compiler did not override when parameters only differed by const/volatile qualifiers
        C:\047rls\reactos\modules\rosapps\applications\net\roshttpd\include\socket.h(132) : see declaration of 'CServerSocket::OnAccept'

partial pick of 0.4.15-dev-2951-g 5d8e834897781944dcaf80de6c3df9c2c51212c7

2 years ago[0.4.10][NETREG] Fix 1 MSVC2010SP1 x86 dbg warning C4065 CORE-18104
Joachim Henze [Tue, 29 Mar 2022 16:23:49 +0000 (18:23 +0200)]
[0.4.10][NETREG] Fix 1 MSVC2010SP1 x86 dbg warning C4065 CORE-18104

the warning can be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuation:
[11432/11751] Building CXX object modules\rosapps\applications\net\netreg\CMakeFiles\netreg.dir\netreg.cpp.obj
C:\047rls\reactos\modules\rosapps\applications\net\netreg\netreg.cpp(188) : warning C4065: switch statement contains 'default' but no 'case' labels

partial pick of 0.4.15-dev-2951-g 5d8e834897781944dcaf80de6c3df9c2c51212c7

2 years ago[0.4.10][D3DCOMPILER_43] Fix 11 MSVC2010SP1 x86 dbg warnings C4133 CORE-18104
Joachim Henze [Mon, 28 Mar 2022 23:53:28 +0000 (01:53 +0200)]
[0.4.10][D3DCOMPILER_43] Fix 11 MSVC2010SP1 x86 dbg warnings C4133 CORE-18104

can be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuation:
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1107) : warning C4133: 'function' : incompatible types - from 'D3D_PRIMITIVE *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1110) : warning C4133: 'function' : incompatible types - from 'D3D_PRIMITIVE_TOPOLOGY *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1126) : warning C4133: 'function' : incompatible types - from 'D3D_TESSELLATOR_OUTPUT_PRIMITIVE *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1129) : warning C4133: 'function' : incompatible types - from 'D3D_TESSELLATOR_PARTITIONING *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1132) : warning C4133: 'function' : incompatible types - from 'D3D_TESSELLATOR_DOMAIN *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1438) : warning C4133: 'function' : incompatible types - from 'D3D_SHADER_INPUT_TYPE *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1441) : warning C4133: 'function' : incompatible types - from 'D3D_RESOURCE_RETURN_TYPE *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1444) : warning C4133: 'function' : incompatible types - from 'D3D_SRV_DIMENSION *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1507) : warning C4133: 'function' : incompatible types - from 'D3D_CBUFFER_TYPE *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1603) : warning C4133: 'function' : incompatible types - from 'D3D_NAME *' to 'DWORD *'
C:\047rls\reactos\dll\directx\wine\d3dcompiler_43\reflection.c(1604) : warning C4133: 'function' : incompatible types - from 'D3D_REGISTER_COMPONENT_TYPE *' to 'DWORD *'

picked from 0.4.14-dev-8-g 36d9e80add6c295ce05a3655eb23b1efd1f3fff7

( for the record: 0.4.13 got this years ago already via 0.4.13-RC-4-g d27a20233d1a199de3e199a9db94ffbdb5390a48 )

2 years ago[0.4.10][CMBATT] Fix 1 MSVC2010SP1 x86 dbg warning C4146 CORE-18104
Joachim Henze [Mon, 28 Mar 2022 22:43:54 +0000 (00:43 +0200)]
[0.4.10][CMBATT] Fix 1 MSVC2010SP1 x86 dbg warning C4146 CORE-18104

can be observed with MSVC 2010SP1 (16.0.40219.1) x86 target in dbg configuation:
C:\047rls\reactos\drivers\bus\acpi\cmbatt\cmbatt.c(983) : warning C4146: unary minus operator applied to unsigned type, result still unsigned

partial pick of 0.4.13-dev-713-g 08c6d21e1fb9af8fc4d43a29e74ac3e05581c096