tests/: remove useless parenthesis

tests/datawizard/variable_size.c

@@ -120,7 +120,7 @@ static int variable_size_compare(void *data_interface_a, void *data_interface_b)
 	struct variable_size_interface *variable_b = data_interface_b;
 
 	/* Two variables are considered compatible if they have the same size */
-	return (variable_a->size == variable_b->size);
+	return variable_a->size == variable_b->size;
 }
 
 static void display_variable_size(starpu_data_handle_t handle, FILE *f)

tests/disk/disk_compute.c

@@ -182,7 +182,7 @@ int dotest(struct starpu_disk_ops *ops, char *base)
 		FPRINTF(stderr, "TEST SUCCESS\n");
 	else
 		FPRINTF(stderr, "TEST FAIL\n");
-	return (try ? EXIT_SUCCESS : EXIT_FAILURE);
+	return try ? EXIT_SUCCESS : EXIT_FAILURE;
 
 enodev:
 	return STARPU_TEST_SKIPPED;

tests/disk/disk_copy.c

@@ -138,7 +138,7 @@ int dotest(struct starpu_disk_ops *ops, void *param)
 		FPRINTF(stderr, "TEST SUCCESS\n");
 	else
 		FPRINTF(stderr, "TEST FAIL\n");
-	return (try ? EXIT_SUCCESS : EXIT_FAILURE);
+	return try ? EXIT_SUCCESS : EXIT_FAILURE;
 
 enodev:
 	return STARPU_TEST_SKIPPED;

tests/disk/disk_pack.c

@@ -236,7 +236,7 @@ int dotest(struct starpu_disk_ops *ops, char *base)
 		FPRINTF(stderr, "TEST SUCCESS\n");
 	else
 		FPRINTF(stderr, "TEST FAIL\n");
-	return (try ? EXIT_SUCCESS : EXIT_FAILURE);
+	return try ? EXIT_SUCCESS : EXIT_FAILURE;
 
 enodev:
 	return STARPU_TEST_SKIPPED;

tests/main/deprecated_func.c

@@ -1,6 +1,6 @@
 /* StarPU --- Runtime system for heterogeneous multicore architectures.
  *
- * Copyright (C) 2010, 2011, 2012, 2013  CNRS
+ * Copyright (C) 2010, 2011, 2012, 2013, 2017  CNRS
  *
  * StarPU is free software; you can redistribute it and/or modify
  * it under the terms of the GNU Lesser General Public License as published by
@@ -112,7 +112,7 @@ int submit_codelet(struct starpu_codelet cl, int where)
 	{
 		FPRINTF(stderr, "success when executing codelet <%s> with where=%d\n", cl.name, where);
 	}
-	return (x != y);
+	return x != y;
 }
 
 int main(int argc, char **argv)

tests/main/insert_task.c

@@ -1,6 +1,6 @@
 /* StarPU --- Runtime system for heterogeneous multicore architectures.
  *
- * Copyright (C) 2011, 2012, 2013, 2015  CNRS
+ * Copyright (C) 2011, 2012, 2013, 2015, 2017  CNRS
  *
  * StarPU is free software; you can redistribute it and/or modify
  * it under the terms of the GNU Lesser General Public License as published by
@@ -115,7 +115,7 @@ enodev:
         }
 
         FPRINTF(stderr, "values: %d (should be %d) %f (should be %f)\n\n", xx, x*_ifactor, ff, f*_ffactor);
-	return (ret == -ENODEV ? ret : xx == x*_ifactor && ff == f*_ffactor);
+	return ret == -ENODEV ? ret : xx == x*_ifactor && ff == f*_ffactor;
 }
 
 int main(int argc, char **argv)

tests/main/insert_task_many.c

@@ -133,8 +133,10 @@ int main(int argc, char **argv)
 	for (loop = 0; loop < nloops; loop++)
 	{
 		for (i = 0; i < NPARAMS; i++)
-			if ((codelet_dyn.dyn_modes[i] & STARPU_W))
+		{
+			if (codelet_dyn.dyn_modes[i] & STARPU_W)
 				expected[i]++;
+		}
 		ret = starpu_task_insert(&codelet_dyn,
 					 STARPU_R, data_handles[0],
 					 STARPU_R, data_handles[1],
@@ -157,8 +159,10 @@ int main(int argc, char **argv)
 
 		/* Same, but using the toomany codelet */
 		for (i = 0; i < NPARAMS; i++)
-			if ((codelet_dyn.dyn_modes[i] & STARPU_W))
+		{
+			if (codelet_dyn.dyn_modes[i] & STARPU_W)
 				expected[i]++;
+		}
 		ret = starpu_task_insert(&codelet_toomany,
 					 STARPU_R, data_handles[0],
 					 STARPU_R, data_handles[1],
@@ -181,8 +185,10 @@ int main(int argc, char **argv)
 
 		/* Same, but using the variable codelet */
 		for (i = 0; i < NPARAMS; i++)
-			if ((codelet_dyn.dyn_modes[i] & STARPU_W))
+		{
+			if (codelet_dyn.dyn_modes[i] & STARPU_W)
 				expected[i]++;
+		}
 		ret = starpu_task_insert(&codelet_variable,
 					 STARPU_R, data_handles[0],
 					 STARPU_R, data_handles[1],

tests/main/starpu_init.c

@@ -74,7 +74,7 @@ static int check_cpu(int env_cpu, int conf_cpu, int expected_cpu, int *cpu)
 	else
 	{
 		FPRINTF(stderr, "Number of CPUS: %3d -- Number of expected CPUs: %3d\n", *cpu, expected_cpu);
-		return (*cpu != expected_cpu);
+		return *cpu != expected_cpu;
 	}
 }