June 2024 - Wine-commits - list.winehq.org

Rémi Bernon : server: Store the cursor clip rect in the shared data.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: 496f6631574aea0bc372276a9547afa7f53af5ca URL: https://gitlab.winehq.org/wine/wine/-/commit/496f6631574aea0bc372276a9547af… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Feb 19 22:25:01 2024 +0100 server: Store the cursor clip rect in the shared data. Based on a patch by Huw Davies. --- include/wine/server_protocol.h | 3 ++- server/protocol.def | 1 + server/queue.c | 26 ++++++++++++++++---------- server/user.h | 1 - server/winstation.c | 4 ++++ 5 files changed, 23 insertions(+), 12 deletions(-) diff --git a/include/wine/server_protocol.h b/include/wine/server_protocol.h index df31a957486..162d00bd65f 100644 --- a/include/wine/server_protocol.h +++ b/include/wine/server_protocol.h @@ -888,6 +888,7 @@ struct shared_cursor int x; int y; unsigned int last_change; + rectangle_t clip; }; typedef volatile struct @@ -6567,7 +6568,7 @@ union generic_reply /* ### protocol_version begin ### */ -#define SERVER_PROTOCOL_VERSION 813 +#define SERVER_PROTOCOL_VERSION 814 /* ### protocol_version end ### */ diff --git a/server/protocol.def b/server/protocol.def index 20ed82982dd..0592dddaa97 100644 --- a/server/protocol.def +++ b/server/protocol.def @@ -904,6 +904,7 @@ struct shared_cursor int x; /* cursor position */ int y; unsigned int last_change; /* time of last position change */ + rectangle_t clip; /* cursor clip rectangle */ }; typedef volatile struct diff --git a/server/queue.c b/server/queue.c index c2e957cfd21..e06f6afe0f7 100644 --- a/server/queue.c +++ b/server/queue.c @@ -410,7 +410,8 @@ static struct message *alloc_hardware_message( lparam_t info, struct hw_msg_sour static int is_cursor_clipped( struct desktop *desktop ) { - rectangle_t top_rect, clip_rect = desktop->cursor.clip; + const desktop_shm_t *desktop_shm = desktop->shared; + rectangle_t top_rect, clip_rect = desktop_shm->cursor.clip; get_top_window_rectangle( desktop, &top_rect ); return !is_rect_equal( &clip_rect, &top_rect ); } @@ -471,8 +472,8 @@ static int update_desktop_cursor_pos( struct desktop *desktop, user_handle_t win int updated; unsigned int time = get_tick_count(); - x = max( min( x, desktop->cursor.clip.right - 1 ), desktop->cursor.clip.left ); - y = max( min( y, desktop->cursor.clip.bottom - 1 ), desktop->cursor.clip.top ); + x = max( min( x, desktop_shm->cursor.clip.right - 1 ), desktop_shm->cursor.clip.left ); + y = max( min( y, desktop_shm->cursor.clip.bottom - 1 ), desktop_shm->cursor.clip.top ); SHARED_WRITE_BEGIN( desktop_shm, desktop_shm_t ) { @@ -537,29 +538,34 @@ static void get_message_defaults( struct msg_queue *queue, int *x, int *y, unsig void set_clip_rectangle( struct desktop *desktop, const rectangle_t *rect, unsigned int flags, int reset ) { const desktop_shm_t *desktop_shm = desktop->shared; - rectangle_t top_rect; + rectangle_t top_rect, new_rect; unsigned int old_flags; int x, y; get_top_window_rectangle( desktop, &top_rect ); if (rect) { - rectangle_t new_rect = *rect; + new_rect = *rect; if (new_rect.left < top_rect.left) new_rect.left = top_rect.left; if (new_rect.right > top_rect.right) new_rect.right = top_rect.right; if (new_rect.top < top_rect.top) new_rect.top = top_rect.top; if (new_rect.bottom > top_rect.bottom) new_rect.bottom = top_rect.bottom; if (new_rect.left > new_rect.right || new_rect.top > new_rect.bottom) new_rect = top_rect; - desktop->cursor.clip = new_rect; } - else desktop->cursor.clip = top_rect; + else new_rect = top_rect; + + SHARED_WRITE_BEGIN( desktop_shm, desktop_shm_t ) + { + shared->cursor.clip = new_rect; + } + SHARED_WRITE_END; old_flags = desktop->cursor.clip_flags; desktop->cursor.clip_flags = flags; /* warp the mouse to be inside the clip rect */ - x = max( min( desktop_shm->cursor.x, desktop->cursor.clip.right - 1 ), desktop->cursor.clip.left ); - y = max( min( desktop_shm->cursor.y, desktop->cursor.clip.bottom - 1 ), desktop->cursor.clip.top ); + x = max( min( desktop_shm->cursor.x, new_rect.right - 1 ), new_rect.left ); + y = max( min( desktop_shm->cursor.y, new_rect.bottom - 1 ), new_rect.top ); if (x != desktop_shm->cursor.x || y != desktop_shm->cursor.y) set_cursor_pos( desktop, x, y ); /* request clip cursor rectangle reset to the desktop thread */ @@ -3745,7 +3751,7 @@ DECL_HANDLER(set_cursor) reply->new_x = desktop_shm->cursor.x; reply->new_y = desktop_shm->cursor.y; - reply->new_clip = desktop->cursor.clip; + reply->new_clip = desktop_shm->cursor.clip; reply->last_change = desktop_shm->cursor.last_change; } diff --git a/server/user.h b/server/user.h index a20aff99284..7ad1e82ae54 100644 --- a/server/user.h +++ b/server/user.h @@ -56,7 +56,6 @@ struct winstation struct global_cursor { - rectangle_t clip; /* cursor clip rectangle */ unsigned int clip_flags; /* last cursor clip flags */ user_handle_t win; /* window that contains the cursor */ }; diff --git a/server/winstation.c b/server/winstation.c index 87981f64167..3e08f4b3337 100644 --- a/server/winstation.c +++ b/server/winstation.c @@ -310,6 +310,10 @@ static struct desktop *create_desktop( const struct unicode_str *name, unsigned shared->cursor.x = 0; shared->cursor.y = 0; shared->cursor.last_change = 0; + shared->cursor.clip.left = 0; + shared->cursor.clip.top = 0; + shared->cursor.clip.right = 0; + shared->cursor.clip.bottom = 0; } SHARED_WRITE_END; }

