Messages in this thread | | | Subject | Re: [PATCH v2] checkpatch: add double empty line check | From | "Eilon Greenstein" <> | Date | Tue, 20 Nov 2012 22:26:23 +0200 |
| |
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
| |