)]}'
{
  "commit": "e5dcd7841828fd4c03dfc8a5c52691ada979b7e2",
  "tree": "843c88476e50896f179d590fdff70ccd6c2ef9f0",
  "parents": [
    "d5bc7c60c72ee239b5c5d3e4aa808d29412f78d8"
  ],
  "author": {
    "name": "Taylor Blau",
    "email": "me@ttaylorr.com",
    "time": "Wed Jan 13 17:25:10 2021 -0500"
  },
  "committer": {
    "name": "Junio C Hamano",
    "email": "gitster@pobox.com",
    "time": "Wed Jan 13 21:53:48 2021 -0800"
  },
  "message": "pack-revindex.c: avoid direct revindex access in \u0027offset_to_pack_pos()\u0027\n\nTo prepare for on-disk reverse indexes, remove a spot in\n\u0027offset_to_pack_pos()\u0027 that looks at the \u0027revindex\u0027 array in \u0027struct\npacked_git\u0027.\n\nEven though this use of the revindex pointer is within pack-revindex.c,\nthis clean up is still worth doing. Since the \u0027revindex\u0027 pointer will be\nNULL when reading from an on-disk reverse index (instead the\n\u0027revindex_data\u0027 pointer will be mmaped to the \u0027pack-*.rev\u0027 file), this\ncall-site would have to include a conditional to lookup the offset for\nposition \u0027mi\u0027 each iteration through the search.\n\nSo instead of open-coding \u0027pack_pos_to_offset()\u0027, call it directly from\nwithin \u0027offset_to_pack_pos()\u0027.\n\nSigned-off-by: Taylor Blau \u003cme@ttaylorr.com\u003e\nSigned-off-by: Junio C Hamano \u003cgitster@pobox.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "a508d5f0a4df27c53b2271aaff4dceb242cecd45",
      "old_mode": 33188,
      "old_path": "pack-revindex.c",
      "new_id": "5e69bc7372f5476429fb11c53d5b4bd24390798c",
      "new_mode": 33188,
      "new_path": "pack-revindex.c"
    }
  ]
}
