[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-alpha: fix the braces
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH] target-alpha: fix the braces |
Date: |
Tue, 18 Mar 2014 14:11:36 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
On 03/18/2014 04:14 AM, Paolo Bonzini wrote:
> Il 17/03/2014 19:07, Richard Henderson ha scritto:
>> On 03/17/2014 04:28 AM, Paolo Bonzini wrote:
>>> Conform to coding style, and avoid further occurrences of bugs due to
>>> misplaced braces.
>>>
>>> Signed-off-by: Paolo Bonzini <address@hidden>
>>> ---
>>> target-alpha/translate.c | 313
>>> +++++++++++++++++++++++++++--------------------
>>> 1 file changed, 180 insertions(+), 133 deletions(-)
>>
>> I was considering going through the alpha translator and dropping all the
>> highly unlikely tests and stuff that can now be handled by the tcg
>> optimizers.
>> In the process I'd touch most of these lines and fix the conformance.
>>
>> But if we'd prefer to fix them all at once, that's fine too.
>>
>> Reviewed-by: Richard Henderson <address@hidden>
>
> As you prefer---I submitted this patch just to follow up on my promise to
> check
> the braces separately from the other day's fix.
Please go ahead with what you've got. Via trivial?
r~