Hi Marcel, is this philosophically consistent with the change you made
to "inspect it" a while back?  You had changed "inspect it" to answer
the PluggableSystemWindow instead of the object being inspected.  Your
argument, IIRC, was that there is "no other way" to get the window.

The only persons who  ever want the PluggableWindow are IDE
developers, who could just change #inspect themself, privately, or
just add a new method and use that..

#inspect has been a USER method in Smalltalk-80 since its inception 30
years ago.  Unfortnately, it now only caters to its 2 or 3 internal
IDE developers (e.g., you and me), instead of hundreds of users who
might use #inspect as an inline debugging tool.

So, my question is, is this the same philosophy with "debug it?"  Or
is this more like the original #inspect which provided better access
to the underlying domain object being debugged?

On Mon, May 9, 2016 at 6:14 AM,  <commits at source.squeak.org> wrote:
> Marcel Taeumel uploaded a new version of System to project The Trunk:
> http://source.squeak.org/trunk/System-mt.826.mcz
>
> ==================== Summary ====================
>
> Name: System-mt.826
> Author: mt
> Time: 9 May 2016, 1:14:47.861723 pm
> UUID: ec4d495c-324c-214f-9944-0e2be7e78ec6
> Ancestors: System-mt.825
>
> Return the result of a debug-it for further processing.
>
> =============== Diff against System-mt.825 ===============
>
> Item was changed:
>   ----- Method: Project>>debugMethod:forReceiver:inContext: (in category 'debugging') -----
>   debugMethod: aCompiledMethod forReceiver: anObject inContext: aContextOrNil
>         "Just execute the method and return the result. We cannot know how interactive debugging works for arbitrary projects."
>
> +       ^ aCompiledMethod
> -       aCompiledMethod
>                 valueWithReceiver: anObject
>                  arguments: (aContextOrNil
>                                                         ifNil: [#()]
> +                                                       ifNotNil: [{aContextOrNil}])!
> -                                                       ifNotNil: [{aContextOrNil}]).!
>
>