Issue 5118 in pharo: Check pointsTo: and InstVarInclude: in squeak

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 5118 in pharo: Check pointsTo: and InstVarInclude: in squeak

pharo
Status: Accepted
Owner: [hidden email]
Labels: Stef

New issue 5118 by [hidden email]: Check pointsTo: and  
InstVarInclude: in squeak
http://code.google.com/p/pharo/issues/detail?id=5118

#pointsTo: should answer true if the receiver has a pointer to the  
argument. Every object (except for SmallIntegers and instances of compact  
classes) holds a pointer to its class in the header. Since the primitive  
only checks the slots, it will not always return true if the argument is  
the object's class. This breaks pointer finding/tracing tools, that's why  
it was changed in Squeak. See how #instVarsInclude: and #pointsTo: is  
implemented there.


Do not close this bug entry! even if nothing happen- I have no time. but I  
do not want people forget.
Of course you can also have a look and provide a fix.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 5118 in pharo: Check pointsTo: and InstVarInclude: in squeak

pharo

Comment #1 on issue 5118 by marianopeck: Check pointsTo: and  
InstVarInclude: in squeak
http://code.google.com/p/pharo/issues/detail?id=5118

Fix in inbox:

Name:  
SLICE-Issue-5118-Check-pointsTo-and-InstVarInclude-in-squeak-MarianoMartinezPeck.1
Author: MarianoMartinezPeck
Time: 22 December 2011, 9:08:15 pm
UUID: 5458e2b3-468c-4276-9bac-d43bd914ac3e
Ancestors:
Dependencies: Kernel-MarianoMartinezPeck.986

Fix to issue


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 5118 in pharo: Check pointsTo: and InstVarInclude: in squeak

pharo
Updates:
        Status: FixReviewNeeded

Comment #2 on issue 5118 by marianopeck: Check pointsTo: and  
InstVarInclude: in squeak
http://code.google.com/p/pharo/issues/detail?id=5118

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 5118 in pharo: Check pointsTo: and InstVarInclude: in squeak

pharo
Updates:
        Status: Integrated

Comment #3 on issue 5118 by marianopeck: Check pointsTo: and  
InstVarInclude: in squeak
http://code.google.com/p/pharo/issues/detail?id=5118

Since this was included in 14264, shouldn't we close the issue? I do it  
just in case..



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker