[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 03/28] linux-user: Reindent signal handling

From: Laurent Vivier
Subject: Re: [Qemu-devel] [PATCH v2 03/28] linux-user: Reindent signal handling
Date: Tue, 24 May 2016 09:31:00 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.0

Le 24/05/2016 à 08:47, Riku Voipio a écrit :
> On tiistaina 24. toukokuuta 2016 3.21.34 EEST, Laurent Vivier wrote:
>> Le 12/05/2016 à 19:47, Peter Maydell a écrit :
>>> From: Timothy E Baldwin <address@hidden>
>>> Some of the signal handling was a mess with a mixture of tabs and 8
>>> space
>>> indents.
>> And adds some braces, but not everywhere.
>> [and fails on checkpatch.pl]
> Reviewing this for unexpected changes is interesting. Found the most
> reasonable way was to compare the original and patched signal.c after
> passing them with "indent" using arguments roughly resembling qemu
> style[1]. This shows the added braces, which as far as I see don't
> change the logic.

Apply the patch, and "git show -w" shows you only non-space changes,
"git show -w --word-diff" gives you the exact changes.

>> If we don't add braces everywhere, perhaps it's better to add them
>> nowhere.
> I think I will merge this as is, since this would mean fallout in
> editing the patches that come on top of this patch.

I have no problem with that.

Reviewed-by: Laurent Vivier <address@hidden>

BTW, I think signal.c should be split in the different target
directories under linux-user (I should have patch somewhere for that).

>> A coccinelle script can help to add them later.
>> Laurent
>>> Signed-off-by: Timothy Edward Baldwin
>>> <address@hidden>
>>> Message-id:
>>> address@hidden
>>> Reviewed-by: Peter Maydell <address@hidden>
>>> [PMM: just rebased]
>>> Signed-off-by: Peter Maydell <address@hidden> ...
> [1] indent -nbad -bap -nbc -bbo -hnl -br -brs -c33 -cd33 -ncdb -ce -ci4 
> -cli0  -d0 -di1 -nfc1 -i4 -ip0 -l80 -lp -npcs -nprs -npsl -sai -saf -saw
> -ncs -nsc -sob -nfca -cp33 -ss -ts4 -il1 -nut # kernel style changed to
> 4 spaces

reply via email to

[Prev in Thread] Current Thread [Next in Thread]