Re: [PATCH 0/8] sequencer refactoring

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Oswald

Thanks for working on this. I've only had time for a quick read of the first 7 patches but there are some worthwhile clean ups here and the series is well structured. I'll try and have a thorough look at the last patch but I'm going to be off line next week so it may take a while.

Best Wishes

Phillip

On 23/03/2023 16:22, Oswald Buddenhagen wrote:
This is a preparatory series for the separately posted 'rebase --rewind' patch,
but I think it has value in itself.


Oswald Buddenhagen (8):
   rebase: simplify code related to imply_merge()
   rebase: move parse_opt_keep_empty() down
   sequencer: pass around rebase action explicitly
   sequencer: create enum for edit_todo_list() return value
   rebase: preserve interactive todo file on checkout failure
   sequencer: simplify allocation of result array in
     todo_list_rearrange_squash()
   sequencer: pass `onto` to complete_action() as object-id
   rebase: improve resumption from incorrect initial todo list

  builtin/rebase.c              |  63 +++++++--------
  builtin/revert.c              |   3 +-
  rebase-interactive.c          |  36 ++++-----
  rebase-interactive.h          |  27 ++++++-
  sequencer.c                   | 139 +++++++++++++++++++---------------
  sequencer.h                   |  15 ++--
  t/t3404-rebase-interactive.sh |  34 ++++++++-
  7 files changed, 196 insertions(+), 121 deletions(-)




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux