Re: [RFC PATCHES 0/7] Add support for showing headers in the hist browser.

From: Namhyung Kim
Date: Tue Jul 08 2014 - 04:06:14 EST


On Tue, Jul 8, 2014 at 3:23 AM, Arnaldo Carvalho de Melo
<acme@xxxxxxxxxx> wrote:
> Em Fri, Jul 04, 2014 at 05:53:02PM +0900, Namhyung Kim escreveu:
>> you ;-) ). But I also think it needs to fix column/header alignment
>> first, hmm...
>
> This one liner should do the trick, i.e. left justify it.

Yes, but it needs some more tweaks. I'll try to fix it soon..

Thanks,
Namhyung


>
> - Arnaldo
>
> diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c
> index 1ec57dd82284..14e5a039bc45 100644
> --- a/tools/perf/util/sort.c
> +++ b/tools/perf/util/sort.c
> @@ -1215,7 +1215,7 @@ static int __sort__hpp_header(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp,
> hse = container_of(fmt, struct hpp_sort_entry, hpp);
> len = hists__col_len(&evsel->hists, hse->se->se_width_idx);
>
> - return scnprintf(hpp->buf, hpp->size, "%*s", len, hse->se->se_header);
> + return scnprintf(hpp->buf, hpp->size, "%-*s", len, hse->se->se_header);
> }
>
> static int __sort__hpp_width(struct perf_hpp_fmt *fmt,
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/