From 7fb8e162baf14c632fb863efe034f298b5d1d174 Mon Sep 17 00:00:00 2001 From: Colomban Wendling Date: Wed, 5 Aug 2020 16:21:43 +0200 Subject: a11y-keyboard: capslock-beep: Try and detect non-buggy libatspi2 Try and avoid the workaround for buggy libatspi2 if we can know the version we're using has it fixed. --- configure.ac | 3 +++ plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c | 17 +++++++---------- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/configure.ac b/configure.ac index c2e1cfd..19dde6b 100644 --- a/configure.ac +++ b/configure.ac @@ -123,6 +123,9 @@ dnl --------------------------------------------------------------------------- PKG_CHECK_MODULES(LIBATSPI, atspi-2 >= $LIBATSPI_REQUIRED_VERSION, [AC_DEFINE(HAVE_LIBATSPI, 1, [Define if libatspi is available]) have_libatspi=yes], have_libatspi=no) +PKG_CHECK_EXISTS([atspi-2 > 2.36.0], + [AC_DEFINE([DESTROYING_ATSPI_LISTENER_DOES_NOT_CRASH], [1], [Define if libatspi does not have bug 22])], + []) dnl --------------------------------------------------------------------------- dnl - Check for D-Bus diff --git a/plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c b/plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c index a1d7c9c..bae815e 100644 --- a/plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c +++ b/plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c @@ -27,10 +27,6 @@ #include #include -/* See https://gitlab.gnome.org/GNOME/at-spi2-core/-/issues/22 */ -#define DESTROYING_ATSPI_LISTENER_CRASHES 1 - - struct MsdA11yKeyboardAtspiPrivate { AtspiDeviceListener *listener; @@ -80,14 +76,15 @@ msd_a11y_keyboard_atspi_init (MsdA11yKeyboardAtspi *self) self->priv->listener = NULL; self->priv->listening = FALSE; -#if DESTROYING_ATSPI_LISTENER_CRASHES +#ifndef DESTROYING_ATSPI_LISTENER_DOES_NOT_CRASH /* init AT-SPI if needed */ atspi_init (); self->priv->listener = atspi_device_listener_new (on_key_press_event, self, NULL); /* leak a reference so that this listener is *never* destroyed, to - * prevent the crash even if our object gets destroyed. */ + * prevent the crash even if our object gets destroyed. + * See https://gitlab.gnome.org/GNOME/at-spi2-core/-/issues/22 */ g_object_ref (self->priv->listener); #endif } @@ -129,7 +126,7 @@ msd_a11y_keyboard_atspi_start (MsdA11yKeyboardAtspi *self) if (self->priv->listening) return; -#if ! DESTROYING_ATSPI_LISTENER_CRASHES +#ifdef DESTROYING_ATSPI_LISTENER_DOES_NOT_CRASH /* init AT-SPI if needed */ atspi_init (); @@ -148,10 +145,10 @@ msd_a11y_keyboard_atspi_stop (MsdA11yKeyboardAtspi *self) if (! self->priv->listening) return; -#if DESTROYING_ATSPI_LISTENER_CRASHES - register_deregister_events (self, FALSE); -#else +#ifdef DESTROYING_ATSPI_LISTENER_DOES_NOT_CRASH g_clear_object (&self->priv->listener); +#else + register_deregister_events (self, FALSE); #endif self->priv->listening = FALSE; } -- cgit v1.2.1