[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH qemu v2] slirp/debug: Print IP addresses in huma
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH qemu v2] slirp/debug: Print IP addresses in human readable form |
Date: |
Tue, 13 Mar 2018 11:25:59 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 03/12/2018 11:49 PM, Alexey Kardashevskiy wrote:
Signed-off-by: Alexey Kardashevskiy <address@hidden>
---
checkpatch.pl complains on every single changed line as it keeps
using tabs - do I need to post 's/\t/ /g'?
No. checkpatch.pl is guidance, but even stronger is 'be consistent to
what you are editing'; we know to ignore the checkpatch warnings on
slirp code as that has historically used tabs. Mixed style, where TABS
occur in the context but your new additions use space, is also okay.
(You CAN clean up the entire slirp files if you want, but get maintainer
buy-in before doing so; and make the indentation cleanup separate from
any other patch, so that a diff with whitespace ignored shows no change).
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org