1 0

Rémi Bernon : server: Get rid of the global cursor structure.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: 472ce7fd1dd048899d2342576f302c4eb3759e6e URL: https://gitlab.winehq.org/wine/wine/-/commit/472ce7fd1dd048899d2342576f302c… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Feb 19 22:27:55 2024 +0100 server: Get rid of the global cursor structure. Based on a patch by Huw Davies. --- server/queue.c | 12 ++++++------ server/user.h | 9 ++------- server/winstation.c | 3 ++- 3 files changed, 10 insertions(+), 14 deletions(-) diff --git a/server/queue.c b/server/queue.c index e06f6afe0f7..1a98714004c 100644 --- a/server/queue.c +++ b/server/queue.c @@ -440,7 +440,7 @@ static struct thread_input *get_desktop_cursor_thread_input( struct desktop *des struct thread_input *input = NULL; struct thread *thread; - if ((thread = get_window_thread( desktop->cursor.win ))) + if ((thread = get_window_thread( desktop->cursor_win ))) { if (thread->queue) input = thread->queue->input; release_object( thread ); @@ -451,9 +451,9 @@ static struct thread_input *get_desktop_cursor_thread_input( struct desktop *des static int update_desktop_cursor_window( struct desktop *desktop, user_handle_t win ) { - int updated = win != desktop->cursor.win; + int updated = win != desktop->cursor_win; struct thread_input *input; - desktop->cursor.win = win; + desktop->cursor_win = win; if (updated && (input = get_desktop_cursor_thread_input( desktop ))) { @@ -495,7 +495,7 @@ static void update_desktop_cursor_handle( struct desktop *desktop, struct thread { if (input == get_desktop_cursor_thread_input( desktop )) { - user_handle_t win = desktop->cursor.win; + user_handle_t win = desktop->cursor_win; /* when clipping send the message to the foreground window as well, as some driver have an artificial overlay window */ if (is_cursor_clipped( desktop )) queue_cursor_message( desktop, 0, WM_WINE_SETCURSOR, win, handle ); queue_cursor_message( desktop, win, WM_WINE_SETCURSOR, win, handle ); @@ -560,8 +560,8 @@ void set_clip_rectangle( struct desktop *desktop, const rectangle_t *rect, unsig } SHARED_WRITE_END; - old_flags = desktop->cursor.clip_flags; - desktop->cursor.clip_flags = flags; + old_flags = desktop->clip_flags; + desktop->clip_flags = flags; /* warp the mouse to be inside the clip rect */ x = max( min( desktop_shm->cursor.x, new_rect.right - 1 ), new_rect.left ); diff --git a/server/user.h b/server/user.h index 7ad1e82ae54..7355177e13d 100644 --- a/server/user.h +++ b/server/user.h @@ -54,12 +54,6 @@ struct winstation struct namespace *desktop_names; /* namespace for desktops of this winstation */ }; -struct global_cursor -{ - unsigned int clip_flags; /* last cursor clip flags */ - user_handle_t win; /* window that contains the cursor */ -}; - struct key_repeat { int enable; /* enable auto-repeat */ @@ -86,9 +80,10 @@ struct desktop struct timeout_user *close_timeout; /* timeout before closing the desktop */ struct thread_input *foreground_input; /* thread input of foreground thread */ unsigned int users; /* processes and threads using this desktop */ - struct global_cursor cursor; /* global cursor information */ unsigned char keystate[256]; /* asynchronous key state */ struct key_repeat key_repeat; /* key auto-repeat */ + unsigned int clip_flags; /* last cursor clip flags */ + user_handle_t cursor_win; /* window that contains the cursor */ const desktop_shm_t *shared; /* desktop session shared memory */ }; diff --git a/server/winstation.c b/server/winstation.c index 3e08f4b3337..300d899638b 100644 --- a/server/winstation.c +++ b/server/winstation.c @@ -292,7 +292,8 @@ static struct desktop *create_desktop( const struct unicode_str *name, unsigned desktop->foreground_input = NULL; desktop->users = 0; list_init( &desktop->threads ); - memset( &desktop->cursor, 0, sizeof(desktop->cursor) ); + desktop->clip_flags = 0; + desktop->cursor_win = 0; memset( desktop->keystate, 0, sizeof(desktop->keystate) ); memset( &desktop->key_repeat, 0, sizeof(desktop->key_repeat) ); list_add_tail( &winstation->desktops, &desktop->entry );

