]> git.ipfire.org Git - thirdparty/git.git/commit - revision.h
revisions: drop unused "opt" parameter in "tweak" callbacks
authorJeff King <peff@peff.net>
Mon, 3 Jul 2023 06:44:16 +0000 (02:44 -0400)
committerJunio C Hamano <gitster@pobox.com>
Fri, 14 Jul 2023 00:24:00 +0000 (17:24 -0700)
commitcc88afad622b8166563cc6bd68de3155ec558ab0
treee697ed0660f81fa02ea2f4616997736321fabcfd
parent506d35f13da352975688b078e89cb978d44bf39c
revisions: drop unused "opt" parameter in "tweak" callbacks

The setup_revision_opt struct has a "tweak" function pointer, which can
be used to adjust parameters after setup_revisions() parses arguments,
but before it finalizes setup. In addition to the rev_info struct, the
callback receives a pointer to the setup_revision_opt, as well.

But none of the existing callbacks looks at the extra "opt" parameter,
leading to -Wunused-parameter warnings.

We could mark it as UNUSED, but instead let's remove it entirely. It's
conceivable that it could be useful for a callback to have access to the
"opt" struct. But in the 13 years that this mechanism has existed,
nobody has used it. So let's just drop it in the name of simplifying.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/diff-tree.c
builtin/log.c
revision.c
revision.h