After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 684667 - Improve the documentation of AtkObject::children-changed
Improve the documentation of AtkObject::children-changed
Status: RESOLVED FIXED
Product: atk
Classification: Platform
Component: docs
unspecified
Other Linux
: Normal normal
: ---
Assigned To: ATK maintainer(s)
ATK maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-09-23 17:01 UTC by Alejandro Piñeiro Iglesias (IRC: infapi00)
Modified: 2013-10-24 10:32 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fixes the bug (1.74 KB, patch)
2012-09-23 17:02 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
committed Details | Review
Documents the possibility of NULL for a child removed (1.37 KB, patch)
2013-10-24 10:27 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
committed Details | Review

Description Alejandro Piñeiro Iglesias (IRC: infapi00) 2012-09-23 17:01:52 UTC
Some meetings ago we concluded that the index can be -1 in some cases. During the Randa Sprint I also realized that the reference to the object can be NULL (ie: it is used on the implementation of gtktreeviewaccessible).

Again, some day we would need to review the concept of children-changed and check if it really worths to emit one signal each time an object appears, as in several cases that can be irrelevant.
Comment 1 Alejandro Piñeiro Iglesias (IRC: infapi00) 2012-09-23 17:02:51 UTC
Created attachment 225029 [details] [review]
Fixes the bug

Added the documentation at the template files, although as I said on 684665, ideally this should be at the source file itself.
Comment 2 Alejandro Piñeiro Iglesias (IRC: infapi00) 2013-09-11 11:21:51 UTC
Have just committed a similar patch to the one I uploaded on comment 1. Closing bug.
Comment 3 Alejandro Piñeiro Iglesias (IRC: infapi00) 2013-10-24 10:20:18 UTC
I had a quick chat with Mario Sanchez and mentioned that he has been having a lot of problems having the accessible child when this is removed. This makes sense, because if the child is being removed, there is a possibility that the child is not available. Looking at the description of the bug, I also mentioned it. Looking at the fix, I only covered the possibility of -1 as index.

Reopening.
Comment 4 Alejandro Piñeiro Iglesias (IRC: infapi00) 2013-10-24 10:27:35 UTC
Created attachment 257999 [details] [review]
Documents the possibility of NULL for a child removed
Comment 5 Alejandro Piñeiro Iglesias (IRC: infapi00) 2013-10-24 10:32:16 UTC
New documentation added. Closing bug (again).