From a3dcd82d96d8f1325f4b0a0d2aee14446ed5a881 Mon Sep 17 00:00:00 2001
From: Sebastian Soto <ssoto@redhat.com>
Date: Thu, 3 Jul 2025 19:35:21 -0400
Subject: [PATCH] Name extendedGlob() anonymous functions

Moves inline function definitions out of the function.
Helps with code readability.

Signed-off-by: Sebastian Soto <ssoto@redhat.com>
---
 copier/copier.go | 116 ++++++++++++++++++++++++-----------------------
 1 file changed, 60 insertions(+), 56 deletions(-)

diff --git a/copier/copier.go b/copier/copier.go
index 38468e2ae..aa0bba6dc 100644
--- a/copier/copier.go
+++ b/copier/copier.go
@@ -51,62 +51,6 @@ func init() {
 // will be concatenated.
 // The matched paths are returned in lexical order, which makes the output deterministic.
 func extendedGlob(pattern string) (matches []string, err error) {
-	subdirs := func(dir string) []string {
-		var subdirectories []string
-		if err := filepath.WalkDir(dir, func(path string, d fs.DirEntry, err error) error {
-			if err != nil {
-				return nil
-			}
-			if d.IsDir() {
-				if rel, err := filepath.Rel(dir, path); err == nil {
-					subdirectories = append(subdirectories, rel)
-				}
-			}
-			return nil
-		}); err != nil {
-			subdirectories = []string{"."}
-		}
-		return subdirectories
-	}
-	expandPatterns := func(pattern string) []string {
-		components := []string{}
-		dir := pattern
-		file := ""
-		for dir != filepath.VolumeName(dir) && dir != string(os.PathSeparator) {
-			dir, file = filepath.Split(dir)
-			if file != "" {
-				components = append([]string{file}, components...)
-			}
-			dir = strings.TrimSuffix(dir, string(os.PathSeparator))
-		}
-		patterns := []string{filepath.VolumeName(dir) + string(os.PathSeparator)}
-		for i := range components {
-			var nextPatterns []string
-			if components[i] == "**" {
-				for _, parent := range patterns {
-					nextSubdirs := subdirs(parent)
-					for _, nextSubdir := range nextSubdirs {
-						nextPatterns = append(nextPatterns, filepath.Join(parent, nextSubdir))
-					}
-				}
-			} else {
-				for _, parent := range patterns {
-					nextPattern := filepath.Join(parent, components[i])
-					nextPatterns = append(nextPatterns, nextPattern)
-				}
-			}
-			patterns = []string{}
-			seen := map[string]struct{}{}
-			for _, nextPattern := range nextPatterns {
-				if _, seen := seen[nextPattern]; seen {
-					continue
-				}
-				patterns = append(patterns, nextPattern)
-				seen[nextPattern] = struct{}{}
-			}
-		}
-		return patterns
-	}
 	patterns := expandPatterns(pattern)
 	for _, pattern := range patterns {
 		theseMatches, err := filepath.Glob(pattern)
@@ -119,6 +63,66 @@ func extendedGlob(pattern string) (matches []string, err error) {
 	return matches, nil
 }
 
+// subDirs returns a recursive search of all directories under the given dir
+func subDirs(dir string) []string {
+	var subdirectories []string
+	if err := filepath.WalkDir(dir, func(path string, d fs.DirEntry, err error) error {
+		if err != nil {
+			return nil
+		}
+		if d.IsDir() {
+			if rel, err := filepath.Rel(dir, path); err == nil {
+				subdirectories = append(subdirectories, rel)
+			}
+		}
+		return nil
+	}); err != nil {
+		subdirectories = []string{"."}
+	}
+	return subdirectories
+}
+
+// expandPatterns fans out a glob pattern so that "**" is expanded to all possible subdirectory paths at that location.
+func expandPatterns(pattern string) []string {
+	components := []string{}
+	dir := pattern
+	file := ""
+	for dir != filepath.VolumeName(dir) && dir != string(os.PathSeparator) {
+		dir, file = filepath.Split(dir)
+		if file != "" {
+			components = append([]string{file}, components...)
+		}
+		dir = strings.TrimSuffix(dir, string(os.PathSeparator))
+	}
+	patterns := []string{filepath.VolumeName(dir) + string(os.PathSeparator)}
+	for i := range components {
+		var nextPatterns []string
+		if components[i] == "**" {
+			for _, parent := range patterns {
+				nextSubdirs := subDirs(parent)
+				for _, nextSubdir := range nextSubdirs {
+					nextPatterns = append(nextPatterns, filepath.Join(parent, nextSubdir))
+				}
+			}
+		} else {
+			for _, parent := range patterns {
+				nextPattern := filepath.Join(parent, components[i])
+				nextPatterns = append(nextPatterns, nextPattern)
+			}
+		}
+		patterns = []string{}
+		seen := map[string]struct{}{}
+		for _, nextPattern := range nextPatterns {
+			if _, seen := seen[nextPattern]; seen {
+				continue
+			}
+			patterns = append(patterns, nextPattern)
+			seen[nextPattern] = struct{}{}
+		}
+	}
+	return patterns
+}
+
 // isArchivePath returns true if the specified path can be read like a (possibly
 // compressed) tarball.
 func isArchivePath(path string) bool {