configure.ac                     |    2 +-
 vcl/unx/gtk3/a11y/atkwrapper.cxx |   23 +++++------------------
 2 files changed, 6 insertions(+), 19 deletions(-)

New commits:
commit c688fd1393dccfb3df39bd05cceed1cf0c401b02
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Mon Feb 21 15:49:52 2022 +0100
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Tue Feb 22 10:18:46 2022 +0100

    gtk3 a11y: Require atk >= 2.28.1, drop code for < 2.15.2
    
    README.md lists RHEL 7 or CentOS 7 as runtime baseline,
    which has ATK 2.28.1 [1].
    
    Therefore, check for that version in configure.ac
    and drop some code for versions < 2.15.2.
    Requiring 2.28.1 as minimum ATK version will allow for
    further cleanup in follow-up commits.
    
    [1] https://centos.pkgs.org/7/centos-x86_64/atk-2.28.1-2.el7.x86_64.rpm.html
    
    Change-Id: Id7084366d2359deb60948b8969c1d3dcc68ddb47
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/130328
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>

diff --git a/configure.ac b/configure.ac
index af1da0103177..b22d8e17c6da 100644
--- a/configure.ac
+++ b/configure.ac
@@ -11639,7 +11639,7 @@ if test "$test_gtk3" = yes -a "x$enable_gtk3" = "xyes" 
-o "x$enable_gtk3_kde5" =
         add_warning 'Non-system cairo combined with gtk3 is assumed to cause 
trouble; proceed at your own risk.'
     fi
     : ${with_system_cairo:=yes}
-    PKG_CHECK_MODULES(GTK3, gtk+-3.0 >= 3.20 gtk+-unix-print-3.0 
gmodule-no-export-2.0 glib-2.0 >= 2.38 cairo)
+    PKG_CHECK_MODULES(GTK3, gtk+-3.0 >= 3.20 gtk+-unix-print-3.0 
gmodule-no-export-2.0 glib-2.0 >= 2.38 atk >= 2.28.1 cairo)
     GTK3_CFLAGS=$(printf '%s' "$GTK3_CFLAGS" | sed -e "s/-I/${ISYSTEM?}/g")
     GTK3_CFLAGS="$GTK3_CFLAGS -DGDK_DISABLE_DEPRECATED 
-DGTK_DISABLE_DEPRECATED"
     FilterLibs "${GTK3_LIBS}"
diff --git a/vcl/unx/gtk3/a11y/atkwrapper.cxx b/vcl/unx/gtk3/a11y/atkwrapper.cxx
index 53ac1e79df98..1c3d7f5843de 100644
--- a/vcl/unx/gtk3/a11y/atkwrapper.cxx
+++ b/vcl/unx/gtk3/a11y/atkwrapper.cxx
@@ -267,24 +267,11 @@ static AtkRole mapToAtkRole( sal_Int16 nRole )
         ATK_ROLE_UNKNOWN,       // TREE_ITEM - registered below
         ATK_ROLE_TREE_TABLE,
         ATK_ROLE_SCROLL_PANE,   // COMMENT - mapped to atk_role_scroll_pane
-        ATK_ROLE_UNKNOWN        // COMMENT_END - mapped to atk_role_unknown
-#if defined(ATK_CHECK_VERSION)
-        //older ver that doesn't define ATK_CHECK_VERSION doesn't have the 
following
-        , ATK_ROLE_DOCUMENT_PRESENTATION
-        , ATK_ROLE_DOCUMENT_SPREADSHEET
-        , ATK_ROLE_DOCUMENT_TEXT
-#if ATK_CHECK_VERSION(2,15,2)
-        , ATK_ROLE_STATIC
-#else
-        , ATK_ROLE_LABEL
-#endif
-#else
-        //older version should fallback to DOCUMENT_FRAME role
-        , ATK_ROLE_DOCUMENT_FRAME
-        , ATK_ROLE_DOCUMENT_FRAME
-        , ATK_ROLE_DOCUMENT_FRAME
-        , ATK_ROLE_LABEL
-#endif
+        ATK_ROLE_UNKNOWN,        // COMMENT_END - mapped to atk_role_unknown
+        ATK_ROLE_DOCUMENT_PRESENTATION,
+        ATK_ROLE_DOCUMENT_SPREADSHEET,
+        ATK_ROLE_DOCUMENT_TEXT,
+        ATK_ROLE_STATIC
     };
 
     static bool initialized = false;

Reply via email to