fix: Should not destroy the timer multiple times

Voxelum · Mar 16, 2024 · a6935c4 · a6935c4
1 parent f5bcf19
commit a6935c4
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/xmcl-runtime/worker/index.ts b/xmcl-runtime/worker/index.ts
@@ -11,6 +11,9 @@ export const createLazyWorker = <T>(factory: () => Worker, methods: Array<keyof
     logger.log(`Awake the worker ${factory}`)
     worker.on('message', (message: 'idle' | WorkerResponse) => {
       if (message === 'idle') {
+        if (destroyTimer) {
+          clearTimeout(destroyTimer)
+        }
         destroyTimer = setTimeout(() => {
           if (threadWorker) {
             logger.log(`Dispose the worker ${factory}`)
@@ -45,6 +48,7 @@ export const createLazyWorker = <T>(factory: () => Worker, methods: Array<keyof
         threadWorker = threadWorker || createWorker()
         if (destroyTimer) {
           clearTimeout(destroyTimer)
+          destroyTimer = undefined
         }
         threadWorker.postMessage({ type: method, id: _id, args })
       })