diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2017-02-13 15:04:05 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2017-02-13 15:22:34 -0500 |
commit | 9ef6839bcce7ca944c1ace4a7247cf13ca92a28f (patch) | |
tree | fc426aa211d391094f890ee82604403b4743a0fa /tools/perf | |
parent | a7c3899c06865c75f8887f33d9043f6e8e780e71 (diff) |
perf tests record: No need to test an array against NULL
It will always evaluate to 'true', as clang warns:
CC /tmp/build/perf/tests/perf-record.o
CC /tmp/build/perf/tests/evsel-roundtrip-name.o
tests/perf-record.c:69:24: error: comparison of array 'argv' equal to a null pointer is always false [-Werror,-Wtautological-pointer-compare]
if (evlist == NULL || argv == NULL) {
^~~~ ~~~~
1 error generated.
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-o4977g6p9b3peak9ct6ef48q@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf')
-rw-r--r-- | tools/perf/tests/perf-record.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/tests/perf-record.c b/tools/perf/tests/perf-record.c index 8f2e1de6d0ea..541da7a68f91 100644 --- a/tools/perf/tests/perf-record.c +++ b/tools/perf/tests/perf-record.c | |||
@@ -66,7 +66,7 @@ int test__PERF_RECORD(int subtest __maybe_unused) | |||
66 | if (evlist == NULL) /* Fallback for kernels lacking PERF_COUNT_SW_DUMMY */ | 66 | if (evlist == NULL) /* Fallback for kernels lacking PERF_COUNT_SW_DUMMY */ |
67 | evlist = perf_evlist__new_default(); | 67 | evlist = perf_evlist__new_default(); |
68 | 68 | ||
69 | if (evlist == NULL || argv == NULL) { | 69 | if (evlist == NULL) { |
70 | pr_debug("Not enough memory to create evlist\n"); | 70 | pr_debug("Not enough memory to create evlist\n"); |
71 | goto out; | 71 | goto out; |
72 | } | 72 | } |