On 5 July 2013 09:47, Frank Shearar <frank.shearar at gmail.com> wrote:
> As it happens there are 249 classes in my starting-point image with a
> nil environment, and somewhere in excess of 1k classes with
> environment == Smalltalk.
>
> One possibly dirty hack that seems to work is to remove the access to
> the instvar and use `self environment` instead (in Class >>
> #declare:).

Er, so should we ever have classes with a nil environment? Is my
proposed change a stinky hack, or the right thing to do to lazily fix
the issue?

frank


> frank
>
> On 5 July 2013 04:30, Chris Muller <asqueaker at gmail.com> wrote:
>> With the image you start with, PianoRollNoteMorph's environment
>> instvar is set correctly?  But then, at some point, it gets set to
>> nil?  Does a halt catch that change?
>>
>> On Thu, Jul 4, 2013 at 4:00 PM, Frank Shearar <frank.shearar at gmail.com> wrote:
>>> If you update at the moment, when you get to loading Morphic-fbs.663
>>> you'll get a walkback because in Class >> #declare:,
>>> PianoRollNoteMorph's environment instvar is nil.
>>>
>>> Which is kind've odd. I thought that all classes MUST have an
>>> environment (and that right now that environment is Smalltalk).
>>>
>>> So I've tried moving Sound and Morphic (the source & destination
>>> packages for the PianoRollNoteMorph move) around in config maps, and
>>> that doesn't help.
>>>
>>> What gives? How can I fix this?
>>>
>>> frank
>>>
>>