)]}'
{
  "commit": "5d224a559ceec2b45811c3f9080744ad8bb2391c",
  "tree": "3245610d307985dd65b5458897866e838a2dec69",
  "parents": [
    "c41a3a937309f0ebfe33e2314a8f5b9838d6f5cc"
  ],
  "author": {
    "name": "David Benjamin",
    "email": "davidben@google.com",
    "time": "Mon Jun 21 15:12:19 2021 -0400"
  },
  "committer": {
    "name": "Boringssl LUCI CQ",
    "email": "boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Wed Jun 23 18:55:26 2021 +0000"
  },
  "message": "runner: Clean up test logic.\n\nThis addresses some feedback in\nhttps://boringssl-review.googlesource.com/c/boringssl/+/48131/1/ssl/test/runner/runner.go#1555,\npulled into a separate CL for clarity:\n\nFirst, take the listener, waitChan, exec.Cmd trio and wrap them into a\nshimProcess type. shimProcess is now responsible for the -port flag, so\nit can manage the TCPListener internally.\n\nNext, take the core test loop and moves it into a doExchanges()\nfunction, so that it can use a more usual early return pattern for\nerrors, rather than thread err \u003d\u003d nil through all the control flow. With\nshimProcess pulled out, doExchanges() can just take a *shimProcess.\n\nFinally, unacted-on err variable has gotten very far from where it\u0027s\nactually used. Rename it to localErr, to align with our\nexpectedLocalError machinery.\n\nChange-Id: I63697a5d79040ad77fa06c125253ec5031aeaf5c\nReviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/48186\nCommit-Queue: David Benjamin \u003cdavidben@google.com\u003e\nReviewed-by: Adam Langley \u003cagl@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "b52a43b760f32cc038fe7d75b631b9d848981add",
      "old_mode": 33188,
      "old_path": "ssl/test/runner/runner.go",
      "new_id": "f9678506c589ae3b86747c58f69ff936a3d424f8",
      "new_mode": 33188,
      "new_path": "ssl/test/runner/runner.go"
    }
  ]
}