+ /*
+ * some hyperlinks are actionable... this is an ATK convention
+ * that seems convenient though possibly poorly documented or unintended.
+ */
+ if (ATK_IS_ACTION (object))
+ {
+ /*
+ * NOTE: we don't cast 'object' to ATK_OBJECT in the call to
+ * spi_action_interface_new(), because of the above convention,
+ * even though it means we may be violating the func prototype.
+ * See discussion in bugzilla bug #120659.
+ * !!!
+ * IMPORTANT! The 'AtkObject' typecast, instead of the cast macro,
+ * is used below, because 'object' may NOT really be an AtkObject;
+ * it will be cast back to a G_OBJECT inside spi_action_interface_new
+ * before use, so this is OK though very ropey coding style.
+ */
+
+ /* Don't aggregate action twice... if this is from AtkHyperlinkImpl */
+ if (!bonobo_object_query_interface (bonobo_object (new_hyperlink), "IDL:Accessibility/Action:1.0",
+ NULL))
+
+ bonobo_object_add_interface (bonobo_object (new_hyperlink),
+ BONOBO_OBJECT (spi_action_interface_new ((AtkObject *) object)));
+ }