Re: [PATCH 4/7] perf completion: modernize style

2013-09-18 Thread Ramkumar Ramachandra
Hi Arnaldo, Arnaldo Carvalho de Melo wrote: > Can you check how is this in the current perf/core branch so that we can > move forward while I process some other patches? perf/completion is now 3 months old, and my more recent patches have already been merged into Linus' tree. Why is perf/completi

Re: [PATCH 4/7] perf completion: modernize style

2013-08-07 Thread Ramkumar Ramachandra
Arnaldo Carvalho de Melo wrote: > Can you check how is this in the current perf/core branch so that we can > move forward while I process some other patches? This series (except this part) was merged into perf/completion of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux 3 weeks ago, so I

Re: [PATCH 4/7] perf completion: modernize style

2013-08-07 Thread Arnaldo Carvalho de Melo
Em Tue, Jul 16, 2013 at 10:52:55PM +0530, Ramkumar Ramachandra escreveu: > Arnaldo Carvalho de Melo wrote: > > Huh? Why would this be more "modern", if both are equivalent, I think > > applying the same rules as we have for C code applies here. > > It's the prevalent style in git.git, and I figure

Re: [PATCH 4/7] perf completion: modernize style

2013-07-16 Thread Arnaldo Carvalho de Melo
Em Thu, Jul 04, 2013 at 06:11:28PM +0530, Ramkumar Ramachandra escreveu: > Signed-off-by: Ramkumar Ramachandra > --- > tools/perf/bash_completion | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tools/perf/bash_completion b/tools/perf/bash_completion > index d2598be..

Re: [PATCH 4/7] perf completion: modernize style

2013-07-16 Thread Ramkumar Ramachandra
Arnaldo Carvalho de Melo wrote: > Huh? Why would this be more "modern", if both are equivalent, I think > applying the same rules as we have for C code applies here. It's the prevalent style in git.git, and I figured that it was picked up from linux.git but didn't check. Drop this part otherwise.

[PATCH 4/7] perf completion: modernize style

2013-07-04 Thread Ramkumar Ramachandra
Signed-off-by: Ramkumar Ramachandra --- tools/perf/bash_completion | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/perf/bash_completion b/tools/perf/bash_completion index d2598be..8c04f5d 100644 --- a/tools/perf/bash_completion +++ b/tools/perf/bash_completion @@ -7,