diff options
| author | Arnaldo Carvalho de Melo <acme@kernel.org> | 2025-12-11 19:17:53 -0300 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2025-12-17 09:30:37 -0300 |
| commit | d311783bc68b011c77a4ef81321de2c94d7deffc (patch) | |
| tree | 85910e4263bfb24ada236994334a7a40dfedd11d | |
| parent | ea1013c1539270e372fc99854bc6e4d94eaeff66 (diff) | |
| download | linux-d311783bc68b011c77a4ef81321de2c94d7deffc.tar.gz linux-d311783bc68b011c77a4ef81321de2c94d7deffc.zip | |
perf list: Remove unused 'sep' variable
It is just being set to the return of strchr() but never used, just
ditch it and with it get rid of a warning about it not being const on
fedora 44.
Reviewed-by: Ian Rogers <irogers@google.com>
Link: https://lore.kernel.org/r/20251211221756.96294-2-acme@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
| -rw-r--r-- | tools/perf/builtin-list.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/perf/builtin-list.c b/tools/perf/builtin-list.c index 5cbca0bacd35..ac7bd0e41aa1 100644 --- a/tools/perf/builtin-list.c +++ b/tools/perf/builtin-list.c @@ -648,7 +648,7 @@ int cmd_list(int argc, const char **argv) } for (i = 0; i < argc; ++i) { - char *sep, *s; + char *s; if (strcmp(argv[i], "tracepoint") == 0) { char *old_pmu_glob = default_ps.pmu_glob; @@ -720,7 +720,7 @@ int cmd_list(int argc, const char **argv) else if (strcmp(argv[i], "pfm") == 0) print_libpfm_events(&print_cb, ps); #endif - else if ((sep = strchr(argv[i], ':')) != NULL) { + else if (strchr(argv[i], ':') != NULL) { char *old_pmu_glob = ps->pmu_glob; char *old_event_glob = ps->event_glob; |