diff options
| author |   Arnaldo Carvalho de Melo <acme@redhat.com> | 2017-02-13 17:04:05 -0300 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2017-02-13 17:22:34 -0300 |
| commit | 9ef6839bcce7ca944c1ace4a7247cf13ca92a28f (patch) | |
| tree | fc426aa211d391094f890ee82604403b4743a0fa /tools/perf/tests/perf-record.c | |
| parent | perf symbols: No need to check if sym->name is NULL (diff) | |
| download | linux-dev-9ef6839bcce7ca944c1ace4a7247cf13ca92a28f.tar.xz linux-dev-9ef6839bcce7ca944c1ace4a7247cf13ca92a28f.zip | |
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/tests/perf-record.c')
| -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) if (evlist == NULL) /* Fallback for kernels lacking PERF_COUNT_SW_DUMMY */ evlist = perf_evlist__new_default(); - if (evlist == NULL || argv == NULL) { + if (evlist == NULL) { pr_debug("Not enough memory to create evlist\n"); goto out; } |