July 2022 - Wine-GitLab - list.winehq.org

[PATCH 0/2] MR461: mshtml document mode fixes
by Gabriel Ivăncescu July 15, 2022

July 15, 2022

For the first commit, note that as of now, there's a lot of tests that will fail on native if mshtml_test.exe is renamed to iexplore.exe, even with *all* the FeatureControl values for `iexplore.exe` removed, which leads me to believe there's a lot of other places where native mshtml hardcodes behavior based on the app name being `iexplore.exe`. For the purposes of testing this, please skip all other tests and execute just the test_strict_mode() test, for example (which *should* fail on native if the executable is renamed to `iexplore.exe` instead). I've actually literally searched the *entire* registry on a temporary VM for `iexplore` and removed all values with it, and it still showed such different behavior after the fact, so it must be hardcoded into mshtml. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/461

2 3

[PATCH v2 0/5] MR384: winsock: Support fallback to ICMP over SOCK_DGRAM on Linux
by Paul Gofman (＠gofman) July 15, 2022

July 15, 2022

ICMP over SOCK_DGRAM is already used in ndispproxy.sys. But apps may also use ICMP directly over SOCK_RAW (Hardspace: Shipbreaker is an example) and that requires admin privileges on Linux (or setcap cap_net_raw+ep on wineserver). AF_INET / SOCK_RAW / IPPROTO_ICMP works on Windows without admin privileges. Comments: - IP, ICMP header structures and chksum() functions are taken from ndisproxy.sys:icmp_echo.c; - ICMP over DGRAM, while designed specifically to allow ICMP without admin permissions, still needs some setup Linux, /proc/sys/net/ipv4/ping_group_range controls that. Such socket creation fails on Testbot Debian machines so I left skip() path in test_icmp(); - Linux substitute local port number instead of provided ICMP packet id, while the referenced game depends on reply id matching the id it provided in request. So the last part performs the fixup. -- v2: ntdll: Compute checksum for ICMP over SOCK_DGRAM IP header. ntdll: Fixup ICMP packet id if SOCK_DGRAM fallback is used. ntdll: Support SOCK_RAW / IPPROTO_ICMP fallback over SOCK_DGRAM. ws2_32/tests: Test ICMP. ws2_32: Add ipv4 raw socket protocol info. https://gitlab.winehq.org/wine/wine/-/merge_requests/384

3 7

[PATCH 0/2] MR469: wined3d: Remove dead code directly using driver interface.
by Jacek Caban (＠jacek) July 15, 2022

July 15, 2022

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

3 3

[PATCH 0/1] MR463: msvcr100: Implement invalid_multiple_scheduling exception.
by Torge Matthies (＠tmatthies) July 15, 2022

July 15, 2022

- Moved from mailing list to GitLab - Compile warnings are fixed @piotr -- https://gitlab.winehq.org/wine/wine/-/merge_requests/463

3 3

[PATCH 0/2] MR468: imm32: regression fix
by Jacek Caban (＠jacek) July 15, 2022

July 15, 2022

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

3 3

[PATCH v4 0/4] MR388: d2d1: Introduce D2DERR_NOT_FOUND.
by Ziqing Hui (＠zhui) July 15, 2022

July 15, 2022

Signed-off-by: Ziqing Hui <zhui(a)codeweavers.com> -- v4: d2d1/tests: Test ID2D1BorderTransform. d2d1/tests: Test ID2D1BlendTransform. d2d1/tests: Add tests for ID2D1OffsetTransform. d2d1/tests: Add tests for ID2D1TransformGraph. https://gitlab.winehq.org/wine/wine/-/merge_requests/388

3 6

[PATCH v4 0/2] MR419: win32u/window: Ignore SetWindowPos() for message-only windows.
by Giovanni Mascellani (＠giomasce) July 15, 2022

July 15, 2022

-- v4: win32u/window: Ignore changing WS_EX_TOPMOST for message-only windows. user32/tests: Check that message-only windows ignore WS_EX_TOPMOST. https://gitlab.winehq.org/wine/wine/-/merge_requests/419

2 5

