[libvirt] [PATCH] nwfilter_gentech_driver: Fix indentation and typo

Martin Kletzander mkletzan at redhat.com
Mon Feb 20 10:27:56 UTC 2017

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.  While we appreciate patches, sending more and more small
patches that are not really needed is tiresome for most parties

If we stumple upon such typo or indentation, we either push it right
away as trivial (only if you have push access, though) or just keep it
in a separate branch and after a while, send it as a big patch which is
just one mail.  That way it doesn't take that much of a time and

Most of the time, though, the time spent on the typo/indentation fixing
could be spent on something better, more useful, and also rewarding.  So
most of the things get fixed just because someone has some patches
already in that area of code.

Have a nice day,
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170220/1a460910/attachment-0001.sig>

More information about the libvir-list mailing list