)]}'
{
  "commit": "01563088dd47e31606cb884221e6ac67285bbe69",
  "tree": "d458bb52f3f9be2ab157204060badb1c898f05fb",
  "parents": [
    "5b401364a7428b1885a504bce41ac7b2cbfbf015"
  ],
  "author": {
    "name": "David Barker",
    "email": "david.barker@argondesign.com",
    "time": "Mon Oct 09 13:59:16 2017 +0100"
  },
  "committer": {
    "name": "David Barker",
    "email": "david.barker@argondesign.com",
    "time": "Thu Nov 02 14:21:41 2017 +0000"
  },
  "message": "OBU: Fix a few bugs\n\n* The function av1_decode_tg_tiles_and_wrapup performs some per-frame\n  initialization; some of this was mistakenly being performed\n  once per tile group instead, leading to strange behaviour\n  (eg, forgetting loop-restoration coefficients, forgetting\n  the boundary information for all but the last tile group, etc.)\n\n  Fix this by pulling all of the initialization code into its\n  own function and calling it only if the initialize_flag is set.\n\n* While fixing the above, I realized that the \u0027context_updated\u0027\n  flag in av1_decode_tg_tile_and_wrapup was not behaving as intended:\n  The idea is that, when using frame parallel mode, we save the\n  frame context early so that the next frame can start decoding.\n  Then we don\u0027t need to store the frame context at the end of\n  the frame, since we already dealt with it at the start of the frame.\n\n  However, this \u0027context_updated\u0027 flag was local to one tile group,\n  ie. it got reset to 0 once we started decoding the second tile group.\n  So we\u0027d end up storing the frame context again at the end of the frame\n  if there was \u003e1 tile group.\n\n  This didn\u0027t break anything, but it is a bit weird. So, to match\n  the original intent, we ditch the \u0027context_updated\u0027 flag and\n  directly check if we\u0027re in frame parallel mode when necessary.\n\n* Fix a bug where we read one byte too much from a tile group\n  OBU when the extended OBU header was used.\n\nBUG\u003daomedia:892\n\nChange-Id: Ifbe561de0de35525d809e23915ac5263273e8de7\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "d8356fa4d24e2208aa58d4e829325749e974a771",
      "old_mode": 33188,
      "old_path": "av1/decoder/decodeframe.c",
      "new_id": "2bf89184cfc336ddc543d8a13cca19a1c41bfee4",
      "new_mode": 33188,
      "new_path": "av1/decoder/decodeframe.c"
    }
  ]
}
