A FogBugz case was edited by Benjamin Van Ryseghem.
Case ID: 10764
Title: EyeInspector never terminate refresh process leading to no garbage collection
Status: Resolved (Fix To Include)
Category: Bug
Project: Tools
Area: Misc
Priority: 1 - Show Stopper !!!!!!
Milestone: Pharo3.0: 30/03/2014
Assigned To: Benjamin Van Ryseghem
URL:
https://pharo.fogbugz.com/default.asp?10764not put a default value here, even when it's required, leading to the impossibility to open the inspector as it is (lack of default information) is wrong. The problem is not not referencing directly a class, is using nil as NullPattern where clearly it is not. If you want to put nil here, so mak the inspector work with nil here, aka than you can open it. Or example. Or do not ask someone (aka, *me*) to fix the spec layout since you need magic invocation to open it.
according to <code>updateInspectorFrom: oldInspector
self updateTitle.
self inspector ifNil: [ ^ self ].
self inspector list
handlesDoubleClick: true;
doubleClickAction: [ self dive ].
self canBeRefreshed ifFalse: [ ^ self ].
self needRebuild: false.
self buildWithSpec: oldInspector spec selector.
oldInspector close. "kill the update process"</code> the oldInspector should be closed anyway, except in the case this method does not support which is when you had a real inspector in the value holder and then you put nil, since the method returns before closing the old inspector.
what is the wrongest thing here is spending more energy trying to find who to blame than to fix the issue!
You are subscribed to this case. If you do not want to receive automatic notifications in the future, unsubscribe (
https://pharo.fogbugz.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&command=view&ixBug=10764) from this case.
_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Locked
