summaryrefslogtreecommitdiffstats
path: root/contrib/persistent-https
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2023-03-17 15:17:08 -0400
committerJunio C Hamano <gitster@pobox.com>2023-03-17 14:17:48 -0700
commit1e5e0974964170a917d6950dda74e82745ab65aa (patch)
tree44453ab06a144271911e7de4b09eaf091690486c /contrib/persistent-https
parent647edf79d6b61391392ed42435b3214c02f539f1 (diff)
downloadgit-1e5e0974964170a917d6950dda74e82745ab65aa.tar.gz
git-1e5e0974964170a917d6950dda74e82745ab65aa.zip
http: mark unused parameter in fill_active_slot() callbacks
We have a generic "fill" function that is used by both the dumb http push and fetch code paths. It takes a void parameter in case the caller wants to pass along extra data, but (since the previous commit) neither does so. So we could simply drop the extra parameter. But since it's good practice to provide a void pointer for in callback functions, we'll leave it here for the future, and just annotate it as unused (to appease -Wunused-parameter). While we're marking it, let's also fix the type in http-walker's function to have the correct "void" type. The original had to cast the function pointer and was technically undefined behavior (though generally OK in practice). Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib/persistent-https')
0 files changed, 0 insertions, 0 deletions