Camille Teruel wrote:
On 22 sept. 2014, at 09:14, Marcus Denker <[email protected]> wrote:

  
Yes, this is why

https://pharo.fogbugz.com/f/cases/13606/testSlotScopeParallelism-is-failing

is faling, too.

There is a root cause for this that in some cases when changing classes that the layout gets not updated correctly...
    
Yes it's been a while that we suffer from this bug. Now that we found a way to reproduce it I could post a fix to review (https://pharo.fogbugz.com/f/cases/13606/testSlotScopeParallelism-is-failing).

To reproduce create two class, one subclass of the other.
Then add a class variable or a shared pool in the superclass.
Then, you can see that the slot scope of the subclass is not correctly updated:
TheSubclass layout slotScope parentScope == TheSubclass superclass layout slotScope. ---> false

  
On 21 Sep 2014, at 03:42, Ben Coman <[email protected]> wrote:

    
In build 40239, if I try to add an instance variable (e.g. 'xx') to AbstractNautilusUI, I get "Error: should not happen" from LayoutEmptyScope(LayoutAbstractScope)>>rebase:to: A tried a few other classes and they are okay. I've tracked this down to occur since build 40127 [Edit, 40147 was incorrect]...

13697 Yet another cleanup of MessageBrowser API
 https://pharo.fogbugz.com/f/cases/13697

13698 Small cleanup of Model and the UI package
 https://pharo.fogbugz.com/f/cases/13698

13696 Nautilus Refactoring 1: Classlist widget
 https://pharo.fogbugz.com/f/cases/13696

Is this a known problem? I can't identify a ticket on the tracker.

cheers -ben

      

I isolated the problem adding instance variables to AbstractNautilusUI and logged issue 14052 for it. 
A hack that fixes/resets the problem is provided, but its just shooting in the dark without understanding the architecture. Thats as far as I can take it.
cheers -ben

https://pharo.fogbugz.com/default.asp?14052

Reply via email to