don't corrupt the identity of AbstractRemoteRef in their finalizers

this made it unreliable for the WeakKeyDict these are typically put in (client_refs)
to have trouble finding them to cleanup the dictionary later
since their hash and identity changed

fixes #15923
may also fix #16091

Author: vtjnash

base/multi.jl

@@ -493,15 +493,13 @@ type Future <: AbstractRemoteRef
     v::Nullable{Any}
 
     Future(w::Int, rrid::RRID) = Future(w, rrid, Nullable{Any}())
-    Future(w::Int, rrid::RRID, v) = (r = new(w,rrid.whence,rrid.id,v); test_existing_ref(r))
+    Future(w::Int, rrid::RRID, v) = (r = new(w,rrid.whence,rrid.id,v); return test_existing_ref(r))
 end
 
 function finalize_future(f::Future)
     if f.where > 0
         isnull(f.v) && send_del_client(f)
         f.where = 0
-        f.whence = 0
-        f.id = 0
         f.v = Nullable{Any}()
     end
     f
@@ -512,7 +510,7 @@ type RemoteChannel{T<:AbstractChannel} <: AbstractRemoteRef
     whence::Int
     id::Int
 
-    RemoteChannel(w::Int, rrid::RRID) = (r = new(w, rrid.whence, rrid.id); test_existing_ref(r))
+    RemoteChannel(w::Int, rrid::RRID) = (r = new(w, rrid.whence, rrid.id); return test_existing_ref(r))
 end
 
 function test_existing_ref(r::Future)
@@ -527,25 +525,23 @@ function test_existing_ref(r::Future)
     end
     client_refs[r] = true
     finalizer(r, finalize_future)
-    r
+    return r
 end
 
 function test_existing_ref(r::RemoteChannel)
     found = getkey(client_refs, r, false)
     !is(found,false) && (client_refs[r] == true) && return found
     client_refs[r] = true
     finalizer(r, finalize_remote_channel)
-    r
+    return r
 end
 
 function finalize_remote_channel(r::RemoteChannel)
     if r.where > 0
         send_del_client(r)
         r.where = 0
-        r.whence = 0
-        r.id = 0
     end
-    r
+    return r
 end
 
 Future(w::LocalProcess) = Future(w.id)