Commit-ID:  27feb761c7211c6c35350277b6c65989b982b377
Gitweb:     https://git.kernel.org/tip/27feb761c7211c6c35350277b6c65989b982b377
Author:     Arnaldo Carvalho de Melo <[email protected]>
AuthorDate: Thu, 15 Mar 2018 15:14:45 -0300
Committer:  Arnaldo Carvalho de Melo <[email protected]>
CommitDate: Tue, 20 Mar 2018 13:19:28 -0300

perf annotate: Move jumps_percent_color to ui_browser

Since all it needs is in ui_browser and annotation structs members.

Cc: Adrian Hunter <[email protected]>
Cc: Andi Kleen <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Jin Yao <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Wang Nan <[email protected]>
Link: https://lkml.kernel.org/n/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
---
 tools/perf/ui/browsers/annotate.c | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/tools/perf/ui/browsers/annotate.c 
b/tools/perf/ui/browsers/annotate.c
index 3bc003fe0b1d..6e2eea09a9b0 100644
--- a/tools/perf/ui/browsers/annotate.c
+++ b/tools/perf/ui/browsers/annotate.c
@@ -81,12 +81,11 @@ static bool disasm_line__filter(struct ui_browser *browser, 
void *entry)
        return false;
 }
 
-static int annotate_browser__jumps_percent_color(struct annotate_browser 
*browser,
-                                                int nr, bool current)
+static int ui_browser__jumps_percent_color(struct ui_browser *browser, int nr, 
bool current)
 {
-       struct annotation *notes = browser__annotation(&browser->b);
+       struct annotation *notes = browser__annotation(browser);
 
-       if (current && (!browser->b.use_navkeypressed || 
browser->b.navkeypressed))
+       if (current && (!browser->use_navkeypressed || browser->navkeypressed))
                return HE_COLORSET_SELECTED;
        if (nr == notes->max_jump_sources)
                return HE_COLORSET_TOP;
@@ -95,11 +94,10 @@ static int annotate_browser__jumps_percent_color(struct 
annotate_browser *browse
        return HE_COLORSET_NORMAL;
 }
 
-static int annotate_browser__set_jumps_percent_color(struct annotate_browser 
*browser,
-                                                    int nr, bool current)
+static int ui_browser__set_jumps_percent_color(struct ui_browser *browser, int 
nr, bool current)
 {
-        int color = annotate_browser__jumps_percent_color(browser, nr, 
current);
-        return ui_browser__set_color(&browser->b, color);
+        int color = ui_browser__jumps_percent_color(browser, nr, current);
+        return ui_browser__set_color(browser, color);
 }
 
 static void disasm_line__write(struct disasm_line *dl, struct ui_browser 
*browser,
@@ -237,8 +235,8 @@ static void annotate_browser__write(struct ui_browser 
*browser, void *entry, int
                                        printed = scnprintf(bf, sizeof(bf), 
"%*d ",
                                                            ab->jumps_width,
                                                            bl->jump_sources);
-                                       prev = 
annotate_browser__set_jumps_percent_color(ab, bl->jump_sources,
-                                                                               
         current_entry);
+                                       prev = 
ui_browser__set_jumps_percent_color(browser, bl->jump_sources,
+                                                                               
   current_entry);
                                        ui_browser__write_nstring(browser, bf, 
printed);
                                        ui_browser__set_color(browser, prev);
                                }

Reply via email to