)]}'
{
  "commit": "bc132fa5c9d7ef74e10f1e47262cdb9481aaf144",
  "tree": "b74d188242705f37d9d180e50c30f60df8a2496f",
  "parents": [
    "c3ad76b7509c0d86eefe4adf09cb499e82a4c345"
  ],
  "author": {
    "name": "Wan-Teh Chang",
    "email": "wtc@google.com",
    "time": "Tue Nov 20 18:06:38 2018 -0800"
  },
  "committer": {
    "name": "Wan-Teh Chang",
    "email": "wtc@google.com",
    "time": "Thu Nov 29 18:16:43 2018 +0000"
  },
  "message": "Do not acquire extra fb references.\n\nThe cm-\u003eref_frame_map and cm-\u003enext_ref_frame_map arrays hold references\nto frame buffers. The policy is that whenever we store a frame buffer\nindex in cm-\u003eref_frame_map or cm-\u003enext_ref_frame_map, we increase the\nref_count of that frame buffer. Conversely, whenever we overwrite an\nelement of these arrays with another value, we need to decrease the\nref_count of the old frame buffer.\n\nUnder this policy, generate_next_ref_frame_map() acquires an extra frame\nbuffer reference in the \"if (mask \u0026 1)\" case. This still works out OK\nbecause both release_frame_buffers() and swap_frame_buffers() compensate\nfor the extra extra frame buffer references, but it makes the code very\nconfusing.\n\nThis CL fixes generate_next_ref_frame_map() so that it does not acquire\nthe extra frame buffer references, and makes the corresponding changes\nto release_frame_buffers() and swap_frame_buffers().\n\nBUG\u003daomedia:2266\n\nChange-Id: I88fb9c06f17fb5d3ec77ca40c063f763af88e9de\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "95820e566e7ff6ef7ff4f34521f8ce1e07024c9f",
      "old_mode": 33188,
      "old_path": "av1/decoder/decodeframe.c",
      "new_id": "a30b267655ff596f4291a18342b025899d28afee",
      "new_mode": 33188,
      "new_path": "av1/decoder/decodeframe.c"
    },
    {
      "type": "modify",
      "old_id": "019db7e806212ee52d066b80423b808d3d46c5b1",
      "old_mode": 33188,
      "old_path": "av1/decoder/decoder.c",
      "new_id": "773305d5f35cf696e2a38dc77f84388db2ad45b8",
      "new_mode": 33188,
      "new_path": "av1/decoder/decoder.c"
    }
  ]
}