Skip to content

Commit

Permalink
Merge pull request #685 from pharo-spec/dev-2.0
Browse files Browse the repository at this point in the history
disable JobList progress bar for the moment
  • Loading branch information
estebanlm authored Feb 13, 2024
2 parents c37e6c5 + c228cb3 commit 204ec25
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions src/NewTools-Core/StPharoJobManager.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
This is a JobManager to work with the Jobs package.
This manager will listen to the announcer and fire the events as it corresponds, allowing the user to interact with the joblist tool.
This is a HACK (hopefully temporal) based on the fact that most of the processing in pharo (for the moment) is hidden deep in the model, far from the UI, and even worst, most of it happens inside the UI Thread, blocking regular processing.
"
Class {
#name : 'StPharoJobManager',
Expand Down Expand Up @@ -40,7 +40,7 @@ StPharoJobManager class >> enable [
{ #category : 'class initialization' }
StPharoJobManager class >> initialize [

self enable
"self enable"
]

{ #category : 'private' }
Expand All @@ -63,10 +63,7 @@ StPharoJobManager class >> jobChanged: ann [
StPharoJobManager class >> jobEnd: ann [

self isHeadless ifTrue: [ ^ self ].
self jobList jobEnd: ann.
"this is just a cleanup... because jobs may leak!"
self jobList isEmpty
ifTrue: [ self application removeProperty: #jobList ]
self jobList jobEnd: ann
]

{ #category : 'private' }
Expand Down

0 comments on commit 204ec25

Please sign in to comment.