Hi Ben, Ah, I absolutely hate those ugly guys, I often terminate them by hand, just to have a clean Process Browser listing ;-) I applied your patch to SHTextStyler>>#styleInBackgroundProcess: and will see what it gives in my current working image. I will report back. Sven
On 27 Mar 2015, at 16:04, Ben Coman <btc@openInWorld.com> wrote:
Probably most people don't look much at Tools > Process Browser. I never did until I doing the delay refactoring to keep track of when the timer event loop was running.
I often find several of the following processes stuck on a semaphore... [ sem notNil ifTrue: [ sem wait. view ifNotNil: [ view stylerStyledInBackground: text ] ] ] forkAt: Processor activePriority.
So I had a little dig. Now trying to fix this multithreaded code at this late stage has some risk of destabilising things, so I propose the following minimal effective change...
[ sem notNil ifTrue: [ (sem waitTimeoutSeconds: 5) ifFalse: [ view ifNotNil: [ view stylerStyledInBackground: text ] ] ] ] forkAt: Processor activePriority.
Can anyone more familiar with SHTextStyler than the half hour I've had comment if they foresee any problem with that?
cheers -ben