1 0

Rémi Bernon : win32u: Use the shared memory for get_clip_cursor.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: 5b013260d149b256b4da36c4caef155ff42a5cb7 URL: https://gitlab.winehq.org/wine/wine/-/commit/5b013260d149b256b4da36c4caef15… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Jun 21 00:31:15 2024 +0200 win32u: Use the shared memory for get_clip_cursor. --- dlls/win32u/input.c | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/dlls/win32u/input.c b/dlls/win32u/input.c index 2fbe21e56af..4d0c2589c1b 100644 --- a/dlls/win32u/input.c +++ b/dlls/win32u/input.c @@ -2555,24 +2555,21 @@ BOOL WINAPI NtUserGetPointerInfoList( UINT32 id, POINTER_INPUT_TYPE type, UINT_P BOOL get_clip_cursor( RECT *rect, UINT dpi ) { - BOOL ret; + struct object_lock lock = OBJECT_LOCK_INIT; + const desktop_shm_t *desktop_shm; + NTSTATUS status; if (!rect) return FALSE; - SERVER_START_REQ( set_cursor ) - { - req->flags = 0; - if ((ret = !wine_server_call( req ))) - *rect = wine_server_get_rect( reply->new_clip ); - } - SERVER_END_REQ; + while ((status = get_shared_desktop( &lock, &desktop_shm )) == STATUS_PENDING) + *rect = wine_server_get_rect( desktop_shm->cursor.clip ); - if (ret) + if (!status) { HMONITOR monitor = monitor_from_rect( rect, MONITOR_DEFAULTTOPRIMARY, 0 ); *rect = map_dpi_rect( *rect, get_monitor_dpi( monitor ), dpi ); } - return ret; + return !status; } BOOL process_wine_clipcursor( HWND hwnd, UINT flags, BOOL reset )

1 0

Rémi Bernon : server: Use a separate variable to determine the message on Alt release.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: 534aff4a6307cd289746c993aed2c2a70be8f57b URL: https://gitlab.winehq.org/wine/wine/-/commit/534aff4a6307cd289746c993aed2c2… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Jun 20 23:59:52 2024 +0200 server: Use a separate variable to determine the message on Alt release. Based on a patch by Huw Davies. --- server/queue.c | 11 +++++------ server/user.h | 1 + server/winstation.c | 1 + 3 files changed, 7 insertions(+), 6 deletions(-) diff --git a/server/queue.c b/server/queue.c index 1a98714004c..c17446a1dd9 100644 --- a/server/queue.c +++ b/server/queue.c @@ -2171,17 +2171,16 @@ static int queue_keyboard_message( struct desktop *desktop, user_handle_t win, c if (input->kbd.flags & KEYEVENTF_KEYUP) { /* send WM_SYSKEYUP if Alt still pressed and no other key in between */ - /* we use 0x02 as a flag to track if some other SYSKEYUP was sent already */ - if ((desktop->keystate[VK_MENU] & 0x82) != 0x82) break; + if (!(desktop->keystate[VK_MENU] & 0x80) || !desktop->alt_pressed) break; message_code = WM_SYSKEYUP; - desktop->keystate[VK_MENU] &= ~0x02; + desktop->alt_pressed = 0; } else { /* send WM_SYSKEYDOWN for Alt except with Ctrl */ if (desktop->keystate[VK_CONTROL] & 0x80) break; message_code = WM_SYSKEYDOWN; - desktop->keystate[VK_MENU] |= 0x02; + desktop->alt_pressed = 1; } break; @@ -2191,7 +2190,7 @@ static int queue_keyboard_message( struct desktop *desktop, user_handle_t win, c if (!(input->kbd.flags & KEYEVENTF_KEYUP)) break; if (!(desktop->keystate[VK_MENU] & 0x80)) break; message_code = WM_SYSKEYUP; - desktop->keystate[VK_MENU] &= ~0x02; + desktop->alt_pressed = 0; break; default: @@ -2201,7 +2200,7 @@ static int queue_keyboard_message( struct desktop *desktop, user_handle_t win, c /* fall through */ case VK_F10: message_code = (input->kbd.flags & KEYEVENTF_KEYUP) ? WM_SYSKEYUP : WM_SYSKEYDOWN; - desktop->keystate[VK_MENU] &= ~0x02; + desktop->alt_pressed = 0; break; } diff --git a/server/user.h b/server/user.h index 7355177e13d..99491293a7c 100644 --- a/server/user.h +++ b/server/user.h @@ -81,6 +81,7 @@ struct desktop struct thread_input *foreground_input; /* thread input of foreground thread */ unsigned int users; /* processes and threads using this desktop */ unsigned char keystate[256]; /* asynchronous key state */ + unsigned char alt_pressed; /* last key press was Alt (used to determine msg on release) */ struct key_repeat key_repeat; /* key auto-repeat */ unsigned int clip_flags; /* last cursor clip flags */ user_handle_t cursor_win; /* window that contains the cursor */ diff --git a/server/winstation.c b/server/winstation.c index 300d899638b..63bdc3389de 100644 --- a/server/winstation.c +++ b/server/winstation.c @@ -294,6 +294,7 @@ static struct desktop *create_desktop( const struct unicode_str *name, unsigned list_init( &desktop->threads ); desktop->clip_flags = 0; desktop->cursor_win = 0; + desktop->alt_pressed = 0; memset( desktop->keystate, 0, sizeof(desktop->keystate) ); memset( &desktop->key_repeat, 0, sizeof(desktop->key_repeat) ); list_add_tail( &winstation->desktops, &desktop->entry );

