commit b4b90fd2cc2b53789bffedc5ae829bd2d9b94009
author David Pursehouse <dpursehouse@collab.net> Mon May 27 14:07:04 2019 +0900
committer David Pursehouse <dpursehouse@collab.net> Mon May 27 14:07:38 2019 +0900
tree af52e25cff0d3f6d2f34fb3dd9d0e5a731fadb0e
parent 9286865ea0dc52c53b2e9cd03806fe7811338d96

StartCommand:  Fix synchronization on non-final field

When all error prone warnings are enabled the SynchronizeOnNonFinalField
bug pattern is reported:

  plugins/replication/src/main/java/com/googlesource/gerrit/plugins/replication/StartCommand.java:103:
  error: [SynchronizeOnNonFinalField] Synchronizing on non-final fields is not safe:
  if the field is ever updated, different threads may end up locking on different objects.
      synchronized (stdout) {
                   ^
    (see https://errorprone.info/bugpattern/SynchronizeOnNonFinalField)

Change-Id: Ib2df20aa28af4edd36ce5b9dfcf7d82c409dab84