On 2/25/2014 11:08 PM, Ivan Gerasimov wrote:
I missed that you remove the strong reference (line 57). I think
it's good to hold the strong reference so that
ReferenceQueue.remove(timeout) will timeout and the test can verify
reliably.
This is an important part.
If we didn't remove the strong reference then both threads would wait
for the specified period of time even without the fix.
The point is to make both threads wake up from lock.wait(timeout), and
we do it by removing the strong reference and forcing it be enqueued
Now it becomes clear what the test does (thanks). It'd be helpful to
add some comments. The test should also verify that only one thread
gets a non-null reference == weakReference and the other should get a
null reference in addition to the wait time > TIMEOUT.
Mandy
