threads / patch / 23882

patchdiff: Support visibility modifiers in the PHP hunk header regexp

Subject: [PATCH] diff: Support visibility modifiers in the PHP hunk header regexp

## tl;dr

One message between May 23, 2010 and May 23, 2010. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Björn Steinbrink· May 23, 2010, 18:05 UTC · lore

Starting with PHP5, class methods can have a visibility modifier, which caused the methods not to be matched by the existing regexp, so extend the regexp to match those modifiers. And while we're at it, allow the "static" modifier as well.

Since the "static" modifier can appear either before or after the visibility modifier, let's just allow any number of modifiers to appear in any order, as that simplifies the regexp and shouldn't cause any false positives.

Signed-off-by: Björn Steinbrink <B.Steinbrink@gmx.de>
---
 userdiff.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)
Show changes to userdiff.c +3 −1
diff --git a/userdiff.c b/userdiff.c
index 67003fb..c49cc1b 100644
--- a/userdiff.c
+++ b/userdiff.c
@@ -45,7 +45,9 @@ PATTERNS("pascal",
 	 "|[-+0-9.e]+|0[xXbB]?[0-9a-fA-F]+"
 	 "|<>|<=|>=|:=|\\.\\."
 	 "|[^[:space:]]|[\x80-\xff]+"),
-PATTERNS("php", "^[\t ]*((function|class).*)",
+PATTERNS("php",
+	 "^[\t ]*(((public|protected|private|static)[\t ]+)*function.*)$\n"
+	 "^[\t ]*(class.*)$",
 	 /* -- */
 	 "[a-zA-Z_][a-zA-Z0-9_]*"
 	 "|[-+0-9.e]+|0[xXbB]?[0-9a-fA-F]+"
-- 
1.7.1.227.g12cb1

← back to recent threads