1 0

Rémi Bernon : server: Use separate functions to update the desktop and input keystates.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: d5b4458c8d34ef5319f31caefcf11fab3eaf9cec URL: https://gitlab.winehq.org/wine/wine/-/commit/d5b4458c8d34ef5319f31caefcf11f… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Jun 16 09:58:53 2023 +0200 server: Use separate functions to update the desktop and input keystates. Based on a patch by Huw Davies. --- server/queue.c | 39 ++++++++++++++++++++++++--------------- 1 file changed, 24 insertions(+), 15 deletions(-) diff --git a/server/queue.c b/server/queue.c index c17446a1dd9..699a8fdf248 100644 --- a/server/queue.c +++ b/server/queue.c @@ -1470,7 +1470,7 @@ static struct timer *set_timer( struct msg_queue *queue, unsigned int rate ) } /* change the input key state for a given key */ -static void set_input_key_state( unsigned char *keystate, unsigned char key, int down ) +static void set_input_key_state( unsigned char *keystate, unsigned char key, unsigned char down ) { if (down) { @@ -1481,34 +1481,33 @@ static void set_input_key_state( unsigned char *keystate, unsigned char key, int } /* update the input key state for a keyboard message */ -static void update_input_key_state( struct desktop *desktop, unsigned char *keystate, - unsigned int msg, lparam_t wparam ) +static void update_key_state( unsigned char *keystate, unsigned int msg, + lparam_t wparam, int desktop ) { - unsigned char key; - int down = 0; + unsigned char key, down = 0, down_val = desktop ? 0xc0 : 0x80; switch (msg) { case WM_LBUTTONDOWN: - down = (keystate == desktop->keystate) ? 0xc0 : 0x80; + down = down_val; /* fall through */ case WM_LBUTTONUP: set_input_key_state( keystate, VK_LBUTTON, down ); break; case WM_MBUTTONDOWN: - down = (keystate == desktop->keystate) ? 0xc0 : 0x80; + down = down_val; /* fall through */ case WM_MBUTTONUP: set_input_key_state( keystate, VK_MBUTTON, down ); break; case WM_RBUTTONDOWN: - down = (keystate == desktop->keystate) ? 0xc0 : 0x80; + down = down_val; /* fall through */ case WM_RBUTTONUP: set_input_key_state( keystate, VK_RBUTTON, down ); break; case WM_XBUTTONDOWN: - down = (keystate == desktop->keystate) ? 0xc0 : 0x80; + down = down_val; /* fall through */ case WM_XBUTTONUP: if (wparam >> 16 == XBUTTON1) set_input_key_state( keystate, VK_XBUTTON1, down ); @@ -1516,7 +1515,7 @@ static void update_input_key_state( struct desktop *desktop, unsigned char *keys break; case WM_KEYDOWN: case WM_SYSKEYDOWN: - down = (keystate == desktop->keystate) ? 0xc0 : 0x80; + down = down_val; /* fall through */ case WM_KEYUP: case WM_SYSKEYUP: @@ -1544,6 +1543,16 @@ static void update_input_key_state( struct desktop *desktop, unsigned char *keys } } +static void update_thread_input_key_state( struct thread_input *input, unsigned int msg, lparam_t wparam ) +{ + update_key_state( input->keystate, msg, wparam, 0 ); +} + +static void update_desktop_key_state( struct desktop *desktop, unsigned int msg, lparam_t wparam ) +{ + update_key_state( desktop->keystate, msg, wparam, 1 ); +} + /* release the hardware message currently being processed by the given thread */ static void release_hardware_message( struct msg_queue *queue, unsigned int hw_id ) { @@ -1569,7 +1578,7 @@ static void release_hardware_message( struct msg_queue *queue, unsigned int hw_i } if (clr_bit) clear_queue_bits( queue, clr_bit ); - update_input_key_state( input->desktop, input->keystate, msg->msg, msg->wparam ); + update_thread_input_key_state( input, msg->msg, msg->wparam ); list_remove( &msg->entry ); free_message( msg ); } @@ -1696,7 +1705,7 @@ static void queue_hardware_message( struct desktop *desktop, struct message *msg unsigned int msg_code; int flags; - update_input_key_state( desktop, desktop->keystate, msg->msg, msg->wparam ); + update_desktop_key_state( desktop, msg->msg, msg->wparam ); last_input_time = get_tick_count(); if (msg->msg != WM_MOUSEMOVE) always_queue = 1; @@ -1736,7 +1745,7 @@ static void queue_hardware_message( struct desktop *desktop, struct message *msg flags = thread ? get_rawinput_device_flags( thread->process, msg ) : 0; if (!win || !thread || (flags & RIDEV_NOLEGACY)) { - if (input && !(flags & RIDEV_NOLEGACY)) update_input_key_state( input->desktop, input->keystate, msg->msg, msg->wparam ); + if (input && !(flags & RIDEV_NOLEGACY)) update_thread_input_key_state( input, msg->msg, msg->wparam ); free_message( msg ); return; } @@ -2528,7 +2537,7 @@ static int get_hardware_message( struct thread *thread, unsigned int hw_id, user if (!win || !win_thread) { /* no window at all, remove it */ - update_input_key_state( input->desktop, input->keystate, msg->msg, msg->wparam ); + update_thread_input_key_state( input, msg->msg, msg->wparam ); list_remove( &msg->entry ); free_message( msg ); continue; @@ -2544,7 +2553,7 @@ static int get_hardware_message( struct thread *thread, unsigned int hw_id, user else { /* for another thread input, drop it */ - update_input_key_state( input->desktop, input->keystate, msg->msg, msg->wparam ); + update_thread_input_key_state( input, msg->msg, msg->wparam ); list_remove( &msg->entry ); free_message( msg ); }

