|Anonymous | Login||12-05-2021 01:46 UTC|
|Main | My View | View Issues | Change Log | Docs|
|Viewing Issue Advanced Details [ Jump to Notes ]||[ View Simple ] [ Issue History ] [ Print ]|
|ID||Category||Severity||Reproducibility||Date Submitted||Last Update|
|0002359||[Squeak] MVC||major||always||12-13-05 12:32||01-20-06 22:59|
|ETA||none||Fixed in Version||3.9||Product Version||3.9|
|Summary||0002359: Debugger in MVC opens in next available Morphic world|
|Description||In Squeak3.9a-6705, "self halt" in MVC displays nothing in MVC. The debugger opens in whatever Morphic world next becomes active.|
|Steps To Reproduce|
|Attached Files||DebuggerMVCSchedulingFix-dtl.cs.gz [^] (1,649 bytes) 12-14-05 05:16|
(0003321 - 1568 - 1670 - 1670 - 1670 - 1670 - 1670)
The problem is in Debugger class>>openOn:context:label:contents:fullView:
From version history:
Last working version was 'ajh 8/6/2003 11:40 Debugger class openOn:context:label:contents:fullView:'.
Error was introduced in 'md 8/9/2005 20:10 Debugger class openOn:context:label:contents:fullView:' which is attempting to fix some problem with opening debuggers from the test runner, by scheduling the debugger in a deferred UI message (which does not work in MVC StandardSystemController).
One subsequent version has been added: 'dao 8/29/2005 10:38 Debugger class openOn:context:label:contents:fullView:'. This appears to be an update to clean up CurrentProjectRefactoring, not related to this bug.
Simple fix is to go back to the ajh version, but still need to address the problem that md was trying to fix, plus add the good patch from dao, plus update DebuggerPlus to match. Better (but ugly) is to schedule deferred UI message only if we are in Morphic. That's what the attached change set (DebuggerMVCSchdulingFix-dtl) does.
Related change: DebuggerPlus used a copy of the earlier ajh version. DebuggerMVCSchdulingFix-dtl updates it to be the same as the method Debugger. I have not tested this, but presume it to be OK based on the fact that the DebuggerPlus matched the earlier (good) Debugger method.
Entropy alert: This change set adds yet another 'Smalltalk isMorphic' to an already ugly method.
Note: I think I did a change set a long time ago that makes deferred UI messages more generally available in MVC. Need to dig that one up.
(0003582 - 15 - 15 - 15 - 15 - 15 - 15)
|will be in 6717|
|12-13-05 12:32||lewis||New Issue|
|12-14-05 05:16||lewis||File Added: DebuggerMVCSchedulingFix-dtl.cs.gz|
|12-14-05 05:20||lewis||Note Added: 0003321|
|01-20-06 22:59||MarcusDenker||Status||new => closed|
|01-20-06 22:59||MarcusDenker||Note Added: 0003582|
|01-20-06 22:59||MarcusDenker||Resolution||open => fixed|
|01-20-06 22:59||MarcusDenker||Fixed in Version||=> 3.9|
| Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
41 total queries executed.|
32 unique queries executed.