On Mon, 2015-07-06 at 18:21 +0200, Michael Haggerty wrote:
<snip> changes applied; will re-roll.
> > +
> > +int safe_create_reflog(const char *refname, struct strbuf *err, int force_create)
> > +{
> > + int ret;
> > + struct strbuf sb = STRBUF_INIT;
> > +
> > + ret = log_ref_setup(refname, &sb, err, force_create);
> > + strbuf_release(&sb);
> > + return ret;
> > +}
> > +
>
> Is it really necessary to have two functions, safe_create_reflog() and
> log_ref_setup()? I don't see any of the callers doing anything special
> with the sb_logfile argument from the latter, so maybe it could be
> inlined into safe_create_reflog()? Maybe I'm overlooking something.
log_ref_write_1 does use the sb_logfile argument.
--
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