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 352257 - improved the function which returns if two objects are identical
improved the function which returns if two objects are identical
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
0.2.x
Other Linux
: Normal normal
: ---
Assigned To: Orca Maintainers
Orca Maintainers
Depends on:
Blocks: 352254
 
 
Reported: 2006-08-21 13:12 UTC by Oana Serb
Modified: 2006-08-23 12:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Improved isSameObject() function to determine if two object are identical (2.05 KB, patch)
2006-08-21 13:24 UTC, Oana Serb
none Details | Review
Added some code comments (2.47 KB, patch)
2006-08-23 06:34 UTC, Oana Serb
committed Details | Review

Description Oana Serb 2006-08-21 13:12:25 UTC
If the objects have same name, index and parent, the function should return TRUE.

In case of tree items for java applications, the function returns FALSE, instead of TRUE, because the parents pointers are different.
Comment 1 Oana Serb 2006-08-21 13:24:20 UTC
Created attachment 71303 [details] [review]
Improved isSameObject() function to determine if two object are identical
Comment 2 Willie Walker 2006-08-22 13:12:59 UTC
This looks OK to me, but I'd like to see some documentation in there saying what is going on.  For example, something such as "When we're looking at children of objects that manage their descendants, we will often get different objects that point to the same logical child.  We want to be able to determine if two objects are in fact pointing to the same child.  If we cannot do so easily (i.e., object equivalence), we examine the hierarchy and the object index at each level."

Something like that, but you can use your own words to describe it.  :-)
Comment 3 Oana Serb 2006-08-23 06:34:47 UTC
Created attachment 71434 [details] [review]
Added some code comments
Comment 4 Willie Walker 2006-08-23 12:02:50 UTC
Thanks!  Please commit.
Comment 5 Oana Serb 2006-08-23 12:31:35 UTC
Patch committed to CVS head.