diff options
author | Wang Nan <[email protected]> | 2015-12-07 02:35:46 +0000 |
---|---|---|
committer | Arnaldo Carvalho de Melo <[email protected]> | 2015-12-07 12:02:12 -0300 |
commit | 979d2cac1144da6b25334a8572c80cde9662105c (patch) | |
tree | 793dcda67a21a845f283bd6ea53f1ebbbf793b30 /tools/perf/builtin-annotate.c | |
parent | 837eeb7569bf2b3bd3b1b82e0e61edb19811036e (diff) |
perf hists browser: Reset selection when refresh
With the following steps:
Step 1: perf report
Step 2: Use UP/DOWN to select an entry, don't press 'ENTER'
Step 3: Use '/' to filter symbols, use a filter which returns
empty result
Step 4: Press 'ENTER'
We see that, even if we have filtered all the symbols (and the main
interface is empty), pressing 'ENTER' still selects one symbol. This
behavior surprises the user.
This patch resets browser->{he_,}selection in hist_browser__refresh()
and lets it choose default selection. In this case
browser->{he_,}selection keeps NULL so user won't see annotation item in
menu.
Signed-off-by: Wang Nan <[email protected]>
Acked-by: Namhyung Kim <[email protected]>
Tested-by: Arnaldo Carvalho de Melo <[email protected]>
Cc: Zefan Li <[email protected]>
Cc: [email protected]
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
Diffstat (limited to 'tools/perf/builtin-annotate.c')
0 files changed, 0 insertions, 0 deletions