Open tinchodias opened 7 months ago
In addition, if this method makes sense:
BlSpace
pulseUntilEmptyTaskQueue: aSpace
timeout: 200 milliSeconds
then we might move it to instance side:
aSpace pulseUntilEmptyTaskQueueTimeout: 200 milliSeconds
Also, BlSpace class>>#pulseUntilEmptyTaskQueue:timeout:
should wait if aSpace universe hasDeferredActions
EDIT: This was fixed in #478
This pass might open the door to remove BlUseAsyncFeatures
which is now only used by Scripter's strategies (sync and async).
Notes about BlDevScripter.
The package is not small:
BlDevScripterStepExecutorSyncStrategy
is the only referencer to BlUseAsyncFeatures
which shows it is not being used (else there should be another reference to BlUseAsyncFeatures at least)
I take as starting point this method: https://github.com/pharo-graphics/Bloc/blob/6cfbca76ec7e5cf0633bf5b2d66e0ba4ce4da291/src/Bloc/BlSpace.class.st#L284
Points to analyze/discuss:
aSpace dispatchEvent: BlSpaceEventsProcessedEvent new
. The space's phase doesn't do it like that.aSpace dispatchEvent: anEvent
byaSpace hostSpace enqueueEvent: anEvent
to emulate whatBlOSWindowEventHandler
orBlMorphicEventHandler
do.