<div class="gmail_quote">On 12 December 2011 17:10, Chris Muller <span dir="ltr"><<a href="mailto:asqueaker@gmail.com">asqueaker@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The halt is there on purpose because it is a long-standing bug with<br>
Avi's original WriteBarrier that, when it occurs, will need to be<br>
closely looked at at that point in the code. It could only possibly<br>
happen when changing the class-definition of a class with instances<br>
behind WB, so it probably wouldn't happen in prod (presumably, your<br>
tests would catch it anyway, right? :).<br></blockquote><div><br></div><div>It did happen in development, and I expect it to happen often, because I do change class definitions while instances of a class behind WB exists. That is exactly what I like in smalltalk - programming the system in runtime, and I am not going to stop doing that. To kill all the instances I would have to clean Seaside seasons and run GC. So it's not a minor thing for me.</div>
<div>So what do I do about it?</div><div>You say it's a long-standing bug with original Avi's code</div><div>- do you know what was the bug about?</div><div>- do you know if the problem exists with the new WB?</div>
<div>- maybe I can look into it, even though so far I didn't notice any problems, I just removed the "halt"</div><div>- I can just switch the WB off like it used to be in Magma 1.2</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
For PointerFinder, I didn't look at your patch, but wouldn't you need<br>
to put the correct code in the -Pharo platform package? I'm fine to<br>
move the PointerFinder reference to the -Squeak package if necessary.<br>
People of Pharo, please let me know!</blockquote><div><br></div><div>I see. It's up to you. The above bothers me more. Just know that PointerFinder is deprecated and I guess is going to be removed.</div></div><br clear="all">
<div><br></div>-- <br>Milan Mimica<br><a href="http://sparklet.sf.net">http://sparklet.sf.net</a><br>