Ensure that DBus errors are freed

Call dbus_error_free in some cases where it is needed. Just pass NULL to
dbus_message_get_args where we aren't using the error.

https://bugzilla.gnome.org/show_bug.cgi?id=698951

Add NULL check to fix crash when receiving a dbus reply after cleanup

socket-adaptor: fix memory leak

Set a destructor for the object data. Other than fixing a leak this
also allows to simplify the code a bit.

https://bugzilla.gnome.org/show_bug.cgi?id=696733

Release as v2.8.0

Release v2.7.91

Add Locale property, mapped to atk_object_get_object_locale

https://bugzilla.gnome.org/show_bug.cgi?id=694368

Release v2.7.90

configure: bump atk dependency to 2.7.5 in order to enusre ATK_ROLE_LEVEL_BAR is available.

Fixes bug 693189.

Release v2.7.5

Update README

The README was very outdated.

Fix various warnings

Support ATK_ROLE_LEVEL_BAR

Simplify autogen.sh

Simply calling autoreconf will do most of what is needed; we don't need
the full gnome-autogen.sh script, and this eliminates incompatibilities
with new versions of automake (ie, 1.13) that the GNOME script did not
check for.

Release v2.7.3

droute: Fix another memory few memory leaks

We need to free the pointer array here, along with the path.

https://bugzilla.gnome.org/show_bug.cgi?id=689887

Stop using deprecated glib functions

GStaticRecMutex is deprecated. Use GRecMutex instead. Also, this means
that we need at least glib 2.32, so specify that in configure.ac.

Re-register if the registry goes away and later returns

Don't return DBUS_HANDLER_RESULT_HANDLED for NameOwnerChanged signals

NameOwnerChanged signals are generic signals sent out by dbus-daemon,
not really specific to this code, so the filter should return
DBUS_HANDLER_RESULT_NOT_YET_HANDLED for them so that libdbus will
propagate them to other filters (specifically the one in at-spi2-core,
which shares a D-Bus connection with at-spi2-atk).

Release v2.7.2

droute: Fix memory leak in path cleanup

    Frees all allocated memory.

    g_ptr_array_free(..., FALSE) was probably being called to avoid a destructor being called on the elements of the array that point to static data. But that method returns an array that must still be g_free()d.

    https://bugzilla.gnome.org/show_bug.cgi?id=688363

socket_embed_hook() should return immediately if spi_global_register is NULL

Remove dbind

Dbind was unused here except formerly in the droute test.

Fix droute test, and remove dbind dependency

The droute test was broken (it had not been updated for recent API
changes and also did not return a non-zero exit code on failure). Also,
it is the only thing in at-spi2-atk that uses dbind, so re-worked it to
call libdbus directly, so that we can remove dbind / avoid having it
both here and in at-spi2-core.

atk-adaptor: don't emit a critical in case the bridge was not initialized

Fixes running with NO_AT_BRIDGE=1

https://bugzilla.gnome.org/show_bug.cgi?id=684334

Up libatspi dependency to 2.5.92

At-spi2-atk fetches a shared D-Bus connection via libatspi and now
expects this connection to be private, so it is incompatible with
versions of at-spi2-core older than 2.5.92.

https://bugzilla.gnome.org/show_bug.cgi?id=686667

Release v2.7.1

Fix unused functions when compiling w/o debug

https://bugzilla.gnome.org/show_bug.cgi?id=678045

Remove empty spi_cache_dispose() function

https://bugzilla.gnome.org/show_bug.cgi?id=678045

droute: Fix compile-time warnings

Unused variables and discarding 'const' qualifier.

https://bugzilla.gnome.org/show_bug.cgi?id=678045

Remove schema

Setting atk-bridge-location can cause problems on multilib builds and
should now be unnecessary since atk-bridge is installed into the normal
library path. No other keys were present in the schema, so removing it.

Release v2.6.0

Register the cache on the main D-Bus connection

In atk_bridge_adaptor_cleanup, droute will deregister the cache object
path from the main DBusConnection, causing libdbus to complain since it
was not registered. Fixes a regression introduced when deferring
registration of the cache.

Fix crashes in atk_bridge_adaptor_cleanup