1 0

Rémi Bernon : server: Move the desktop keystate to shared memory.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: 2eeb4d5192e808c18e5949cc1c14b51b22fee342 URL: https://gitlab.winehq.org/wine/wine/-/commit/2eeb4d5192e808c18e5949cc1c14b5… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Jun 21 00:11:11 2024 +0200 server: Move the desktop keystate to shared memory. Based on a patch by Huw Davies. --- include/wine/server_protocol.h | 3 +- server/protocol.def | 1 + server/queue.c | 81 ++++++++++++++++++++++++++---------------- server/winstation.c | 2 +- 4 files changed, 55 insertions(+), 32 deletions(-)

1 0

Rémi Bernon : win32u: Use the shared data if possible for NtUserGetAsyncKeyState.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: 9a7408e7719d6a16ea06c3e1371a306e8e200611 URL: https://gitlab.winehq.org/wine/wine/-/commit/9a7408e7719d6a16ea06c3e1371a30… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Jun 15 15:49:55 2024 +0200 win32u: Use the shared data if possible for NtUserGetAsyncKeyState. Based on a patch by Huw Davies. --- dlls/win32u/hook.c | 2 -- dlls/win32u/input.c | 44 +++++++++++--------------------------------- dlls/win32u/message.c | 16 +++------------- dlls/win32u/ntuser_private.h | 8 -------- dlls/win32u/sysparams.c | 2 -- dlls/win32u/win32u_private.h | 1 - dlls/win32u/winstation.c | 2 -- 7 files changed, 14 insertions(+), 61 deletions(-) diff --git a/dlls/win32u/hook.c b/dlls/win32u/hook.c index fab2b960fa5..eca9e7d6ce9 100644 --- a/dlls/win32u/hook.c +++ b/dlls/win32u/hook.c @@ -345,8 +345,6 @@ static LRESULT call_hook( struct win_hook_params *info, const WCHAR *module, siz if (params != info) free( params ); } - if (info->id == WH_KEYBOARD_LL || info->id == WH_MOUSE_LL) - InterlockedIncrement( &global_key_state_counter ); /* force refreshing the key state cache */ return ret; } diff --git a/dlls/win32u/input.c b/dlls/win32u/input.c index 4d0c2589c1b..a582826d18e 100644 --- a/dlls/win32u/input.c +++ b/dlls/win32u/input.c @@ -406,7 +406,6 @@ static const KBDTABLES kbdus_tables = static LONG clipping_cursor; /* clipping thread counter */ -LONG global_key_state_counter = 0; BOOL grab_pointer = TRUE; BOOL grab_fullscreen = FALSE; @@ -810,52 +809,31 @@ static void check_for_events( UINT flags ) */ SHORT WINAPI NtUserGetAsyncKeyState( INT key ) { - struct user_key_state_info *key_state_info = get_user_thread_info()->key_state; - INT counter = global_key_state_counter; - BYTE prev_key_state; - SHORT ret; + const desktop_shm_t *desktop_shm; + struct object_lock lock = OBJECT_LOCK_INIT; + NTSTATUS status; + BYTE state = 0; + SHORT ret = 0; if (key < 0 || key >= 256) return 0; check_for_events( QS_INPUT ); - if (key_state_info && !(key_state_info->state[key] & 0xc0) && - key_state_info->counter == counter && NtGetTickCount() - key_state_info->time < 50) - { - /* use cached value */ - return 0; - } - else if (!key_state_info) - { - key_state_info = calloc( 1, sizeof(*key_state_info) ); - get_user_thread_info()->key_state = key_state_info; - } + while ((status = get_shared_desktop( &lock, &desktop_shm )) == STATUS_PENDING) + state = desktop_shm->keystate[key]; - ret = 0; + if (status) return 0; + if (!(state & 0x40)) return (state & 0x80) << 8; + + /* Need to make a server call to reset the last pressed bit */ SERVER_START_REQ( get_key_state ) { req->async = 1; req->key = key; - if (key_state_info) - { - prev_key_state = key_state_info->state[key]; - wine_server_set_reply( req, key_state_info->state, sizeof(key_state_info->state) ); - } if (!wine_server_call( req )) { if (reply->state & 0x40) ret |= 0x0001; if (reply->state & 0x80) ret |= 0x8000; - if (key_state_info) - { - /* force refreshing the key state cache - some multithreaded programs - * (like Adobe Photoshop CS5) expect that changes to the async key state - * are also immediately available in other threads. */ - if (prev_key_state != key_state_info->state[key]) - counter = InterlockedIncrement( &global_key_state_counter ); - - key_state_info->time = NtGetTickCount(); - key_state_info->counter = counter; - } } } SERVER_END_REQ; diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index d1f49ca3ed9..b7062414658 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -3469,7 +3469,7 @@ NTSTATUS send_hardware_message( HWND hwnd, UINT flags, const INPUT *input, LPARA struct send_message_info info; int prev_x, prev_y, new_x, new_y; NTSTATUS ret; - BOOL wait, affects_key_state = FALSE; + BOOL wait; info.type = MSG_HARDWARE; info.dest_tid = 0; @@ -3495,10 +3495,6 @@ NTSTATUS send_hardware_message( HWND hwnd, UINT flags, const INPUT *input, LPARA req->input.mouse.flags = input->mi.dwFlags; req->input.mouse.time = input->mi.time; req->input.mouse.info = input->mi.dwExtraInfo; - affects_key_state = !!(input->mi.dwFlags & (MOUSEEVENTF_LEFTDOWN | MOUSEEVENTF_LEFTUP | - MOUSEEVENTF_RIGHTDOWN | MOUSEEVENTF_RIGHTUP | - MOUSEEVENTF_MIDDLEDOWN | MOUSEEVENTF_MIDDLEUP | - MOUSEEVENTF_XDOWN | MOUSEEVENTF_XUP)); break; case INPUT_KEYBOARD: if (input->ki.dwFlags & KEYEVENTF_SCANCODE) @@ -3523,7 +3519,6 @@ NTSTATUS send_hardware_message( HWND hwnd, UINT flags, const INPUT *input, LPARA req->input.kbd.flags = input->ki.dwFlags & ~KEYEVENTF_SCANCODE; req->input.kbd.time = input->ki.time; req->input.kbd.info = input->ki.dwExtraInfo; - affects_key_state = TRUE; break; case INPUT_HARDWARE: req->input.hw.msg = input->hi.uMsg; @@ -3553,13 +3548,8 @@ NTSTATUS send_hardware_message( HWND hwnd, UINT flags, const INPUT *input, LPARA } SERVER_END_REQ; - if (!ret) - { - if (affects_key_state) - InterlockedIncrement( &global_key_state_counter ); /* force refreshing the key state cache */ - if ((flags & SEND_HWMSG_INJECTED) && (prev_x != new_x || prev_y != new_y)) - user_driver->pSetCursorPos( new_x, new_y ); - } + if (!ret && (flags & SEND_HWMSG_INJECTED) && (prev_x != new_x || prev_y != new_y)) + user_driver->pSetCursorPos( new_x, new_y ); if (wait) { diff --git a/dlls/win32u/ntuser_private.h b/dlls/win32u/ntuser_private.h index 39943e2261b..d95ea03d45f 100644 --- a/dlls/win32u/ntuser_private.h +++ b/dlls/win32u/ntuser_private.h @@ -116,7 +116,6 @@ struct user_thread_info HHOOK hook; /* Current hook */ UINT active_hooks; /* Bitmap of active hooks */ struct received_message_info *receive_info; /* Message being currently received */ - struct user_key_state_info *key_state; /* Cache of global key state */ struct imm_thread_data *imm_thread_data; /* IMM thread data */ HKL kbd_layout; /* Current keyboard layout */ UINT kbd_layout_id; /* Current keyboard layout ID */ @@ -134,13 +133,6 @@ static inline struct user_thread_info *get_user_thread_info(void) return CONTAINING_RECORD( NtUserGetThreadInfo(), struct user_thread_info, client_info ); } -struct user_key_state_info -{ - UINT time; /* Time of last key state refresh */ - INT counter; /* Counter to invalidate the key state */ - BYTE state[256]; /* State for each key */ -}; - struct hook_extra_info { HHOOK handle; diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 34de10168e2..9a06db2c7f6 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -6334,8 +6334,6 @@ static void thread_detach(void) destroy_thread_windows(); user_driver->pThreadDetach(); - free( thread_info->key_state ); - thread_info->key_state = 0; free( thread_info->rawinput ); cleanup_imm_thread(); diff --git a/dlls/win32u/win32u_private.h b/dlls/win32u/win32u_private.h index f7ba2c1a423..e2e35d6c40d 100644 --- a/dlls/win32u/win32u_private.h +++ b/dlls/win32u/win32u_private.h @@ -90,7 +90,6 @@ extern void unregister_imm_window( HWND hwnd ); extern BOOL grab_pointer; extern BOOL grab_fullscreen; extern BOOL destroy_caret(void); -extern LONG global_key_state_counter; extern HWND get_active_window(void); extern HWND get_capture(void); extern BOOL get_cursor_pos( POINT *pt ); diff --git a/dlls/win32u/winstation.c b/dlls/win32u/winstation.c index 7168a614735..1676539c69c 100644 --- a/dlls/win32u/winstation.c +++ b/dlls/win32u/winstation.c @@ -441,11 +441,9 @@ BOOL WINAPI NtUserSetThreadDesktop( HDESK handle ) if (ret) /* reset the desktop windows */ { struct user_thread_info *thread_info = get_user_thread_info(); - struct user_key_state_info *key_state_info = thread_info->key_state; get_session_thread_data()->shared_desktop = find_shared_session_object( locator ); thread_info->client_info.top_window = 0; thread_info->client_info.msg_window = 0; - if (key_state_info) key_state_info->time = 0; if (was_virtual_desktop != is_virtual_desktop()) update_display_cache( FALSE ); } return ret;

