)]}'
{
  "commit": "64b8bbdf4cae812301e382fa6690e7f0328351e5",
  "tree": "2d5054b28d87b432432a20fab5e0ea5ba9491e3d",
  "parents": [
    "bc2e897dc710272401d6f299625c754168c6c9af"
  ],
  "author": {
    "name": "Rupert Swarbrick",
    "email": "rupert.swarbrick@argondesign.com",
    "time": "Mon Oct 16 15:53:07 2017 +0100"
  },
  "committer": {
    "name": "Sebastien Alaiwan",
    "email": "sebastien.alaiwan@allegrodvt.com",
    "time": "Thu Oct 19 05:04:15 2017 +0000"
  },
  "message": "Don\u0027t compute rtile width/height in av1_get_rest_ntiles\n\nRestoration units are a fixed square size (in cm-\u003erst_info[plane]) for\nalmost the entire image. The only special case is for tiles at the\nright hand edge or the bottom row, which might expand or be cropped.\n\nThe av1_get_rest_ntiles function was implementing the cropping\nbehaviour when the image happened to be less than one restoration unit\nwide or high (but not the expansion behaviour), but the result was\nnever useful: if you want to get the size of a restoration tile in\norder to divide by it to work out what tile you\u0027re on, the fixed\nsquare size is what you want. If you need to know how big this\nparticular tile is, call av1_get_rest_tile_limits.\n\nAs well as removing the output arguments from\nav1_get_rest_tile_limits, this patch also removes the tile_width and\ntile_height fields from the RestorationInternal structure. Note that\nthe tile size which is what you actually need is accessible as\nrst-\u003ersi-\u003erestoration_tilesize. (In practice, these were almost always\nthe same anyway).\n\nThis patch also has a couple of other small cleanups. Firstly, it\nmoves the subsampling_y field out of\nCONFIG_STRIPED_LOOP_RESTORATION. It\u0027s not actually needed when you\u0027re\nnot doing striped loop restoration, but this gets rid of lots of\nhorrible #if/#endif lines at callsites for av1_get_rest_tile_limits.\n\nSecondly, it simplifies the code in init_rest_search_ctxt (and fixes\nsome tautologous assertions). Now that YV12_BUFFER_CONFIG has a more\nuniform layout, there\u0027s a simpler way to set things up, so we use\nthat.\n\nChange-Id: I3c32d8ea0abe119dc86b9efa7564b27dde2151dc\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "0f9c81bd35bff37892b247f7807edfe364d79b6a",
      "old_mode": 33188,
      "old_path": "av1/common/restoration.c",
      "new_id": "d18497fbaef05d837ccd37e34b742afd0a92f4bf",
      "new_mode": 33188,
      "new_path": "av1/common/restoration.c"
    },
    {
      "type": "modify",
      "old_id": "23a53879eac4278bea8fffeefb63f4a41b9388f7",
      "old_mode": 33188,
      "old_path": "av1/common/restoration.h",
      "new_id": "60fbd3aface04605e7b66f142b100ebcf18c3d6f",
      "new_mode": 33188,
      "new_path": "av1/common/restoration.h"
    },
    {
      "type": "modify",
      "old_id": "aa400f88549647120d16d244c79db3ee883cf49f",
      "old_mode": 33188,
      "old_path": "av1/encoder/pickrst.c",
      "new_id": "0ac80ab377f18b2352103a7cebfce023a74ce81b",
      "new_mode": 33188,
      "new_path": "av1/encoder/pickrst.c"
    }
  ]
}