[Pharo-dev] Puzzled

Clément Bera bera.clement at gmail.com
Thu Apr 3 16:50:31 EDT 2014


Hey,

I remembered removing the initializeAutoRefresh code from the debugger.
Perhaps I didn't commit that package. Or I forgot this one.


2014-04-03 12:34 GMT-07:00 Pharo4Stef <pharo4Stef at free.fr>:

> ok now I understand: an endless loop inside the debugger creation. I do
> not understand why we did not address it with clement
> because we open the debugger and other when we fixed the logic of the
> inspector (to avoid polling refresh).
>
> Stef
> On 03 Apr 2014, at 21:13, Pharo4Stef <pharo4Stef at free.fr> wrote:
>
> >
> > | context process debugger  |
> > context := [ 20 factorial ] asContext.
> >
> > process := Process
> >               forContext: context
> >               priority: Processor userInterruptPriority.
> > [
> >       debugger := Smalltalk tools debugger new
> >                                               process: process
> >                                               controller: nil
> >                                               context: context.
> >       Transcript show: debugger stack class ; cr
> > ] ensure: [ process terminate]
> >
> >
> > When I look at some of the stuck processes
> >
> > SpecDebugger>>initializePresenter
> >       super initializePresenter.
> >       self flag: 'some of this logic could be moved to the stack widget'.
> >       self flag: 'The toolbar should not be updated when the list
> changes, but when an action is perormed.'.
> >       self stack whenListChanged: [ :aList |
> >               aList isEmpty ifFalse: [ self stack setSelectedItem: aList
> first ].
> >               "Updating the toolbar will result in changing the button
> widgets.
> >               If the widget hasn't been opened, there will be no spec,
> which leads to an error."
> >               self spec ifNotNil: [
> >                       self updateToolbar ] ].
> >
> >       self stack whenSelectedItemChanged: [:aContext |
> >               self updateCodeFromContext: aContext.
> >               self updateInspectorsFromContext: aContext.
> >               self stack updateForSelectionChanged ].
> >
> >       self contextInspector initializeAutoRefresh.
> >
> >                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^
> >
> > And initializeAutoRefresh does not exist in the latest image.
> > I have no idea why the debugger may work may be this method is not used
> at all.
> >
> > Can somebody else confirm because I have the impression to fall into a
> rat nest.
> > stef
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.pharo.org/pipermail/pharo-dev_lists.pharo.org/attachments/20140403/34944638/attachment-0002.html>


More information about the Pharo-dev mailing list