)]}'
{
  "commit": "462f5cae0f764569da4e6a7ab40ee1d3b85353ed",
  "tree": "cd6cd190540a07f3d74089be6fe99c5ec4f1fcd0",
  "parents": [
    "c15087d17bd3c146696bfe6abf86322d79bf61ca"
  ],
  "author": {
    "name": "Ævar Arnfjörð Bjarmason",
    "email": "avarab@gmail.com",
    "time": "Sun Mar 28 15:15:49 2021 +0200"
  },
  "committer": {
    "name": "Junio C Hamano",
    "email": "gitster@pobox.com",
    "time": "Sun Mar 28 19:03:10 2021 -0700"
  },
  "message": "fetch-pack: don\u0027t needlessly copy fsck_options\n\nChange the behavior of the .gitmodules validation added in\n5476e1efde (fetch-pack: print and use dangling .gitmodules,\n2021-02-22) so we\u0027re using one \"fsck_options\".\n\nI found that code confusing to read. One might think that not setting\nup the error_func earlier means that we\u0027re relying on the \"error_func\"\nnot being set in some code in between the two hunks being modified\nhere.\n\nBut we\u0027re not, all we\u0027re doing in the rest of \"cmd_index_pack()\" is\nfurther setup by calling fsck_set_msg_types(), and assigning to\ndo_fsck_object.\n\nSo there was no reason in 5476e1efde to make a shallow copy of the\nfsck_options struct before setting error_func. Let\u0027s just do this\nsetup at the top of the function, along with the \"walk\" assignment.\n\nSigned-off-by: Ævar Arnfjörð Bjarmason \u003cavarab@gmail.com\u003e\nSigned-off-by: Junio C Hamano \u003cgitster@pobox.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "2b2266a4b7debf9064efdbd56b732031b04456c2",
      "old_mode": 33188,
      "old_path": "builtin/index-pack.c",
      "new_id": "5ad80b85b473c1f785bc7b409d99c556f64a6757",
      "new_mode": 33188,
      "new_path": "builtin/index-pack.c"
    }
  ]
}