<div dir="ltr"><div dir="ltr">On Sat, Mar 20, 2021 at 10:37 AM David T. Lewis <<a href="mailto:lewis@mail.msen.com" target="_blank">lewis@mail.msen.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br>
On Thu, Mar 18, 2021 at 03:05:29PM +0100, <a href="mailto:stes@PANDORA.BE" target="_blank">stes@PANDORA.BE</a> wrote:<br>
> <br>
> <br>
> If a good solution exists, it's better of course than a quick <br>
> (and possibly) wrong short term solution ...<br>
> <br>
> Thanks for having looked into this.<br>
> <br>
> David Stes<br>
><br>
<br>
Hi David,<br>
<br>
I found a "good enough" solution. Sources are updated in SVN r3799.<br>
<br>
Dave<br></blockquote><div><br></div><div>Wow, I didn't realize until just now you're still maintaining the interpreter sources on SVN. I mean I knew you did it somewhere, I just never bothered to look at it 😳</div><div><br></div><div>Also found your VMMaker-dtl.423 with the changes. I just did the same hack in SqueakJS. It was good enough to not crash the image anymore.</div><div><br></div><div>I also had to add a VMMaker version to vm attribute 1004, because of ImageSegmentTest>>testSelectors 😬</div><div><br></div><div>Then 2 out of 3 objects-as-methods test were working, the 3rd one revealed a bug, with the second line in testAddNumbers erroring.</div><div><br></div><div>Tracking that one down was slightly tricky because it looked like it had to do with the perform. But after banging on it for a while, I found that if the method/object was found in the cache, the current message selector global would not be updated. The only place where that is used is in primitiveInvokeObjectAsMethod. 😜</div><div><br></div><div>Now all 3 tests are green. Yay! Thanks for the original report, David S!</div><div><br></div><div>- Vanessa -</div><div> </div></div></div>