Unexpected results with escaping

[jacobcase]

Hello,

UPDATE: Looks like all of the multi-term patterns were not working as expected due to spacing. They all work as expected when that's fixed. All that leaves is the first pattern I put where I would expect it not to match.

I am building a single string out of a list of glob patterns I want to match to pass into glob.Compile().
An example might be glob.Compile("{"+strings.Join(globPatterns, ",")+"}", '.')

It seems though it may be impossible to have a , in one of the patterns. I wrote a simple test table to demonstrate what my assumption would be and what the actual results were in a comment above each test case. The last case I listed also surprised me, because the Compile documentation suggests that \ should escape { and }.

type T struct {
	Pattern string
	Input   string
}

func main() {
	tests := []T{
		{
			// Should not match since ',' is a separator (it does)
			Pattern: "test,pattern",
			Input:   "test,pattern",
		},
		{
			// Should match (and it does)
			Pattern: "test\\,pattern",
			Input:   "test,pattern",
		},
		{
			// Should not match (it doesn't)
			Pattern: "{ test,pattern }",
			Input:   "test,pattern",
		},
		{
			// Should match (it doesn't)
			Pattern: "{ test\\,pattern }",
			Input:   "test,pattern",
		},
		{
			// Should match (it doesn't)
			Pattern: "{ \\{test\\} }",
			Input:   "{test}",
		},
	}

	for _, t := range tests {
		g, err := glob.Compile(t.Pattern)
		if err != nil {
			panic(err)
		}

		fmt.Printf("%+v, %v\n", t, g.Match(t.Input))
	}
}

Which prints the following:

./globtest
{Pattern:test,pattern Input:test,pattern}, true
{Pattern:test\,pattern Input:test,pattern}, true
{Pattern:{ test,pattern } Input:test,pattern}, false
{Pattern:{ test\,pattern } Input:test,pattern}, false
{Pattern:{ \{test\} } Input:{test}}, false

Are my assumptions in my comments correct? If so, how would you suggest they be addressed? I would be willing to take a crack at it if you would like.

[jacobcase]

Additionally, should the Compile documentation be updated to include , in the list of special characters that need escaped? QuoteMeta also appears to not escape ,.