April 2025 - Wine-GitLab - list.winehq.org

Re: [PATCH v4 0/1] MR7806: winewayland: Implement relative motion accumulator
by Alexandros Frantzis (＠afrantzis) April 17, 2025

April 17, 2025

Alexandros Frantzis (@afrantzis) commented about dlls/winewayland.drv/wayland_pointer.c: > pointer_handle_axis_discrete > }; > > +/********************************************************************** > + * wayland_motion_delta_to_window > + * > + * Converts the surface-local delta to window (logical) coordinate delta. > + */ > +static void wayland_motion_delta_to_window(struct wayland_surface *surface, > + double surface_x, double surface_y, > + double *window_x, double *window_y) Nit: Align with parameters above -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7806#note_101262

1 0

Re: [PATCH v2 0/1] MR7833: mf: Update state and start clock for both paused and stopped.
by Nikolay Sivov (＠nsivov) April 17, 2025

April 17, 2025

I'm using this demo player based on mfplay https://github.com/microsoft/Windows-classic-samples/tree/main/Samples/Win7…, and start->pause->restart works fine there for the audio. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7833#note_101261

1 0

Re: [PATCH v2 0/1] MR7833: mf: Update state and start clock for both paused and stopped.
by Nikolay Sivov (＠nsivov) April 17, 2025

April 17, 2025

This is already working using OnClockRestart(). How does it break in your testing? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7833#note_101260

1 0

[PATCH 0/4] MR7826: ntdll: Fix some more Valgrind warnings.
by Sven Baars (＠sbaars) April 17, 2025

April 17, 2025

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

3 5

[PATCH v3 0/7] MR7836: bcrypt: Partial BCryptKeyDerivation implementation (PBKDF2).
by Piotr Caban (＠piotr) April 17, 2025

April 17, 2025

-- v3: bcrypt: Add BCryptKeyDerivation partial implementation (PBKDF2 algorithm). https://gitlab.winehq.org/wine/wine/-/merge_requests/7836

2 8

[PATCH v2 0/7] MR7836: bcrypt: Partial BCryptKeyDerivation implementation (PBKDF2).
by Piotr Caban (＠piotr) April 17, 2025

April 17, 2025

-- v2: bcrypt: Add BCryptKeyDerivation partial implementation (PBKDF2 algorithm). bcrypt: Reorganize hash_handle_from_desc helper so it can be reused. bcrypt: Handle PBKDF2 in BCryptGenerateSymmetricKey. bcrypt: Handle PBKDF2 in BCryptGetProperty. bcrypt: Add PBKDF2 algorithm provider. bcrypt: Fix BcryptDeriveKeyPBKDF2 with NULL salt. include: Add some PBKDF2 related definitions. https://gitlab.winehq.org/wine/wine/-/merge_requests/7836

3 9

[PATCH 0/7] MR7836: bcrypt: Partial BCryptKeyDerivation implementation (PBKDF2).
by Piotr Caban (＠piotr) April 17, 2025

April 17, 2025

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

3 9

Re: [PATCH v5 0/2] MR7623: winegstreamer: Use provided PTS and duration in video_decoder.
by Brendan McGrath (＠redmcg) April 17, 2025

April 17, 2025

On Thu Apr 17 07:54:50 2025 +0000, Rémi Bernon wrote: > Have you tried GstVideoTimeCodeMeta / do you know what it does? No, I didn't look at that. But taking a look now, it looks like it might be good for Video PTS/DTS, but wouldn't work for audio. I'm not sure for duration either. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7623#note_101244

1 0

Re: [PATCH v5 0/2] MR7623: winegstreamer: Use provided PTS and duration in video_decoder.
by Rémi Bernon April 17, 2025

April 17, 2025

On Wed Apr 16 08:20:27 2025 +0000, Brendan McGrath wrote: > Oops (again). I just tried using the `preserve_timestamps` boolean and > it doesn't work. The `timestamp/x-wg-transform` is a flag that lets > `video_decoder.c` know that an input timestamp has been provided (and > that it should use it). It will otherwise fall back to the behavior > defined by `provide_timestamps`. > But if we want to get rid of the need for the GUID, I guess we could > move the existing logic in `video_decoder.c` (for generating our own > timestamp) in to `wg_transform.c`. Hmm, I'm not sure that moving timestamp generation to the unix side is better. Can't we simply fill the sample timestamps from the input provided timestamps instead of the GStreamer-generated timestamps we don't really care about? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7623#note_101242

1 0

Re: [PATCH v5 0/2] MR7623: winegstreamer: Use provided PTS and duration in video_decoder.
by Rémi Bernon April 17, 2025

April 17, 2025

On Wed Apr 16 01:09:24 2025 +0000, Brendan McGrath wrote: > On well formed media, that `dts < 0` should only ever be true once (as > after adding ts_offset, we should never get a value below 0 again). It's > mainly there in case we get media with malformed timestamps (and dts > goes backwards). IMO the offset should only be updated once, and there should be a check and warning/fixme message if that assumption is broken. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7623#note_101241

1 0