@matheusbsbs / matheus-bagliano / commits / a66fad2d28

reftable: ignore unused argc/argv in test functions

There are several reftable test "main" functions that don't look at their argc/argv. They don't technically need to take these parameters, as they are called individually by cmd__reftable(). But it probably makes sense to keep them all consistent for now. In the long run these will probably all get converted to the unit-test framework anyway. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Jeff King committed Aug 17, 2024 at 04:23 UTC a66fad2d28f362cd0f91f13d98a7ddb8bc599c3e

3 files changed +3 -3

reftable/block_test.c

+1 -1

@@ -116,7 +116,7 @@ static void test_block_read_write(void)

116 }

117 }

118

119 -int block_test_main(int argc, const char *argv[])

119 +int block_test_main(int argc UNUSED, const char *argv[] UNUSED)

120 {

121 RUN_TEST(test_block_read_write);

122 return 0;

reftable/readwrite_test.c

+1 -1

@@ -951,7 +951,7 @@ static void test_corrupt_table(void)

951 strbuf_release(&buf);

952 }

953

954 -int readwrite_test_main(int argc, const char *argv[])

954 +int readwrite_test_main(int argc UNUSED, const char *argv[] UNUSED)

955 {

956 RUN_TEST(test_log_zlib_corruption);

957 RUN_TEST(test_corrupt_table);

reftable/stack_test.c

+1 -1

@@ -1084,7 +1084,7 @@ static void test_reftable_stack_compaction_concurrent_clean(void)

1084 clear_dir(dir);

1085 }

1086

1087 -int stack_test_main(int argc, const char *argv[])

1087 +int stack_test_main(int argc UNUSED, const char *argv[] UNUSED)

1088 {

1089 RUN_TEST(test_empty_add);

1090 RUN_TEST(test_read_file);