diff --git a/configure.ac b/configure.ac index 82382447fde7c411f61dbd62f7db388a6a8d9cf9..8786651c012fae85a069dd543d4031d50b0fa274 100644 --- a/configure.ac +++ b/configure.ac @@ -466,7 +466,7 @@ if test "$enable_warn" != "no"; then # We will do this by hand instead and only default to the macro for unknown compilers case "$ax_cv_c_compiler_vendor" in gnu | clang) - CFLAGS="$CFLAGS -Wall -Wextra -Wno-unused-parameter" + CFLAGS="$CFLAGS -Wall -Wextra -Wshadow -Wno-unused-parameter" ;; intel) CFLAGS="$CFLAGS -w2 -Wunused-variable" diff --git a/src/queue.c b/src/queue.c index 38f8620fdc75d744df31513792e96323dbf83647..af4dfa3c94470814d4f6e7f53687a2fcba69d6dd 100644 --- a/src/queue.c +++ b/src/queue.c @@ -188,7 +188,7 @@ struct task *queue_gettask(struct queue *q, const struct task *prev, /* Set some pointers we will use often. */ int *qtid = q->tid; struct task *qtasks = q->tasks; - const int qcount = q->count; + const int old_qcount = q->count; /* Data for the sliding window in which to try the task with the best overlap with the previous task. */ @@ -201,7 +201,7 @@ struct task *queue_gettask(struct queue *q, const struct task *prev, int ind = -1; /* Loop over the queue entries. */ - for (int k = 0; k < qcount; k++) { + for (int k = 0; k < old_qcount; k++) { if (k < queue_search_window) { window[window_count].ind = k; window[window_count].tid = qtid[k]; diff --git a/tests/test125cells.c b/tests/test125cells.c index c7e01693b45f76fa21ffd397289fc06ad36af03d..a385a7c890fe27ed11d3c5d87d6903fa6d254516 100644 --- a/tests/test125cells.c +++ b/tests/test125cells.c @@ -564,7 +564,7 @@ int main(int argc, char *argv[]) { /* Start the test */ ticks time = 0; - for (size_t i = 0; i < runs; ++i) { + for (size_t n = 0; n < runs; ++n) { const ticks tic = getticks(); @@ -642,7 +642,7 @@ int main(int argc, char *argv[]) { time += toc - tic; /* Dump if necessary */ - if (i == 0) { + if (n == 0) { sprintf(outputFileName, "swift_dopair_125_%s.dat", outputFileNameExtension); dump_particle_fields(outputFileName, main_cell, solution, 0);