On Fri, Nov 15, 2024 at 08:11:01AM +0100, Patrick Steinhardt wrote:
> On Fri, Nov 15, 2024 at 12:54:28AM +0800, shejialuo wrote:
> > + if (strbuf_read_file(&ref_content, iter->path.buf, 0) < 0 ) {
>
> Nit: there's a space too much here now.
>
I will improve this in the next version.
> > + /*
> > + * Ref file could be removed by another concurrent process. We should
> > + * ignore this error and continue to the next ref.
> > + */
> > + if (errno == ENOENT)
> > + goto cleanup;
> > +
> > + ret = error_errno(_("cannot read ref file '%s': %s"),
> > + iter->path.buf, strerror(errno));
> > + goto cleanup;
> > + }
>
> You report `errno` twice. This should be:
>
> ret = error_errno(_("cannot read ref file '%s'"), iter->path.buf);
>
> Other than that this version looks good to me, thanks!
>
Opps, I didn't think about it, I just copied it. I will fix this in the
next version.
> Patrick
