Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756802Ab1EWRH7 (ORCPT ); Mon, 23 May 2011 13:07:59 -0400 Received: from oproxy6-pub.bluehost.com ([67.222.54.6]:49958 "HELO oproxy6-pub.bluehost.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1756324Ab1EWRH6 (ORCPT ); Mon, 23 May 2011 13:07:58 -0400 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=xenotime.net; h=Received:Date:From:To:Cc:Subject:Message-Id:In-Reply-To:References:Organization:X-Mailer:Mime-Version:Content-Type:Content-Transfer-Encoding:X-Identified-User; b=er5uLM9hXWGi9lwQ/euHRKaNvN8MGPd2Q5VBSE48i6qJGeFHuyI7qRG1DnwnreH/qFIM8j4qXdBwQLEuaTiT2vzHCWCRNFmFlHJwCjPW3DhCugLSuSMsga9PQZY+TH9J; Date: Mon, 23 May 2011 10:07:55 -0700 From: Randy Dunlap To: anish singh Cc: Joe Perches , davej@redhat.com, rostedt@goodmis.org, apw@shadowen.org, akpm@linux-foundation.org, vapier@gentoo.org, linux-kernel@vger.kernel.org, man.k1983@gmail.com Subject: Re: [patch v3] checkpatch: Signature format verification Message-Id: <20110523100755.9e77d043.rdunlap@xenotime.net> In-Reply-To: References: <1306006721.1622.18.camel@anish-desktop> <1306059528.2761.3.camel@anish-desktop> <1306164094.2292.25.camel@anish-desktop> <1306167105.8687.11.camel@Joe-Laptop> <1306168598.8687.27.camel@Joe-Laptop> <20110523093920.8f1e5cd0.rdunlap@xenotime.net> <1306168975.8687.29.camel@Joe-Laptop> Organization: YPO4 X-Mailer: Sylpheed 2.7.1 (GTK+ 2.16.6; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Identified-User: {1807:box742.bluehost.com:xenotime:xenotime.net} {sentby:smtp auth 50.53.38.135 authed with rdunlap@xenotime.net} Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1638 Lines: 42 On Mon, 23 May 2011 22:36:12 +0530 anish singh wrote: > On Mon, May 23, 2011 at 10:12 PM, Joe Perches wrote: > > > On Mon, 2011-05-23 at 09:39 -0700, Randy Dunlap wrote: > > > On Mon, 23 May 2011 09:36:38 -0700 Joe Perches wrote: > > > > On Mon, 2011-05-23 at 22:03 +0530, anish singh wrote: > > > > > \s[a-zA-Z]* => this is a regular expression for name & space b/w > > > > > Last/First name > > > > > Ex: anish kumar > > > > > .* => It is used to grep whatever follows name...in our context > > > > > mail-id > > > > > Ex: abc@bbc.com. > > > > > So, $1 will contain ...'anish kumar ' > > > > Unless the email contains a quote or a period like: > > > > Signed-off-by: "J. Random Contributor" > > > and quotes are not required (i.e., are optional) AFAIK. > > > > And still should be accepted. > > > if ($1 !~ /^\s+([a-zA-Z\s\"]*.*)/i) { > WARN("Space required after $sign\n" . > $herecurr); > } > if ($1 !~ /[\sa-zA-Z\"]*\s<.*>/i) { > WARN("Space required b/w Full Name & Mail-id:\n" . "b/w" is not a good abbreviation (for "between" ???). > $herecurr); > } > Qoute has been taken care too now. --- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code *** -- 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/