Patrick Steinhardt <ps@xxxxxx> writes: > While the actual logic to update the transaction is handled in > `update_refs_stdin()`, the transaction itself is started and committed > in `cmd_update_ref()` itself. This makes it hard to handle transaction > abortion and commits as part of `update_refs_stdin()` itself, ... It is not offhand clear why it makes any difference, especially because there is only a single caller of update_refs_stdin() function, but let's see how this move makes things easier in [8/9] and [9/9]. The patch itself looks like a bug-free no-op change. Thanks.
