No need to destroy endpoint factories in JavaScript

zeroc-ice · Oct 28, 2024 · 2f7a83b · 2f7a83b
1 parent ffaef5e
commit 2f7a83b
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 17 deletions.
diff --git a/js/src/Ice/EndpointFactoryManager.js b/js/src/Ice/EndpointFactoryManager.js
@@ -110,9 +110,4 @@ export class EndpointFactoryManager {
         s.endEncapsulation();
         return e;
     }
-
-    destroy() {
-        this._factories.forEach(factory => factory.destroy());
-        this._factories = [];
-    }
 }
diff --git a/js/src/Ice/InstanceExtensions.js b/js/src/Ice/InstanceExtensions.js
@@ -360,9 +360,6 @@ Instance.prototype.destroy = async function () {
         if (this._locatorManager) {
             this._locatorManager.destroy();
         }
-        if (this._endpointFactoryManager) {
-            this._endpointFactoryManager.destroy();
-        }
 
         if (this._initData.properties.getPropertyAsInt("Ice.Warn.UnusedProperties") > 0) {
             const unusedProperties = this._initData.properties.getUnusedProperties();

diff --git a/js/src/Ice/TcpEndpointFactory.js b/js/src/Ice/TcpEndpointFactory.js
@@ -29,10 +29,6 @@ export class TcpEndpointFactory {
         return e;
     }
 
-    destroy() {
-        this._instance = null;
-    }
-
     clone(instance) {
         return new TcpEndpointFactory(instance);
     }

diff --git a/js/src/Ice/WSEndpointFactory.js b/js/src/Ice/WSEndpointFactory.js
@@ -29,9 +29,4 @@ export class WSEndpointFactory {
         e.initWithStream(s);
         return e;
     }
-
-    destroy() {
-        this._delegate.destroy();
-        this._instance = null;
-    }
 }