Date: Thu, 17 May 2012 04:34:17 -0400 (EDT)
Message-Id: <20120517.043417.1244517151794670965.davem@davemloft.net>
To: jchapman@katalix.com
Cc: joe@perches.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH net-next] net: l2tp: Standardize logging styles
From: David Miller <davem@davemloft.net>
In-Reply-To: <4FB4ACD1.10509@katalix.com>
References: <1337198156.10019.8.camel@joe2Laptop>
	<4FB4ACD1.10509@katalix.com>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 957
Lines: 24

From: James Chapman <jchapman@katalix.com>
Date: Thu, 17 May 2012 08:46:25 +0100

> This is a good improvement. Thanks!
> 
> Signed-off-by: James Chapman <jchapman@katalix.com>

James, please don't quote an entire patch when ACK'ing a patch like.
Do you realize that this makes the whole patch have to be processed by
every recipient subscribed to these mailing lists as well as sites
that monitor discussions on patches such as patchwork?

Even if you want to comment on some portions of the patch, you should
only quote the very specific hunks you wish to comment upon.

It is absolutely never appropriate to quote an entire patch in an
email response, really.

Anyways I'll apply this patch, thanks everyone.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/