wt-status: drop unused status parameter
The v2_fix_up_changed() function doesn't actually need to see the wt_status struct. It's possible that could change in the future, but this is a static-local function with one caller. It would be easy to read-add it back then. Let's drop the unused parameter. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Jeff King committed
May 9, 2019 at 17:30 UTC
13a178129ffe91b4d2443c7950a399a634488225
1 file changed
+2
-4
wt-status.c
+2
-4
@@ -2045,9 +2045,7 @@ static void wt_porcelain_v2_submodule_state(
2045
/*
2046
* Fix-up changed entries before we print them.
2047
*/
2048
-static void wt_porcelain_v2_fix_up_changed(
2049
- struct string_list_item *it,
2050
- struct wt_status *s)
2048
+static void wt_porcelain_v2_fix_up_changed(struct string_list_item *it)
2049
{
2050
struct wt_status_change_data *d = it->util;
2051
@@ -2107,7 +2105,7 @@ static void wt_porcelain_v2_print_changed_entry(
2105
char submodule_token[5];
2106
char sep_char, eol_char;
2107
2110
- wt_porcelain_v2_fix_up_changed(it, s);
2108
+ wt_porcelain_v2_fix_up_changed(it);
2109
wt_porcelain_v2_submodule_state(d, submodule_token);
2110
2111
key[0] = d->index_status ? d->index_status : '.';