From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 11 Jan 2011 19:13:01 GMT
Subject: Re: [PATCH] userdiff: match Pascal class methods
Message-ID: <7vipxv45ky.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1294736039-5912-1-git-send-email-zapped@mail.ru>

Alexey Shumkin <zapped@mail.ru> writes:

> Class declarations were already covered by the second pattern, but class methods have the 'class' keyword in front too. Account for it.

Too long a line (which I could re-wrap locally but I'd rather not be in
the business of doing that for everybody).

>  PATTERNS("pascal",
> -	 "^((procedure|function|constructor|destructor|interface|"
> +	 "^(((class[ \t]+)?(procedure|function)|constructor|destructor|interface|"
>  		"implementation|initialization|finalization)[ \t]*.*)$"

Earlier we took "^procedure frotz", "^function frotz", etc. and now we
also take "^class procedure frotz", "^class function frotz", but not
"^class constructor frotz"---am I reading the patterns correctly?