Add Stack>>do: method - Githubissues

martinmcclure / mist

Filetree code for the Mist project (mist-project.org)

32 stars 5 forks source link

Add Stack>>do: method #3

Closed timfel closed 12 years ago

timfel commented 12 years ago

I haven't tested on Pharo, but this was missing in Squeak. If it is present in Pharo, I'll adapt it

martinmcclure commented 12 years ago

In Pharo, Stack is a subclass of LinkedList, which implements #do:. Unfortunately, in Pharo Stack does not understand #linkedList. Two options:

Start a Squeak branch. Seems like a big hammer for a small problem.
Implement our own stack class for Mist to use. Seems like a waste, but I guess it's only until Mist is self-hosted (which will be a while).

Opinions?

martinmcclure commented 12 years ago

Tim, please try the issue3-stack branch code in Squeak. I've made FogScopeStack use an OrderedCollection rather than a Stack, hopefully avoiding the problem altogether.

timfel commented 12 years ago

That branch works, thanks :)

martinmcclure commented 12 years ago

Thanks for the confirmation; I've merged the issue branch to master.