Mariano don't you like all these ifNil around in case⦠and sometimes this funny logic because one object is missing. ;) So the if is not broken⦠is a tragedy: you know the story where at the end everybody die = no future. We should for example, replace all the MC string concat by an object like in Squeak. Now to answer your question I think that the answer is not about if this is broken or not but what scenario does it support. So do we have cases where we want to get the class of a method that is not bound to a class? When methodClass is accessed? Stef
Replacing something is not the same as removing, I agree. But if the hacks or workaround were for a particular very good reason and some point in time, with certain image and libraries, why to keep it NOW in Pharo? Say ReferenceStream had some hacks around the system. Now we have removed it. Why should we keep the hacks in Pharo itself? No problem if someone wants to install ReferenceStream again in Pharo. But if you do so, bring with you all the hacks/workarounds to make it work. We will not keep the garbage in the main image. Take this very example of #methodClass. Why would be need to keep the "self literals > 0" ? (if the reasons are what Eliot said). If then Newspeak wants to be built on top of Pharo, great, bring the changes (like this one) with you.
Finally, I am not afraid/shame of rollbacking. I prefer to clean something and then discover that it was actually needed than do nothing. In fact, I would be able to document why it was needed ;)
frank
-- Mariano http://marianopeck.wordpress.com
-- Mariano http://marianopeck.wordpress.com