[PATCH 0/4] MR467: comctl32/button: Pass the destination rectangle to GetThemePartSize() when painting check boxes.
by Zhiyi Zhang (＠zhiyi) July 15, 2022

July 15, 2022

The destination rectangle may be smaller than that of theme parts, in which case, the draw size returned is adjusted to fit the destination rectangle. Without passing a destination rectangle to GetThemePartSize(), the function assumes that there is no limit for destination. This fixes winecfg check boxes missing the upper edge in 192 DPI. You may notice that the check box bitmap is slightly shrank and so the shape is a bit off. However, this is what happens on Windows as well when check box height is set to 8 DLUs. GetThemePartSize() without destination rectangles are also used in other controls, but manual tests showed that those controls don't shrink their theme background as check boxes do so they are left unchanged. Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/467

2 4

[PATCH v2 0/1] MR466: po: Update Korean translation.
by Byeongsik Jeon (＠bsjeon) July 15, 2022

July 15, 2022

Signed-off-by: Byeongsik Jeon <bsjeon(a)hanmail.net> -- v2: po: Update Korean translation. https://gitlab.winehq.org/wine/wine/-/merge_requests/466

3 3

[PATCH 0/1] MR466: po: Update Korean translation.
by Byeongsik Jeon (＠bsjeon) July 15, 2022

July 15, 2022

Signed-off-by: Byeongsik Jeon <bsjeon(a)hanmail.net> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/466

2 1

[PATCH 0/3] MR465: server: Various socket fixes.
by Zebediah Figura (＠zfigura) July 15, 2022

July 15, 2022

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

2 3

[PATCH v3 0/5] MR388: d2d1: Introduce D2DERR_NOT_FOUND.
by Ziqing Hui (＠zhui) July 15, 2022

July 15, 2022

Signed-off-by: Ziqing Hui <zhui(a)codeweavers.com> -- v3: d2d1/tests: Test ID2D1BorderTransform. d2d1/tests: Test ID2D1BlendTransform. d2d1: Implement ID2D1OffsetTransform. d2d1/tests: Add tests for ID2D1OffsetTransform. d2d1/tests: Add tests for ID2D1TransformGraph. https://gitlab.winehq.org/wine/wine/-/merge_requests/388

3 8

Re: [PATCH v2 0/1] MR462: secur32: Fix GnuTLS system priority file override. - approved
by Alexandre Julliard (＠julliard) July 14, 2022

July 14, 2022

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

1 0

Re: [PATCH v4 0/1] MR447: explorer: Fix memory leaks. - approved
by Alexandre Julliard (＠julliard) July 14, 2022

July 14, 2022

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

1 0

[PATCH v2 0/1] MR458: winevulkan: Update to VK spec version 1.3.221.
by Georg Lehmann (＠DadSchoorse) July 14, 2022

July 14, 2022

-- v2: winevulkan: Update to VK spec version 1.3.221. https://gitlab.winehq.org/wine/wine/-/merge_requests/458

3 2

Re: [PATCH v2 0/1] MR459: maintainers: Add trailing slashes to directories. - approved
by Alexandre Julliard (＠julliard) July 14, 2022

July 14, 2022

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

1 0

Re: [PATCH v2 0/1] MR453: opengl32/tests: Test wglCopyContext(). - approved
by Alexandre Julliard (＠julliard) July 14, 2022

July 14, 2022

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

1 0

