diff options
| author | Arnaldo Carvalho de Melo <[email protected]> | 2015-10-22 21:10:52 +0000 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <[email protected]> | 2015-10-22 21:10:52 +0000 |
| commit | f06cff7c59b6b252d667435d7baad48687b41002 (patch) | |
| tree | eee276fd27aeae2e92e2aff91cb4a3af87a792fa /tools/perf/builtin-script.c | |
| parent | perf ui tui: Register the error callbacks before initializing the widgets (diff) | |
| download | kernel-f06cff7c59b6b252d667435d7baad48687b41002.tar.gz kernel-f06cff7c59b6b252d667435d7baad48687b41002.zip | |
perf annotate: Don't die() when finding an invalid config option
The perf_config() infrastructure we inherited from git calls die() when
the provided config callback returns -1, meaning some key in a config
section is unexpected, that seems ok for a stdio based tool, but in
--tui we end up messing up the output, so just tell the user about the
error, wait for a keystroke and return 0, being more resilient and
proceeding with what we managed to parse.
That die() needs to die, tho :-)
Cc: Adrian Hunter <[email protected]>
Cc: Borislav Petkov <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Frederic Weisbecker <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Stephane Eranian <[email protected]>
Cc: Wang Nan <[email protected]>
Link: http://lkml.kernel.org/n/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
Diffstat (limited to 'tools/perf/builtin-script.c')
0 files changed, 0 insertions, 0 deletions
