[PATCH 07/10] files-backend: drop refs parameter from split_symref_update()
- From
Jeff King <peff@peff.net>
- Date
- Feb 14, 2019, 05:50 UTC
- Message-ID
- <20190214055045.GG20578@sigill.intra.peff.net>
- In-Reply-To
- <20190214054736.GA20091@sigill.intra.peff.net>
This parameter was added in fcc42ea0c9 (split_symref_update(): add a files_ref_store argument, 2016-09-04) without comment, but never used. The splitting is purely mechanical, and doesn't depend on the particular ref-store. Let's drop this parameter in the name of simplicity.
Signed-off-by: Jeff King <peff@peff.net> --- refs/files-backend.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/refs/files-backend.c b/refs/files-backend.c index dd8abe9185..ef053f716c 100644 --- a/refs/files-backend.c +++ b/refs/files-backend.c @@ -2254,8 +2254,7 @@ static int split_head_update(struct ref_update *update, * Note that the new update will itself be subject to splitting when * the iteration gets to it. */ -static int split_symref_update(struct files_ref_store *refs, - struct ref_update *update, +static int split_symref_update(struct ref_update *update, const char *referent, struct ref_transaction *transaction, struct string_list *affected_refnames, @@ -2449,7 +2448,7 @@ static int lock_ref_for_update(struct files_ref_store *refs, * of processing the split-off update, so we * don't have to do it here. */ - ret = split_symref_update(refs, update, + ret = split_symref_update(update, referent.buf, transaction, affected_refnames, err); if (ret)
-- 2.21.0.rc1.567.g648f076c3f