2001-12-10 Michael Meeks <michael@ximian.com>
[platform/core/uifw/at-spi2-atk.git] / ChangeLog
index 90db9f4..92845b6 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
+2001-12-10  Michael Meeks  <michael@ximian.com>
+
+       * configure.in: use cspi/libspi.pc.in instead.
+
+       * Makefile.am (pkgconfig_DATA): upd. to match.
+
+       * test/simple-at.c (report_focus_event): kill hacks around
+       bad return values, use putenv not setenv
+
+       * libspi/desktop.c (impl_desktop_get_child_at_index): don't
+       fire ChildGone - I killed it.
+
+       * libspi/component.c
+       (impl_accessibility_component_get_extents): remove
+       bogus return.
+
+       * idl/Accessible.idl: kill ChildGone exception.
+
+       * cspi/*.[ch]: kill int return from ref / unref.
+       
+       * cspi/spi_main.c (cspi_object_add_check): fold into
+       (cspi_object_add): here.
+
+       * cspi/spi_component.c (AccessibleComponent_getExtents):
+       handle exceptions elegantly.
+
+       * cspi/spi-private.h (cspi_check_ev_return),
+       (cspi_return_if_fail): impl.
+
+       * cspi/spi_accessible.c: use extensively.
+       (AccessibleStateSet_equals): add direct compare check.
+
+       * cspi/spi_selection.c: tolerate NULL objs, and check
+       exceptions before return.
+
+       * cspi/spi-util.c (cspi_warn_ev): rename to
+       (cspi_check_ev): this & don't pass ev in; no point.
+
+2001-12-10  Michael Meeks  <michael@ximian.com>
+
+       * test/test-simple.c: use putenv not setenv
+       (get_environment_vars): more chatty if you're not using
+       the fun bits.
+
+       * test/simple-at.c: do the setenv so more people see the
+       nice demo easily.
+
+2001-12-10  Bill Haneman <bill.haneman@sun.com>
+
+       * libspi/registry.c:
+       Changed check of CORBA_environment on notification to be a warning
+       instead of an error for the moment, since we can recover from this
+       error when caused by a queued notification from a dead app.
+       
+       * libspi/value.c:
+       Fixes for spi_value, use G_TYPE_DOUBLE for atk_value values (as
+       used by all current implementors of AtkValue), and coerce to
+       CORBA_float. (Proper general case fix may require change to Value.idl).
+
+2001-12-09  Bill Haneman <bill.haneman@sun.com>
+
+       * cspi/spi.h:
+       * cspi/spi_event.c:
+       * cspi/spi_accessible.c:
+       Documentation fixes: added user_data params to documentation for 
+       listeners and callbacks.  
+
+       * cspi/spi_accessible.c: 
+       Changed AccessibleStateSet_compare to return a
+       StateSet rather than return the difference set into a third parameter.
+       
+       
+2001-12-09  Bill Haneman <bill.haneman@sun.com>
+
+       * configure.in:
+       Replace use of AM_PROG_XML_I18N_TOOLS macro with AC_PROG_INTLTOOL.
+       remove 'dnl' comment line from AC_OUTPUT (autoconf doesn't like
+       it). 
+
+       * Makefile.am:
+       Remove subdirectory po from SUBDIRS for now.
+
+       * at-bridge/bridge.c:
+       Beefed up a couple of debug printouts.
+       One-line fix for signal notification name formatting.
+
+       * libspi/accessible.c:
+       Added assertion to spi_accessible_new ().
+
+       * libspi/application.c:
+       Put #ifdef qualifiers around a printf.
+
+       * libspi/value.c:
+       Fixed derivation (from BONOBO_OBJECT to SPI_BASE).
+
+       * registryd/Makefile.am:
+       * util/Makefile.am:
+       Replaces use of XML_I18N_MERGE_SERVER_RULE with
+       INTLTOOL_SERVER_RULE macro.
+
+       * test/simple-at.c:
+       Added an event listener for several ATK signals, for
+       testing and debugging.  Removed a few g_warnings.
+       Added test output if focussed item implements the Value interface.
+
+       * test/test-simple.c:
+       Added test_value () function (known not to pass at the moment, so
+       not actually called yet.
+       
+2001-12-08  Michael Meeks  <michael@ximian.com>
+
+       * at-bridge/bridge.c (bridge_focus_tracker),
+       (emit_eventv): fix the same ref. leak.
+       (register_atk_event_listeners): don't leak.
+
+       * cspi/spi-listener-impl.c (cspi_event): more 
+       protection from client code.
+
+       * test/test-simple.c (test_desktop): unref the app.
+
+       * test/Makefile.am: upd. TESTS.
+
+       * at-bridge/bridge.c (bridge_exit_func): release the registry.
+
+       * test/keysynth-demo.c (keysynth_exit): unref the listeners.
+
+       * cspi/spi_accessible.c (Accessible_queryInterface): constify.
+
+       * cspi/spi_registry.c: constify in strings.
+       (deregisterGlobalEventListenerAll): don't release the ref.
+
+       * cspi/spi.h: start removing redundant / conflicting
+       gtkdoc comments, making the API readable at a glance :-)
+       
+       * cspi/spi_event.c (AccessibleEventListener_unref): impl.
+
+       * test/keysynth-demo.c (keysynth_exit, main): upd.
+
+       * test/simple-at.c (main, simple_at_exit): upd.
+
+       * test/test-simple.c (unutterable_horror): kill.
+       (utterable_normal_derefs): kill.
+       (main): upd & do a setenv before gtk_init, so that we
+       can use memprof - and to save the pain of remembering
+       this, if we're running under memprof - don't exit.
+
+       * cspi/spi_main.c (SPI_exit): return exit status
+       for cspi.
+       (SPI_event_quit): impl.
+       (SPI_event_main): simplify.
+       (cspi_is_gnome_app): kill.
+       (SPI_init): kill isGNOMEapp arg.
+
+       * at-bridge/bridge.c (register_atk_event_listeners):
+       re-instate the unref.
+       (bridge_exit_func): don't re-activate the registry,
+       protect vs. theoretical re-entry, use the
+       AT_BRIDGE_SHUTDOWN env. var to determine whether to
+       assert a clean shutdown - for regression tests.
+
+2001-12-08  Abel Cheung  <maddog@linux.org.hk>
+
+       * configure.in: Comment out dummy i18n support for now,
+         otherwise it fails to build.
+       * idl/Makefile.am: Add Selection.idl and Hypertext.idl to
+         EXTRA_DIST. They are missing in tarball.
+
+2001-12-07  Michael Meeks  <michael@ximian.com>
+
+       * util/magnifier.c: kill bonobo activation reg. race.
+
+       * libspi/component.c
+       (impl_accessibility_component_get_extents): make it match
+       it's sig.
+
+       * libspi/registry.c (_registry_notify_listeners):
+       re-remove X headers & reconcile conflicts.
+       (spi_listener_struct_free): bad conflict resolve.
+       (impl_accessibility_registry_register_application):
+       more bad merging fixed.
+       (impl_accessibility_registry_deregister_application): ditto.
+       (spi_registry_new): ditto.
+       (spi_registry_init): don't have an applications field.
+
+[ merge fixups2 branch to here ]
+2001-12-07  Michael Meeks  <michael@ximian.com>
+
+       * test/simple-at.c (report_focus_event),
+       (report_focussed_accessible): remove g_warning debug.
+
+2001-12-07  Michael Meeks  <michael@ximian.com>
+
+       * test/test-simple.c (validate_tree): comment out some checks.
+       (test_desktop): app now not registered idly - horay.
+
+2001-12-07  Michael Meeks  <michael@ximian.com>
+
+       * cspi/spi_registry.c
+       (registerAccessibleKeystrokeListener): clean, stop it leaking
+       great swathes of memory, and device event controller references.
+       ensure we always pass valid sequences.
+       (deregisterAccessibleKeystrokeListener): ditto.
+       (generateKeyEvent): upd.
+
+       * at-bridge/bridge.c (bridge_key_listener): handle
+       registry dead exception gracefully.
+       (bridge_focus_tracker, emit_eventv): free ev.
+
+       * test/simple-at.c (report_focus_event): fix polarity.
+
+       * libspi/application.c (notify_listeners): kill the ev
+       argument, and make sure we init the ev we use.
+       (spi_application_object_event_listener): split out a chunk
+       (get_atk_object_ref): here & fixup.
+       (reverse_lookup_name_for_toolkit_event): make const.
+       (spi_application_toolkit_event_listener): fixup, kill leaks
+       ref problems etc.
+
+       * at-bridge/bridge.c (register_atk_event_listeners):
+       don't unref the noop object - causes grief.
+
+       * libspi/registry.c: remove the X headers.
+       turn off the debug churn.
+
+       * libspi/accessible.c (spi_accessible_new_return): only
+       release the ref if we are supposed to [doh]
+
+       * test/simple-at.c (report_button_press, report_focus_event):
+       guard vs. unexpected exit.
+
+       * at-bridge/bridge.c (gtk_module_init): prune printf.
+       (bridge_exit_func): fix app unref.
+
+       * libspi/Makefile.am: don't install registry or desktop.h
+
+       * libspi/*.h: update includes.
+
+       * libspi/application.h: don't include ourself, or
+       other redundant headers.
+
+2001-12-06  Michael Meeks  <michael@ximian.com>
+
+       * libspi/accessible.c
+       (get_accessible_from_servant): impl.
+       Upd. all CORBA impl.s to use it killing cut and paste.
+
+       * cspi/spi_main.c (cspi_cleanup): guard vs. double
+       invokes.
+       (SPI_init): register atexit cleanup handler here.
+
+2001-12-06  Michael Meeks  <michael@ximian.com>
+
+       * at-bridge/bridge.c (gtk_module_init): g_error on
+       activation exception
+
+2001-12-06  Michael Meeks  <michael@ximian.com>
+
+       * at-bridge/bridge.c (gtk_module_init): some sort of
+       confusing merge clash - reverted.
+       (bridge_idle_init): ditto.
+
+       * test/test-simple.c (main, utterable_normal_derefs):
+       merge from Bill - somehow not committed.
+
+2001-12-06  Michael Meeks  <michael@ximian.com>
+
+       * libspi/value.c (spi_value_class_init): upd.
+       (spi_value_finalize): kill.
+
+       * libspi/table.c (spi_table_finalize): kill.
+       (spi_table_interface_new): upd.
+       (spi_table_class_init): upd.
+       (get_table_from_servant): impl.
+       Upd. all CORBA impl.s to use it killing yet more cut and paste.
+
+       * libspi/selection.c (spi_selection_finalize): kill.
+       (spi_selection_interface_new): upd.
+       (spi_selection_class_init): upd.
+       (get_selection_from_servant): impl.
+       Upd. all CORBA impl.s to use it killing yet more cut and paste.
+
+       * libspi/relation.c (spi_relation_finalize): kill.
+       (spi_relation_class_init, spi_relation_new): upd
+
+       * libspi/image.c (spi_image_finalize): kill.
+       (spi_image_class_init): upd.
+       (spi_image_interface_new): upd.
+       (get_image_from_servant): impl.
+       Upd. all CORBA impl.s to use it killing yet more cut and paste.
+
+       * libspi/hyperlink.c (spi_hyperlink_class_init): upd.
+       (spi_hyperlink_finalize): kill.
+       (spi_hyperlink_new): upd.
+       (get_hyperlink_from_servant): impl.
+       Upd. all CORBA impl.s to use it killing yet more cut and paste.
+
+       * libspi/hypertext.c (spi_hypertext_interface_new): upd.
+       (spi_hypertext_finalize): kill.
+       (spi_hypertext_class_init): upd.
+       (get_hypertext_from_servant): impl.
+       Upd. all CORBA impl.s to use it killing yet more cut and paste.
+
+       * libspi/editabletext.c (spi_editable_text_finalize): kill.
+       (spi_editable_text_interface_new): upd.
+       (get_editable_text_from_servant): impl.
+       Upd. all CORBA impl.s to use it killing yet more cut and paste.
+       
+       * libspi/text.c (get_text_from_servant): impl.
+       All impls - update to use it killing more cut and paste,
+       also fix bad string returns on duff args ( a somewhat
+       impossible case to handle - but hey ).
+       (spi_text_construct): impl.
+       (spi_text_interface_new): upd.
+       (spi_text_object_finalize): kill.
+
+       * libspi/component.c
+       (accessibility_component_object_finalize): kill.
+       (spi_component_class_init): upd.
+       (get_component_from_servant): impl. helper to kill cut & paste.
+       (impl_accessibility_component_contains): 
+       (impl_accessibility_component_get_accessible_at_point): 
+       (impl_accessibility_component_get_position): 
+       (impl_accessibility_component_get_extents): 
+       (impl_accessibility_component_get_mdi_z_order): 
+       (impl_accessibility_component_get_layer): 
+       (spi_component_interface_new): upd.
+
+       * libspi/action.c (spi_action_finalize): kill.
+       (spi_action_class_init): upd. inherit from SpiObject.
+       (spi_action_interface_new): upd.
+       (get_action_from_servant): impl. helper to reduce cut & paste.
+       (impl_getKeyBinding, impl_getName, impl_doAction),
+       (impl_getDescription): Use it everywhere.
+
+       * cspi/spi_main.c (get_live_refs): a pointer hash.
+       (spi_object_hash, spi_object_equal): comment out.
+       There are issues with CORBA object hashing that need
+       solving.
+
+       * libspi/application.c (spi_application_init),
+       (spi_application_new): upd.
+
+       * libspi/accessible.c (spi_accessible_object_finalize): kill.
+       (spi_accessible_class_init): upd.
+       s/accessible/object/ in epv methods.
+
+2001-12-05  Bill Haneman <bill.haneman@sun.com>
+
+       * at-bridge/bridge.c:
+       Reorder the bridge initialization into the bridge gtk_module_init,
+       an immediate call to bridge_register_app, and an idle_add for
+       register_atk_event_listeners (the only code that *must* wait until
+       GTK+ is initialized) via bridge_idle_add. Bridge_register_app does
+       the activation, creates the application's Accessible object, and
+       queues the bridge_idle_init function.  Restored call to
+       Accessibility_Registry_deregisterApplication().  
+       Prior to making this call we obtain a new reference to the
+       registry, which is a temporary hack that helps us handle apps that
+       shut down the bonobo mainloop before we get there.
+
+       * libspi/registry.c:
+       Fix string leakage in debug block, and don't segv if a an app
+       dies before its event notifications have been propagated.
+
+       * cpi/spi_util.c:
+       Modify cspi_check_ev() to call g_warning() rather than g_error(),
+       as stopgap measure until we get proper exception handling when 
+       objects die during or before interrogation.  
+
+       * cspi/spi_main.c:
+       * cspi/spi.h:
+       Moved use of isGNOMEApp flag from SPI_event_main to SPI_init.
+       Also changed default value of is_gnome_app to TRUE.
+
+       * test/simple-at.c:
+       * test/keysynth-demo.c:
+       Changed use of SPI_init and SPI_event_main to reflect API change above.
+
+       * test/simple-at.c:
+       Made report_focus_event check the warning value from getName
+       before reporting on focussed accessible; simplifies handling of
+       notifications from dead objects.
+
+       * test/test-simple.c: (Changes not in this patch since source is
+       not in CVS ?)
+       
+2001-12-05  Michael Meeks  <michael@ximian.com>
+
+       * registryd/registryd.c (main): protect against
+       bonobo-activation startup race condition.
+
+       * cspi/spi_main.c (cspi_object_add_check): cope
+       with the exception elegantly.
+
+       * idl/Accessible.idl: decl. ChildGone
+
+       * libspi/desktop.c (spi_desktop_dispose): impl.
+       (spi_desktop_class_init): upd.
+       (spi_desktop_add_application),
+       (spi_desktop_remove_application),
+       (abnormal_application_termination): impl.
+       (impl_desktop_get_child_at_index): re-impl.
+
+       * libspi/registry.c
+       (impl_accessibility_registry_deregister_application),
+       (impl_accessibility_registry_register_application): 
+       kill excessive debug clutter.
+
+       * test/test-simple.c (test_misc): test some NULL
+       tolerance. (main): upd.
+
+       * cspi/spi_main.c (cspi_accessible_is_a): a NULL
+       implements no interfaces - so let's not crash.
+
+       * cspi/spi_accessible.c (Accessible_queryInterface): ditto.
+
+       * libspi/accessible.c (spi_accessible_new_return): impl.
+       helper to calm the cut and paste, to cope with
+       NULL AtkObjects effectively and efficiently, and to kill
+       the referencing bugs.
+
+       * libspi/table.c (impl_getAccessibleAt),
+       (impl_getRowHeader, impl_getColumnHeader): upd.
+
+       * libspi/hyperlink.c (impl_getObject): upd.
+
+       * libspi/accessible.c
+       (impl_accessibility_accessible_get_child_at_index): upd.
+
+       * libspi/component.c
+       (impl_accessibility_component_get_accessible_at_point): upd.
+
+       * test/test-simple.c (test_editable_text, test_text): impl.
+       some simple tests.
+
+       * cspi/spi_editabletext.c
+       (AccessibleEditableText_insertText): constify.
+
+       * test/test-simple.c (focus_me): make more robust.
+
+       * libspi/accessible.c: remove some printfs in methods that
+       we test and work fine, so we can get a clean tree dump.
+
+       * test/test-simple.c (validate_accessible),
+       (validate_tree): make tree dumping pretty too.
+
+       * at-bridge/bridge.c (bridge_signal_listener): upd. so
+       it builds without debug on.
+
+       * test/test-simple.c (validate_accessible): fix the most
+       ludicrous bug.
+
+       * cspi/spi_main.c (spi_object_release): leave the nicely
+       blanked memory around if in debug mode.
+
+       * cspi/spi-listener-impl.c (cspi_event): use add_check.
+
+[committed to fixups2]
+2001-12-05  Michael Meeks  <michael@ximian.com>
+
+       * libspi/editabletext.c
+       (spi_editable_text_class_init): g_type_class_peek_parent,
+       not interface peek - so we finalize cleanly.
+
+       * libspi/accessible.c (spi_accessible_new): kill debug.
+
+       * test/test-simple.c (validate_accessible): fix ref.
+       leak, print strings before freeing them; sigh, comment a
+       chunk out due to strangeness.
+       (validate_tree): unref the child after use.
+
+       * cspi/spi_component.c (AccessibleComponent_getLayer):
+       add missing break statement, switch on zlayer not the
+       uninitialized retval.
+
+       * cspi/spi_main.c (cspi_object_add): add assert, local ref
+       before (possibly re-entering) remote unref.
+
+       * test/simple-at.c (main): unref the desktop.
+
+       * cspi/spi_accessible.c (cspi_accessible_is_a): move 
+       into:
+       
+       * cspi/spi_main.c (cspi_accessible_is_a): here to 
+       isolate bonobo API usage more.
+
+       * libspi/registry.c (impl_accessibility_registry_register_application): 
+       prune bogus comment.
+
+2001-12-04  Michael Meeks  <michael@ximian.com>
+
+       * test/test-simple.c (validate_tree): upd.
+       IndexInParent check to make it more lucid.
+       (validate_accessible): dump the tree.
+
+       * libspi/accessible.c (de_register_public_ref),
+       (get_public_refs): impl.
+       (spi_accessible_new): make more efficient & intuitive.
+       Now IndexInParent check works.
+
+2001-12-04  Michael Meeks  <michael@ximian.com>
+
+       * cspi/spi-util.h: remove.
+
+       * cspi/spi.h: move SPI_freeString here
+
+       * libspi/component.c (impl_accessibility_component_get_layer): 
+       warning fixage.
+       (impl_accessibility_component_get_mdi_z_order): ditto,
+       return -1 on error - is this ok ?
+
+       * test/test-simple.c (test_application): impl.
+
+       * cspi/spi_accessible.c (Accessible_getApplication),
+       (Accessible_isApplication): impl.
+
+       * libspi/registry.c (spi_listener_struct_free): release
+       the CORBA reference too.
+
+       * libspi/accessible.c
+       (impl_accessibility_accessible_get_parent): fix ref leak.
+
+       * libspi/registry.c (spi_registry_new): make the registry
+       immortal - we don't want it to go away ever.
+
+       * libspi/application.c
+       (spi_accessible_application_finalize): impl. cleanup.
+
+       * cspi/spi_registry.c
+       (deregisterGlobalEventListenerAll),
+       (deregisterGlobalEventListener),
+       (deregisterAccessibleKeystrokeListener): fix ref leak.
+
+       * cspi/spi_main.c (SPI_init): remove redundant ref.
+
+       * test/test-simple.c (unutterable_horror): impl. to force a
+       clean shutdown.
+       (main): use it.
+
+       * libspi/registry.c
+       (impl_accessibility_registry_deregister_application): release
+       the application when we de-register it.
+
+       * at-bridge/bridge.c (bridge_exit_func): comment out shutdown
+       until we have the infastructure to get it right.
+
+2001-12-03  Michael Meeks  <michael@ximian.com>
+
+       * libspi/Makefile.am: only build eventlistener.[ch]
+
+       * libspi/accessible.c
+       (impl_accessibility_accessible_get_child_at_index): don't leak.
+
+       * at-bridge/bridge.c (bridge_exit_func): cope with an exit
+       before we hit the idle handler.
+
+       * cspi/spi-util.c (cspi_check_ev): g_error is far nicer for
+       debugging than exit.
+
+       * at-bridge/bridge.c (bridge_focus_tracker): fix ref issue.
+
+       * libspi/listener.c (impl_notify_event): check source
+       not NIL before unref.
+
+       * libspi/registry.c (impl_registry_notify_event): ditto.
+
+       * libspi/eventlistener.c
+       (impl_accessible_event_notify_event): unref the source.
+
+       * libspi/accessibleeventlistener.c
+       (impl_accessible_event_notify_event): unref the source.
+
+       * idl/Event.idl: remove 'hash_id' attribute - of no
+       concievable use.
+
+       * libspi/application.c (notify_listeners): upd list
+       iteration & sig.
+       (spi_application_object_event_listener), 
+       (spi_application_toolkit_event_listener): upd to do correct
+       CORBA ref counting.
+
+       * libspi/registry.c
+       (_registry_notify_listeners): update.
+       (parse_event_type): this method also leaks like a sieve.
+       (impl_accessibility_registry_deregister_global_event_listener): 
+       fix ETYPE_WINDOW case.
+
+       * libspi/application.c
+       (impl_accessibility_application_register_object_event_listener): 
+       kill leak.
+
+       * libspi/hyperlink.c (impl_getObject): fix ref mistake.
+
+       * libspi/hypertext.c (impl_getLink): ditto.
+
+       * cspi/spi_main.c (cspi_object_add): remove
+       bogus CORBA_Object_duplicate.
+
+       * cspi/spi-listener-impl.c (cspi_event): dup/ref
+       on the object_add.
+
+       * cspi/spi_accessible.c
+       (Accessible_getRelationSet): fix leak / referencing issue.
+
+       * cspi/spi_main.c (spi_object_hash, spi_object_equal): impl.
+       (cspi_object_add, cspi_object_unref): upd.
+       (spi_object_release, get_live_refs): impl.
+       (cspi_cleanup): simplify.
+
+       * *.[ch]: s/boolean/SpiBoolean/
+
+       * cspi/spi_accessible.c (role_names): actualy order so it
+       corresponds with the enum !
+
+       * test/test-simple.c (test_roles): check some role strings
+       vs. enums to detect offset errors.
+
+2001-12-01  Michael Meeks  <michael@ximian.com>
+
+       * cspi/spi_main.c (cspi_object_add_check): impl. helper.
+
+       * cspi/spi_accessible.c (Accessible_getChildAtIndex):
+       check ev before 
+       (Accessible_getChildAtIndex): use cspi_add_check so we
+       check the ev before using a (possibly) invalid value.
+
+       * libspi/accessible.c
+       (impl_accessibility_accessible_get_parent): don't segv
+       on a NULL parent.
+
+       * at-bridge/bridge.c (gtk_module_init): upd.
+       (bridge_property_event_listener): don't leak strings
+       and events !
+       (bridge_state_event_listener),
+       (bridge_property_event_listener): don't use a static
+       buffer in case of recursive event emission &
+       re-enterancy - split out the swathe of cut and paste
+       into:
+       (emit_eventv): and fix 2 ref leaks here & comply to the
+       coding style.
+       (bridge_register_app): rename to 
+       (bridge_idle_init): this and kill arg parsing, split
+       various inits & bonobo-activation registration to module
+       init time, kill referencing error.
+       (bridge_exit_func): fix the same leak.
+       (bridge_focus_tracker): don't leak event structures
+       and strings.
+       (bridge_signal_listener): more leakage and cut & paste
+       fixage.
+       (bridge_key_listener): return the result - FIXME is
+       this polarity correct ?
+       (accessibility_keystroke_from_atk_key_event): rename
+       (accessibility_init_keystroke_from_atk_key_event): and stop
+       leakage.
+       (gtk_module_init): return a value - not that it is checked by
+       Gtk - but hey.
+       (bridge_key_listener): register my horror at casting between
+       IDL defined structs.
+       (bridge_get_registry): kill - not used.
+
+       * cspi/spi_accessible.c (Accessible_getRole): make return const.
+
+       * test/test-simple.c (test_roles): impl.
+       (test_desktop, validate_accessible, validate_tree): impl.
+
+       * cspi/spi_accessible.c (role_names): fixup bogus
+       MAX_ROLES setup, make constants const for efficient linkage
+       (AccessibleRole_getName): conform to coding style, make
+       return 'const'
+
+2001-11-30  Michael Meeks  <michael@ximian.com>
+
+       * cspi/spi.h: add missing prototype.
+
+       * libspi/registry.c (spi_listener_struct_new): fix sig.
+
+       * test/Makefile.am: add test-simple
+
+       * test/test-simple.c: add
+
+       * test/test.sh: add.
+
+       * cspi/spi_main.c (SPI_init): make SPI_inited module global,
+
+       * at-bridge/bridge.c (gtk_module_init): don't do an idle init,
+       do it straight - since otherwise we often get an activation race.
+
+[ work before Bill duplicated a chunk of it ... ]
+2001-11-30  Michael Meeks  <michael@ximian.com>
+
+       * libspi/keystrokelistener.c
+       (spi_keystroke_listener_class_init): fixup arg
+       count (doh).
+
+       * libspi/eventlistener.c
+       (spi_event_listener_class_init): ditto.
+
+       * tests/test.sh: add.
+
+       * cspi/spi-listener-impl.h: update macros.
+
+       * cspi/spi-listener-impl.c
+       (cspi_event_listener_remove_callback): add
+       correct pre-conditions.
+
+       * cspi/spi_main.c (SPI_exit): guard vs. double
+       exits, exit the mainloop nicely.
+       (SPI_event_main): make more sense.
+       (SPI_init): move the atexit evilness here so we
+       always do it (yuck).
+       (SPI_event_main): upd.
+
+       * test/simple-at.c: update for listener changes.
+
+       * registryd/registryd.c (main): prune kruft.
+
+       * libspi/libspi.h: upd.
+
+       * libspi/eventlistener.[ch]: rename broken macros.
+
+       * libspi/accessibleeventlistener.[ch]: remove.
+
+       * libspi/*.[ch]: IS_SPI_... -> SPI_IS_...
+
+       * test/test-simple.c: Add - automated tests.
+
+       * cspi/spi-listener-impl.c (cspi_event),
+       (cspi_key_event): add user_data to invoke.
+       (cspi_keystroke_listener_finalize): impl.
+       (cspi_keystroke_listener_class_init): upd.
+
+       * cspi/spi_event.c
+       (createAccessibleEventListener),
+       (AccessibleEventListener_addCallback),
+       (createAccessibleKeystrokeListener),
+       (AccessibleKeystrokeListener_addCallback): upd.
+
+       * libspi/deviceeventcontroller.c
+       (spi_device_event_controller_check_key_event): add public proto.
+
+       * libspi/registry.c
+       (impl_accessibility_registry_deregister_global_event_listener): 
+       always init listeners.
+       (_registry_notify_listeners): yes it is.
+
+       * cspi/spi-listener-impl.c (cspi_key_event): 
+       move debug to here.
+       (cspi_event_listener_finalize): impl.
+       (cspi_event_listener_class_init): hook up.
+       (event_handler_new, event_handler_free),
+       (event_list_remove_by_callback): impl.
+       (cspi_event_listener_remove_callback),
+       (cspi_keystroke_listener_remove_callback),
+       (cspi_keystroke_listener_add_callback),
+       (cspi_event_listener_add_callback): upd.
+
+       * libspi/keystrokelistener.c
+       (keystroke_listener_object_finalize): kill
+       (spi_keystroke_listener_class_init): upd.
+       (spi_keystroke_listener_add_callback),
+       (spi_keystroke_listener_remove_callback): kill,
+       use signals.
+       (impl_key_event): prune drasticaly.
+
+       * libspi/eventlistener.c
+       (spi_event_listener_add_callback),
+       (spi_event_listener_remove_callback): kill, use
+       signals.
+       (impl_accessible_event_notify_event): upd.
+       (spi_event_listener_object_finalize): remove.
+
+       * libspi/Makefile.am: use eventlistener.[ch]
+
+2001-12-04  Bill Haneman  <bill.haneman@sun.com>
+
+       * libspi/registry.c (_registry_notify_listeners):
+       Changed listener loop iteration to use preferred convention.
+       Got rid of string memory leaks caused be calling g_strconcat
+       inside function calls.
+
+       * libspi/registry.c (parse_event_type):
+       Stopped g_strconcat memory leaks, free the g_strsplit return,
+       g_strdup the split pieces when used, etc.
+
+2001-12-04  Bill Haneman  <bill.haneman@sun.com>
+
+       * idl/State.idl: 
+       Change method 'compare' to return a 'difference set' rather than
+       taking a StateSet as an out parameter (more Java-friendly).
+
+       * idl/Accessible.idl:
+       Added CORBA struct 'BoundingBox', to faciliate API changes below:
+       
+       * idl/Component.idl:
+       * idl/Image.idl:
+       Changed methods 'getExtents' and 'getImageExtents' in these
+       interfaces to return a BoundingBox structure.  
+       (getPosition and getSize are redundant and possibly should be
+       removed, rather than creating a corresponding Point struct.)
+
+       * cspi/spi_component.c:
+       Modify implementation of getExtents to reflect the above IDL
+       change.
+       
+       * cspi/spi_image.c:
+       * cspi/spi.h:
+       Add (missing) AccessibleImage_getExtents () API.
+
+       * docs/reference/cspi/at-spi-cspi-sections.txt:
+       Added AccessibleImage_getImageExtents () to docs.
+       
 2001-12-03  Bill Haneman  <bill.haneman@sun.com>
 
 2001-12-03  Bill Haneman  <bill.haneman@sun.com>
 
-        * idl/Component.idl:
+       * idl/Component.idl:
        Added two methods, getLayer () and getMDIZOrder (), to reflect
        the new ATK methods atk_object_get_layer and
        atk_object_get_mdi_z_order. (Which arguably should be part of the
        AtkComponent API and AtkObject).  Also added an enum,
        ComponentLayer, for the first method's return value.
 
        Added two methods, getLayer () and getMDIZOrder (), to reflect
        the new ATK methods atk_object_get_layer and
        atk_object_get_mdi_z_order. (Which arguably should be part of the
        AtkComponent API and AtkObject).  Also added an enum,
        ComponentLayer, for the first method's return value.
 
+       * libspi/component.c:
+       Added implementations for these methods.
+
        * cspi/spi_component.c:
        * cspi/spi.h:
        Added cspi binding methods for above, 
        * cspi/spi_component.c:
        * cspi/spi.h:
        Added cspi binding methods for above,