Issue 4862 in pharo: HashedCollection rehashAll crashes the VM

Status: Accepted
Owner: [hidden email]
CC: marianopeck
Labels: Type-Bug Milestone-1.4 Importance-High

New issue 4862 by [hidden email]: HashedCollection rehashAll crashes  
the VM
http://code.google.com/p/pharo/issues/detail?id=4862

HashedCollection rehashAll crashes the VM

This is related to the change of Mariano?


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

Comment #1 on issue 4862 by marianopeck: HashedCollection rehashAll crashes  
the VM
http://code.google.com/p/pharo/issues/detail?id=4862

Indeed. This is a big problem. I am trying to solve it. I will keep you  
informed.


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

Comment #2 on issue 4862 by marianopeck: HashedCollection rehashAll crashes  
the VM
http://code.google.com/p/pharo/issues/detail?id=4862

Ok. I solve it. I will explain it after lunch.


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

Comment #3 on issue 4862 by marianopeck: HashedCollection rehashAll crashes  
the VM
http://code.google.com/p/pharo/issues/detail?id=4862

Hi Marcus. The whole explanation of the issue is here:  
http://forum.world.st/New-problem-with-rehash-WAS-Re-MethodDictionary-fast-rehash-td3847208.html

This fix is based in Levente Fix:

Name: Kernel-ul.630
Author: ul
Time: 27 September 2011, 6:09:18.907 pm
UUID: 87434805-2f3c-c446-b12b-3af3c0043cea
Ancestors: Kernel-nice.629

Fix for the recently introduced MethodDictionary bug reported by Mariano  
Martinez Peck on the Pharo list and some other things.
- added MethodDictionary class >> #newForCapacity: to reduce the complexity  
of the various methods which create new instances with a given capacity  
(#grow, #rehash, #removeAll). These changes make some comments unnecessary.
- fix: #sizeFor: should allow 75% load factor

In addition, I have also fixed the tests testRehashPreservesCapacity and  
testRehashPreservesElements.  And some small cleaning.
Now with this fix you should be able to do HashedCollection rehashAll  
without problems.

Slice in inbox:

Name:  
SLICE-Issue-4862-HashedCollection-rehashAll-crashes-the-VM-MarianoMartinezPeck.1
Author: MarianoMartinezPeck
Time: 27 September 2011, 8:16:27 pm
UUID: 994c97e7-16ab-40d8-9d5f-533652e131ee
Ancestors:
Dependencies: Kernel-MarianoMartinezPeck.942,  
KernelTests-MarianoMartinezPeck.355

Fix to issue 4862



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

Updates:
        Status: FixToInclude

Comment #4 on issue 4862 by marianopeck: HashedCollection rehashAll crashes  
the VM
http://code.google.com/p/pharo/issues/detail?id=4862

(No comment was entered for this change.)


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

Updates:
        Status: Integrated

Comment #5 on issue 4862 by [hidden email]: HashedCollection  
rehashAll crashes the VM
http://code.google.com/p/pharo/issues/detail?id=4862

14172


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