[libvirt] [PATCH] nwfilter_gentech_driver: Fix indentation and typo
Laine Stump
laine at laine.org
Mon Feb 20 15:19:54 UTC 2017
On 02/20/2017 05:27 AM, Martin Kletzander wrote:
> On Fri, Feb 17, 2017 at 07:39:41PM +0530, Nitesh Konkar wrote:
>> Signed-off-by: Nitesh Konkar <nitkon12 at linux.vnet.ibm.com>
>> ---
>> src/nwfilter/nwfilter_gentech_driver.c | 4 ++--
>> src/nwfilter/nwfilter_gentech_driver.h | 2 +-
>> 2 files changed, 3 insertions(+), 3 deletions(-)
>>
>
> You are sending quite some amount of patches that do no functional
> change. That takes review time and adds a lot of unnecesary
> communication.
My issue with patches that only change formatting is that any touched
lines will have the history of their last functional change obscured by
the formatting change, making it more difficult to do forensic analysis
of the reasoning behind the code's existence (which is often very useful
when trying to fix a regression without causing a different regression).
These changes can also e the cause of merge errors when backporting
future bug fixes onto older maintenance branches.
Sometimes the advantage of making the change outweighs the potential
extra burden, but not always. Clean consistent code is nice to have, but
it doesn't need to be perfect.
More information about the libvir-list
mailing list