)]}'
{
  "commit": "034deee6c7630fad36ce2db5b9d1ab4afd78a09e",
  "tree": "9a3a744fc3f966550f45c389e44620c71ed71b73",
  "parents": [
    "871f9009d3fca0ac70d4df7f75c82b1e31213add"
  ],
  "author": {
    "name": "Phillip Wood",
    "email": "phillip.wood@dunelm.org.uk",
    "time": "Wed Jul 15 16:22:01 2026 +0100"
  },
  "committer": {
    "name": "Junio C Hamano",
    "email": "gitster@pobox.com",
    "time": "Wed Jul 15 12:16:21 2026 -0700"
  },
  "message": "sequencer: simplify pick_one_commit()\n\nUnless we\u0027re rebasing, all we do in pick_one_commit() is call\ndo_pick_commit() and return its result. Simplify the code by returning\nearly if we\u0027re not rebasing so that we don\u0027t have to repeatedly call\nis_rebase_i() in the rest of the function. Note that there are a couple\nof conditions that do not call is_rebase_i() but they check for either\nan \"edit\" or a \"fixup\" command, both of which imply we\u0027re rebasing.\n\nThe only block that does not return early is the one guarded by\n\"!res\". Move the return into that block to make it clear that after\nrecording the commit as rewritten, all we do is return from the\nfunction.\n\nAs the conditional blocks are all mutually exclusive (either the\nconditions are mutually exclusive, or an earlier conditional block\nthat would match a later one contains a \"return\" statement) chain\nthem together with \"else if\" to make that clear.\n\nWhile we could remove \"res\" from the conditions below \"if (!res)\"\nthey are left alone because, when we start using an enum in the next\ncommit, it makes it clear that these clauses are handling cases where\nthere are conflicts.\n\nSigned-off-by: Phillip Wood \u003cphillip.wood@dunelm.org.uk\u003e\nSigned-off-by: Junio C Hamano \u003cgitster@pobox.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "8f3eed205e7c7e54d16f12b1e4f130b9e2f3d2c9",
      "old_mode": 33188,
      "old_path": "sequencer.c",
      "new_id": "9016af9b5d78345670861cb869b8dc52933de6ae",
      "new_mode": 33188,
      "new_path": "sequencer.c"
    }
  ]
}
