Re: [PATCH 2/8] perf tools: Make __hpp__fmt() receive an additional len argument

2014-08-12 Thread Namhyung Kim
Hi Arnaldo, On Mon, 11 Aug 2014 10:17:38 -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, Aug 11, 2014 at 05:05:33PM +0900, Namhyung Kim escreveu: >> Hi Arnaldo, >> >> On Sat, 2 Aug 2014 10:30:26 -0300, Arnaldo Carvalho de Melo wrote: >> > Em Thu, Jul 31, 2014 at 02:47:36PM +0900, Namhyung Kim es

Re: [PATCH 2/8] perf tools: Make __hpp__fmt() receive an additional len argument

2014-08-11 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 11, 2014 at 05:05:33PM +0900, Namhyung Kim escreveu: > Hi Arnaldo, > > On Sat, 2 Aug 2014 10:30:26 -0300, Arnaldo Carvalho de Melo wrote: > > Em Thu, Jul 31, 2014 at 02:47:36PM +0900, Namhyung Kim escreveu: > >> So that it can properly handle alignment requirements later. To do > >> t

Re: [PATCH 2/8] perf tools: Make __hpp__fmt() receive an additional len argument

2014-08-11 Thread Namhyung Kim
Hi Arnaldo, On Sat, 2 Aug 2014 10:30:26 -0300, Arnaldo Carvalho de Melo wrote: > Em Thu, Jul 31, 2014 at 02:47:36PM +0900, Namhyung Kim escreveu: >> So that it can properly handle alignment requirements later. To do >> that, add percent_color_len_snprintf() fucntion to help coloring of >> overhea

Re: [PATCH 2/8] perf tools: Make __hpp__fmt() receive an additional len argument

2014-08-02 Thread Arnaldo Carvalho de Melo
Em Thu, Jul 31, 2014 at 02:47:36PM +0900, Namhyung Kim escreveu: > So that it can properly handle alignment requirements later. To do > that, add percent_color_len_snprintf() fucntion to help coloring of > overhead columns. Can you elaborate on this description? What is not possible to do before

[PATCH 2/8] perf tools: Make __hpp__fmt() receive an additional len argument

2014-07-30 Thread Namhyung Kim
So that it can properly handle alignment requirements later. To do that, add percent_color_len_snprintf() fucntion to help coloring of overhead columns. Signed-off-by: Namhyung Kim --- tools/perf/ui/browsers/hists.c | 14 -- tools/perf/ui/gtk/hists.c | 8 +--- tools/perf/u