- wine-gitlab - winehq.org

Re: [PATCH v2 0/6] MR6136: wined3d: Feed FFP shader constants through push constant buffers, part 5. - approved
by Alexandre Julliard (＠julliard) 31 Jul '24

31 Jul '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6136

1 0

[PATCH v6 0/2] MR6145: comctl32: Implement PSN_QUERYINITIALFOCUS
by Jacob Czekalla (＠JacobCzekalla) 31 Jul '24

31 Jul '24

Adding PSN_QUERYINITIALFOCUS helped fix some focus issues with the property sheet in bug 54862. Previously the listview in the tab control did not get focus from the start when it should have. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54862 -- v6: comctl32: Add handling for PSN_QUERYINITIALFOCUS in prop.c. comctl32/tests: Add test for PSN_QUERYINITIALFOCUS for the propsheet. https://gitlab.winehq.org/wine/wine/-/merge_requests/6145

2 2

[PATCH v9 0/6] MR6004: comctl32: A NULL string fix.
by Aida Jonikienė (＠DodoGTA) 31 Jul '24

31 Jul '24

-- v9: user32/edit: Add some WM_ASKCBFORMATNAME tests. user32/tests: Add tests for NULL strings with edit control GETTEXT message. comctl32/tests: Add tests for NULL strings with edit control GETTEXT message. user32: Correctly handle NULL string in GETTEXT/CBFORMAT AtoW wrapper. user32: Fail on NULL string for edit control WM_GETTEXT message. comctl32: Fail on NULL string for edit control WM_GETTEXT message. https://gitlab.winehq.org/wine/wine/-/merge_requests/6004

4 12

Re: [PATCH v6 0/1] MR3410: advpack: Add nullcheck for parameter that can be null (Coverity)
by Fabian Maurer (＠DarkShadow44) 31 Jul '24

31 Jul '24

On Wed Jul 31 16:19:30 2024 +0000, Nikolay Sivov wrote: > If this is coming from analyzer report, I wouldn't bother. Yeah, it's from an analyzer. But I don't see the harm in making it NULL safe, it seems like a simple change to me. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3410#note_77432

1 0

Re: [PATCH v6 0/1] MR3410: advpack: Add nullcheck for parameter that can be null (Coverity)
by Nikolay Sivov (＠nsivov) 31 Jul '24

31 Jul '24

If this is coming from analyzer report, I wouldn't bother. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3410#note_77430

1 0

Re: [PATCH v6 0/1] MR3410: advpack: Add nullcheck for parameter that can be null (Coverity)
by Fabian Maurer (＠DarkShadow44) 31 Jul '24

31 Jul '24

@huw Sorry for the random ping, but I'm still looking for someone to review this change, would you mind taking a look? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3410#note_77429

1 0

[PATCH 0/6] MR6136: wined3d: Feed FFP shader constants through push constant buffers, part 5.
by Elizabeth Figura (＠zfigura) 31 Jul '24

31 Jul '24

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/6136

4 8

Re: [PATCH v13 0/1] MR6150: win32u: fix a regression bug that virtual desktop not effective.
by Zhiyi Zhang (＠zhiyi) 31 Jul '24

31 Jul '24

Zhiyi Zhang (@zhiyi) commented about dlls/win32u/sysparams.c: > return STATUS_SUCCESS; > } > > -/* parse the desktop size specification */ > -static BOOL parse_size( const WCHAR *size, DWORD *width, DWORD *height ) I don't think you're going in the right direction. Why did you remove all this code? You should be able to parse the resolution from the command line and set it as the resolution for the virtual desktop window and resolve the regression. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6150#note_77420

1 0

[PATCH v5 0/2] MR6145: comctl32: Implement PSN_QUERYINITIALFOCUS
by Jacob Czekalla (＠JacobCzekalla) 31 Jul '24

31 Jul '24

Adding PSN_QUERYINITIALFOCUS helped fix some focus issues with the property sheet in bug 54862. Previously the listview in the tab control did not get focus from the start when it should have. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54862 -- v5: comctl32: Add handling for PSN_QUERYINITIALFOCUS in prop.c. comctl32/tests: Add test for PSN_QUERYINITIALFOCUS for the propsheet. https://gitlab.winehq.org/wine/wine/-/merge_requests/6145

4 5

[PATCH v3 0/2] MR6146: comctl32: Create all prop sheet pages on initialization.
by Jacob Czekalla (＠JacobCzekalla) 31 Jul '24

31 Jul '24

Fixes bug 54861. The UK Kalender program was crashing when a new event was created as stated in the bug report. The fix is to create all the property sheet pages from the beginning instead of when they are selected. -- v3: comctl32: Create all prop sheet pages on initialization. comctl32/tests: Add test for propsheet page creation when propsheet gets initialized. https://gitlab.winehq.org/wine/wine/-/merge_requests/6146

3 7

[PATCH v6 0/1] MR5523: msdelta: Add stubs for ApplyDeltaA() and ApplyDeltaW().
by Vijay Kiran Kamuju (＠infyquest) 31 Jul '24

31 Jul '24

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56007 Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56576 -- v6: msdelta: Add stubs for ApplyDeltaA() and ApplyDeltaW(). https://gitlab.winehq.org/wine/wine/-/merge_requests/5523

