From: Han-Wen Nienhuys Date: Tue, 13 Jul 2021 08:28:38 GMT Subject: Re: [PATCH v6? 00/17] refs API: get rid of errno setting entirely Message-ID: In-Reply-To: On Sun, Jul 11, 2021 at 6:30 PM Ævar Arnfjörð Bjarmason wrote: > > After starting a review of v5 of Han-Wen's where I was coming up with > squashes-on-top I continued with it and saw if I could get rid of > errno setting entirely in refs. thanks for taking this further! Is there a place where I can pull this code (applying patches is bothersome). It adds resolve_refs_unsafe_with_errno() to the public API in refs.h, and I think it has an ugly signature, but I suppose it's better than having the output be implicit through the global errno variable. I read over your patches, and they seem OK to me, modulo small comments I posted. -- Han-Wen Nienhuys - Google Munich I work 80%. Don't expect answers from me on Fridays. -- Google Germany GmbH, Erika-Mann-Strasse 33, 80636 Munich Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Paul Manicle, Halimah DeLaine Prado