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. ---> falseOn 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 |
- [Pharo-dev] AbstractNautilusUI "error, should not happ... Ben Coman
- Re: [Pharo-dev] [issue tracker] back porting 4 to 3 Marcus Denker
- Re: [Pharo-dev] [issue tracker] back porting 4 to ... Ben Coman
- Re: [Pharo-dev] [issue tracker] back porting 4 to ... Camille Teruel
- Re: [Pharo-dev] [issue tracker] back porting 4... Ben Coman
- Re: [Pharo-dev] AbstractNautilusUI "error, should... Nicolai Hess
