- wine-gitlab - winehq.org

Re: [PATCH v2 0/1] MR9234: ntdll: Call cache_inproc_sync() inside the lock.
by Elizabeth Figura (＠zfigura) 20 Oct '25

20 Oct '25

> Nvm, I guess the entire sync retrieval and caching needs to be atomic. Does it? I'm not sure I see how; I think you're right that the mmap doesn't need to be in the lock. I don't know that it's really important, given how rare it'll be in practice, but theoretically it could be outside the lock. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9234#note_119099

1 0

Re: [PATCH v2 0/1] MR9234: ntdll: Call cache_inproc_sync() inside the lock.
by Rémi Bernon (＠rbernon) 20 Oct '25

20 Oct '25

On Mon Oct 20 20:56:40 2025 +0000, Rémi Bernon wrote: > Sorry to have missed that. > Fwiw I think the lock (and probably a different one, there's no need to > lock the fd cache) should be taken again within `cache_inproc_sync` but > only for the things where it's actually necessary. > It does rather costly calls, like `mmap` and there's things like > `InterlockedCompareExchangePointer` in it which are rather pointless otherwise. Nvm, I guess the entire sync retrieval and caching needs to be atomic. Anyway, still I think the compare exchange looks confusing if it always happens inside a lock. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9234#note_119098

1 0

[PATCH 0/1] MR9235: Draft: winex11: Retrieve XIM result as UTF8.
by Bernhard Kölbl (＠besentv) 20 Oct '25

20 Oct '25

The goal here is to make XIM input locale independent, so changing LC_MESSAGES and LC_CTYPE (to change code pages on PE side) can be done without breaking the IME, when set locale isn't set up on the host system. To achieve this, I switched XmbLookupString to Xutf8LookupString, because it *seems* to achieve what I wrote above. Marked as draft because idk if this makes sense at all. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9235

3 2

Re: [PATCH v2 0/1] MR9234: ntdll: Call cache_inproc_sync() inside the lock.
by Rémi Bernon (＠rbernon) 20 Oct '25

20 Oct '25

Sorry to have missed that. Fwiw I think the lock (and probably a different one, there's no need to lock the fd cache) should be taken again within `cache_inproc_sync` but only for the things where it's actually necessary. It does rather costly calls, like `mmap` and there's things like `InterlockedCompareExchangePointer` in it which are rather pointless otherwise. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9234#note_119096

1 0

Re: [PATCH v2 0/1] MR9234: ntdll: Call cache_inproc_sync() inside the lock.
by Elizabeth Figura (＠zfigura) 20 Oct '25

20 Oct '25

The original revision submitted as !7226 did not have this bug; it was introduced in !9091. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9234#note_119091

1 0

[PATCH 0/1] MR9234: ntdll: Call cache_inproc_sync() inside the lock.
by Elizabeth Figura (＠zfigura) 20 Oct '25

20 Oct '25

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

2 1

[PATCH 0/1] MR9233: user32: Move GetWindowDisplayAffinity() stub to win32u.
by Paul Gofman (＠gofman) 20 Oct '25

20 Oct '25

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

2 1

[PATCH 0/1] MR9232: include: Implement C_ASSERT() using the C23 static_assert() if available.
by Brendan Shanks (＠bshanks) 20 Oct '25

20 Oct '25

C23 deprecates _Static_assert(). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9232

2 1

[PATCH 0/1] MR9223: wine.inf: Create [fonts] section in win.ini for 16-bit apps.
by Dmitry Timoshkov (＠dmitry) 20 Oct '25

20 Oct '25

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

3 4

[PATCH v2 0/1] MR9104: scrrun: Implement MoveFolder().
by Elizabeth Figura (＠zfigura) 20 Oct '25

20 Oct '25

-- v2: scrrun: Implement MoveFolder(). https://gitlab.winehq.org/wine/wine/-/merge_requests/9104

2 1

2025

2024

2023

2022

wine-gitlab