1 0

Zhiyi Zhang : profapi: Add stub dll.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: 28040129930f2d2f717bdb3718365bc1ebf67822 URL: https://gitlab.winehq.org/wine/wine/-/commit/28040129930f2d2f717bdb3718365b… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Thu May 9 11:02:48 2024 +0800 profapi: Add stub dll. --- configure | 2 ++ configure.ac | 1 + dlls/profapi/Makefile.in | 2 ++ dlls/profapi/profapi.spec | 17 +++++++++++++++++ 4 files changed, 22 insertions(+) diff --git a/configure b/configure index 5a8debc5629..a939cb640ca 100755 --- a/configure +++ b/configure @@ -1369,6 +1369,7 @@ enable_pidgen enable_powrprof enable_printui enable_prntvpt +enable_profapi enable_propsys enable_psapi enable_pstorec @@ -22517,6 +22518,7 @@ wine_fn_config_makefile dlls/powrprof enable_powrprof wine_fn_config_makefile dlls/printui enable_printui wine_fn_config_makefile dlls/prntvpt enable_prntvpt wine_fn_config_makefile dlls/prntvpt/tests enable_tests +wine_fn_config_makefile dlls/profapi enable_profapi wine_fn_config_makefile dlls/propsys enable_propsys wine_fn_config_makefile dlls/propsys/tests enable_tests wine_fn_config_makefile dlls/psapi enable_psapi diff --git a/configure.ac b/configure.ac index 231ec7f17d6..4d958d18193 100644 --- a/configure.ac +++ b/configure.ac @@ -3025,6 +3025,7 @@ WINE_CONFIG_MAKEFILE(dlls/powrprof) WINE_CONFIG_MAKEFILE(dlls/printui) WINE_CONFIG_MAKEFILE(dlls/prntvpt) WINE_CONFIG_MAKEFILE(dlls/prntvpt/tests) +WINE_CONFIG_MAKEFILE(dlls/profapi) WINE_CONFIG_MAKEFILE(dlls/propsys) WINE_CONFIG_MAKEFILE(dlls/propsys/tests) WINE_CONFIG_MAKEFILE(dlls/psapi) diff --git a/dlls/profapi/Makefile.in b/dlls/profapi/Makefile.in new file mode 100644 index 00000000000..83c679227d0 --- /dev/null +++ b/dlls/profapi/Makefile.in @@ -0,0 +1,2 @@ +MODULE = profapi.dll +IMPORTLIB = profapi diff --git a/dlls/profapi/profapi.spec b/dlls/profapi/profapi.spec new file mode 100644 index 00000000000..529aaac2ff8 --- /dev/null +++ b/dlls/profapi/profapi.spec @@ -0,0 +1,17 @@ +101 stub -noname @ +102 stub -noname @ +103 stub -noname @ +104 stub -noname @ +105 stub -noname @ +106 stub -noname @ +107 stub -noname @ +108 stub -noname @ +109 stub -noname @ +110 stub -noname @ +111 stub -noname @ +112 stub -noname @ +113 stub -noname @ +114 stub -noname @ +115 stub -noname @ +116 stub -noname @ +117 stub -noname @

