)]}'
{
  "commit": "edf6ccaeb3c66baa331aa55fe0fcdc5c4697f3e7",
  "tree": "11e66ff0ba3d6d50bc0ff0603fd62a7269a72a7b",
  "parents": [
    "608fa51d55ea09caa31c2a4ec2d365db858cd86f"
  ],
  "author": {
    "name": "Nathan Harold",
    "email": "nharold@google.com",
    "time": "Fri Sep 07 11:50:05 2018 -0700"
  },
  "committer": {
    "name": "Nathan Harold",
    "email": "nharold@google.com",
    "time": "Fri Sep 07 11:56:08 2018 -0700"
  },
  "message": "Don\u0027t Use NPE for Flow Control in getCellLocation\n\nRemove catch of NPE from getCellLocation and do explicit\nnull checks. This will make it easier to differentiate\nbetween actual problems and local NPEs. In pre-P, there\nwere multiple conditions that would intentionally return\nnull over the binder interface. In Post-P null is returned\nwhen location is disabled, which means that null is a very\ncommon return value. We shouldn\u0027t use NPE to handle the\ncase where location is disabled.\n\nBug: 114294321\nTest: manual w/ SL4A (no automated tests for TM)\nChange-Id: Ie5b3afa56f280ff901ff9fa7c93f958d754f109b\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "e6173c0717d4170cf70e66ac8e4327ee0453b8d8",
      "old_mode": 33188,
      "old_path": "telephony/java/android/telephony/TelephonyManager.java",
      "new_id": "3672671ccb5cb78501b20ebe75b06567b218fa4f",
      "new_mode": 33188,
      "new_path": "telephony/java/android/telephony/TelephonyManager.java"
    }
  ]
}