+1 :-)

Best,
Marcel
Am 21.12.2019 05:13:06 schrieb Chris Muller <asqueaker at gmail.com>:
Thanks for the idea.  Even though #isInitialized was in Tools, we always have Tools loaded, so it would still have been occupying the methods list.  The tools should bend over backward to let the class library be as clean as it can be.

I liked the idea to put an error message in there, but was not quite right for this case, though.  If you try it, you'll see why Array empty is the right choice.   :)

 - Chris

On Fri, Dec 20, 2019 at 10:06 PM <commits at source.squeak.org [mailto:commits at source.squeak.org]> wrote:

Chris Muller uploaded a new version of Tools to project The Inbox:
http://source.squeak.org/inbox/Tools-cmm.928.mcz [http://source.squeak.org/inbox/Tools-cmm.928.mcz]

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

Name: Tools-cmm.928
Author: cmm
Time: 20 December 2019, 10:06:10.77308 pm
UUID: e1a28262-590a-443a-83d8-6e994b0f676b
Ancestors: Tools-mt.925

Fix DictionaryInspector to allow debugging of Dictionary initialization, and render the entries in order for OrderedDictionary's.

=============== Diff against Tools-mt.925 ===============

Item was changed:
  ----- Method: DictionaryInspector>>calculateKeyArray (in category 'selecting') -----
  calculateKeyArray
+       "Recalculate the KeyArray from the object being inspected."
+       keyArray := [ object keysInOrder ] ifError:
+               [ "Can occur when debugging Dictionary new"
+               Array empty ].
+       selectionIndex := 0!
-       "Recalculate the KeyArray from the object being inspected"
-
-       keyArray := object keysSortedSafely asArray.
-       selectionIndex := 0.
- !


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