[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAM9d7cjDNFDarYFLRXbO0o_HLJjU2FzPLc4vwuvOdyJoxB6jrQ@mail.gmail.com>
Date: Tue, 8 Jul 2014 17:05:44 +0900
From: Namhyung Kim <namhyung@...nel.org>
To: Arnaldo Carvalho de Melo <acme@...nel.org>
Cc: Jiri Olsa <jolsa@...hat.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Adrian Hunter <adrian.hunter@...el.com>,
Corey Ashford <cjashfor@...ux.vnet.ibm.com>,
David Ahern <dsahern@...il.com>,
Don Zickus <dzickus@...hat.com>,
Frederic Weisbecker <fweisbec@...il.com>,
Ingo Molnar <mingo@...nel.org>, Jiri Olsa <jolsa@...nel.org>,
Mike Galbraith <efault@....de>,
Paul Mackerras <paulus@...ba.org>,
Peter Zijlstra <peterz@...radead.org>,
Stephane Eranian <eranian@...gle.com>
Subject: Re: [RFC PATCHES 0/7] Add support for showing headers in the hist browser.
On Tue, Jul 8, 2014 at 3:23 AM, Arnaldo Carvalho de Melo
<acme@...nel.org> 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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists