From a6488f04deba974e556290a84763fe4918ce9660 Mon Sep 17 00:00:00 2001
From: naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Fri, 27 Oct 2017 08:40:40 +0000
Subject: Revert "Dir.glob with FNM_EXTGLOB is optimized [Feature #13873]"

This reverts commit r60341,r60342,r60344,r60345.
Breaking compabitility of the order of result breaks many tests.
To avoid such effort to fix tests, the order should be kept.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60478 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 dir.c | 92 ++++++++-----------------------------------------------------------
 1 file changed, 10 insertions(+), 82 deletions(-)

(limited to 'dir.c')

diff --git a/dir.c b/dir.c
index da38ee8145..2c0ff03c53 100644
--- a/dir.c
+++ b/dir.c
@@ -287,8 +287,6 @@ bracket(
 #define UNESCAPE(p) (escape && *(p) == '\\' ? (p) + 1 : (p))
 #define ISEND(p) (!*(p) || (pathname && *(p) == '/'))
 #define RETURN(val) return *pcur = p, *scur = s, (val);
-#define FNMATCH_ALLOC_N(type, n) ((type *)malloc(sizeof(type) * (n)))
-#define FNMATCH_FREE(ptr) free(ptr)
 
 static int
 fnmatch_helper(
@@ -312,11 +310,8 @@ fnmatch_helper(
 
     int r;
 
-    if (period && *s == '.') { /* leading period */
-	int c = *UNESCAPE(p);
-	if (c != '.' && (!(flags & FNM_EXTGLOB) || c != '{')) RETURN(FNM_NOMATCH);
-    }
-
+    if (period && *s == '.' && *UNESCAPE(p) != '.') /* leading period */
+	RETURN(FNM_NOMATCH);
 
     while (1) {
 	switch (*p) {
@@ -350,57 +345,6 @@ fnmatch_helper(
 	    }
 	    goto failed;
 	  }
-
-	  case '{':
-	    if (flags & FNM_EXTGLOB) {
-		size_t len = pend - p;
-		char *buf = FNMATCH_ALLOC_N(char, len);
-		const char *rbrace = NULL;
-		while (p < pend) {
-		    const char *t = ++p;
-		    int nest = 0;
-		    while (p < pend && !(*p == ',' && nest == 0)) {
-			if (*p == '{') nest++;
-			if (*p == '}') {
-			    if (nest == 0) {
-				if (!rbrace) rbrace = p;
-				goto rest;
-			    }
-			    nest--;
-			}
-			if (*p == '\\' && escape) {
-			    if (++p >= pend) break;
-			}
-			Inc(p, pend, enc);
-		    }
-		    if (!rbrace) {
-			rbrace = p;
-			while (rbrace < pend && !(*rbrace == '}' && nest == 0)) {
-			    if (*rbrace == '{') nest++;
-			    if (*rbrace == '}') nest--;
-			    if (*rbrace == '\\' && escape) {
-				if (++p >= pend) break;
-			    }
-			    Inc(rbrace, pend, enc);
-			}
-		    }
-		  rest:
-		    memcpy(buf, t, p-t);
-		    buf[p-t]=0;
-		    strlcpy(buf+(p-t), rbrace+1, len-(p-t));
-		    {
-			const char *pp = buf, *ss = s;
-			r = fnmatch_helper((const char **)&pp, &ss, flags|FNM_DOTMATCH, enc);
-		    }
-		    if (r == 0) {
-			p = buf;
-			FNMATCH_FREE(buf);
-			RETURN(0);
-		    }
-		    if (p >= rbrace) break;
-		}
-		FNMATCH_FREE(buf);
-	    }
 	}
 
 	/* ordinary */
@@ -1481,12 +1425,6 @@ has_magic(const char *p, const char *pend, int flags, rb_encoding *enc)
 	  case '[':
 	    return MAGICAL;
 
-	  case '{':
-	    if (flags & FNM_EXTGLOB) {
-		return MAGICAL;
-	    }
-	    break;
-
 	  case '\\':
 	    if (escape && p++ >= pend)
 		continue;
@@ -2333,13 +2271,6 @@ push_pattern(const char *path, VALUE ary, void *enc)
     rb_ary_push(ary, name);
 }
 
-struct push_glob_args {
-    struct glob_args glob;
-    int flags;
-    int fd;
-};
-static int push_caller(const char *path, VALUE val, void *enc);
-
 static int
 ruby_brace_expand(const char *str, int flags, ruby_glob_func *func, VALUE arg,
 		  rb_encoding *enc, VALUE var)
@@ -2348,7 +2279,7 @@ ruby_brace_expand(const char *str, int flags, ruby_glob_func *func, VALUE arg,
     const char *p = str;
     const char *pend = p + strlen(p);
     const char *s = p;
-    const char *lbrace = NULL, *rbrace = NULL;
+    const char *lbrace = 0, *rbrace = 0;
     int nest = 0, status = 0;
 
     while (*p) {
@@ -2367,18 +2298,9 @@ ruby_brace_expand(const char *str, int flags, ruby_glob_func *func, VALUE arg,
 
     if (lbrace && rbrace) {
 	size_t len = strlen(s) + 1;
-	char *buf;
+	char *buf = GLOB_ALLOC_N(char, len);
 	long shift;
 
-	if (func == push_caller && !strchr(lbrace, '/')) {
-	    /* Now it reaches file basename entry. */
-	    /* Handle braces in glob_helper */
-	    struct push_glob_args *a = (struct push_glob_args *)arg;
-	    a->flags |= FNM_EXTGLOB;
-	    return glob_call_func(func, s, arg, enc);
-	}
-
-	buf = GLOB_ALLOC_N(char, len);
 	if (!buf) return -1;
 	memcpy(buf, s, lbrace-s);
 	shift = (lbrace-s);
@@ -2442,6 +2364,12 @@ ruby_brace_glob(const char *str, int flags, ruby_glob_func *func, VALUE arg)
     return ruby_brace_glob_with_enc(str, flags, func, arg, rb_ascii8bit_encoding());
 }
 
+struct push_glob_args {
+    struct glob_args glob;
+    int flags;
+    int fd;
+};
+
 static int
 push_caller(const char *path, VALUE val, void *enc)
 {
-- 
cgit v1.2.3