Re: [PATCH v3 1/4] perf diff: color the Delta column

2013-11-29 Thread Jiri Olsa
On Thu, Nov 28, 2013 at 10:54:25PM +0530, Ramkumar Ramachandra wrote: > Jiri Olsa wrote: > > these colors are not consistent with colors in baseline, > > moreover all negative values are shown as red > > > > - please check get_percent_color function (used for baseline), > > it checks the

Re: [PATCH v3 1/4] perf diff: color the Delta column

2013-11-29 Thread Jiri Olsa
On Thu, Nov 28, 2013 at 10:54:25PM +0530, Ramkumar Ramachandra wrote: Jiri Olsa wrote: these colors are not consistent with colors in baseline, moreover all negative values are shown as red - please check get_percent_color function (used for baseline), it checks the percentage against

Re: [PATCH v3 1/4] perf diff: color the Delta column

2013-11-28 Thread Ramkumar Ramachandra
Jiri Olsa wrote: > these colors are not consistent with colors in baseline, > moreover all negative values are shown as red > > - please check get_percent_color function (used for baseline), > it checks the percentage against following values: > > #define MIN_GREEN 0.5 > #define

Re: [PATCH v3 1/4] perf diff: color the Delta column

2013-11-28 Thread Jiri Olsa
On Wed, Nov 27, 2013 at 04:32:31PM +0530, Ramkumar Ramachandra wrote: > Color the numbers in the Delta column either green or red depending on > whether the number is positive or negative. > > Cc: Jiri Olsa > Cc: Namhyung Kim > Cc: Arnaldo Carvalho de Melo > Signed-off-by: Ramkumar Ramachandra

Re: [PATCH v3 1/4] perf diff: color the Delta column

2013-11-28 Thread Jiri Olsa
On Wed, Nov 27, 2013 at 04:32:31PM +0530, Ramkumar Ramachandra wrote: Color the numbers in the Delta column either green or red depending on whether the number is positive or negative. Cc: Jiri Olsa jo...@redhat.com Cc: Namhyung Kim namhy...@kernel.org Cc: Arnaldo Carvalho de Melo

Re: [PATCH v3 1/4] perf diff: color the Delta column

2013-11-28 Thread Ramkumar Ramachandra
Jiri Olsa wrote: these colors are not consistent with colors in baseline, moreover all negative values are shown as red - please check get_percent_color function (used for baseline), it checks the percentage against following values: #define MIN_GREEN 0.5 #define MIN_RED

[PATCH v3 1/4] perf diff: color the Delta column

2013-11-27 Thread Ramkumar Ramachandra
Color the numbers in the Delta column either green or red depending on whether the number is positive or negative. Cc: Jiri Olsa Cc: Namhyung Kim Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/builtin-diff.c | 37 - 1 file

[PATCH v3 1/4] perf diff: color the Delta column

2013-11-27 Thread Ramkumar Ramachandra
Color the numbers in the Delta column either green or red depending on whether the number is positive or negative. Cc: Jiri Olsa jo...@redhat.com Cc: Namhyung Kim namhy...@kernel.org Cc: Arnaldo Carvalho de Melo a...@redhat.com Signed-off-by: Ramkumar Ramachandra artag...@gmail.com ---