From 069c15d256ed5308d09a7a16d64c8af2416bed58 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ren=C3=A9=20Scharfe?= Date: Thu, 4 Sep 2025 19:58:25 +0200 Subject: [PATCH] object-name: declare pointer type of extend_abbrev_len()'s 2nd parameter MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Expose the expected type of the second parameter of extend_abbrev_len() instead of casting a void pointer internally. Just a single caller passes in a void pointer, the rest pass the correct type. Let the compiler help keeping it that way. Signed-off-by: René Scharfe Signed-off-by: Junio C Hamano --- object-name.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/object-name.c b/object-name.c index 88d1313028..6c6c38d3cc 100644 --- a/object-name.c +++ b/object-name.c @@ -691,10 +691,9 @@ static inline char get_hex_char_from_oid(const struct object_id *oid, return hex[oid->hash[pos >> 1] & 0xf]; } -static int extend_abbrev_len(const struct object_id *oid, void *cb_data) +static int extend_abbrev_len(const struct object_id *oid, + struct min_abbrev_data *mad) { - struct min_abbrev_data *mad = cb_data; - unsigned int i = mad->init_len; while (mad->hex[i] && mad->hex[i] == get_hex_char_from_oid(oid, i)) i++;