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. --- plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) (limited to 'plugins') 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