When freeing data, atk_bridge_adaptor_cleanup frees the wrong pointer,
resulting in a crash. Also, we need to close D-Bus connections before
removing the last reference, or libdbus will abort.

https://bugzilla.gnome.org/show_bug.cgi?id=684434

Release v2.5.92

Fix an errant g_object_unref

If an object is deregistered but not in the cache, then only unref it if
it was found in the queue of objects to be added (ie, it is possible
that it is leased, rather than queued to be cached, in which case it
should not be unrefed).

Don't leak pending calls

https://bugzilla.gnome.org/show_bug.cgi?id=684077

Don't leak message

https://bugzilla.gnome.org/show_bug.cgi?id=684077

Don't leak strings

https://bugzilla.gnome.org/show_bug.cgi?id=684077

Close the connection when we are done with it

Since it is no longer a shared connection

https://bugzilla.gnome.org/show_bug.cgi?id=684077

Make cleanup more robust

https://bugzilla.gnome.org/show_bug.cgi?id=684042

atk-adaptor: plug some refcount leaks

cache->add_traversal holds GObject references, so when we remove/pop
items from it, we should also drop the reference, unless it's transfered
to another queue.

https://bugzilla.gnome.org/show_bug.cgi?id=683979

Remove erroneous debug print introduced in last commit

Only initialize the cache when an AT is running

If no AT is running, we shouldn't need to be maintaining the cache, so
only initialize when an AT is active. Note that this change does not
deinitialize the cache when an AT is no longer running. Deinitializing
and setting spi_global_cache to NULL might be a good idea, but trying to
keep risk to a minimum this close to 3.6.

Fix build for removal of po/ directory

Various things missed.

Removed po directory

