[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v2 4/5] iotests: Make 083 less flak
From: |
Eric Blake |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v2 4/5] iotests: Make 083 less flaky |
Date: |
Fri, 10 Nov 2017 12:26:49 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 11/10/2017 11:29 AM, Max Reitz wrote:
>>>> It seems that in this patch you're indenting with spaces but this file
>>>> uses tabs.
>>>
>>> Yes, but tabs are wrong. :-)
>>
>> I actually agree with you, but don't mix them in the file :-)
>
> I can whistle and say here, too, that Eric liked it. O:-)
I don't really pay attention to which files have pre-existing TABs.
You're right that preserving whole-file TABs is a bit nicer from
consistency than reformatting a file wholesale; but then you have to
tell checkpatch that preserving TABs was intentional. Mixed mode
indentation is not as consistent, but at least keeps checkpatch happy
without effort, and may make it easier for a patch down the road to
finally do wholesale conversion of the rest of the file to avoid TABs.
So when it comes to a file with existing TABs, I'm okay whether the
patch preserves TABs (with documentation that it is doing so
intentionally) or switches to mixed-mode spaces.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-block] [PATCH v2 1/5] iotests: Make 030 less flaky, (continued)
[Qemu-block] [PATCH v2 5/5] iotests: Make 136 less flaky, Max Reitz, 2017/11/09
Re: [Qemu-block] [PATCH v2 0/5] iotests: Make some tests less flaky, Max Reitz, 2017/11/09