perf annotate: Move jumps_percent_color to ui_browser

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

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jin Yao <yao.jin@linux.intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: https://lkml.kernel.org/n/tip-9f8c2f9aetbibcw33d615y9o@git.kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent bc1c0f3d
...@@ -81,12 +81,11 @@ static bool disasm_line__filter(struct ui_browser *browser, void *entry) ...@@ -81,12 +81,11 @@ static bool disasm_line__filter(struct ui_browser *browser, void *entry)
return false; return false;
} }
static int annotate_browser__jumps_percent_color(struct annotate_browser *browser, static int ui_browser__jumps_percent_color(struct ui_browser *browser, int nr, bool current)
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; return HE_COLORSET_SELECTED;
if (nr == notes->max_jump_sources) if (nr == notes->max_jump_sources)
return HE_COLORSET_TOP; return HE_COLORSET_TOP;
...@@ -95,11 +94,10 @@ static int annotate_browser__jumps_percent_color(struct annotate_browser *browse ...@@ -95,11 +94,10 @@ static int annotate_browser__jumps_percent_color(struct annotate_browser *browse
return HE_COLORSET_NORMAL; return HE_COLORSET_NORMAL;
} }
static int annotate_browser__set_jumps_percent_color(struct annotate_browser *browser, static int ui_browser__set_jumps_percent_color(struct ui_browser *browser, int nr, bool current)
int nr, bool current)
{ {
int color = annotate_browser__jumps_percent_color(browser, nr, current); int color = ui_browser__jumps_percent_color(browser, nr, current);
return ui_browser__set_color(&browser->b, color); return ui_browser__set_color(browser, color);
} }
static void disasm_line__write(struct disasm_line *dl, struct ui_browser *browser, 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 ...@@ -237,8 +235,8 @@ static void annotate_browser__write(struct ui_browser *browser, void *entry, int
printed = scnprintf(bf, sizeof(bf), "%*d ", printed = scnprintf(bf, sizeof(bf), "%*d ",
ab->jumps_width, ab->jumps_width,
bl->jump_sources); bl->jump_sources);
prev = annotate_browser__set_jumps_percent_color(ab, bl->jump_sources, prev = ui_browser__set_jumps_percent_color(browser, bl->jump_sources,
current_entry); current_entry);
ui_browser__write_nstring(browser, bf, printed); ui_browser__write_nstring(browser, bf, printed);
ui_browser__set_color(browser, prev); ui_browser__set_color(browser, prev);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment