> Mostly because it's not terribly practical to check in anything that outputs more than one or two lines. I think we should adopt the vkd3d_bytecode_buffer scheme and track the status on the vkd3d_string_buffer instead of attempting to propagate errors up from vkd3d_string_buffer_resize().
Agreed. What I don't like is that no check of allocation failure is done at all.
--
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/310#note_43932
--
v2: dinput: Get rid of duplicated winetest helpers.
include: Avoid using strcmp directly in wine/test.h.
include: Introduce winetest wrappers for vprintf and GetTickCount.
include: Pass a temporary buffer to winetest_elapsed.
include: Rename winetest_printf to winetest_print_location.
include: Initialize winetest global counters to 0.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3691
Fixes a bug where the statusbar doesn't correctly paint when the parent window is resized. This can be seen in wine notepad.
--
v5: comctl32/status: Invalidate entire status bar on WM_SIZE.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3667
> > > POSIX allows to use the `%m$` syntax and avoid repeat many times the same argument, but unfortunately the standard `printf()` doesn't...
> >
> > Sadly, it seems that MinGW in ANSI mode doesn't implement this feature, otherwise I'd be glad to depend on it anyway.
>
> Could we perhaps use \_vsprintf_p() on the only non-POSIX platform we care about?
I wasn't aware that existed. I'll have to check, but it sounds promising. Goodness only knows why they had to add a separate export for that.
--
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/310#note_43922
Currently handle_foreground_lost() unacquires devices, but nothing on the path of NOTIFY_FOREGROUND_LOST unregisters raw input, and dinput continues hogging all the input preventing focused window (e. g., message box) from getting it.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3688