On Fri, Feb 17, 2012 at 14:31, Anthony Liguori aligu...@us.ibm.com wrote:
On 02/11/2012 03:44 AM, Blue Swirl wrote:
On Fri, Feb 10, 2012 at 17:47, Anthony Liguorialigu...@us.ibm.com
wrote:
On 02/09/2012 10:02 PM, malc wrote:
On Fri, 10 Feb 2012, Evgeny Voevodin wrote:
On 02/09/2012
On Fri, Feb 17, 2012 at 15:26, Anthony Liguori aligu...@us.ibm.com wrote:
On 02/17/2012 08:55 AM, Markus Armbruster wrote:
Anthony Liguorialigu...@us.ibm.com writes:
I really hate having these discussions. I would almost rather we just
pay the one-time cost of re-indenting so we can stop
Am 18.02.2012 10:13, schrieb Blue Swirl:
On Fri, Feb 17, 2012 at 15:26, Anthony Liguori aligu...@us.ibm.com
wrote:
Well this was what I was getting at in my previous comments. If we just
need to reindent 10 files with a few random changes here and there,
then
maybe that isn't so bad.
But
On 02/18/2012 04:56 AM, Stefan Weil wrote:
Am 18.02.2012 10:13, schrieb Blue Swirl:
On Fri, Feb 17, 2012 at 15:26, Anthony Liguori aligu...@us.ibm.com wrote:
Well this was what I was getting at in my previous comments. If we just
need to reindent 10 files with a few random changes here and
On Sat, Feb 18, 2012 at 14:18, Anthony Liguori anth...@codemonkey.ws wrote:
On 02/18/2012 04:56 AM, Stefan Weil wrote:
Am 18.02.2012 10:13, schrieb Blue Swirl:
On Fri, Feb 17, 2012 at 15:26, Anthony Liguori aligu...@us.ibm.com
wrote:
Well this was what I was getting at in my previous
Am 18.02.2012 10:13, schrieb Blue Swirl:
One way to handle this is gradual reformatting, every time when code
is touched, only changes towards common CODING_STYLE are allowed.
That's what we've been doing wrt braces and I appreciate us not
cluttering the history with reformatting commits.
On 02/18/2012 08:53 AM, Andreas Färber wrote:
I think only 'git blame' output would be affected and that is not 100%
reliable anyway, considering for example code movement.
I use repo.or.cz's blame function quite often to find out who to cc or
what commit to mention, and (valid) typo fixes
On 02/11/2012 03:44 AM, Blue Swirl wrote:
On Fri, Feb 10, 2012 at 17:47, Anthony Liguorialigu...@us.ibm.com wrote:
On 02/09/2012 10:02 PM, malc wrote:
On Fri, 10 Feb 2012, Evgeny Voevodin wrote:
On 02/09/2012 06:59 PM, Andreas F?rber wrote:
Disable warnings for spaces before opening
Anthony Liguori aligu...@us.ibm.com writes:
I really hate having these discussions. I would almost rather we just
pay the one-time cost of re-indenting so we can stop debating about
this.
For folks that feel strongly about this, please submit the following:
An indent command that takes
On 02/17/2012 08:55 AM, Markus Armbruster wrote:
Anthony Liguorialigu...@us.ibm.com writes:
I really hate having these discussions. I would almost rather we just
pay the one-time cost of re-indenting so we can stop debating about
this.
For folks that feel strongly about this, please submit
On Thu, Feb 9, 2012 at 14:59, Andreas Färber afaer...@suse.de wrote:
Disable warnings for spaces before opening parenthesis in
hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
Signed-off-by: Andreas Färber afaer...@suse.de
Cc: Blue Swirl blauwir...@gmail.com
Cc: malc av1...@comtv.ru
---
On Fri, Feb 10, 2012 at 17:47, Anthony Liguori aligu...@us.ibm.com wrote:
On 02/09/2012 10:02 PM, malc wrote:
On Fri, 10 Feb 2012, Evgeny Voevodin wrote:
On 02/09/2012 06:59 PM, Andreas F?rber wrote:
Disable warnings for spaces before opening parenthesis in
On 02/09/2012 10:02 PM, malc wrote:
On Fri, 10 Feb 2012, Evgeny Voevodin wrote:
On 02/09/2012 06:59 PM, Andreas F?rber wrote:
Disable warnings for spaces before opening parenthesis in
hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
Why audio files are such a special thing?
Because they are
Disable warnings for spaces before opening parenthesis in
hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
Signed-off-by: Andreas Färber afaer...@suse.de
Cc: Blue Swirl blauwir...@gmail.com
Cc: malc av1...@comtv.ru
---
scripts/checkpatch.pl |5 +
1 files changed, 5 insertions(+), 0
Am 09.02.2012 15:59, schrieb Andreas Färber:
Disable warnings for spaces before opening parenthesis in
hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
Signed-off-by: Andreas Färber afaer...@suse.de
Cc: Blue Swirl blauwir...@gmail.com
Cc: malc av1...@comtv.ru
---
For testing I used the
On Thu, 9 Feb 2012, Andreas F?rber wrote:
Disable warnings for spaces before opening parenthesis in
hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
Signed-off-by: Andreas F?rber afaer...@suse.de
Cc: Blue Swirl blauwir...@gmail.com
Cc: malc av1...@comtv.ru
Thanks.
---
scripts/checkpatch.pl
On 02/09/2012 06:59 PM, Andreas Färber wrote:
Disable warnings for spaces before opening parenthesis in
hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
Why audio files are such a special thing?
Isn't it be better to revert a patch that introduced checkpatch.pl errors?
--
Kind regards,
Evgeny
On Fri, 10 Feb 2012, Evgeny Voevodin wrote:
On 02/09/2012 06:59 PM, Andreas F?rber wrote:
Disable warnings for spaces before opening parenthesis in
hw/{ac97,adlib,cs4231a,es1370,gus,sb16}.c.
Why audio files are such a special thing?
Because they are consistently formatted the way they
18 matches
Mail list logo