1 0

Arkadiusz Hiler : bcp47langs: Add stub dll.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: 2030973a3f23a84468f06c1c2d856a7cb4710adb URL: https://gitlab.winehq.org/wine/wine/-/commit/2030973a3f23a84468f06c1c2d856a… Author: Arkadiusz Hiler <ahiler(a)codeweavers.com> Date: Tue Apr 23 14:02:20 2024 +0300 bcp47langs: Add stub dll. --- configure | 2 ++ configure.ac | 1 + dlls/bcp47langs/Makefile.in | 2 ++ dlls/bcp47langs/bcp47langs.spec | 71 +++++++++++++++++++++++++++++++++++++++++ 4 files changed, 76 insertions(+) diff --git a/configure b/configure index a939cb640ca..4a60f6a048a 100755 --- a/configure +++ b/configure @@ -1006,6 +1006,7 @@ enable_authz enable_avicap32 enable_avifil32 enable_avrt +enable_bcp47langs enable_bcrypt enable_bcryptprimitives enable_bluetoothapis @@ -21952,6 +21953,7 @@ wine_fn_config_makefile dlls/avifil32 enable_avifil32 wine_fn_config_makefile dlls/avifil32/tests enable_tests wine_fn_config_makefile dlls/avifile.dll16 enable_win16 wine_fn_config_makefile dlls/avrt enable_avrt +wine_fn_config_makefile dlls/bcp47langs enable_bcp47langs wine_fn_config_makefile dlls/bcrypt enable_bcrypt wine_fn_config_makefile dlls/bcrypt/tests enable_tests wine_fn_config_makefile dlls/bcryptprimitives enable_bcryptprimitives diff --git a/configure.ac b/configure.ac index 4d958d18193..0870961c041 100644 --- a/configure.ac +++ b/configure.ac @@ -2459,6 +2459,7 @@ WINE_CONFIG_MAKEFILE(dlls/avifil32) WINE_CONFIG_MAKEFILE(dlls/avifil32/tests) WINE_CONFIG_MAKEFILE(dlls/avifile.dll16,enable_win16) WINE_CONFIG_MAKEFILE(dlls/avrt) +WINE_CONFIG_MAKEFILE(dlls/bcp47langs) WINE_CONFIG_MAKEFILE(dlls/bcrypt) WINE_CONFIG_MAKEFILE(dlls/bcrypt/tests) WINE_CONFIG_MAKEFILE(dlls/bcryptprimitives) diff --git a/dlls/bcp47langs/Makefile.in b/dlls/bcp47langs/Makefile.in new file mode 100644 index 00000000000..5eb4caaf0e7 --- /dev/null +++ b/dlls/bcp47langs/Makefile.in @@ -0,0 +1,2 @@ +MODULE = bcp47langs.dll +IMPORTLIB = bcp47langs diff --git a/dlls/bcp47langs/bcp47langs.spec b/dlls/bcp47langs/bcp47langs.spec new file mode 100644 index 00000000000..a054c1486a9 --- /dev/null +++ b/dlls/bcp47langs/bcp47langs.spec @@ -0,0 +1,71 @@ +1 stub GetUnIsoRegionCode +@ stub AppendUserLanguageInputMethods +@ stub AppendUserLanguageInternal +@ stub AppendUserLanguages +@ stub Bcp47BufferFromLcid +@ stub Bcp47FindClosestLanguage +@ stub Bcp47FromCompactTagInternal +@ stub Bcp47FromHkl +@ stub Bcp47FromLcid +@ stub Bcp47GetAbbreviation +@ stub Bcp47GetDirectionality +@ stub Bcp47GetDistance +@ stub Bcp47GetExtensionSingletons +@ stub Bcp47GetExtensionSubstring +@ stub Bcp47GetIsoLanguageCode +@ stub Bcp47GetIsoScriptCode +@ stub Bcp47GetLanguageName +@ stub Bcp47GetMuiForm +@ stub Bcp47GetNeutralForm +@ stub Bcp47GetNlsForm +@ stub Bcp47GetSubtagMapInternal +@ stub Bcp47GetUnIsoRegionCode +@ stub Bcp47IsWellFormed +@ stub Bcp47Normalize +@ stub ClearApplicationLanguageOverride +@ stub ClearApplicationManifestLanguages +@ stub ClearHttpAcceptLanguageOptOut +@ stub ClearUserDisplayLanguageOverride +@ stub ClearUserLocaleFromLanguageProfileOptOut +@ stub CompactTagFromBcp47Internal +@ stub DllGetActivationFactory +@ stub DllGetClassObject +@ stub GetApplicationLanguageOverride +@ stub GetApplicationLanguages +@ stub GetApplicationManifestLanguages +@ stub GetAppropriateUserLocaleForUserLanguages +@ stub GetAppropriateUserPreferredAndDisplayLanguagesForUser +@ stub GetClosestMatchingUserLanguage +@ stub GetDisplayLanguagesForAllUsers +@ stub GetFontFallbackLanguageList +@ stub GetHttpAcceptLanguageOptOut +@ stub GetInputMethodOverrideForUser +@ stub GetPendingUserDisplayLanguage +@ stub GetRelevantLocalesFromLanguageTags +@ stub GetSerializedUserLanguageProfile +@ stub GetSerializedUserLanguagesForUser +@ stub GetStartingUserDisplayLanguage +@ stub GetUserDisplayLanguageOverride +@ stub GetUserLanguageInputMethods +@ stub GetUserLanguageInputMethodsForUser +@ stub GetUserLanguages +@ stub GetUserLanguagesForAllUsers +@ stub GetUserLanguagesForUser +@ stub GetUserLocaleFromLanguageProfileOptOut +@ stub IsTransientLcid +@ stub LanguageListAsMuiForm +@ stub LcidFromBcp47 +@ stub RemoveInputsForAllLanguagesInternal +@ stub RemoveUserLanguageInputMethods +@ stub ResolveLanguages +@ stub SetApplicationLanguageOverride +@ stub SetApplicationManifestLanguages +@ stub SetHttpAcceptLanguageOptOut +@ stub SetInputMethodOverride +@ stub SetPreviousUserDisplayLanguages +@ stub SetStartingUserDisplayLanguage +@ stub SetUserDisplayLanguageOverride +@ stub SetUserLanguageInputMethods +@ stub SetUserLanguagesInternal +@ stub SetUserLanguagesInternalCore +@ stub SetUserLocaleFromLanguageProfileOptOut

