[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] hw/display/cirrus_vga: Fix code mis-indentation
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH] hw/display/cirrus_vga: Fix code mis-indentation |
Date: |
Wed, 3 Jun 2020 08:13:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 5/30/20 7:41 PM, Thomas Huth wrote:
> On 29/05/2020 18.54, Philippe Mathieu-Daudé wrote:
>> While replacing fprintf() by qemu_log_mask() in commit
>> 2b55f4d3504, we incorrectly used a 'tab = 4 spaces'
>> alignment, leading to misindented new code. Fix now.
>>
>> Reported-by: Peter Maydell <peter.maydell@linaro.org>
>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>> ---
>> hw/display/cirrus_vga.c | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/hw/display/cirrus_vga.c b/hw/display/cirrus_vga.c
>> index 92c197cdde..212d6f5e61 100644
>> --- a/hw/display/cirrus_vga.c
>> +++ b/hw/display/cirrus_vga.c
>> @@ -1032,9 +1032,9 @@ static void cirrus_bitblt_start(CirrusVGAState * s)
>> } else {
>> if (s->cirrus_blt_mode & CIRRUS_BLTMODE_TRANSPARENTCOMP) {
>> if (s->cirrus_blt_pixelwidth > 2) {
>> - qemu_log_mask(LOG_GUEST_ERROR,
>> - "cirrus: src transparent without colorexpand "
>> - "must be 8bpp or 16bpp\n");
>> + qemu_log_mask(LOG_GUEST_ERROR,
>> + "cirrus: src transparent without
>> colorexpand "
>> + "must be 8bpp or 16bpp\n");
>> goto bitblt_ignore;
>> }
>> if (s->cirrus_blt_mode & CIRRUS_BLTMODE_BACKWARDS) {
>>
>
> I think it would be better to fix the TABs in the whole surounding area,
> too. Or maybe even in the whole file. Otherwise this problem will happen
> soon again...
Well, this is a small step to unbreak alignment mistake from latest pull
request because my editor was using a 1tab=4spaces configuration, and
QEMU style expects 1tab=8spaces:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg707941.html
Fixing the whole area is a different task IMO :)
Can this go via qemu-trivial?
>
> Thomas
>
>
- Re: [PATCH] hw/display/cirrus_vga: Fix code mis-indentation,
Philippe Mathieu-Daudé <=