mirror of
https://github.com/git/git
synced 2024-09-29 21:27:13 +00:00
unit-tests: ignore unused argc/argv
All of the unit test programs have their own cmd_main() function, but none of them actually look at the argc/argv that is passed in. In the long run we may want them to handle options for the test harness. But we'd probably do that with a shared harness cmd_main(), dispatching to the individual tests. In the meantime, let's annotate the unused parameters to avoid triggering -Wunused-parameter. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
7046c85cce
commit
df9d638c24
|
@ -33,7 +33,7 @@
|
|||
"\x10\x11\x12\x13\x14\x15\x16\x17\x18\x19\x1a\x1b\x1c\x1d\x1e\x1f" \
|
||||
"\x7f"
|
||||
|
||||
int cmd_main(int argc, const char **argv) {
|
||||
int cmd_main(int argc UNUSED, const char **argv UNUSED) {
|
||||
TEST_CHAR_CLASS(isspace, " \n\r\t");
|
||||
TEST_CHAR_CLASS(isdigit, DIGIT);
|
||||
TEST_CHAR_CLASS(isalpha, LOWER UPPER);
|
||||
|
|
|
@ -38,7 +38,7 @@ static void check_hash_data(const void *data, size_t data_length,
|
|||
"SHA1 and SHA256 (%s) works", #literal); \
|
||||
} while (0)
|
||||
|
||||
int cmd_main(int argc, const char **argv)
|
||||
int cmd_main(int argc UNUSED, const char **argv UNUSED)
|
||||
{
|
||||
struct strbuf aaaaaaaaaa_100000 = STRBUF_INIT;
|
||||
struct strbuf alphabet_100000 = STRBUF_INIT;
|
||||
|
|
|
@ -20,7 +20,7 @@ static void t_calloc_100(struct mem_pool *pool)
|
|||
check(pool->mp_block->end != NULL);
|
||||
}
|
||||
|
||||
int cmd_main(int argc, const char **argv)
|
||||
int cmd_main(int argc UNUSED, const char **argv UNUSED)
|
||||
{
|
||||
TEST(setup_static(t_calloc_100, 1024 * 1024),
|
||||
"mem_pool_calloc returns 100 zeroed bytes with big block");
|
||||
|
|
|
@ -69,7 +69,7 @@ static void test_prio_queue(int *input, size_t input_size,
|
|||
#define TEST_INPUT(input, result) \
|
||||
test_prio_queue(input, ARRAY_SIZE(input), result, ARRAY_SIZE(result))
|
||||
|
||||
int cmd_main(int argc, const char **argv)
|
||||
int cmd_main(int argc UNUSED, const char **argv UNUSED)
|
||||
{
|
||||
TEST(TEST_INPUT(((int []){ 2, 6, 3, 10, 9, 5, 7, 4, 5, 8, 1, DUMP }),
|
||||
((int []){ 1, 2, 3, 4, 5, 5, 6, 7, 8, 9, 10 })),
|
||||
|
|
|
@ -145,7 +145,7 @@ static void test_u16_roundtrip(void)
|
|||
check_int(in, ==, out);
|
||||
}
|
||||
|
||||
int cmd_main(int argc, const char *argv[])
|
||||
int cmd_main(int argc UNUSED, const char *argv[] UNUSED)
|
||||
{
|
||||
TEST(test_common_prefix(), "common_prefix_size works");
|
||||
TEST(test_parse_names_normal(), "parse_names works for basic input");
|
||||
|
|
|
@ -452,7 +452,7 @@ static void t_default_write_opts(void)
|
|||
}
|
||||
|
||||
|
||||
int cmd_main(int argc, const char *argv[])
|
||||
int cmd_main(int argc UNUSED, const char *argv[] UNUSED)
|
||||
{
|
||||
TEST(t_default_write_opts(), "merged table with default write opts");
|
||||
TEST(t_merged_logs(), "merged table with multiple log updates for same ref");
|
||||
|
|
|
@ -142,7 +142,7 @@ static void t_merged_iter_pqueue_top(void)
|
|||
merged_iter_pqueue_release(&pq);
|
||||
}
|
||||
|
||||
int cmd_main(int argc, const char *argv[])
|
||||
int cmd_main(int argc UNUSED, const char *argv[] UNUSED)
|
||||
{
|
||||
TEST(t_pq_record(), "pq works with record-based comparison");
|
||||
TEST(t_pq_index(), "pq works with index-based comparison");
|
||||
|
|
|
@ -532,7 +532,7 @@ static void t_reftable_index_record_roundtrip(void)
|
|||
strbuf_release(&in.u.idx.last_key);
|
||||
}
|
||||
|
||||
int cmd_main(int argc, const char *argv[])
|
||||
int cmd_main(int argc UNUSED, const char *argv[] UNUSED)
|
||||
{
|
||||
TEST(t_reftable_ref_record_comparison(), "comparison operations work on ref record");
|
||||
TEST(t_reftable_log_record_comparison(), "comparison operations work on log record");
|
||||
|
|
|
@ -75,7 +75,7 @@ static void t_infix_walk(void)
|
|||
tree_free(root);
|
||||
}
|
||||
|
||||
int cmd_main(int argc, const char *argv[])
|
||||
int cmd_main(int argc UNUSED, const char *argv[] UNUSED)
|
||||
{
|
||||
TEST(t_tree_search(), "tree_search works");
|
||||
TEST(t_infix_walk(), "infix_walk works");
|
||||
|
|
|
@ -105,7 +105,7 @@ static void t_addstr(struct strbuf *buf, const void *data)
|
|||
check_str(buf->buf + orig_len, text);
|
||||
}
|
||||
|
||||
int cmd_main(int argc, const char **argv)
|
||||
int cmd_main(int argc UNUSED, const char **argv UNUSED)
|
||||
{
|
||||
if (!TEST(t_static_init(), "static initialization works"))
|
||||
test_skip_all("STRBUF_INIT is broken");
|
||||
|
|
|
@ -24,7 +24,7 @@ static void check_strcmp_offset(const char *string1, const char *string2,
|
|||
expect_offset), \
|
||||
"strcmp_offset(%s, %s) works", #string1, #string2)
|
||||
|
||||
int cmd_main(int argc, const char **argv)
|
||||
int cmd_main(int argc UNUSED, const char **argv UNUSED)
|
||||
{
|
||||
TEST_STRCMP_OFFSET("abc", "abc", 0, 3);
|
||||
TEST_STRCMP_OFFSET("abc", "def", -1, 0);
|
||||
|
|
|
@ -227,7 +227,7 @@ static void t_detach(void)
|
|||
free(detached);
|
||||
}
|
||||
|
||||
int cmd_main(int argc, const char **argv)
|
||||
int cmd_main(int argc UNUSED, const char **argv UNUSED)
|
||||
{
|
||||
TEST(t_static_init(), "static initialization");
|
||||
TEST(t_dynamic_init(), "dynamic initialization");
|
||||
|
|
|
@ -308,7 +308,7 @@ static void run_t_trailer_iterator(void)
|
|||
}
|
||||
}
|
||||
|
||||
int cmd_main(int argc, const char **argv)
|
||||
int cmd_main(int argc UNUSED, const char **argv UNUSED)
|
||||
{
|
||||
run_t_trailer_iterator();
|
||||
return test_done();
|
||||
|
|
Loading…
Reference in a new issue