Re: [PATCH] userdiff: match Pascal class methods
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 11, 2011, 19:13 UTC
- 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?