From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Eilon Greenstein" Subject: Re: [PATCH v2] checkpatch: add double empty line check Date: Tue, 20 Nov 2012 22:26:23 +0200 Message-ID: <1353443183.10779.12.camel@lb-tlvb-eilong.il.broadcom.com> References: <20121120115239.GA7955@dm> <1353421624.6559.9.camel@lb-tlvb-eilong.il.broadcom.com> <20121120144329.GE7955@dm> <1353424027.6559.15.camel@lb-tlvb-eilong.il.broadcom.com> <20121120154443.GK7955@dm> <1353427570.6559.21.camel@lb-tlvb-eilong.il.broadcom.com> <20121120161417.GA17797@dm> <20121120163607.GB17797@dm> <1353438635.10779.10.camel@lb-tlvb-eilong.il.broadcom.com> <20121120193249.GH17797@dm> <20121120201142.GI17797@dm> Reply-To: eilong@broadcom.com Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: "Joe Perches" , "David Rientjes" , "linux-kernel@vger.kernel.org" , netdev To: "Andy Whitcroft" Return-path: In-Reply-To: <20121120201142.GI17797@dm> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Tue, 2012-11-20 at 20:11 +0000, Andy Whitcroft wrote: > > > > Actually the version I sent should indeed cope with the deleted lines > > regardless of order. It was cirtainly intended to. > > ... and I think I thought of a couple more corner cases neither solution > will find. So I am going to go away and make up a proper set of tests > for this apparently simple change. As it is really annoying when it > false positives. I will post against when I have something which works. > Thanks Andy! I will assist in testing it on all the scenarios I have created once you post it. I appreciate you taking the time to look into adding a test for this issue. Thanks, Eilon