Merge pull request #947 from RobertOrthofer/944-cleanup-workers

terminate workers on destroy
This commit is contained in:
Michael Gevlich 2025-11-17 12:53:23 +04:00 committed by GitHub
commit 67c68990af
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 16 additions and 4 deletions

View File

@ -1433,6 +1433,8 @@ class Renderer {
}
public destroy() {
this.labelWorker.destroy();
for (let i in this.controls) {
this.controls[i].remove();
}

View File

@ -1988,6 +1988,18 @@ export class Planet extends RenderNode {
this.quadTreeStrategy.clearRenderedNodes();
}
/**
* Destroy planet.
* @public
*/
public override destroy() {
this._terrainWorker.destroy();
this._plainSegmentWorker.destroy();
this.renderer?.destroy()
this.onremove();
super.destroy();
}
// function checkTerrainCollision(entity) {
// let _tempTerrPoint = new Vec3();
// let nodes = globus.planet._renderedNodes;

View File

@ -62,10 +62,8 @@ export class BaseWorker<T> {
w.onmessage = null;
w.terminate();
}
//@ts-ignore
this._pendingQueue = null;
//@ts-ignore
this._workerQueue = null;
this._pendingQueue = [];
this._workerQueue = [];
}
public get pendingQueue(): T[] {