On Wed, Jan 18, 2012 at 12:25 PM, Henrik Johansen < henrik.s.johansen@veloxit.no> wrote:
On Jan 18, 2012, at 12:14 47PM, Mariano Martinez Peck wrote:
Well....it seems I found the problem. And in fact, it has alredy been solved by Levente in Squeak...I waste so many hours...and the fix was there..
So, can you try changing to
sizeFor: numberOfElements "Return the minimum capacity of a dictionary that can hold numberOfElements elements. At least 25% of the array must be empty and the return value must be a power of 2."
^(numberOfElements * 4 // 3) asLargerPowerOfTwo max: 1
And see if it works? it works hereâ¦
It should not be necessary. The real bug is that Integer >> isPowerOfTwo does not check for the edge case:
0 isPowerOfTwo true
A small fix:
Integer >> #isPowerOfTwo "Return true if the receiver is an integral power of two." ^ self ~= 0 and: [(self bitAnd: self-1) = 0]
Indeed :) now...even if that's correct, do you think there could be side effect in other places? -- Mariano http://marianopeck.wordpress.com