From 2c06b8d75023df1b3a5b92030a2a482fe3b2ce72 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bj=C3=B6rn=20Svensson?= Date: Fri, 22 Sep 2023 14:42:56 +0200 Subject: [PATCH] Fix Clang compiler warnings When building using a newer Clang with additional diagnostics the following warnings are given: -Wunused-but-set-variable -Wstrict-prototypes These issues are corrected in this commit. Reproducable using: CC="clang-13" CFLAGS="-Wall -pedantic" cmake .. --- src/slug.c | 2 -- tests/bench.c | 6 +++--- tests/bench.h | 4 ++-- tests/check_tree.c | 2 +- 4 files changed, 6 insertions(+), 8 deletions(-) diff --git a/src/slug.c b/src/slug.c index 5f89207..6974f53 100644 --- a/src/slug.c +++ b/src/slug.c @@ -90,7 +90,6 @@ int r3_slug_parse(r3_slug_t *s, const char *needle, int needle_len, const char * return 0; } - int cnt = 0; int state = 0; const char * p = offset; @@ -122,7 +121,6 @@ int r3_slug_parse(r3_slug_t *s, const char *needle, int needle_len, const char * if (s->pattern) { s->pattern_len = p - s->pattern; } - cnt++; state--; p++; break; diff --git a/tests/bench.c b/tests/bench.c index 9f961e0..dd8ecc3 100644 --- a/tests/bench.c +++ b/tests/bench.c @@ -17,7 +17,7 @@ -unsigned long unixtime() { +unsigned long unixtime(void) { struct timeval tp; if (gettimeofday((struct timeval *) &tp, (NULL)) == 0) { return tp.tv_sec; @@ -25,7 +25,7 @@ unsigned long unixtime() { return 0; } -double microtime() { +double microtime(void) { struct timeval tp; long sec = 0L; double msec = 0.0; @@ -93,7 +93,7 @@ void bench_append_csv(char *filename, int countOfB, ...) { -int main() +int main(void) { R3Node * n = r3_tree_create(1); diff --git a/tests/bench.h b/tests/bench.h index 6bbad50..08680d6 100644 --- a/tests/bench.h +++ b/tests/bench.h @@ -22,9 +22,9 @@ typedef struct { double end; } bench; -unsigned long unixtime(); +unsigned long unixtime(void); -double microtime(); +double microtime(void); void bench_start(bench *b); diff --git a/tests/check_tree.c b/tests/check_tree.c index b2ddf8c..19b6c14 100644 --- a/tests/check_tree.c +++ b/tests/check_tree.c @@ -304,7 +304,7 @@ START_TEST (test_node_construct_and_free) } END_TEST -static R3Node * create_simple_str_tree() { +static R3Node * create_simple_str_tree(void) { R3Node * n; n = r3_tree_create(10); r3_tree_insert_path(n, "/zoo", NULL);