Fix the continue back into GTK/Event-Loop/C-code

> +    run [
> +        <category: 'accessing'>
> +
> +        [ | task sem |
> +          sem := Semaphore new.
> +          [ queue peek.
> +            [ queue isEmpty ] whileFalse: [
> +              task := queue next.
> +              [ task ensure: [ sem signal ] ] fork"At: Processor userBackgroundPriority".
> +              sem wait ] ] repeat ] fork
> +    ]

> On Mon, Oct 21, 2013 at 12:55:16PM +0200, Gwenaël Casaccio wrote:
>> Hi,
>>
>> here is a patch that fix the continue back into GTK/Event-Loop/C-code
>> (with the debugger and text widget).
> thank you very much!
>
>
>> -        self isLastContextSelected ifFalse: [ self stepToSelectedContext ].
>> -        debugger step.
>> -        self updateContextWidget
>> +        TaskQueue uniqueInstance add: [
>> +            self isLastContextSelected ifFalse: [ self stepToSelectedContext ].
>> +            debugger step.
>> +            self updateContextWidget ]
> after having looked at the blox code, what do you think about something
> like this:
>
>
> self debugWith: [debugger step]
>
> and
>
> debugWith: aBlock [
> TaskQueue uniqueInstance add: [
> self isLastContextSelected ..
> aBlock value.
> self updateContextWidget
> ]
> ]
>
> ?
>
>> +    run [
>> +        <category: 'accessing'>
>> +
>> +        [ | task sem |
>> +          sem := Semaphore new.
>> +          [ queue peek.
>> +            [ queue isEmpty ] whileFalse: [
>> +              task := queue next.
>> +              [ task ensure: [ sem signal ] ] fork"At: Processor userBackgroundPriority".
>> +              sem wait ] ] repeat ] fork
>> +    ]
> * What prevents the VM to gc this process?

> On Mon, Oct 21, 2013 at 12:55:16PM +0200, Gwenaël Casaccio wrote:
>> Hi,
>>
>> here is a patch that fix the continue back into GTK/Event-Loop/C-code
>> (with the debugger and text widget).
> Good Morning,
>
> I think I manage to break the TaskQueue
>
> Repeat:
> Open Workspace
> '1234' do: [:each | each asUpperCase] ALT+D/DebugIt
> Step until the BlockClosure is executed
>
> select "each asUpperCase" CTRL+I => UndefinedObject
> select "each" CTRL+I => Character
>
> in the Inspector type
> self upperCase CTRL+P Debugger comes up
> self upperCase CTRL+P nothing
>
> Result:
> Stepping in the debugger is broken, DoIt's don't
> appear to work either.

>
> Proposition:
> * Name the forked process (so one can identify it)
> * Keep the process in a local variable so one can
>    more easily find the process.
>
> thanks
> holger
>
>

> On Mon, Oct 21, 2013 at 12:55:16PM +0200, Gwenaël Casaccio wrote:
>> Hi,
>>
>> here is a patch that fix the continue back into GTK/Event-Loop/C-code
>> (with the debugger and text widget).
> Good Morning,
>
> I think I manage to break the TaskQueue
>
> Repeat:
> Open Workspace
> '1234' do: [:each | each asUpperCase] ALT+D/DebugIt
> Step until the BlockClosure is executed
>
> select "each asUpperCase" CTRL+I => UndefinedObject
> select "each" CTRL+I => Character
>
> in the Inspector type
> self upperCase CTRL+P Debugger comes up
> self upperCase CTRL+P nothing
>
> Result:
> Stepping in the debugger is broken, DoIt's don't
> appear to work either.
>
>
> Proposition:
> * Name the forked process (so one can identify it)
> * Keep the process in a local variable so one can
>    more easily find the process.