Re: [PATCH 10/10] perf sort: Consolidate sort_entry__setup_elide()

2013-04-03 Thread Namhyung Kim
On Wed, 3 Apr 2013 14:09:44 -0300, Arnaldo Carvalho de Melo wrote: > Em Wed, Apr 03, 2013 at 09:26:19PM +0900, Namhyung Kim escreveu: >> From: Namhyung Kim >> >> The same code was duplicate to places, factor them out to common >> sort__setup_elide(). > > Looks ok, applying after fixing up fuzzes

Re: [PATCH 10/10] perf sort: Consolidate sort_entry__setup_elide()

2013-04-03 Thread Arnaldo Carvalho de Melo
Em Wed, Apr 03, 2013 at 09:26:19PM +0900, Namhyung Kim escreveu: > From: Namhyung Kim > > The same code was duplicate to places, factor them out to common > sort__setup_elide(). Looks ok, applying after fixing up fuzzes due to this being at the end of the patchseries. Things like this that are

[PATCH 10/10] perf sort: Consolidate sort_entry__setup_elide()

2013-04-03 Thread Namhyung Kim
From: Namhyung Kim The same code was duplicate to places, factor them out to common sort__setup_elide(). Signed-off-by: Namhyung Kim --- tools/perf/builtin-diff.c | 4 +--- tools/perf/builtin-report.c | 20 +--- tools/perf/builtin-top.c| 4 +--- tools/perf/util/sort.c

[PATCH 10/10] perf sort: Consolidate sort_entry__setup_elide()

2013-04-03 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com The same code was duplicate to places, factor them out to common sort__setup_elide(). Signed-off-by: Namhyung Kim namhy...@kernel.org --- tools/perf/builtin-diff.c | 4 +--- tools/perf/builtin-report.c | 20 +---

Re: [PATCH 10/10] perf sort: Consolidate sort_entry__setup_elide()

2013-04-03 Thread Arnaldo Carvalho de Melo
Em Wed, Apr 03, 2013 at 09:26:19PM +0900, Namhyung Kim escreveu: From: Namhyung Kim namhyung@lge.com The same code was duplicate to places, factor them out to common sort__setup_elide(). Looks ok, applying after fixing up fuzzes due to this being at the end of the patchseries. Things

Re: [PATCH 10/10] perf sort: Consolidate sort_entry__setup_elide()

2013-04-03 Thread Namhyung Kim
On Wed, 3 Apr 2013 14:09:44 -0300, Arnaldo Carvalho de Melo wrote: Em Wed, Apr 03, 2013 at 09:26:19PM +0900, Namhyung Kim escreveu: From: Namhyung Kim namhyung@lge.com The same code was duplicate to places, factor them out to common sort__setup_elide(). Looks ok, applying after fixing