On Wed Feb 14, 2024 at 6:46 PM AEDT, Patrick Steinhardt wrote: > + SWAP(refname, ref->refname); > + SWAP(refname_cap, ref->refname_cap); > reftable_ref_record_release(ref); > + SWAP(refname, ref->refname); > + SWAP(refname_cap, ref->refname_cap); Likewise here, what are your thoughts about reversing the SWAP arguments for the last two invocations?