> On 7 Aug 2019, at 18:49, Roger Riggs <roger.ri...@oracle.com> wrote: > > <snip> > > It seemed unnecessary/redundant since except for isRunning it is updated > inside a synchronize(lock).
Pardon, what seemed unnecessary? > The synchronized(socketList) is also unnecessary since it is already > synchronized on lock. You are right, this is redundant, albeit harmless. It might be a leftover from one of the design iterations we've had when synchronization was more fine-grained. I'll remove all the synchronization on `socketList`. Thanks.
