March 12, 2026
1:52 p.m.
On Thu Mar 12 18:48:27 2026 +0000, Brendan McGrath wrote:
Yes. I originally had two `LeaveCriticalSection` (one for each path) but decided that was unnecessary as the `IMFMediaEngineNotify_EventNotify` simply writes the event to a queue (thus is brief). But seems I only removed the one `LeaveCriticalSection` when I made that change. There is no queue, it's a user callback. Please leave it the way it was, I believe I tested this manually before.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/10306#note_132077