The script and data-switch context menu are only meaningful when it deals with a data file. So add a check so that it cannot be shown when perf-top is run.
Signed-off-by: Namhyung Kim <[email protected]> --- tools/perf/ui/browsers/hists.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c index 05e94feba3cb..0affffeed89c 100644 --- a/tools/perf/ui/browsers/hists.c +++ b/tools/perf/ui/browsers/hists.c @@ -2309,7 +2309,7 @@ skip_annotation: socked_id); } /* perf script support */ - if (browser->he_selection) { + if (is_report_browser(hbt) && browser->he_selection) { if (sort__has_thread) { nr_options += add_script_opt(browser, &actions[nr_options], @@ -2332,10 +2332,12 @@ skip_annotation: NULL, browser->selection->sym); } } - nr_options += add_script_opt(browser, &actions[nr_options], - &options[nr_options], NULL, NULL); - nr_options += add_switch_opt(browser, &actions[nr_options], - &options[nr_options]); + if (is_report_browser(hbt)) { + nr_options += add_script_opt(browser, &actions[nr_options], + &options[nr_options], NULL, NULL); + nr_options += add_switch_opt(browser, &actions[nr_options], + &options[nr_options]); + } nr_options += add_exit_opt(browser, &actions[nr_options], &options[nr_options]); -- 2.6.4
