Re: [PATCH 5/4] perf ui/tui: Print helpline message as is

From: Arnaldo Carvalho de Melo
Date: Tue Jan 12 2016 - 10:24:08 EST


Em Mon, Jan 11, 2016 at 07:53:14PM +0900, Namhyung Kim escreveu:
> When a tip message contains a percent sign, it was treated printf format
> specifier so broken string was printed like below.
>
> Tip: Limit to show entries above 577nly: perf report --percent-limit 5
> ^^^
>
> As ui_browser__show receives format string, pass additional "%s" so that
> the help (tip) message can be printed as is.
>
> Tip: Limit to show entries above 5% only: perf report --percent-limit 5

Applied.

> Signed-off-by: Namhyung Kim <namhyung@xxxxxxxxxx>
> ---
> tools/perf/ui/browsers/hists.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c
> index 901d481e6cea..08c09ad755d2 100644
> --- a/tools/perf/ui/browsers/hists.c
> +++ b/tools/perf/ui/browsers/hists.c
> @@ -480,7 +480,7 @@ static int hist_browser__run(struct hist_browser *browser, const char *help)
>
> hists__browser_title(browser->hists, hbt, title, sizeof(title));
>
> - if (ui_browser__show(&browser->b, title, help) < 0)
> + if (ui_browser__show(&browser->b, title, "%s", help) < 0)
> return -1;
>
> while (1) {
> --
> 2.6.4