On 1 Oct 2016, at 22:17, David T. Lewis lewis@mail.msen.com wrote:
On Sat, Oct 01, 2016 at 05:14:14PM +0200, Levente Uzonyi wrote:
On Sat, 1 Oct 2016, Bert Freudenberg wrote:
That name is a pharoism. In Squeak it's still findSubstring:in:startingAt:matchTable:. Perhaps the recent merge has overwritten it.
Levente
This naming inconsistency is a long standing annoyance that should be fixed. As of Feb 2011, I thought that we had reached agreement to do the necessary cleanup in the Pharo image:
http://lists.squeakfoundation.org/pipermail/vm-dev/2011-February/006781.html
Maybe that was just wishful thinking on my part. I can no longer find the original email exchange, but I think it was on the pharo list.
Maybe someone could open an issue on the Pharo bug tracker to make this problem go away?
The original problem was addressed by Nicolas and Andreas:
http://bugs.squeak.org/view.php?id=6366
Meanwhile a workaround was introduced in Pharo, resulting in the renamed method.
But once fixed in the VM, the Pharo workaround was no longer required, and should have been removed so that the Squeak and Pharo images would not require different VMMaker versions:
http://bugs.squeak.org/view.php?id=7479 http://bugs.squeak.org/view.php?id=7479
well.. this kind of things is the reason why there should not be a fork anymore :) I just didn’t remember that change and then I didn’t work on merge it :P
I will address this tomorrow.
Esteban
Dave