On Feb 28, 2013, at 3:09 PM, Clément Bera <[email protected]> wrote:
> Hello,
>
> I was looking for VM tests in the Pharo image and I found the class
> IslandVMTweaksTestCase. The strange thing is its comments :
> "Test case for some tweaks to the VM that Islands requires. These tests are
> largely for documentation; with an un-tweaked VM, the tests mostly still
> succeed, albeit with possible memory corruption."
>
> Does that really mean if I run those tests it might result into memory
> corruption ?
>
> I don't think it is a good idea to ship Pharo with this memory corruption
> tests in IslandVMTweaksTestCase. I suggest that if the tests may really
> corrupt memory we remove them from Pharo, or if they cannot corrupt memory
> then we fix the memory corruption comment.
>
Yes, we should not ship code like that.
Marcus
