diff options
| author | Marc Laukien <marc@zeroc.com> | 2004-02-20 20:07:25 +0000 |
|---|---|---|
| committer | Marc Laukien <marc@zeroc.com> | 2004-02-20 20:07:25 +0000 |
| commit | 20ce462d81e0885ef6c34e3aab08adb8bf53d82f (patch) | |
| tree | 76da7f5f7c9964c2a9e3bc7e651f2e1cd109e377 /java/src/IceInternal/BasicStream.java | |
| parent | more ami (diff) | |
| download | ice-20ce462d81e0885ef6c34e3aab08adb8bf53d82f.tar.bz2 ice-20ce462d81e0885ef6c34e3aab08adb8bf53d82f.tar.xz ice-20ce462d81e0885ef6c34e3aab08adb8bf53d82f.zip | |
more ami
Diffstat (limited to 'java/src/IceInternal/BasicStream.java')
| -rw-r--r-- | java/src/IceInternal/BasicStream.java | 37 |
1 files changed, 13 insertions, 24 deletions
diff --git a/java/src/IceInternal/BasicStream.java b/java/src/IceInternal/BasicStream.java index 92679446dcf..7226a5c296a 100644 --- a/java/src/IceInternal/BasicStream.java +++ b/java/src/IceInternal/BasicStream.java @@ -40,23 +40,22 @@ public class BasicStream _messageSizeMax = _instance.messageSizeMax(); // Cached for efficiency. } -/* - * Do NOT use a finalizer for BasicStream - this causes a - * severe performance penalty! - * - protected void - finalize() - throws Throwable + // + // Do NOT use a finalizer, this would cause a severe performance + // penalty! We must make sure that destroy() is called instead, to + // reclaim resources. + // + public void + destroy() { - if(_buf != null) - { - _bufferManager.reclaim(_buf); - } - - super.finalize(); + _bufferManager.reclaim(_buf); + _buf = null; } -*/ + // + // This function allows this object to be reused, rather than + // reallocated. + // public void reset() { @@ -73,16 +72,6 @@ public class BasicStream } } - // - // Must be called in order to reclaim the buffer - // - public void - destroy() - { - _bufferManager.reclaim(_buf); - _buf = null; - } - public IceInternal.Instance instance() { |