We were allowing for another thread to add an item to the cache after we'd already tried to pull out the SoftReference. Since the SoftReference had been pulled out before the item was added, it was nil, but since contains? returned true, we assumed that it was not nil. Signed-off-by: Ambrose Bonnaire-Sergeant <firstname.lastname@example.org>
…as inefficient in execution time, though this one might be inefficient in memory. More testing needs to be done aorund the two implementations.
Signed-off-by: fogus <email@example.com>