[PATCH 1/7] perf hists: Separate out hist print functions

2012-08-19 Thread Namhyung Kim
From: Namhyung Kim Separate out those functions into ui/stdio/hist.c. This is required for upcoming changes. Signed-off-by: Namhyung Kim --- tools/perf/Makefile| 3 +- tools/perf/ui/stdio/hist.c | 648 +++ tools/perf/util/hist.c | 677 ++---

Re: [PATCH 1/7] perf hists: Separate out hist print functions

2012-08-09 Thread Namhyung Kim
Hi, Arnaldo On Thu, 9 Aug 2012 16:18:27 -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, Aug 06, 2012 at 05:57:36PM +0900, Namhyung Kim escreveu: >> From: Namhyung Kim >> >> Separate out those functions into ui/hist.c. This is required for >> upcoming changes. > > Isn't it better to further sepa

Re: [PATCH 1/7] perf hists: Separate out hist print functions

2012-08-09 Thread Arnaldo Carvalho de Melo
Em Mon, Aug 06, 2012 at 05:57:36PM +0900, Namhyung Kim escreveu: > From: Namhyung Kim > > Separate out those functions into ui/hist.c. This is required for > upcoming changes. Isn't it better to further separate it by introducing the ui/stdio/ directory since these functions use fprintf? - Arna

[PATCH 1/7] perf hists: Separate out hist print functions

2012-08-06 Thread Namhyung Kim
From: Namhyung Kim Separate out those functions into ui/hist.c. This is required for upcoming changes. Signed-off-by: Namhyung Kim --- tools/perf/Makefile| 4 +- tools/perf/ui/hist.c | 648 ++ tools/perf/util/hist.c | 677 ++--