Fix another crash when removing hung processes

There was another instance where we were freeing the wrong data.

Remove the correct datum when clearing a hung process

When clearing hung processes, we were passing the wrong variable to
g_list_remove, meaning that the datum would never be removed from the
list, even though it had been freed, resulting in a crash.

Fix a crash when parsing events

If an event listener specifies a detail for an event, but an event
arrives with an empty detail, then libatspi might crash.

build: Use gobject-introspection's Makefile instead of rolling our own

Invoking the GI compiler through libtool would cause us to embed an
RPATH in our binaries.

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

Fix atspi_device_listener_new_simple and some other warnings

Re-register keystroke listeners if the registry goes away and returns

Fix atspi_accessible_get_index_in_parent when the parent is not cached

Atspi_accessible_get_index_in_parent was returning -1 if the object's
parent is NULL, regardless of whether the accessible_parent field is
currently valid.

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

Release v2.7.2

[registry] Fix crash handling DeregisterKeystrokeListener

atspi: Readd dbus-1 to introspection generation

It is presently used in AtspiApplication.

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

Some introspection fixes to make vapigen happy

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

Fix gtk-doc for AtspiRegistry.

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

Don't distribute dbind-config.h

Dbind-config.h is generated by configure and should not be part of the
release.

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

Only call an event listener once per event

If an event listener is registered for an event more than once (ie,
registering both the event and its superset), then only notify it once.

Release v2.7.1

Updated Slovak translation

Don't output a warning when a p2p connection isn't available

Update cache in response to role change notifications

We update the cache in response to name, description, and parent
property-change signals, but we were missing code to handle signals for
role changes.

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

Release as v2.6.0

Release v2.5.92

Don't leak dbus connections

By using a private connection and closing it.

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

Don't check for NULL when calling _atspi_application_new

g_object_new will error out rather than return NULL when out of memory,
so no need to check its return value.

Don't leak the display string

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

Make build silent

So we can see warnings better.

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

Don't leak errors

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

Exit without a SIGABRT when unable to open X display

At-spi2-registryd was calling g_error when unable to open the default
display, which led to a SIGABRT and a core dump. This behavior is
inconsistent with what is generally done by GNOME apps, so let's call
g_warning() and exit(1) instead.

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

Fixing 'atspi_accessible_get_id' annotation

Fix some annotations compilation warnings

Release v2.5.91

Updated Polish translation

Add glib-2.0 to requires in at-spi2.pc.in

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

Fix build with -fno-common

Add license to atspi-constants.h

Release v2.5.5

Allow to build out of source directory

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

Add atspi_action_get_localized_name

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

Code clean-ups

There isn't really any point in checking for GLib functions returning NULL
due to lack of memory, since they just abort instead.

Release v2.5.4

Fix abort on an error while removing a match rule

Consistently return a value - from joerg@netbsd.org

Release v2.5.3

atspi-misc: Prevent leak

We need to free the data that XGetWindowProperty returns back to us.

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

autogen.sh: Honor NOCONFIGURE=1

See http://people.gnome.org/~walters/docs/build-api.txt

Release v2.52

Fix crash in atspi_deregister_keystroke_listener

Atspi_registry_keystroke_listener has code to copy a keyset before passing
to dbind, in order to handle NULL keystrings, but this code was not present
in atspi_deregister_keystroke_listener.

https://bugs.freedesktop.org/show_bug.cgi?id=50578

_ATSPI_DBUS_CHECK_SIG: remove warning when no message received

Add G_BEGIN_DECLS and G_END_DECLS macros for prototypes to fix c++ linkage

Add a check for a null message when returning a hyperlink

Release v2.5.1

Fix for bug 675004 - object:state-changed:defunct events are emitted as object:state-change:defunct

Fix type in atspi_document_get_attribute_value

    atspi_document_get_attribute_value was failing because of a typo.

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

Increase start-up timeout to 15 seconds

It seems that a five-second time-out is often not sufficient.

Fix GError sent when receiving a reply of the wrong type

When a method reply of the wrong type was received, the parameters were
still sent backwards when formatting the GError.

Set the application start-up timeout to 5 seconds by default

The application timeout is now implemented in 2.4 but is not settable.
Instead, there is a default of 5 seconds, so the default should be
consistent in 2.5.
Also, document the default values.

Add startup_time parameter to atspi_set_timeout

Add a parameter to allow timeouts not to be enforced until an
application has been running for a given amount of time. Might help
prevent timeout exceptions for applications that get bogged down
performing tasks at initialization.

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

Add atspi_set_timeout

Release v2.4.0

hindi translation by Chandan Kumar

Fixed typo. Closes bug #605662

Release v2.3.92

Fix crash fetching error when retrieving a property

Increase the keystroke event timeout

It seems that it is currently possible for at-spi2-atk to get bogged
down when sending a keystroke notification, so 1 second is a short time
to wait for a reply. It may be possible to mitigate this in the future,
in which case we could reduce the timeout again, but not going to do
this for 3.4. See comment on
https://bugzilla.gnome.org/show_bug.cgi?id=671976.

[registry] Send a ping to a process when marking as hung

Sometimes keys that should be captured by event listeners are instead
passed through to the application. It seems that at-spi2-registryd might
not always be removing the process from the list of hung processes. I
have not reproduced this recently, and I'm not sure where the root
problem is. Hoping that it will help to send the process a ping and use
the reply as a queue that it is un-hung (previously we were relying on
receiving a reply to the original key event notification).

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

Malayalam translation by Anish

Finnish translation update from l10n.laxstrom.name/wiki/Gnome_3.4 translation sprint

Add initial Khmer translation

Fix a couple of NEWS typos

Release v2.3.91

