From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] devtools: fix printing subject of checked patch Date: Mon, 01 Oct 2018 23:47:27 +0200 Message-ID: <3296063.PHNZ8QPBa7@xps> References: <20181001203813.1798-1-thomas@monjalon.net> <20181001204808.GA12992@hmswarspite.think-freely.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: Neil Horman To: dev@dpdk.org Return-path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id 377DB5B34 for ; Mon, 1 Oct 2018 23:47:30 +0200 (CEST) In-Reply-To: <20181001204808.GA12992@hmswarspite.think-freely.org> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 01/10/2018 22:48, Neil Horman: > On Mon, Oct 01, 2018 at 10:38:13PM +0200, Thomas Monjalon wrote: > > If checkpatches.sh is not run with verbose option (-v), > > the patch subject is printed as headline of errors only > > if there is an error reported by checkpatch.pl, not with other checks. > > The headline is moved to a function which is called after each check > > if there is an error and if it has not already be printed. > > > > One more addition, in verbose mode, checkpatch.pl is now announced > > as done for other checks. > > > > Signed-off-by: Thomas Monjalon > Acked-by: Neil Horman Applied