Hi Christoph,

it has "always" been that way.. I suppose that its (a) for performance reasonse and (b) you are "stopping time" when debugging anyway. :-)

Best,
Marcel
Am 01.10.2020 13:56:55 schrieb Thiede, Christoph <christoph.thiede at student.hpi.uni-potsdam.de>:
Hi Marcel,

just a basic question - why do we disable stepping in Debuggers? :-)

Best,
Christoph
[http://www.hpi.de/]
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von commits at source.squeak.org <commits at source.squeak.org>
Gesendet: Dienstag, 29. September 2020 10:20:36
An: squeak-dev at lists.squeakfoundation.org; packages at lists.squeakfoundation.org
Betreff: [squeak-dev] The Trunk: Tools-mt.990.mcz
 
Marcel Taeumel uploaded a new version of Tools to project The Trunk:
http://source.squeak.org/trunk/Tools-mt.990.mcz [http://source.squeak.org/trunk/Tools-mt.990.mcz]

==================== Summary ====================

Name: Tools-mt.990
Author: mt
Time: 29 September 2020, 10:20:33.276444 am
UUID: e7da396b-a8ec-7a44-8f15-5ba7d0d30b37
Ancestors: Tools-mt.989

In debuggers, forward model-wake-up call to update inspectors. This fixes the bug where a 'nil' receiver had no fields in the receiver inspector. For example, try debug-it on any expression in the workspace to trigger that bug.

Note that the underlying challenge is that the inspectors in debuggers are not stepping. Field updates are triggered manually via #updateInspectors. An inspector's initial object is 'nil'. So, "changing" that to 'nil' will not update the field list in inspectors bc. it is considered redundant and will be ignored there. While stepping is not required to produce the initial field list, the model-wake-up call will usually trigger that initial update in a stand-alone inspector.

That's why I think that it is not necessary to change anything in the inspector code but only here in the debugger.

=============== Diff against Tools-mt.989 ===============

Item was added:
+ ----- Method: Debugger>>modelWakeUpIn: (in category 'user interface') -----
+ modelWakeUpIn: aWindow
+
+        super modelWakeUpIn: aWindow.
+        self updateInspectors.!


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20201001/cf0c84c9/attachment.html>