i18n support has been removed from at-spi2-atk
(See commits ed260a6 and bc07db9, bug #678035)

Correctly cast hyperlinks in GetURI

https://bugzilla.gnome.org/show_bug.cgi?id=683182

Release v2.5.91

Updated Polish translation

Updated Latvian translation

Removing gtk+3 directory from AC_CONFIG_FILES on configure.ac

That directory was removed on a previous commit

https://bugzilla.gnome.org/show_bug.cgi?id=678315

Fix for Bug 678315 - Remove the bridge module for Gtk+ 3

Release v2.5.90

When a client is removed, don't continue looking through the list of clients

https://bugzilla.gnome.org/show_bug.cgi?id=679297

Don't leak GError when option parsing fails

https://bugzilla.gnome.org/show_bug.cgi?id=679296

Don't iterate over the list twice when removing events

https://bugzilla.gnome.org/show_bug.cgi?id=679295

Release v2.5.5

Allow to build out of source directory

https://bugzilla.gnome.org/show_bug.cgi?id=680280

Implement GetLocalizedName for actions

https://bugzilla.gnome.org/show_bug.cgi?id=680598

Release v2.5.4

Only send property-change signals relevant for caching, absent listeners

Previously, all property-change signals were passed on, regardless of any
registered listeners, because some of them are used by libatspi to keep the
cache up-to-date. However, gtk sends many notifications for property changes
that would have no impact on libatspi's caching code, so we should not waste
time building and sending D-Bus messages for them unless something is
interested in the event.

https://bugzilla.gnome.org/show_bug.cgi?id=674719

Only add items to the cache on children-changed if they are passed in

Gtk tree views, for instance, send children-changed signals without passing
an object. If a toolkit does this, then assume that it is doing so because
fetching the object would be expensive, and/or the object is transient, so
just have the cache bypass the signal rather than ref the child in order to
add it. Helps with performance when creating large gtk+ tree views.

https://bugzilla.gnome.org/show_bug.cgi?id=674719

Don't try to set up the server with a main loop if there's no server

Only create a directory and a socket when requested

Currently, atk-bridge clutters XDG_RUNTIME_DIR with directories for sockets,
which is especially annoying since gtk currently does not call
atk_bridge_adaptor_cleanup, so the directories never go away. This change does
not really solve the problem--it only hides it from users who do not really
need AT-SPI--but, nevertheless, there is no reason to create a directory or a
socket if nothing has requested it, so doing this lazily makes sense.

atk-adaptor: plug a refcount leak

https://bugzilla.gnome.org/show_bug.cgi?id=679285

atk-adaptor: plug a refcount leak

https://bugzilla.gnome.org/show_bug.cgi?id=679285

Don't create a sub-directory for the socket when running as root

If an application is running as root but within a user's runtime directory,
then creating a temporary directory will result in the directory being owned
by root, and the user will not have permission to search the directory to
access the socket, so an AT running as the normal user will not be able to
connect to the application running as root.
Fixes regression introduced by the fix for BGO#678348.

Fix for bug #679013 - AtspiCollection should be implemented for all containers

Release v2.5.3

Add atkbridge.symbols to EXTRA_DIST

Fix some more warnings

Reinstate gnome_accessibility_module_init/shutdown for the gtk 3 module

Removing gnome_accessibility_module_init/shutdown will break Unity, for one,
and there is no real reason to remove it, other than it being an old API that
is to be deprecated and should not be needed anymore, but the new API is not
necessarily finalized, so might as well reinstate the old method for now until
we are certain what the new API will be and can then request that, ie, Unity be
patched accordingly.

Use XDG_RUNTIME_DIR to hold sockets, and do not make a world-writable dir

If we use XDG_RUNTIME_DIR, then the directory should be owned by the
appropriate user, so it should not need to be world-writable. Hopefully this
won't break accessibility for administrative apps on some distro.

https://bugzilla.gnome.org/show_bug.cgi?id=678348

Only register events at beginning if AT is listening

https://bugzilla.gnome.org/show_bug.cgi?id=678475

Make extern the globals in introspection.h

To fix duplicate symbol _spi_org_a11y_atspi_Accessible in
../atk-adaptor/adaptors/.libs/libatk-bridge-adaptors.a(libatk_bridge_adaptors_la-accessible-adaptor.o)
and .libs/libatk_bridge_2_0_la-introspection.o

more uint -> guint fixes

Missed one with the last commit.

bridge.c: change 'uint' for 'guint'

'uint' seems to work on Linux (GNU extension?) but it breaks on Mac OS,
so use 'guint' instead.

Remove duplicate code from GTK+ modules

Now that the AT_NO_BRIDGE check lives in atk-bridge-2.0

Put declarations at the beginning of the block

Reset inited variable on cleanup

Move NO_AT_BRIDGE test to the library

https://bugzilla.gnome.org/show_bug.cgi?id=678125

Fix undeclared G_BEGIN_DECL usage

Fix compile-time warnings

Unused variables, assignment used as truth values and unused functions.

https://bugzilla.gnome.org/show_bug.cgi?id=678045

Namespace the exported API

https://bugzilla.gnome.org/show_bug.cgi?id=678051

Export the atk-bridge-2.0 library

https://bugzilla.gnome.org/show_bug.cgi?id=678051

Link the GTK+ modules against the private library

https://bugzilla.gnome.org/show_bug.cgi?id=678037

Remove unneeded X11 dependency

https://bugzilla.gnome.org/show_bug.cgi?id=678036

Move adaptor_*() functions into their own header

https://bugzilla.gnome.org/show_bug.cgi?id=678037

Move GTK+ modules to their own source files

https://bugzilla.gnome.org/show_bug.cgi?id=678037

Remove unused variables

https://bugzilla.gnome.org/show_bug.cgi?id=678039

Remove unused functions

https://bugzilla.gnome.org/show_bug.cgi?id=678040

Create an uninstalled library

https://bugzilla.gnome.org/show_bug.cgi?id=678037

Export adaptor_init() and adaptor_cleanup()

https://bugzilla.gnome.org/show_bug.cgi?id=678037

build: Quiet by default

https://bugzilla.gnome.org/show_bug.cgi?id=678041

Remove duplicate use of DBUS_LIBS

https://bugzilla.gnome.org/show_bug.cgi?id=678038

Remove unused X11 display code

https://bugzilla.gnome.org/show_bug.cgi?id=678036

po: No files left to translate

https://bugzilla.gnome.org/show_bug.cgi?id=678035

No need to include i18n support

There's nothing to translate here.

https://bugzilla.gnome.org/show_bug.cgi?id=678035

Release v2.5.2

Fix for bug 677211 - The collection interface's MATCH_ANY fails for states

Update autogen.sh, in order to support automake 1.12