Here is the next set of 20 patches of those you already reviewed.
I cut this patch off just before patch #40 in the previous series.
These are the patches numbered 20-39 in the original series.
I think I have addressed your concerns here so If you could take a quick look
and hopefully bless them that would be awesome!
On Tue, Jul 15, 2014 at 4:33 PM, Ronnie Sahlberg <sahlb...@google.com> wrote:
> This is the next 20 patches from my originally big patch series and follow
> the previous 19 patches that is now in juns tree.
> These patches were numbered 20-39 in the original 48-patch series.
> Changes since these patches were in the original series:
> - Addressing concerns from mhagger's review
> Ronnie Sahlberg (20):
> refs.c: change ref_transaction_create to do error checking and return
> refs.c: update ref_transaction_delete to check for error and return
> refs.c: make ref_transaction_begin take an err argument
> refs.c: add transaction.status and track OPEN/CLOSED/ERROR
> 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
> 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 lock_ref_sha1 static
> 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
> branch.c | 30 +++---
> builtin/commit.c | 24 +++--
> builtin/receive-pack.c | 96 +++++++++++++-------
> builtin/replace.c | 15 +--
> builtin/tag.c | 15 +--
> builtin/update-ref.c | 11 ++-
> fast-import.c | 53 +++++++----
> refs.c | 242
> refs.h | 78 ++++++++++++----
> sequencer.c | 27 ++++--
> walker.c | 59 +++++++-----
> 11 files changed, 403 insertions(+), 247 deletions(-)
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html