- Wine-gitlab - list.winehq.org

[PATCH v5 0/5] MR874: mfmediaengine: Avoid vertically flipping video.
by Rémi Bernon 14 Oct '22

14 Oct '22

This shows that media session samples are correctly oriented, but that mfmediaengine renders them upside down. -- v5: mfmediaengine: Remove vertical flipping of video frames. mfmediaengine/tests: Check IMFMediaEngine_TransferVideoFrames output orientation. mfmediaengine/tests: Pass a device manager and output format to create_media_engine. mf/tests: Check sample grabber RGB / NV12 orientation. mf/tests: Factor test grabber callback implementations. https://gitlab.winehq.org/wine/wine/-/merge_requests/874

3 9

[PATCH 0/1] MR1062: d2d1: Use a gcc pragma instead of a compiler flag for -fexcess-precision.
by Alexandre Julliard (＠julliard) 14 Oct '22

14 Oct '22

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

4 3

Re: [PATCH v5 0/5] MR944: win32u: Make desktop resize logic asynchronous.
by Rémi Bernon 14 Oct '22

14 Oct '22

On Thu Oct 13 19:11:33 2022 +0000, Rémi Bernon wrote: > Sorry, I'll have a look. I'm having a hard time reproducing it, is this with some specific tests? The first commit indeed changes some behavior, where it was only repositioning the current process windows before and is now repositioning all windows from the desktop thread. I suspect that it could be about the message being sent to the desktop, which is now waiting for all windows to reposition by broadcasting a message itself. If they aren't actively processing messages, it could probably lock up. I created https://gitlab.winehq.org/wine/wine/-/merge_requests/1056 to address that issue, but I cannot confirm if it fixes anything. Other than that I experienced some random and seldom crashes running d3d9:device for instance, but I'm not sure to see how it's related. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/944#note_10671

1 0

[PATCH v2 0/5] MR29: vkd3d-shader/hlsl: Support complex implicit casts, complex explicit casts and complex broadcasts. (PART 1/2)
by Francisco Casas (＠fcasas) 14 Oct '22

14 Oct '22

Hello, this is the first part of a patch series to support: * Complex broadcasts. * Complex implicit casts between component-wise equal types. * Complex explicit casts between component-wise compatible types. This first part of the series includes mostly tests. Following patches in: https://gitlab.winehq.org/fcasas/vkd3d/-/commits/complex_broadcasts_2/ This supersedes !16 . -- v2: tests: Test explicit casts between types that are component-wise tests: Test implicit casts between types that are equal component-wise. vkd3d-shader/hlsl: Always go through implicit conversion in assignments. tests: Test for invalid complex broadcasts. tests: Set ULPs to 2 in normalize() test. https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/29

5 10

[PATCH 0/1] MR1057: wbemprox: Add HyperVisorPresent property to win32_ComputerSystem
by Louis Lenders (＠xe) 14 Oct '22

14 Oct '22

Native Access 2 queries for this property -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1057

5 10

[PATCH 0/1] MR1064: wbemprox: Add HypervisorPresent property to win32_ComputerSystem.
by Louis Lenders (＠xe) 14 Oct '22

14 Oct '22

Native Access 2 queries for this property -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1064

4 4

[PATCH 0/1] MR1060: include: Remove interfaces already define in msxml6.idl
by Alistair Leslie-Hughes (＠alesliehughes) 14 Oct '22

14 Oct '22

The removed interfaces cannot be found in the windows SDK, so making it one step close to being the same. Signed-off-by: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1060

3 2

[PATCH 0/1] MR1058: po: Update Finnish translation.
by Lauri Kenttä 14 Oct '22

14 Oct '22

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

2 2

[PATCH 0/1] MR1056: win32u: Asynchronously send the WM_DISPLAYCHANGE to the desktop window.
by Rémi Bernon 14 Oct '22

14 Oct '22

I'm not sure this would solve the problem from https://gitlab.winehq.org/wine/wine/-/merge_requests/944?commit_id=bc09f081… as I'm not yet able to reproduce it, but I suspect it might help. In any case I don't think there's any good reason for this call to be synchronous with send_message. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1056

2 2

Re: [PATCH v2 0/10] MR969: combase: CoMsgWaitForMultipleHandles message pump.
by Kevin Puetz (＠puetzk) 13 Oct '22

13 Oct '22

> I'd suggest moving some of the existing code into helper functions first. You asked for that the first time too. I've tried, but I just couldn't find any good seams to carve it along besides consolidating the PeekMessage calls (e7e4c3d8eee915975e958521b8caffe94ef7a34d). The 3 major hunks (message pump, the IMessageFilter_MessagePending stuff, and polling the handles for completions/APCs) all tie into how the loop exits and the post conditions (re-posting QuitMessage, writing `*index` (or leaving it unmodified), errors, translating previous Wait* result), and the order is quite important (because of side-effects like consuming/preserving QS_ALLPOSTMESSAGE). If you have any ideas I'd love to hear them. I could certainly pull those 3 major hunks out into functions, but when they have get passed in pointers to a bunch of outside-the-loop variables, and their return value ends up being decoded into ends up with stuff like `if(hr == S_FALSE) break; else if(FAILED(hr)) goto err;` then the flow control just turned into spaghetti I hated even more than what it started as. > ten commits per MR is rather too many (especially when they're non trivial). Yeah, for lack of coming up with decent helper functions I tried to dice it *really* finely separate testable commits that changed only one behavior. I think they are all correct and improvements on the status quo, but the very first commit fixes the same bug (short-timeout race) for MTAs that the last finally fixes for STAs, so it felt weird to propose actually merging it partially. And yet that's really the order in which the steps make the most sense to read (in terms of minimizing extraneous code change); getting the MTA out of the loop, so that it's *only* the mesage loops, then building toward toward the case where everything is happening in the right order and there's a correct place to put the timeout. The part that I dislike about this series is temporarily introducing use of MWMO_INPUTAVAILABLE; that was a detour born of trying to split the last 3 commits apart from each other, so there's only one behavior change at a time. 1. WAIT_OBJECT_* signaled after dispatching a message 2. WAIT_IO_COMPLETION after dispatching a message (rather than just once at the start) 3. making sure to we're out of relevant messages before escalating WAIT_TIMOUET to RPC_S_CALLPENDING). Required some careful work to *preserve* bugs until the next commit. I guess there's a few things in the middle that basically just refactor/add tests without changing any observable behavior, so I'll move those to the front and make a separate MR, just to cut down the size of this one... And I guess I could also put a lot of the new tests in as just new `todo_wine` tests, separate from the actual changes that fix them. That would at least get a lot of lines of code out of this MR, though not much of the conceptual complexity. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/969#note_10623

1 0