diff options
author | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2022-09-23 13:54:09 +0200 |
---|---|---|
committer | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2022-09-23 16:19:21 +0200 |
commit | 5c41653364d7be01293819b08b0d3d7799d910f5 (patch) | |
tree | ffd8db1da07df327a0f0d4f8d0356891c5b7d405 /hugofs/glob_test.go | |
parent | 281554ee97eb243af097611aaf6bfec8940ad6d1 (diff) |
Consolidate the glob case logic
Looking at the code as a whole, we ended up with a little to much "buttons". It turns out that doing case insensitive matching (lower both pattern and strings to match) performs just fine. Or at least, it
gives the penalty to the people who uses mixed case filenames.
```
GetGlob/Default_cache-10 10.6ns ± 2% 10.6ns ± 1% ~ (p=0.657 n=4+4)
GetGlob/Filenames_cache,_lowercase_searchs-10 10.6ns ± 2% 10.6ns ± 0% ~ (p=1.000 n=4+4)
GetGlob/Filenames_cache,_mixed_case_searchs-10 29.7ns ± 1% 29.6ns ± 1% ~ (p=0.886 n=4+4)
GetGlob/GetGlob-10 13.7ns ± 1% 13.7ns ± 0% ~ (p=0.429 n=4+4)
name old alloc/op new alloc/op delta
GetGlob/Default_cache-10 0.00B 0.00B ~ (all equal)
GetGlob/Filenames_cache,_lowercase_searchs-10 0.00B 0.00B ~ (all equal)
GetGlob/Filenames_cache,_mixed_case_searchs-10 5.00B ± 0% 5.00B ± 0% ~ (all equal)
GetGlob/GetGlob-10 0.00B 0.00B ~ (all equal)
name old allocs/op new allocs/op delta
GetGlob/Default_cache-10 0.00 0.00 ~ (all equal)
GetGlob/Filenames_cache,_lowercase_searchs-10 0.00 0.00 ~ (all equal)
GetGlob/Filenames_cache,_mixed_case_searchs-10 1.00 ± 0% 1.00 ± 0% ~ (all equal)
GetGlob/GetGlob-10
```
Diffstat (limited to 'hugofs/glob_test.go')
-rw-r--r-- | hugofs/glob_test.go | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/hugofs/glob_test.go b/hugofs/glob_test.go index fbb276445..a6ae85fc8 100644 --- a/hugofs/glob_test.go +++ b/hugofs/glob_test.go @@ -52,14 +52,16 @@ func TestGlob(t *testing.T) { create("UPPER/sub/style.css") create("root/UPPER/sub/style.css") + c.Assert(collect(filepath.FromSlash("/jsonfiles/*.json")), qt.HasLen, 2) + c.Assert(collect("**.json"), qt.HasLen, 5) c.Assert(collect("**"), qt.HasLen, 8) c.Assert(collect(""), qt.HasLen, 0) c.Assert(collect("jsonfiles/*.json"), qt.HasLen, 2) c.Assert(collect("*.json"), qt.HasLen, 1) c.Assert(collect("**.xml"), qt.HasLen, 1) - c.Assert(collect(filepath.FromSlash("/jsonfiles/*.json")), qt.HasLen, 2) - c.Assert(collect("UPPER/sub/style.css"), qt.HasLen, 1) + c.Assert(collect("root/UPPER/sub/style.css"), qt.HasLen, 1) + c.Assert(collect("UPPER/sub/style.css"), qt.HasLen, 1) } |