Enable caching on a call to atspi_accessible_set_cache_mask

Previously, caching was always disabled unless atspi_event_main() was
called to start a main loop, since we need to handle the case of a
simple application/script that performs an action and then sleeps rather
than entering a main loop. In such a case, libatspi will not be able to
detect that an event has been fired to update the cache, so caching
should be disabled. However, an application may want to start a main
loop through some other mechanism than by calling atspi-event_main(),
and caching should be done in such a case. So we'll assume that, if
atspi_accessible_set_cache_mask is called, then the application expects
caching to work and will run a main loop one way or another. Note that
it is possible for a script to run a main loop in some instances but not
others, and, in such a case, if it is not using atspi_event_main(), then
it should set caching to ATSPI_CACHE_NONE when not in a main loop and
back to ATSPI_CACHE_DEFAULT when running a main loop.

[l10n] Added Estonian (et) to LINGUAS

[l10n] Updated Estonian translation

Don't cache data for transient objects

If an object is known to be transient, then ignore its cache, since it
seems better not to rely on a toolkit to send updates for transient
objects.
Note that this isn't really a complete solution, since it only disables
the cache if STATE_TRANSIENT is known to be present. If AddAccessible
was not sent for the object and atspi_accessible_get_state_set has not
been called, then the state set is not known, so STATE_TRANSIENT is not
"known to be present..." It may be a good idea to fetch the state set if
it is not known, or check the parent for STATE_MANAGES_DESCENDANTS,
although in the latter case it is also possible that the parent's state
set is not known, and it may be needed to check the parent's parent and
so on.

Adding Uzbek@cyrillic translation.

Reduce the method call timeout slightly

It is likely a good idea for the method call timeout to be lower than
the registry daemon's keystroke listener timeout. This way, if an AT
makes a method call that times out while in a keystroke event listener,
it will still have time to exit the listener before the registry daemon
presumes it hung.

Send key events to listeners assumed to be hung, but don't block

Orca would sometimes fail to notice that its modifier key was released
since it was presumed to be hung and would not be sent the event. So,
rather than ignore event listeners assumed to be hung, we'll send them
the notifications for the keys that they're listening to but not wait
for a reply, instead assuming that they are not consumed.

Expect a signed int for GetIndexInParent, per the spec

Fix atspi_table_get_row_column_extents_at_index

atspi_table_get_row_column_extents_at_index was trying to read
parameters from the dbus message in the wrong order and also not
returning the row.

Fix message displayed when a call returns the wrong type

The message displayed by dbind for a type mismatch had its parameters
inverted.

Invalidate states when receiving a focus event

Sometimes we get "focus" events from gtk, but the cache still does not
indicate that the object has STATE_FOCUSED. Really should get to the
bottom of this but don't have time to investigate more right now. Might
be some kind of issue with the accessible being added to the cache
before it is fully initialized--perhaps AddAccessible really doesn't
make sense.

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

Release v2.3.90

Don't g_object_unref a NULL object for event variant

If we get an event with what appears to be an accessible as the variant
but we do not succeed in marshalling the accessible (possibly because it
is the null path), do not unref the null object. Stops a glib warning.

If a keystroke listener does not respond, disable until it responds

If an AT is hung or is suspended while having a keystroke listener
registered, then this should not lock up the desktop, as would happen if
we wait indefinitely for the keystroke listener to respond.

At-spi-bus-launcher signal emission fixes

Emit PropertiesChanged from the correct interface. Also, create
GVariantBuilders on the stack; fixes memory leaks.

Release v2.3.5

For IsScreenReaderEnabled, listen for the correct gsettings change signal

Fix atspi_collection_get_matches_to prototype

For atspi_collection_get_matches_to, rename restrict to limit_scope

Restrict can be a keyword. Don't use it as a variable name.
Also update D-Bus introspection.

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

Updated Norwegian bokmål translation

Add ScreenReaderEnabled d-bus property and proxy for GNOME setting

Add a ScreenReaderEnabled property to org.a11y.Bus, and proxy the
org.gnome.desktop.a11y.applications screen-reader-enabled gsettings key.
This will allow Orca to monitor the key without depending on gsettings
and a GNOME-specific key.

Call g_settings_sync after updating IsEnabled

Ensure that the GSettings key is updated immediately upon receiving a
D-Bus message to set the property.

Have dbind_method_call_reentrant_va check the signature of replies

When making a method call, we should check that the signature of the
reply matches what we expect and throw an error if not. Otherwise we may
crash.

Set correct end offset in atspi_text_get_attributes

Fixes bug 668334

Release v2.3.4

Avoid making calls on processes thought to be hung

If a method call times out, record the process as being hung, and send a
ping. Consider the process to be no longer hung when a response to the
ping is received.

Hopefully this will prevent long delays if an AT makes consecutive calls
on an application that is not responding.

Using value instead of memory address when calling _atspi_dbus_call

The following functions are changed: atspi_selection_select_child,
atspi_selection_deselect_child and atspi_selection_is_child_selected

Fixes bug 667254

Allow applications to query keyboard and mouse listeners

Add methods to list the currently-active keyboard and mouse listeners
and signals to be fired when listeners are registered or deregistered.

Part of the fix for http://bugzilla.gnome.org/show_bug.cgi?id=666870

Warn if a device listener returns a non-bool, rather than crash

Revert part of last commit

Modifiers and hw_code are still sent as int16 for now, so it was wrong
to remove the code to parse them as int16, at least for the time being.

Add some type checks for dbus messages, and remove some old code

Added a couple of message signature checks where it was still TODO (and
might have led to a crash if the wrong signature was sent). Also removed
some cases where two different signature types were accepted for API
compatibility reasons--the old API is no longer used.

Add new Japanese translation