- Wine-GitLab - list.winehq.org

Re: [PATCH v3 0/6] MR553: mf: Some attribute tweaks to fix Resampler transform regressions. - approved
by Alexandre Julliard (＠julliard) Aug. 2, 2022

Aug. 2, 2022

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

1 0

[PATCH v8 0/8] MR391: scrrun: Implement MoveFolder().
by Robert Wilhelm (＠sloper42) Aug. 2, 2022

Aug. 2, 2022

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52128 Signed-off-by: Robert Wilhelm <robert.wilhelm(a)gmx.net> -- v8: scrrun: Move directories only in MoveFolder(). scrrun: Support wildcards in MoveFolder(). scrrun: Move source dir into destination dir if destination ends with separator in MoveFolder(). scrrun: Check that source is directory in MoveFolder(). scrrun: Check for non-existant source in MoveFolder(). https://gitlab.winehq.org/wine/wine/-/merge_requests/391

3 19

[PATCH v15 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion
by Tobias Görgens Aug. 2, 2022

Aug. 2, 2022

Fix stub DllGetVersion implementation to read Dll version and put it in correct structure. Implement GetDllVersion by using DllGetVersion function and return version -- v15: cabinet/tests: add tests for DllGetVersion & GetDllVersion https://gitlab.winehq.org/wine/wine/-/merge_requests/564

2 4

[PATCH v3 0/4] MR565: Implement NCryptEncrypt and BCryptEncrypt
by Santino Mazza (＠tati1454) Aug. 2, 2022

Aug. 2, 2022

Some installer, like the visual studio installer, also make use of this function, instead of just NCryptVerifySignature. -- v3: ncrypt: Initial implementation for NCryptEncrypt. bcrypt: Initial RSA encryption implementation. bcrypt/tests: Test for BCryptEncrypt with RSA keys. https://gitlab.winehq.org/wine/wine/-/merge_requests/565

3 11

Re: [PATCH v2 0/3] MR511: webservices: Support error strings and faults in error.
by Hans Leidekker (＠hans) Aug. 2, 2022

Aug. 2, 2022

Otherwise this patchset looks good. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/511#note_5508

1 0

[PATCH 0/1] MR516: webservices: Write the correct headers when a request message is addressed.
by Shaun Ren (＠shaunren) Aug. 2, 2022

Aug. 2, 2022

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

3 2

Re: [PATCH v3 0/1] MR515: webservices: Handle in/out parameters properly in WsCall. - approved
by Hans Leidekker (＠hans) Aug. 2, 2022

Aug. 2, 2022

This merge request was approved by Hans Leidekker. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/515

1 0

[PATCH v14 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion
by Tobias Görgens Aug. 2, 2022

Aug. 2, 2022

Fix stub DllGetVersion implementation to read Dll version and put it in correct structure. Implement GetDllVersion by using DllGetVersion function and return version -- v14: cabinet/tests: add tests for DllGetVersion & GetDllVersion https://gitlab.winehq.org/wine/wine/-/merge_requests/564

1 1

[PATCH v12 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion
by Tobias Görgens Aug. 2, 2022

Aug. 2, 2022

Fix stub DllGetVersion implementation to read Dll version and put it in correct structure. Implement GetDllVersion by using DllGetVersion function and return version -- v12: cabinet/tests: add tests for DllGetVersion & GetDllVersion https://gitlab.winehq.org/wine/wine/-/merge_requests/564

1 1

[PATCH v11 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion
by Tobias Görgens Aug. 2, 2022

Aug. 2, 2022

Fix stub DllGetVersion implementation to read Dll version and put it in correct structure. Implement GetDllVersion by using DllGetVersion function and return version -- v11: cabinet/tests: add tests for DllGetVersion & GetDllVersion https://gitlab.winehq.org/wine/wine/-/merge_requests/564

1 1

Re: [PATCH v2 0/3] MR511: webservices: Support error strings and faults in error.
by Hans Leidekker (＠hans) Aug. 2, 2022

Aug. 2, 2022

Hans Leidekker (@hans) commented about dlls/webservices/error.c: > WS_HEAP *heap; > ULONG prop_count; > struct prop prop[ARRAY_SIZE( error_props )]; > + ULONG error_strs_count; > + ULONG error_strs_size; /* Maximum length of the error_strs array */ > + WS_STRING *error_strs; The error_ prefix is redundant. 'strings' would be clearer. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/511#note_5494

1 0

[PATCH v10 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion
by Tobias Görgens Aug. 2, 2022

Aug. 2, 2022

Fix stub DllGetVersion implementation to read Dll version and put it in correct structure. Implement GetDllVersion by using DllGetVersion function and return version -- v10: cabinet/tests: add tests for DllGetVersion & GetDllVersion https://gitlab.winehq.org/wine/wine/-/merge_requests/564

1 1

Re: [PATCH v2 0/3] MR511: webservices: Support error strings and faults in error.
by Hans Leidekker (＠hans) Aug. 2, 2022

Aug. 2, 2022

First patch should be merged with the second. Tests and include changes could be separate patches. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/511#note_5492

1 0

[PATCH v9 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion
by Tobias Görgens Aug. 2, 2022

Aug. 2, 2022

Fix stub DllGetVersion implementation to read Dll version and put it in correct structure. Implement GetDllVersion by using DllGetVersion function and return version -- v9: cabinet/tests: add tests for DllGetVersion & GetDllVersion https://gitlab.winehq.org/wine/wine/-/merge_requests/564

2 3

Re: [PATCH v8 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion
by Tobias Görgens Aug. 2, 2022

Aug. 2, 2022

On Tue Aug 2 05:00:46 2022 +0000, Chip Davis wrote: > > Is there any way to manually trigger the test bot? > You can submit patches yourself to [the > testbot](https://testbot.winehq.org/). Note that you will need an > account on the testbot in order to do this, which I believe is separate > from your account here. > > the pipeline fails due to a (more or less) unrelated issue in another > dll (msi). They have a function with the same name, DllGetVersion. > That is because `DllGetVersion()` is also the name of an optional > function supported by COM server DLLs. That's what those definitions in > `<shlwapi.h>` are for. > > Could it be that the dev defined it in shlwapi.h? The issue is that I > can't define my function correctly with this definition (probably the > reason why the first implementation used it, wrongly), but they can't > use it without (or probably can, but didn't). > Or, you could give `cabinet`'s `DllGetVersion()` a new name, and then in > `cabinet.spec`, export the new name under the correct name: > ``` > 2 stdcall -private DllGetVersion(ptr) cabinet_dll_get_version > ``` > (Why did you remove the ordinals? The commit in which this happened has > somehow vanished, but I remember seeing that.) OK, I'll try to submit the patch manually to the test bot to see if it works now as expected :) Oh, I see, o I should leave the definition of DllGetVersion in it I guess. Renaming the function in cabinet.dll seems more like a workaround than a fix tbh, but I'll do as you say. Removing the ordinals didn't have a specific reason, I just noticed that in some spec files there are ordinals and in some there are just @'s. I couldn't find anything on the difference, so I swapped them. Especially since I don't know how to numerate the imports, this seemed easier and worked fine. Do the ordinals have any meaning/is there a reason to keep them? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/564#note_5485

1 0

[PATCH v3 0/7] MR536: xactengine3_7: Add helper function to add entries
by Alistair Leslie-Hughes (＠alesliehughes) Aug. 2, 2022

Aug. 2, 2022

Signed-off-by: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> -- v3: xactengine3_7: Return error on invalid notification value. xactengine3_7: Implement callback for supported messages xactengine3_7: Record context for each notications xactengine3_7: Map IXACT3Wave interfaces xactengine3_7: Map IXACT3Cue interfaces xactengine3_7: Map SoundBank interfaces xactengine3_7: Add helper function to add entries https://gitlab.winehq.org/wine/wine/-/merge_requests/536

3 12

[PATCH v3 0/2] MR521: ddraw: Don't account video memory for sysmem surfaces' draw textures.
by Stefan Dösinger Aug. 2, 2022

Aug. 2, 2022

Star Trek Starfleet Academy does not like it when available video memory goes down after creating a system memory resource. It destroys all its textures and recreates them, and in some sitations forgets to recreate one or another texture, resulting in rendering bugs. I suspect the game is trying to detect focus loss by monitoring for unexpected video memory changes. --- I am open to renaming WINED3DUSAGE_PRIVATE to something else, like WINED3DUSAGE_NO_VIDMEM_ACCOUNTING. This particular name is ugly long though. -- v3: ddraw/tests: Add some video memory accounting tests. https://gitlab.winehq.org/wine/wine/-/merge_requests/521

3 5

[PATCH v2 0/6] MR543: wined3d: Cache the output mode lists where possible.
by Zebediah Figura (＠zfigura) Aug. 2, 2022

Aug. 2, 2022

-- v2: wined3d: Build the device list only once in wined3d_output_find_closest_matching_mode(). dxgi: Build the device list only once in dxgi_output_get_display_mode_list(). d3d8: Cache the output mode list. d3d9: Cache the output mode list. wined3d: Build a list of wined3d_display_mode structures in wined3d_output_get_mode[_count](). wined3d: Factor out mode_matches_filter(). https://gitlab.winehq.org/wine/wine/-/merge_requests/543

5 10

[PATCH v2 0/7] MR544: ws2_32: More TCP reset patches.
by Zebediah Figura (＠zfigura) Aug. 2, 2022

Aug. 2, 2022

-- v2: ws2_32: Translate AFD_POLL_RESET to FD_CLOSE plus WSAECONNABORTED in WSAEnumNetworkEvents(). server: Translate AFD_POLL_RESET to FD_CLOSE plus WSAECONNABORTED in window messages. server: Return ERROR_CONNECTION_RESET when trying to recv() on a reset socket. ws2_32/tests: Work around a Linux bug. ws2_32/tests: Test send() after TCP reset. http.sys: Keep connection sockets open after sending a 400 response. ws2_32/tests: Test sending data to a peer is closed(). https://gitlab.winehq.org/wine/wine/-/merge_requests/544

2 7

[PATCH v2 0/11] MR557: mciqtz32: Correct video window behavior by creating default window.
by Akihiro Sagawa (＠sgwaki) Aug. 2, 2022

Aug. 2, 2022

This is a new try of mciqtz32's video window patch. Comparing to the previous !370, I added a lot of tests regarding the video window behavior. Some portion of the previous patch are dropped to avoid complexity, I'll add that part later. -- v2: mciqtz32: Show the default video window when switching from another one. mciqtz32: Hide the default video window when switching to another one. mciqtz32: Reset the video size when changing video destination. mciqtz32: Correct initial video window dimensions. mciqtz32: Fix MCI_DGV_WHERE_WINDOW behavior. mciqtz32: Correct video window behavior by creating default window. winmm/tests: Add tests for destination of video window. winmm/tests: Add tests for dimensions of video window. winmm/tests: Add tests for window style of video window. mciqtz32: Don't hide video window when stopping. winmm/tests: Add tests for visibility of video window. https://gitlab.winehq.org/wine/wine/-/merge_requests/557

3 15

Re: [PATCH v8 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion
by Chip Davis (＠cdavis5e) Aug. 2, 2022

Aug. 2, 2022

> Is there any way to manually trigger the test bot? You can submit patches yourself to [the testbot](https://testbot.winehq.org/). Note that you will need an account on the testbot in order to do this, which I believe is separate from your account here. > the pipeline fails due to a (more or less) unrelated issue in another dll (msi). They have a function with the same name, DllGetVersion. That is because `DllGetVersion()` is also the name of an optional function supported by COM server DLLs. That's what those definitions in `<shlwapi.h>` are for. > Could it be that the dev defined it in shlwapi.h? The issue is that I can't define my function correctly with this definition (probably the reason why the first implementation used it, wrongly), but they can't use it without (or probably can, but didn't). Or, you could give `cabinet`'s `DllGetVersion()` a new name, and then in `cabinet.spec`, export the new name under the correct name: ``` 2 stdcall -private DllGetVersion(ptr) cabinet_dll_get_version ``` (Why did you remove the ordinals? The commit in which this happened has somehow vanished, but I remember seeing that.) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/564#note_5469

1 0

[PATCH v4 0/6] MR561: winex11.drv: Use DEVMODEW instead of x11drv_display_setting.
by Rémi Bernon Aug. 2, 2022

Aug. 2, 2022

In preparation for moving display placement to win32u, for https://gitlab.winehq.org/wine/wine/-/merge_requests/551. The idea for win32u to enumerate and work with full modes, with driver extra data, to do the display placement logic and validation, and pass the modes array for all adapters at once to the driver to apply them. -- v4: winex11.drv: Constify set_current_mode DEVMODEW parameter. winex11.drv: Iterate display settings using dmDriverExtra. winex11.drv: Use DEVMODEW instead of x11drv_display_setting. winex11.drv: Update devmode dmPosition instead of new_rect. winex11.drv: Use DM_POSITION field bit to mark placed displays. winex11.drv: Use a local desired_rect variable when placing displays. https://gitlab.winehq.org/wine/wine/-/merge_requests/561

2 7

[PATCH v5 0/6] MR550: comctl32/listview: Fix LVS_OWNERDATA list view multi select.
by Angelo Haller (＠szanni) Aug. 2, 2022

Aug. 2, 2022

The following patches fix sending of the LVN_ODSTATECHANGED notification for LVS_OWNERDATA list views, adding more refined tests in the process and fixing various bugs. This is v5, with the added comment in 3/6 as requested on the mailing list. These are the patches sent as a response to v4 by Zhiyi Zhang on the mailing list, unmodified apart from 3/6 or f24f4b7fd. --- Warning: I have had access to the Windows Research Kernel (WRK) 1.2 ~10 years ago. These changes are regarding comctrl32 & tests which are NOT part of the WRK. As outlined in https://wiki.winehq.org/Developer_FAQ this should therefore satisfy the requirement of ONLY submitting patches to components I have NOT had access to. -- v5: comctl32/tests: Add more ownerdata listview tests. https://gitlab.winehq.org/wine/wine/-/merge_requests/550

3 9

[PATCH 0/1] MR571: ntdll: Fix floating point registers not being restored under Wow64.
by Brendan Shanks (＠bshanks) Aug. 1, 2022

Aug. 1, 2022

This bug showed up on macOS Wow64 where XMM registers weren't being restored after an exception. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/571

2 2

Re: [PATCH v8 0/1] MR564: cabinet: implement DllGetVersion & GetDllVersion
by Tobias Görgens Aug. 1, 2022

Aug. 1, 2022

On Mon Aug 1 21:34:19 2022 +0000, Tobias G��rgens wrote: > I'll slowly add/improve commits after the test results are available, I > don't want to delete every commit again lol > Is there any way to manually trigger the test bot? the pipeline fails due to an unrelated issue in another dll. They have a function with the same name, DllGetVersion. Could it be that the dev defined it in shlwapi.h? The issue is that I can't declare my function correctly with this definition (probably the reason why the first implementation used it, wrongly). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/564#note_5462

1 0