-
Olivier Fourdan authored
The xwayland-piglit.sh script spawns weston, runs run-piglit.sh and finally kills weston. However, this whole script is running with “-e” meaning that any error will cause the script to exit immediately. As a result, if run-piglit.sh exits with a non-zero code such as 77 for skipping the test, the script will exit prematurely leaving weston running, and meson will simply wait until the timeout kicks in, and fail eventually instead of skipping the test as it should. Fix this by removing the option to exit immediately prior to spawn the script run-piglit.sh. Closes: xorg/xserver#1204 Suggested-by:
Michel Dänzer <mdaenzer@redhat.com> Signed-off-by:
Olivier Fourdan <ofourdan@redhat.com> Reviewed-by:
Michel Dänzer <mdaenzer@redhat.com>
f37d11cd -
Simon Ser authored
Passing -noTouchPointerEmulation results in an error about the flag not being recognized. Signed-off-by:
Simon Ser <contact@emersion.fr> Fixes: 7d34b1f2 ("xwayland: add -noTouchPointerEmulation")
ccffe90c -
Two different functions in xwayland-cursor.c and xwayland-input.c use the same name xwl_seat_update_cursor() which is confusing when reading the code. Rename xwl_seat_update_cursor() to xwl_seat_update_all_cursors() in xwayland-cursor.c to help with readability of the code. No functional change. Signed-off-by:
Olivier Fourdan <ofourdan@redhat.com> Reviewed-by:
Simon Ser <contact@emersion.fr> Reviewed-by:
Carlos Garnacho <carlosg@gnome.org>
e6401e60 -
It just make more sense to keep xwl_cursor_release() with the rest of the cursor code. No functional change. Signed-off-by:
Olivier Fourdan <ofourdan@redhat.com> Reviewed-by:
Simon Ser <contact@emersion.fr> Reviewed-by:
Carlos Garnacho <carlosg@gnome.org>
672fe9e2 -
The code to clear a cursor pending frame callback was duplicated in multiple places in the code. Introduce a new xwl_cursor_clear_frame_cb() function and remove the duplicated code. No functional change. Signed-off-by:
Olivier Fourdan <ofourdan@redhat.com> Reviewed-by:
Simon Ser <contact@emersion.fr> Reviewed-by:
Carlos Garnacho <carlosg@gnome.org>
c93c2e77
- hw/xwayland/xwayland-cursor.c 25 additions, 15 deletionshw/xwayland/xwayland-cursor.c
- hw/xwayland/xwayland-cursor.h 2 additions, 0 deletionshw/xwayland/xwayland-cursor.h
- hw/xwayland/xwayland-input.c 2 additions, 16 deletionshw/xwayland/xwayland-input.c
- hw/xwayland/xwayland.c 1 addition, 0 deletionshw/xwayland/xwayland.c
- test/scripts/xwayland-piglit.sh 2 additions, 0 deletionstest/scripts/xwayland-piglit.sh