Re: [PATCH v2 0/3] MR448: riched20: Implement ITextDocument::Undo and ITextDocument::Redo (#53289) - approved
by Alexandre Julliard (＠julliard) July 14, 2022

July 14, 2022

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

1 0

Re: [PATCH v2 0/4] MR456: ntdll: Implement socket NtReadFile and NtWriteFile directly on top of recv_socket and send_socket. - approved
by Alexandre Julliard (＠julliard) July 14, 2022

July 14, 2022

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

1 0

[PATCH v2 0/3] MR450: wined3d: Do not pause transform feedback after every draw call.
by Zebediah Figura (＠zfigura) July 14, 2022

July 14, 2022

test_index_buffer_offset() occasionally fails (on some machines and/or Mesa versions) due to a bug in radeonsi. Although the bug has since been fixed upstream, I ended up writing these patches for it, and since they reduce GL calls I'm inclined to believe they're a good idea regardless. The relevant Mesa bug is <https://gitlab.freedesktop.org/mesa/mesa/-/issues/6138>. -- v2: wined3d: Do not pause transform feedback after every draw call. wined3d: Pause transform feedback in wined3d_context_gl_draw_textured_quad(). wined3d: Pause transform feedback in wined3d_context_gl_draw_shaded_quad(). https://gitlab.winehq.org/wine/wine/-/merge_requests/450

3 4

Re: [PATCH v2 0/3] MR449: wined3d: Fix client_map_count refcounting errors. - approved
by Alexandre Julliard (＠julliard) July 14, 2022

July 14, 2022

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

1 0

[PATCH 0/1] MR451: ntdll: Only check casefold flag when filesystem supports it.
by Tuupertunut (＠Tuupertunut) July 14, 2022

July 14, 2022

Currently the wine function `get_dir_case_sensitivity_stat` calls `ioctl` syscall with command `EXT2_IOC_GETFLAGS` and tries to read a flag `EXT4_CASEFOLD_FL` from the result. It does this on all filesystems without checking if the filesystem supports that flag. This patch adds a check: only do the call on filesystems that are known to support `EXT4_CASEFOLD_FL`. To my knowledge these are EXT4 and F2FS. Advantages: - The `EXT4_CASEFOLD_FL` flag is nonstandard and other filesystems may use the same binary value to represent another flag. This patch prevents flag compatibility issues with other filesystems. - This fixes (or works around) an issue on FUSE filesystems using libfuse 3.10 or older and kernel 5.13 or newer, where unsupported `ioctl` calls always succeed and return random (uninitialized) data (see https://github.com/libfuse/libfuse/issues/640) On such setups Wine randomly fails to read files because it thinks that the `EXT4_CASEFOLD_FL` is set when really it is just random data. Ubuntu 22.04 LTS has settled on libfuse 3.10 and kernel 5.15 so this issue will be around for years to come. Disadvantages: - The flag will not be read on possible other current and future filesystems that support it. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/451

3 6

[PATCH 0/6] MR464: Implement _StructuredTaskCollection::_Schedule{,_loc}
by Torge Matthies (＠tmatthies) July 14, 2022

July 14, 2022

- Based on !463 - _RunAndWait and exception handling implemented in later commits @piotr -- https://gitlab.winehq.org/wine/wine/-/merge_requests/464

2 6

Re: [PATCH 0/4] MR384: winsock: Support fallback to ICMP over SOCK_DGRAM on Linux
by Paul Gofman (＠gofman) July 14, 2022

July 14, 2022

> GitLab > > gitlab-to-mail <https://gitlab.winehq.org/project_2_bot> started a new > discussion > <https://gitlab.winehq.org/wine/wine/-/merge_requests/384#note_4039>: > > Zebediah Figura replied on the mailing list: > > |On 7/5/22 14:53, Paul Gofman wrote: @@ -602,9 +623,68 @@ static > NTSTATUS try_recv( int fd, struct async_recv_ioctl *async, ULONG_PTR > *siz } status = (hdr.msg_flags & MSG_TRUNC) ? STATUS_BUFFER_OVERFLOW : > STATUS_SUCCESS; + if (async->fixup_type == > SOCK_PROT_FIXUP_ICMP_OVER_DGRAM) + { + unsigned int tot_len = > sizeof(struct ip_hdr) + ret; + size_t len = hdr.msg_iov[0].iov_len; + > char *buf = hdr.msg_iov[0].iov_base; + struct cmsghdr *cmsg; + struct > ip_hdr ip_h; + + if (ret + sizeof(ip_h) > len) + status = > STATUS_BUFFER_OVERFLOW; + + if (len < sizeof(ip_h)) + { + ret = len; + > } + else + { + ret = min( ret, len - sizeof(ip_h) ); + memmove( buf + > sizeof(ip_h), buf, ret ); + ret += sizeof(ip_h); + } + memset( &ip_h, > 0, sizeof(ip_h) ); + ip_h.v_hl = (4 << 4) | (sizeof(ip_h) >> 2); + > ip_h.tot_len = htons( tot_len ); + ip_h.protocol = 1; + ip_h.saddr = > unix_addr.in.sin_addr.s_addr; + + for (cmsg = CMSG_FIRSTHDR( &hdr ); > cmsg; cmsg = CMSG_NXTHDR( &hdr, cmsg )) + { + if (cmsg->cmsg_level != > IPPROTO_IP) continue; + switch (cmsg->cmsg_type) + { +#if > defined(IP_TTL) + case IP_TTL: + ip_h.ttl = *(BYTE *)CMSG_DATA( cmsg > ); + break; +#endif +#if defined(IP_TOS) + case IP_TOS: + ip_h.tos = > *(BYTE *)CMSG_DATA( cmsg ); + break; +#endif +#if defined(IP_PKTINFO) > + case IP_PKTINFO: + { + struct in_pktinfo *info; + + info = (struct > in_pktinfo *)CMSG_DATA( cmsg ); + ip_h.daddr = info->ipi_addr.s_addr; > + break; + } +#endif + } + } + memcpy( buf, &ip_h, min( sizeof(ip_h), > len )); + } Would you mind making this a helper function? try_recv() > is already large and this seems to be pretty self-contained logic.| |Sure.| > |> if (async->control) > { > + if (async->fixup_type == > SOCK_PROT_FIXUP_ICMP_OVER_DGRAM) > + FIXME( "May return extra control > headers.\n" ); > + What is this for?| When initializing the fixup on server I am setting IP_RECVTTL, IP_RECVTOS and IP_PKTINFO socket options. If the app is calling recvmsg() it may get the headers it didn't request. My reasoning that such a situation doesn't deserve further code complication to avoid but deserves a fixme. > |> @@ -737,17 +818,26 @@ static NTSTATUS sock_recv( HANDLE handle, > HANDLE event, PIO_APC_ROUTINE apc, voi > } > } > > - SERVER_START_REQ( > recv_socket ) > + do > { > - req->force_async = force_async; > - > req->async = server_async( handle, &async->io, event, apc, apc_user, > iosb_client_ptr(io) ); > - req->oob = !!(unix_flags & MSG_OOB); > - > status = wine_server_call( req ); > - wait_handle = > wine_server_ptr_handle( reply->wait ); > - options = reply->options; > > - nonblocking = reply->nonblocking; > - } > - SERVER_END_REQ; > + > SERVER_START_REQ( recv_socket ) > + { > + req->force_async = > force_async; > + req->async = server_async( handle, &async->io, event, > apc, apc_user, iosb_client_ptr(io) ); > + req->oob = !!(unix_flags & > MSG_OOB); > + req->fixup_type = async->fixup_type; > + status = > wine_server_call( req ); > + if (status == > STATUS_MORE_PROCESSING_REQUIRED) > + { > + async->fixup_type = > reply->fixup_type; > + continue; > + } > + wait_handle = > wine_server_ptr_handle( reply->wait ); > + options = reply->options; > > + nonblocking = reply->nonblocking; > + } > + SERVER_END_REQ; > + } > while (status == STATUS_MORE_PROCESSING_REQUIRED); > > alerted = > status == STATUS_ALERTED; > if (alerted) I'm missing something; why do > we need to call this in a loop? Why does the client needs to > communicate the fixup type to the server in the first place?| The async may be initiated on the server and delivered to the process (through SIGUSR1)�� before we set async->fixup_type. It might be not likely in practice, but I double verified that it is possible by injecting Sleep(...) before setting fixup type to async structure from server. So I need to make sure that fixup_type in async structure is correctly set before try_recv() can be reached. So idea is, fixup_type is "no fixup" by default, no extra server call is involved in normal case. But if fixup is required and client does not yet know about it server doesn't�� do anything and just returns fixup type with STATUS_MORE_PROCESSING_REQUIRED. > |> diff --git a/server/protocol.def b/server/protocol.def > index > a8beccaf468..e1e5f3d9faa 100644 > --- a/server/protocol.def > +++ > b/server/protocol.def > @@ -1446,17 +1446,23 @@ enum server_fd_type > > file_pos_t count; /* count of bytes to unlock */ > @END > > - > /* > Perform a recv on a socket */ > @REQ(recv_socket) > int oob; /* are we > receiving OOB data? */ > async_data_t async; /* async I/O parameters > */ > int force_async; /* Force asynchronous mode? */ > + int > fixup_type; /* protocol fixup type returned previously */ > @REPLY > > obj_handle_t wait; /* handle to wait on for blocking recv */ > > unsigned int options; /* device open options */ > int nonblocking; /* > is socket non-blocking? */ > + int fixup_type; /* protocol fixup type > (see below) */ > @END > +enum sock_prot_fixup_type > +{ > + > SOCK_PROT_FIXUP_NONE, > + SOCK_PROT_FIXUP_ICMP_OVER_DGRAM, > +}; > Making this an enum strikes me as odd in general. A bitmask might make > more sense, but frankly I'd just leave it as a dedicated field until > we have something else that needs fixups. Do you have any plans for > introducing other types of fixups?| Well, originally I did something between the lines of this (had icmp_over_dgram field in request / reply). But then I looked at that and it felt odd that general recv_socket() has some special icmp related fields, it felt like naming that "fixup_type" tells what it is without going into very specific ICMP fixup details looking weird at this level. If you feel like it is better to name it, e. g., "icmp_over_dgram" I can change it back. I don't currently have any other types of fixups in mind. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/384#note_4045

3 6

[PATCH v2 0/1] MR427: ntdll: Preserve rbx register when calling DLL entry point on x64.
by Paul Gofman (＠gofman) July 14, 2022

July 14, 2022

Assassin's Creed Origins clobbers rbx in its main module (second of three) TLS callbacks. That is apparent right in the beginning of the TLS callback disassembly when rbx is set to '4' unconditionally without any prior save. That leads to a fault in call_tls_callbacks() which is still in __TRY block and gets handled. However the third TLS callback is not executed and that leads to intermittent hangs later on. It is rather involved to make a TLS callback test in Wine testsuite as there is no portable way to generate a custom TLS callback. I've made a test program (based on the example here: https://lallouslab.net/2017/05/30/using-cc-tls-callbacks-in-visual-studio-w…) and compiled it with MSVC. The source code is here: https://gist.github.com/gofman/3287a953bcab3a5c888a8d494461cb8a. The program calls all the callbacks on Windows 10 here. There is also a similar wrapper already there for i386 on another occasion. -- v2: ntdll: Preserve rbx register when calling DLL entry point on x64. https://gitlab.winehq.org/wine/wine/-/merge_requests/427

2 2

[PATCH 0/1] MR462: secur32: Fix GnuTLS system priority file override.
by Hans Leidekker (＠hans) July 14, 2022

July 14, 2022

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53344 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/462

2 1

Re: [PATCH 0/1] MR363: user32/tests: Test for window exposure behaviours.
by Jinoh Kang (＠iamahuman) July 14, 2022

July 14, 2022

Jinoh Kang (@iamahuman) commented about dlls/user32/tests/msg.c: > DeleteObject( hrgn2 ); > } > > +static void visualize_region_differences( HWND hwnd, HWND hother, HRGN hrgn_expect, HRGN hrgn_actual ) I included this function since it was really helpful for me while testing debugging the implementation--it eliminates the need to interpret the rectangle list manually. I can remove this if it clutters the test, though. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/363#note_4317

1 0

Re: [PATCH 0/2] MR314: ntdll/tests: Preserve EFLAGS across syscall on x64. - closed
by Jinoh Kang (＠iamahuman) July 14, 2022

July 14, 2022

This merge request was closed by Jinoh Kang. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/314

1 0

Re: [PATCH 0/2] MR314: ntdll/tests: Preserve EFLAGS across syscall on x64.
by Jinoh Kang (＠iamahuman) July 14, 2022

July 14, 2022

Superseded by MR !282 (probably). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/314#note_4315

1 0

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) July 14, 2022

July 14, 2022

Jinoh Kang (@iamahuman) commented about server/async.c: > return woken; > } > > +static int cancel_blocking( struct process *process, struct thread *thread, client_ptr_t iosb ) > +{ > + struct async *async; > + int woken = 0; > + > +restart: > + LIST_FOR_EACH_ENTRY( async, &process->asyncs, struct async, process_entry ) > + { > + if (async->terminated || async->canceled) continue; There is a pre-existing bug where cancellation requests are ignored for alerted asyncs (i.e. async instances that are not actually completed and can go back from the "terminated" state to the pending state). Perhaps it's worth addressing that bug first? https://www.winehq.org/pipermail/wine-devel/2022-March/210053.html -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3836

2 1

[PATCH 0/1] MR459: maintainers: Add trailing slashes to directories.
by Jeremy White (＠jwhite) July 14, 2022

July 14, 2022

The gitlab scripts operate without access to the wine tree. Requiring a trailing slash on patterns simplifies the logic to automatically assign reviewers to MRs. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/459

2 1

[PATCH v3 0/2] MR419: win32u/window: Ignore SetWindowPos() for message-only windows.
by Giovanni Mascellani (＠giomasce) July 14, 2022

July 14, 2022

-- v3: win32u/window: Ignore changing WS_EX_TOPMOST for message-only windows. user32/tests: Check that message-only windows ignore WS_EX_TOPMOST. https://gitlab.winehq.org/wine/wine/-/merge_requests/419

2 2

[PATCH 0/1] MR458: winevulkan: Update to VK spec version 1.3.221.
by Georg Lehmann (＠DadSchoorse) July 14, 2022

July 14, 2022

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

2 1

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) July 14, 2022

July 14, 2022

On Thu Jul 14 06:48:37 2022 +0000, Daniel Lehman wrote: > in the exchange, it sounded like you had or were working on a fix. were > you asking me to fix it or wait for your fix? Well, it was actually more of a question than a suggestion. I just wondered if it would be a good idea to get I/O cancellation right first, lest rolling in another cancellation function would result in some unknown regression due to falsely indicated success from that function. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_4299

1 0

[PATCH v5 0/1] MR398: xactengine3_7/tests: Test notifications when loading a wave bank.
by Giovanni Mascellani (＠giomasce) July 14, 2022

July 14, 2022

"The King of Fighters '98 Ultimate Match Final Edition" depends on this behavior. At least, the build I have is; it seems other builds are not. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> -- v5: xactengine3_7/tests: Test notifications when loading a wave bank. https://gitlab.winehq.org/wine/wine/-/merge_requests/398

3 2

[PATCH 0/2] MR455: rpcrt4/tests: Add I_RpcBindingInqLocalClientPID() tests.
by Zhiyi Zhang (＠zhiyi) July 14, 2022

July 14, 2022

Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/455

3 3

[PATCH 0/3] MR448: riched20: Implement ITextDocument::Undo and ITextDocument::Redo (#53289)
by Jinoh Kang (＠iamahuman) July 14, 2022

July 14, 2022

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

3 4

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Daniel Lehman (＠dlehman25) July 14, 2022

July 14, 2022

On Sun Jul 10 17:41:42 2022 +0000, Jinoh Kang wrote: > There is a pre-existing bug where cancellation requests are ignored for > alerted asyncs (i.e. async instances that are not actually completed and > can go back from the "terminated" state to the pending state). Perhaps > it's worth addressing that bug first? https://www.winehq.org/pipermail/wine-devel/2022-March/210053.html in the exchange, it sounded like you had or were working on a fix. were you asking me to fix it or wait for your fix? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_4275

1 0

[PATCH 0/3] MR457: ntdll/tests: Add RtlFirstFreeAce tests
by Alistair Leslie-Hughes (＠alesliehughes) July 14, 2022

July 14, 2022

Signed-off-by: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/457

2 3

[PATCH 0/4] MR456: ntdll: Implement socket NtReadFile and NtWriteFile directly on top of recv_socket and send_socket.
by Zebediah Figura (＠zfigura) July 14, 2022

July 14, 2022

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

2 4

[PATCH 0/7] MR454: shell32: Correctly check SVSI_EDIT.
by Zhiyi Zhang (＠zhiyi) July 14, 2022

July 14, 2022

The value of SVSI_EDIT is 0x3. Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/454

2 7

[PATCH 0/1] MR453: opengl32/tests: Test wglCopyContext().
by Zhiyi Zhang (＠zhiyi) July 14, 2022

July 14, 2022

Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/453

2 1

[PATCH v3 0/1] MR447: explorer: Fix memory leaks.
by Zhiyi Zhang (＠zhiyi) July 14, 2022

July 14, 2022

Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> -- v3: explorer: Fix memory leaks. https://gitlab.winehq.org/wine/wine/-/merge_requests/447

2 2

[PATCH v2 0/1] MR447: explorer: Fix memory leaks.
by Zhiyi Zhang (＠zhiyi) July 14, 2022

July 14, 2022

Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> -- v2: opengl32/tests: Test wglCopyContext(). https://gitlab.winehq.org/wine/wine/-/merge_requests/447

2 1

[PATCH 0/1] MR452: win32u: Send WM_NCPAINT as notify message in send_ncpaint().
by Paul Gofman (＠gofman) July 14, 2022

July 14, 2022

The test shows that during RedrawWindow( ... RDW_ERASENOW ...) WM_NCPAINT delivery is not waited on the RedrawWindow() thread if the window is on another thread. It is still delivered directly to window procedure without showing up in PeekMessage(), and is processed synchronously if window is on the same thread. That corresponds to SendNotifyMessage() behaviour. Fixes Warhammer: Vermintide 2 hanging during the game start. The game redraws desktop window (RedrawWindow(NULL, NULL, NULL, RDW_INVALIDATE | RDW_ERASE | RDW_ALLCHILDREN | RDW_ERASENOW | RDW_FRAME)). During that all the windows are getting updated and might be hitting that SendMessage in question. This hangs when it comes to the game's launcher's (minimized) window which is another process and doesn't process message queue at this point. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/452

2 1

[PATCH 0/3] MR450: wined3d: Do not pause transform feedback after every draw call.
by Zebediah Figura (＠zfigura) July 14, 2022

July 14, 2022

test_index_buffer_offset() occasionally fails (on some machines and/or Mesa versions) due to a bug in radeonsi. Although the bug has since been fixed upstream, I ended up writing these patches for it, and since they reduce GL calls I'm inclined to believe they're a good idea regardless. The relevant Mesa bug is <https://gitlab.freedesktop.org/mesa/mesa/-/issues/6138>. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/450

2 3

[PATCH 0/3] MR449: wined3d: Fix client_map_count refcounting errors.
by Zebediah Figura (＠zfigura) July 14, 2022

July 14, 2022

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

2 3

Re: [PATCH v5 0/5] MR399: XMLHttpRequest improvements - approved
by Alexandre Julliard (＠julliard) July 13, 2022

July 13, 2022

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

1 0

Re: [PATCH v2 0/1] MR443: wined3d: Defer destroying Vulkan compute pipelines. - approved
by Alexandre Julliard (＠julliard) July 13, 2022

July 13, 2022

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

1 0

[PATCH v2 0/6] MR265: kernelbase: Implement PrefetchVirtualMemory.
by Jinoh Kang (＠iamahuman) July 13, 2022

July 13, 2022

Signed-off-by: Jinoh Kang <jinoh.kang.kr(a)gmail.com> -- v2: kernelbase: Implement PrefetchVirtualMemory. kernel32/tests: Add tests for PrefetchVirtualMemory. ntdll: Implement NtSetInformationVirtualMemory (intra-process case). ntdll/tests: Add tests for NtSetInformationVirtualMemory. include: Add definition for NtSetInformationVirtualMemory. include: Move WIN32_MEMORY_RANGE_ENTRY from winnt.h to winbase.h. https://gitlab.winehq.org/wine/wine/-/merge_requests/265

3 7