perf tools: Use ARRAY_SIZE() instead of ad hoc equivalent, spotted by array_size...

Fix the following coccicheck warning:

tools/perf/util/trace-event-parse.c:209:35-36: WARNING: Use ARRAY_SIZE

ARRAY_SIZE(arr) is a macro provided in tools/include/linux/kernel.h,
which not only measures the size of the array, but also makes sure
that `arr` is really an array.

It has been tested with gcc (Debian 8.3.0-6) 8.3.0.

Signed-off-by: Guo Zhengkui <[email protected]>
Cc: Alexander Shishkin <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Mark Rutland <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Link: http://lore.kernel.org/lkml/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>

diff --git a/tools/perf/util/trace-event-parse.c b/tools/perf/util/trace-event-parse.c
index 9634f0ae57bea0d3fda57abdd6c4673282bffa00..c9c83a40647c7e0ab8bc2d54dbb55361498353cd 100644 (file)
--- a/tools/perf/util/trace-event-parse.c
+++ b/tools/perf/util/trace-event-parse.c
@@ -206,7 +206,7 @@ unsigned long long eval_flag(const char *flag)
        if (isdigit(flag[0]))
                return strtoull(flag, NULL, 0);
 
-       for (i = 0; i < (int)(sizeof(flags)/sizeof(flags[0])); i++)
+       for (i = 0; i < (int)(ARRAY_SIZE(flags)); i++)
                if (strcmp(flags[i].name, flag) == 0)
                        return flags[i].value;