1 0

Arkadiusz Hiler : apisetschema: Add api-ms-win-appmodel-runtime-internal-l1-1-1.
by Alexandre Julliard 21 Jun '24

21 Jun '24

Module: wine Branch: master Commit: 5a903c0f1ac9f4d209c038f93c341c6ee9d7a987 URL: https://gitlab.winehq.org/wine/wine/-/commit/5a903c0f1ac9f4d209c038f93c341c… Author: Arkadiusz Hiler <ahiler(a)codeweavers.com> Date: Tue Apr 23 14:11:32 2024 +0300 apisetschema: Add api-ms-win-appmodel-runtime-internal-l1-1-1. --- dlls/apisetschema/apisetschema.spec | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/apisetschema/apisetschema.spec b/dlls/apisetschema/apisetschema.spec index c7da8a6a806..c46918b6fa8 100644 --- a/dlls/apisetschema/apisetschema.spec +++ b/dlls/apisetschema/apisetschema.spec @@ -1,3 +1,4 @@ +apiset api-ms-win-appmodel-runtime-internal-l1-1-1 = kernelbase.dll apiset api-ms-win-appmodel-runtime-l1-1-2 = kernelbase.dll apiset api-ms-win-base-bootconfig-l1-1-0 = advapi32.dll apiset api-ms-win-base-util-l1-1-0 = advapi32.dll

1 0