- Removing BufferManager class and BasicStream.destroy(), which reduces the

    complexity of many existing finalizers.
- Removed code that cleans up cached objects.
- For compatibility with C#, finalizers no longer call methods on other
    objects.

1 files changed, 5 insertions, 2 deletions

diff --git a/java/src/IceInternal/ThreadPool.java b/java/src/IceInternal/ThreadPool.java
index 67256e507ad..ac14d9fd97a 100644
--- a/java/src/IceInternal/ThreadPool.java
+++ b/java/src/IceInternal/ThreadPool.java
@@ -128,6 +128,10 @@ public final class ThreadPool
     {
         assert(_destroyed);
 
+	/**
+	  * For compatibility with C#, we do not invoke methods on other objects
+	  * in a finalizer.
+	  *
 	try
 	{
 	    if(_selector != null)
@@ -154,6 +158,7 @@ public final class ThreadPool
 	    String s = "exception in `" + _prefix + "' while calling close():\n" + sw.toString();
 	    _instance.logger().error(s);
 	}
+	  */
 
         super.finalize();
     }
@@ -1150,8 +1155,6 @@ public final class ThreadPool
 		}
 	    }
 
-            stream.destroy();
-
             if(TRACE_THREAD)
             {
                 trace("run() terminated");