)]}'
{
  "commit": "69c5f17f11a2afe6b56b04f5a4377e4332858cde",
  "tree": "a1a822e23ebc0e63f0183b1dca1978108099404e",
  "parents": [
    "116761ba9cdee81e0d7b4671f14f9bd256f2cb36"
  ],
  "author": {
    "name": "Jeff King",
    "email": "peff@peff.net",
    "time": "Thu Oct 06 09:23:19 2022 -0400"
  },
  "committer": {
    "name": "Junio C Hamano",
    "email": "gitster@pobox.com",
    "time": "Thu Oct 06 09:59:17 2022 -0700"
  },
  "message": "attr: drop DEBUG_ATTR code\n\nSince its inception in d0bfd026a8 (Add basic infrastructure to assign\nattributes to paths, 2007-04-12), the attribute code carries a little\nbit of debug code that is conditionally compiled only when DEBUG_ATTR is\nset. But since you have to know about it and make a special build of Git\nto use it, it\u0027s not clear that it\u0027s helping anyone (and there are very\nfew mentions of it on the list over the years).\n\nMeanwhile, it causes slight headaches. Since it\u0027s not built as part of a\nregular compile, it\u0027s subject to bitrot. E.g., this was dealt with in\n712efb1a42 (attr: make it build with DEBUG_ATTR again, 2013-01-15), and\nit currently fails to build with DEVELOPER\u003d1 since e810e06357 (attr:\ntighten const correctness with git_attr and match_attr, 2017-01-27).\n\nAnd it causes confusion with -Wunused-parameter; the \"what\" parameter of\nfill_one() is unused in a normal build, but needed in a debug build.\n\nLet\u0027s just get rid of this code (and the now-useless parameter).\n\nSigned-off-by: Jeff King \u003cpeff@peff.net\u003e\nSigned-off-by: Junio C Hamano \u003cgitster@pobox.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "8250b0695321a804be3d06777570702aee1e5f2e",
      "old_mode": 33188,
      "old_path": "attr.c",
      "new_id": "42ad6de8c7c61432e4a3c950d940eee433680a74",
      "new_mode": 33188,
      "new_path": "attr.c"
    }
  ]
}
