From 99ebeaee0a6acb48a5c65618aecdecb3bbfaabbc Mon Sep 17 00:00:00 2001
From: nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Thu, 10 Oct 2002 07:00:07 +0000
Subject: * lib/weakref.rb (WeakRef::@@final): use Hash#delete.

* lib/weakref.rb (WeakRef::__getobj__): examin if alive or not by
  ID_REV_MAP to deal with recycled object.  [ruby-dev:18472]

* lib/weakref.rb (WeakRef::weakref_alive?): ditto.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@2948 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 lib/weakref.rb | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

(limited to 'lib/weakref.rb')

diff --git a/lib/weakref.rb b/lib/weakref.rb
index 6861fde5f5..2e2a55ebe6 100644
--- a/lib/weakref.rb
+++ b/lib/weakref.rb
@@ -25,15 +25,15 @@ class WeakRef<Delegator
       rids = ID_MAP[id]
       if rids
 	for rid in rids
-	  ID_REV_MAP[rid] = nil
+	  ID_REV_MAP.delete(rid)
 	end
-	ID_MAP[id] = nil
+	ID_MAP.delete(id)
       end
       rid = ID_REV_MAP[id]
       if rid
-	ID_REV_MAP[id] = nil
+	ID_REV_MAP.delete(id)
 	ID_MAP[rid].delete(id)
-	ID_MAP[rid] = nil if ID_MAP[rid].empty?
+	ID_MAP.delete(rid) if ID_MAP[rid].empty?
       end
     ensure
       Thread.critical = __old_status
@@ -53,11 +53,11 @@ class WeakRef<Delegator
       Thread.critical = __old_status
     end
     ID_MAP[@__id].push self.__id__
-    ID_REV_MAP[self.id] = @__id
+    ID_REV_MAP[self.__id__] = @__id
   end
 
   def __getobj__
-    unless ID_MAP[@__id]
+    unless ID_REV_MAP[self.__id__] == @__id
       raise RefError, "Illegal Reference - probably recycled", caller(2)
     end
     begin
@@ -68,11 +68,7 @@ class WeakRef<Delegator
   end
 
   def weakref_alive?
-    if ID_MAP[@__id]
-      true
-    else
-      false
-    end
+    ID_REV_MAP[self.__id__] == @__id
   end
 end
 
-- 
cgit v1.2.3