Re: [PATCH v6 5/7] refs: new public ref function: safe_create_reflog
On Mon, 2015-07-06 at 18:21 +0200, Michael Haggerty wrote:
<snip> changes applied; will re-roll.
Show 16 quoted lines
> > +
> > +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.
David Turner <dturner@twopensource.com>