On Fri Mar 22 21:12:59 2024 +0000, Zebediah Figura wrote:
> You could probably avoid the duplication with a separate "%empty |
> [int]" rule. Or alternatively use "arrays" and add an explicit check for
> count > 1.
> Is C_INTEGER correct here, or should we allow compile-time constants?
It is correct. We should check for negative values only. Constant evaluated expressions are not allowed.
--
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/739#note_65991
This is a split from MR3572. It's needed for Underworld Island (2150830) video seeking.
Changes compared to the patches in MR3572:
1. Avoid making session state changes when in paused state.
2. Add tests to show that the MF_MEDIA_ENGINE_EVENT_SEEKING notification is most likely blocking because notify->seeking_event_received is TRUE right after a SetCurrentTime() call. Same for MF_MEDIA_ENGINE_EVENT_TIMEUPDATE when in paused state.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5130
First part of Proton shared memory series. The full branch can be seen at https://gitlab.winehq.org/rbernon/wine/-/commits/mr/shared-memories.
--
v28: win32u: Use the desktop shared data for GetCursorPos.
server: Move the last cursor time to the desktop session object.
server: Move the cursor position to the desktop session object.
win32u: Open desktop shared objects from session mapping.
server: Return the desktop object info in get_thread_desktop.
server: Allocate shared session object for desktops.
win32u: Open the global session shared mapping.
include: Add ReadNoFence64 inline helpers.
server: Create a global session shared mapping.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3103
--
v7: explorer: Restore display settings on process exit.
winex11.drv: Process RRNotify events in xrandr14_get_id.
user32/tests: Test that display settings are restored on process exit.
https://gitlab.winehq.org/wine/wine/-/merge_requests/5060
With some intermediate refactoring to make the code simpler.
I believe the failures that happened previously were coming from how NtGdiDdDDIOpenAdapterFromLuid initialized desc->hAdapter in win32u while winex11 was then relying on it. I missed this detail before and it should be working fine now that the vulkan path is only used to retrieve the physical device.
--
v2: win32u: Move D3DKMT vulkan implementation out of winex11.
winex11: Introduce a new find_adapter_from_handle helper.
winex11: Introduce a new get_vulkan_physical_device helper.
winex11: Initialize D3DKMT vulkan instance only once.
win32u: Open adapters in NtGdiDdDDIEnumAdapters2 outside of the display devices lock.
https://gitlab.winehq.org/wine/wine/-/merge_requests/5306
Brendan Shanks (@bshanks) commented about configure.ac:
> WINE_CHECK_SONAME(vulkan, vkGetInstanceProcAddr)
> if test "x$ac_cv_lib_soname_vulkan" = "x"
> then
> - WINE_CHECK_SONAME(MoltenVK, vkGetInstanceProcAddr)
> + WINE_CHECK_SONAME(MoltenVK, vkGetInstanceProcAddr, [AC_DEFINE_UNQUOTED(SONAME_LIBVULKAN,[$ac_cv_lib_soname_MoltenVK])])
This is causing build errors for me:
```
../dlls/win32u/vulkan.c:91:35: error: use of undeclared identifier 'libMoltenVK'
if (!(vulkan_handle = dlopen( SONAME_LIBVULKAN, RTLD_NOW )))
^
include/config.h:773:26: note: expanded from macro 'SONAME_LIBVULKAN'
#define SONAME_LIBVULKAN libMoltenVK.dylib
^
../dlls/win32u/vulkan.c:93:37: error: use of undeclared identifier 'libMoltenVK'
ERR( "Failed to load %s\n", SONAME_LIBVULKAN );
^
include/config.h:773:26: note: expanded from macro 'SONAME_LIBVULKAN'
#define SONAME_LIBVULKAN libMoltenVK.dylib
^
2 errors generated.
make[1]: *** [dlls/win32u/vulkan.o] Error 1
```
I think quotes are needed around `$ac_cv_lib_soname_MoltenVK`, that results in `SONAME_LIBVULKAN` being correctly quoted.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5365#note_65955