On Tue Apr 11 18:16:38 2023 +0000, Zebediah Figura wrote:
> This seems unrelated and unnecessary.
Yes, this was just to make sure not to release the same buffer twice.
At least, with this assignment removed, my test program from bugzilla behaves the same.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2439#note_30160
Jinoh Kang (@iamahuman) commented about dlls/winex11.drv/xim.c:
> - XNStatusAttributes, status,
> - XNClientWindow, win,
> - XNFocusWindow, win,
> - XNDestroyCallback, &destroy,
> - NULL);
> - }
> - else
> - {
> - xic = XCreateIC(xim,
> - XNInputStyle, ximStyle,
> - XNClientWindow, win,
> - XNFocusWindow, win,
> - XNDestroyCallback, &destroy,
> - NULL);
> - }
> + preedit = XVaCreateNestedList( 0, XNFontSet, fontSet,
Why is this code assuming that `fontSet` is not `NULL`, regardless of `(ximStyle & (XIMPreeditNothing | XIMPreeditNone)) == 0`? It appears to me that the code that actually sets `x11drv_thread_data()->font_set` does so only if the condition is true.
(Yes, I'm aware that later commit changes font set handling, but bisection points to this one as the first bad commit.)
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2562#note_30159
On my Steam Deck, this reduces the time it takes to initialize the wg_parser radically (by around 1 second). This helps in the game WILD HEARTS, which doesn't present while loading help videos during gameplay, causing large stutters.
Because GStreamer can randomly access the file with no known pattern on our side, I opted to implement this by buffering 4 chunks so that interleaved reads to different areas of the file don't cause us to discard and reload cached data more than we need to.
--
v5: winegstreamer: Cache wg_parser input data.
https://gitlab.winehq.org/wine/wine/-/merge_requests/2390
In the current implementation, the main UI will be blocked
after the uninstaller is launched. Put the uninstall process
in a thread to avoid that, and popup a message while trying
to launch multiple uninstallers.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2571
Based on !2524.
This adds the remaining stubs needed for Crazy Machines 3 to work and decode preview images correctly in online mode.
--
v7: msvcr100: Use Context blocking functions in reader_writer_lock class.
msvcr100: Add ExternalContextBase::Unblock implementation.
https://gitlab.winehq.org/wine/wine/-/merge_requests/1979