From: Junio C Hamano Date: Mon, 15 Jan 2024 17:19:30 GMT Subject: Re: [PATCH v2 1/2] revision: ensure MERGE_HEAD is a ref in prepare_show_merge Message-ID: In-Reply-To: Patrick Steinhardt writes: >> Which would mean that we do not need a separate "other_head" >> variable, and instead can use "MERGE_HEAD". > > There is no need for this, is there? We have already resolved the ref to > an object ID, so why not use that via `add_pending_oid()`? add_pending_oid() and its callees use the name only for human consumption (e.g., in error messages), as they all already have the object name.