Re: [PATCH v8 30/44] refs.c: add transaction.status and track OPEN/CLOSED/ERROR

2014-05-22 Thread Ronnie Sahlberg
On Wed, May 21, 2014 at 3:22 PM, Jonathan Nieder wrote: > Ronnie Sahlberg wrote: > >> Please pull my ref-transactions branch. > > I'm at bd5736cb (2014-05-21 13:46) now. > >> On Wed, May 21, 2014 at 3:00 PM, Jonathan Nieder wrote: >>> Ronnie Sahlberg wrote: > --- a/refs.c +++ b/refs.c >

Re: [PATCH v8 30/44] refs.c: add transaction.status and track OPEN/CLOSED/ERROR

2014-05-21 Thread Jonathan Nieder
Ronnie Sahlberg wrote: > Please pull my ref-transactions branch. I'm at bd5736cb (2014-05-21 13:46) now. > On Wed, May 21, 2014 at 3:00 PM, Jonathan Nieder wrote: >> Ronnie Sahlberg wrote: >>> --- a/refs.c >>> +++ b/refs.c >>> @@ -3308,6 +3308,12 @@ struct ref_update { >>> const char ref

Re: [PATCH v8 30/44] refs.c: add transaction.status and track OPEN/CLOSED/ERROR

2014-05-21 Thread Ronnie Sahlberg
Please pull my ref-transactions branch. On Wed, May 21, 2014 at 3:00 PM, Jonathan Nieder wrote: > Ronnie Sahlberg wrote: > >> --- a/refs.c >> +++ b/refs.c >> @@ -3308,6 +3308,12 @@ struct ref_update { >> const char refname[FLEX_ARRAY]; >> }; >> >> +enum ref_transaction_status { >> + RE

Re: [PATCH v8 30/44] refs.c: add transaction.status and track OPEN/CLOSED/ERROR

2014-05-21 Thread Jonathan Nieder
Ronnie Sahlberg wrote: > --- a/refs.c > +++ b/refs.c > @@ -3308,6 +3308,12 @@ struct ref_update { > const char refname[FLEX_ARRAY]; > }; > > +enum ref_transaction_status { > + REF_TRANSACTION_OPEN = 0, > + REF_TRANSACTION_CLOSED = 1, > + REF_TRANSACTION_ERROR = 2, What is

[PATCH v8 30/44] refs.c: add transaction.status and track OPEN/CLOSED/ERROR

2014-05-15 Thread Ronnie Sahlberg
Track the status of a transaction in a new status field. Check the field for sanity, i.e. that status must be OPEN when _commit/_create/_delete or _update is called or else die(BUG:...) Signed-off-by: Ronnie Sahlberg --- refs.c | 31 ++- 1 file changed, 30 insertions(