Re: [Qemu-devel] [PATCH] Fix net.c coding style (with regard to tabs)

[Kevin Wolf <kwolf@redhat.com>]

Jean-Christophe Dubois schrieb:
> Le lundi 22 juin 2009 10:29:10 Kevin Wolf, vous avez écrit :
>> Jean-Christophe DUBOIS schrieb:
>>> This patch will replace existing tab in net.c with spaces
>>> in accordance with qemu CODING_STYLE.
>>>
>>> Note: This first patch is more of a test to see if it
>>> will be accepted and if it is worth spending my time on
>>> fixing the same issue in other files.
>>>
>>> Signed-off-by: Jean-Christophe DUBOIS <jcd@tribudubois.net>
>> I wouldn't do this. We should just pay attention to do it right in code
>> touched by new patches.
> 
> Some people are speaking about not killing 2 birds with one stone (and I don't 
> speak about the kitten ;-) ) and in general I would think you should fix one 
> problem per patch. I am not sure it is a good idea to ask somebody fixing one 
> line in a file to also fix all the coding style issues of this same file.

Agreed, this is obviously nonsense. I'm just talking about the lines
touched by the patch. What I mean is really just "Do it right in new
code, leave old code alone". You might argue that this results in files
with mixed indentation, but in my experience files using tabs are
already mixed in most cases.

Anyway, just my personal opinion and I'm not the one who's to decide if
we should have such whitespace cleanup patches.

Kevin