These patches are the remaining patches that need review in this series. Please review them. On Thu, May 29, 2014 at 9:07 AM, Ronnie Sahlberg <sahlberg@xxxxxxxxxx> wrote: > This patch series can also be found at > https://github.com/rsahlberg/git/tree/ref-transactions > > > Ronnie please review these remaining patches in this series. > > > Sahlberg (44): > refs.c: constify the sha arguments for > ref_transaction_create|delete|update > refs.c: allow passing NULL to ref_transaction_free > refs.c: add a strbuf argument to ref_transaction_commit for error > logging > refs.c: add an err argument to repack_without_refs > refs.c: make ref_update_reject_duplicates take a strbuf argument for > errors > refs.c: add an err argument to delete_ref_loose > refs.c: make update_ref_write update a strbuf on failure > update-ref.c: log transaction error from the update_ref > refs.c: remove the onerr argument to ref_transaction_commit > refs.c: change ref_transaction_update() to do error checking and > return status > refs.c: change ref_transaction_create to do error checking and return > status > refs.c: ref_transaction_delete to check for error and return status > tag.c: use ref transactions when doing updates > replace.c: use the ref transaction functions for updates > commit.c: use ref transactions for updates > sequencer.c: use ref transactions for all ref updates > fast-import.c: change update_branch to use ref transactions > branch.c: use ref transaction for all ref updates > refs.c: change update_ref to use a transaction > refs.c: free the transaction before returning when number of updates > is 0 > refs.c: ref_transaction_commit should not free the transaction > fetch.c: clear errno before calling functions that might set it > fetch.c: change s_update_ref to use a ref transaction > fetch.c: use a single ref transaction for all ref updates > receive-pack.c: use a reference transaction for updating the refs > fast-import.c: use a ref transaction when dumping tags > walker.c: use ref transaction for ref updates > refs.c: make write_ref_sha1 static > refs.c: make lock_ref_sha1 static > refs.c: add transaction.status and track OPEN/CLOSED/ERROR > refs.c: remove the update_ref_lock function > refs.c: remove the update_ref_write function > refs.c: remove lock_ref_sha1 > refs.c: make prune_ref use a transaction to delete the ref > refs.c: make delete_ref use a transaction > refs.c: pass the ref log message to _create/delete/update instead of > _commit > refs.c: pass NULL as *flags to read_ref_full > refs.c: pack all refs before we start to rename a ref > refs.c: move the check for valid refname to lock_ref_sha1_basic > refs.c: call lock_ref_sha1_basic directly from commit > refs.c: add a new flag for transaction delete for refs we know are > packed only > refs.c: pass a skip list to name_conflict_fn > refs.c: make rename_ref use a transaction > refs.c: remove forward declaration of write_ref_sha1 > > branch.c | 30 ++-- > builtin/commit.c | 24 ++- > builtin/fetch.c | 29 +-- > builtin/receive-pack.c | 21 +-- > builtin/replace.c | 15 +- > builtin/tag.c | 15 +- > builtin/update-ref.c | 32 ++-- > cache.h | 2 + > fast-import.c | 42 +++-- > lockfile.c | 21 ++- > refs.c | 468 +++++++++++++++++++++++++++++-------------------- > refs.h | 54 +++--- > sequencer.c | 24 ++- > t/t3200-branch.sh | 2 +- > walker.c | 51 +++--- > 15 files changed, 491 insertions(+), 339 deletions(-) > > -- > 2.0.0.rc3.510.g20c254b > -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html