[squeak-dev] The Trunk: Tools-mt.990.mcz

Marcel Taeumel marcel.taeumel at hpi.de
Wed Oct 7 07:04:52 UTC 2020


Hi Christoph, hi all.

Every time you provide an option, you expect the user to make a decision. Easy-to-use interfaces might be found on a different path. ;-) Just my two cents.

Best,
Marcel
Am 06.10.2020 10:40:46 schrieb Thiede, Christoph <christoph.thiede at student.hpi.uni-potsdam.de>:
Hi Marcel,

just an idea - as we are talking about adding possible mode checkboxes to an inspector in [1], we could also add a checkbox for stepping there. We already have such a menu item in the Process Browser ... And then we could add a preference for choosing the default stepping boolean. Hypothetically. :-)

Best,
Christoph

[1] http://forum.world.st/The-Trunk-Tools-eem-994-mcz-td5122902.html [http://forum.world.st/The-Trunk-Tools-eem-994-mcz-td5122902.html]
[http://www.hpi.de/]
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel
Gesendet: Donnerstag, 1. Oktober 2020 13:58:32
An: squeak-dev; packages at lists.squeakfoundation.org
Betreff: Re: [squeak-dev] The Trunk: Tools-mt.990.mcz
 
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/20201007/589e3c13/attachment.html>


More information about the Squeak-dev mailing list