Hi Junio,

On Mon, 29 Aug 2016, Junio C Hamano wrote:

> Johannes Schindelin <johannes.schinde...@gmx.de> writes:
> 
> >     strbuf_addf(&buf, "%s\n", head);
> >     if (write_in_full(fd, buf.buf, buf.len) < 0)
> > -           die_errno(_("Could not write to %s"), git_path_head_file());
> > +           return error_errno(_("Could not write to %s"),
> > +                              git_path_head_file());
> 
> Same comment around a left-over lockfile applies to this.  An extra
> rollback being minimally intrusive also applies here, I think.

Sure. I added rollbacks in case of failure.

Ciao,
Dscho

Reply via email to