On Dec 4, 2015, at 12:48 AM, Daniel Fuchs <daniel.fu...@oracle.com> wrote:
> The only other possible issue I spotted is in the test: > > Because readers[0] is set by one thread and read by both then using > an array without synchronization is not thread safe. I would > advise using an AtomicReference instead. The only point of the threads is to provoke a race condition so I don’t know that it matters in this case. Nonetheless I updated the test case per the suggestion and verified that it still provokes the failure in the build not containing the patch. Thanks, Brian
