Commit eb31228b authored by Guo Zhengkui's avatar Guo Zhengkui Committed by Arnaldo Carvalho de Melo
Browse files

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



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: default avatarGuo Zhengkui <guozhengkui@vivo.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lore.kernel.org/lkml/20220307034008.4024-1-guozhengkui@vivo.com


Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 1f48989c

tools/perf/util/trace-event-parse.c

+1 −1
Original line number Diff line number Diff line
@@ -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;