riot-ci/RIOT
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

tests: fix cppcheck findings

Browse Source 
- bitarithm_timings: use more void, decrease range
- struct_tm_utility: add cppcheck-suppress invalidscanf
This commit is contained in:
Ludwig Ortmann 2014-08-01 12:41:16 +02:00
parent 60fd4caf37
commit d667c4964a
2 changed files with 21 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
tests/bitarithm_timings/main.c
View File

@ -57,14 +57,17 @@ static void run_test(const char *name, unsigned (*test)(unsigned))
i = 1; i = 1;
} }
volatile unsigned r;
for (unsigned j = 0; j < PER_ITERATION; ++j) { for (unsigned j = 0; j < PER_ITERATION; ++j) {
volatile unsigned r;
r = test(i); r = test(i);
r = test(-1u - i);
r = test(~i);
r = test(~(-1u - i));
}
(void) r; (void) r;
r = test(-1u - i);
(void) r;
r = test(~i);
(void) r;
r = test(~(-1u - i));
(void) r;
}
++count; ++count;
} while (done == 0); } while (done == 0);

22
tests/struct_tm_utility/main.c
View File

@ -26,6 +26,8 @@
*/ */
#include <stdio.h> #include <stdio.h>
#include <stdlib.h>
#include <stdbool.h>
#include "shell.h" #include "shell.h"
#include "posix_io.h" #include "posix_io.h"
@ -44,10 +46,18 @@ static const char DAY_NAMES[7][3] = {
}; };
static const char BOOL_NAMES[2][3] = { "NO", "YES" }; static const char BOOL_NAMES[2][3] = { "NO", "YES" };
bool proper_atoi(const char *a, int *i)
{
char *end;
*i = strtol(a, &end, 0);
return (a != end) && (*end == '\0');
}
static void cmd_days_in(int argc, char **argv) static void cmd_days_in(int argc, char **argv)
{ {
int mon; int mon;
if ((argc != 2) || (sscanf(argv[1], "%d", &mon) != 1) || (mon < 1) || (mon > 12)) { if ((argc != 2) || (proper_atoi(argv[1], &mon) != 1) || (mon < 1) || (mon > 12)) {
printf("Usage: %s <Month[1..12]>\n", argv[0]); printf("Usage: %s <Month[1..12]>\n", argv[0]);
} }
else { else {
@ -59,7 +69,7 @@ static void cmd_days_in(int argc, char **argv)
static void cmd_leap_year(int argc, char **argv) static void cmd_leap_year(int argc, char **argv)
{ {
int year; int year;
if ((argc != 2) || (sscanf(argv[1], "%d", &year) != 1)) { if ((argc != 2) || (proper_atoi(argv[1], &year) != 1)) {
printf("Usage: %s <Year>\n", argv[0]); printf("Usage: %s <Year>\n", argv[0]);
} }
else { else {
@ -71,7 +81,7 @@ static void cmd_leap_year(int argc, char **argv)
static void cmd_doomsday(int argc, char **argv) static void cmd_doomsday(int argc, char **argv)
{ {
int year; int year;
if ((argc != 2) || (sscanf(argv[1], "%d", &year) != 1)) { if ((argc != 2) || (proper_atoi(argv[1], &year) != 1)) {
printf("Usage: %s <Year>\n", argv[0]); printf("Usage: %s <Year>\n", argv[0]);
} }
else { else {
@ -83,9 +93,9 @@ static void cmd_doomsday(int argc, char **argv)
static void cmd_day(int argc, char **argv) static void cmd_day(int argc, char **argv)
{ {
int year, mon, day; int year, mon, day;
if ((argc != 4) || (sscanf(argv[1], "%d", &year) != 1) if ((argc != 4) || (proper_atoi(argv[1], &year) != 1)
|| (sscanf(argv[2], "%d", &mon) != 1) || (proper_atoi(argv[2], &mon) != 1)
|| (sscanf(argv[3], "%d", &day) != 1)) { || (proper_atoi(argv[3], &day) != 1)) {
printf("Usage: %s <Year> <Month[1..12]> <Day[1..31]>\n", argv[0]); printf("Usage: %s <Year> <Month[1..12]> <Day[1..31]>\n", argv[0]);
} }
else { else {