Guessing this is why WeakFinalizersTest&gt;&gt;testNewFinalizationSupported is failing. Someone asked me to make a new trunk image; is it safe to do that with this test erroring out? Sounds like it is.<div><br><div class="gmail_quote">
On Tue, Sep 21, 2010 at 6:22 AM, Igor Stasenko <span dir="ltr">&lt;<a href="mailto:siguctua@gmail.com">siguctua@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
On 21 September 2010 06:20, David T. Lewis &lt;<a href="mailto:lewis@mail.msen.com">lewis@mail.msen.com</a>&gt; wrote:<br>
&gt;<br>
&gt; On Tue, Sep 21, 2010 at 03:28:35AM +0300, Igor Stasenko wrote:<br>
&gt;&gt;<br>
&gt;&gt; I am also uploaded a slice of system changes with new classes &amp; rest<br>
&gt;&gt; of stuff into inbox:<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; CollectionsTests-Igor.Stasenko.169<br>
&gt;&gt; Collections-Igor.Stasenko.370<br>
&gt;&gt; Kernel-Igor.Stasenko.478<br>
&gt;&gt; System-Igor.Stasenko.358<br>
&gt;&gt;<br>
&gt;&gt; note, that despite of these changes, system are still using old<br>
&gt;&gt; WeakRegistry instance,<br>
&gt;&gt; and obviously &#39;WeakRegistry default&#39; needs to be modified to be<br>
&gt;&gt; replaced with new functionality.<br>
&gt;<br>
&gt; These inbox changes are out of date with respect to the trunk, so<br>
&gt; would need to be merged with care.<br>
<br>
</div>Yeah, i did not updated the image before applying changesets.<br>
<div class="im"><br>
&gt; But aside from that, is there<br>
&gt; any reason that your changes should *not* be moved into trunk now?<br>
&gt; If no objections, then we should include them now.<br>
&gt;<br>
<br>
</div>There are a couple of doits, which should be invoked after loading changes:<br>
<br>
WeakArray restartFinalizationProcess<br>
and of couse<br>
Smalltalk recreateSpecialObjectsArray<br>
<br>
Btw, i found that in image i was using,<br>
Smalltalk specialObjectsArray size<br>
was 50<br>
while in #recreateSpecialObjectsArray method there was already 55 entries<br>
(and with weak finalization it should be 56).<br>
<div class="im"><br>
&gt; I understand that the VM changes are not yet available in the<br>
&gt; official VMs, but this should be corrected the next time that new<br>
&gt; VMs are distributed.<br>
&gt;<br>
&gt; Dave<br>
&gt;<br>
&gt;<br>
</div>Thanks, David.<br>
<div><div></div><div class="h5"><br>
<br>
--<br>
Best regards,<br>
Igor Stasenko AKA sig.<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Casey Ransberger<br>
</div>