2 1

Re: [PATCH v2 0/1] MR6170: programs/cmd: Fix test failures for SET /P command.
by eric pouech (＠epo) 31 Jul '24

31 Jul '24

1) It's not a high priority for me to fix white space discrepancies in printing (unless there's a program in the wild that makes use of it) (I would be more worried about a space before the = as it wwould impact the saved environment variable) 2) would you mind adding a test case for this so that we don't lose this use case? 3) I strongly believe SET (and not only SET /P) shall rely on parsing the parameters with the usual helper functions instead of doing their own parsing 4) there's also a more annoying issue like not expanding in SET the potential variables inside quotes (like SET "%foo%=bar", and IIRC Wine doesn't expand %foo%, but it does expand in SET %foo%=bar) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6170#note_77392

1 0

Re: [PATCH v10 0/3] MR5104: include: Add missing Task Scheduler Trigger definitions - approved
by Alexandre Julliard (＠julliard) 30 Jul '24

30 Jul '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5104

1 0

Re: [PATCH v6 0/2] MR6163: shell32: Fix FO_MOVE when destination has conflicting directory. - approved
by Alexandre Julliard (＠julliard) 30 Jul '24

30 Jul '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6163

1 0

Re: [PATCH v2 0/8] MR6177: mshtml: Use JS bindings for more objects. - approved
by Alexandre Julliard (＠julliard) 30 Jul '24

30 Jul '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6177

1 0

Re: [PATCH v2 0/1] MR6176: sechost: Check both lpServiceName and lpServiceProc for NULL in StartServiceCtrlDispatcher(). - approved
by Alexandre Julliard (＠julliard) 30 Jul '24

30 Jul '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6176

1 0

Re: [PATCH v4 0/1] MR6053: d3dx9/effect: Trace the pass count header field. - approved
by Alexandre Julliard (＠julliard) 30 Jul '24

30 Jul '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6053

1 0

Re: [PATCH v2 0/1] MR6170: programs/cmd: Fix test failures for SET /P command.
by Vijay Kiran Kamuju (＠infyquest) 30 Jul '24

30 Jul '24

We are missing spaces in wine, if have spaces after = and before ' or ". -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6170#note_77379

1 0

[PATCH v3 0/1] MR6175: ntdll: Raise exception on failed CS wait.
by Paul Gofman (＠gofman) 30 Jul '24

30 Jul '24

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57011 -- v3: ntdll: Raise exception on failed CS wait. https://gitlab.winehq.org/wine/wine/-/merge_requests/6175

2 3

[PATCH 0/8] MR6151: mshtml: window.performance improvements.
by Gabriel Ivăncescu (＠insn) 30 Jul '24

30 Jul '24

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/6151

3 12

Re: [PATCH v2 0/1] MR6170: programs/cmd: Fix test failures for SET /P command.
by eric pouech (＠epo) 30 Jul '24

30 Jul '24

- are we missing new lines? (assuming new test cases that don't pass) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6170#note_77370

1 0

Re: [PATCH v2 0/1] MR6170: programs/cmd: Fix test failures for SET /P command.
by Vijay Kiran Kamuju (＠infyquest) 30 Jul '24

30 Jul '24

The following tests fail: set /p abc= 'Value=' 'Value=' set /p abc= "Value=" Value= on wine: ../../wine cmd.exe.so Z:\home\admin\devel\wineiq64-build\programs\cmd>set /p abc= 'Value=' 'Value=' Z:\home\admin\devel\wineiq64-build\programs\cmd>set /p abc= "Value=" "Value=" -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6170#note_77369

1 0

[PATCH v4 0/2] MR6160: comctl32: Fix sorting for listview.
by Jacob Czekalla (＠JacobCzekalla) 30 Jul '24

30 Jul '24

The listview bug is caused by passing the incorrect index to the LISTVIEW_SortItems callers custom compare function. If we pass the copied array indices, which are different because of the sorting, the caller is going to use those indices for their unsorted array which don't match. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56140 -- v4: comctl32: Fix sorting for listview. comctl32/tests: Add test for listview sorting order. https://gitlab.winehq.org/wine/wine/-/merge_requests/6160

4 6

[PATCH v3 0/2] MR6059: mf: Retry PROCESSINPUTNOTIFY if TRANSFORM_TYPE_NOT_SET is returned.
by Brendan McGrath (＠redmcg) 30 Jul '24

30 Jul '24

Send INVALIDATEMEDIATYPE to allow the transform type to be set before retrying PROCESSINPUTNOTIFY. -- v3: mf: Retry PROCESSINPUTNOTIFY if TRANSFORM_TYPE_NOT_SET is returned. mf/tests: Add tests for custom evr presenter. https://gitlab.winehq.org/wine/wine/-/merge_requests/6059

3 3

[PATCH v2 0/1] MR6053: d3dx9/effect: Trace the pass count header field.
by Nikolay Sivov (＠nsivov) 30 Jul '24

30 Jul '24

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- v2: d3dx9/effect: Document one remaining header field. https://gitlab.winehq.org/wine/wine/-/merge_requests/6053

4 6

2025

2024

2023

2022

wine-gitlab