)]}'
{
  "commit": "70f1f3a860fe9e276a3ed216bf2b179ea0e8cde3",
  "tree": "b8f0eecd792a9fa76d11c9c99e3ddb35ab12f639",
  "parents": [
    "2091fdfc99f2cec60ae3fda2538ae1993b0fc8b8"
  ],
  "author": {
    "name": "David Pursehouse",
    "email": "dpursehouse@collab.net",
    "time": "Mon May 27 13:58:31 2019 +0900"
  },
  "committer": {
    "name": "David Pursehouse",
    "email": "dpursehouse@collab.net",
    "time": "Mon May 27 13:58:31 2019 +0900"
  },
  "message": "ExportReviewNotes: Fix synchronization on non-final field\n\nWhen all error prone warnings are enabled the SynchronizeOnNonFinalField\nbug pattern is reported:\n\n  plugins/reviewnotes/src/main/java/com/googlesource/gerrit/plugins/reviewnotes/ExportReviewNotes.java:110:\n  error: [SynchronizeOnNonFinalField] Synchronizing on non-final fields is not safe:\n  if the field is ever updated, different threads may end up locking on different objects.\n    synchronized (changes) {\n                 ^\n    (see https://errorprone.info/bugpattern/SynchronizeOnNonFinalField)\n\nChange-Id: Ie671af628f1ce60de53165048b80d88e2c4d26ae\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "06975a052e6777bb5dde9a4617d7ef885543fa6b",
      "old_mode": 33188,
      "old_path": "src/main/java/com/googlesource/gerrit/plugins/reviewnotes/ExportReviewNotes.java",
      "new_id": "447cb77bd194a284456e898ee7ddbfd43e227429",
      "new_mode": 33188,
      "new_path": "src/main/java/com/googlesource/gerrit/plugins/reviewnotes/ExportReviewNotes.java"
    }
  ]
}