diff options
author | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2017-05-31 13:10:46 +0300 |
---|---|---|
committer | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2017-06-03 09:22:57 +0200 |
commit | 0907a5c1c293755e6bf297246f07888448d81f8b (patch) | |
tree | beb4af6ccb9907ec2d7ccfdd53d04b92f07e095d /hugolib | |
parent | 3d9c4f513b0443648d7e88995e351df1739646d2 (diff) |
all: Temporarily revert to BurntSushi for TOML front matter handling
We still have go-toml as a transitive dependency, and it is the way to go eventually, but we care about speed, so let us wait that one out.
Note that the issue this fixes is about taxonomies, but I guess this is a general issue for sites with many pages that uses TOML as front matter.
```
benchmark old ns/op new ns/op delta
BenchmarkFrontmatterTags/TOML:1-4 23206 8543 -63.19%
BenchmarkFrontmatterTags/TOML:11-4 80117 18495 -76.92%
BenchmarkFrontmatterTags/TOML:21-4 140676 28727 -79.58%
benchmark old allocs new allocs delta
BenchmarkFrontmatterTags/TOML:1-4 173 60 -65.32%
BenchmarkFrontmatterTags/TOML:11-4 625 138 -77.92%
BenchmarkFrontmatterTags/TOML:21-4 1106 210 -81.01%
benchmark old bytes new bytes delta
BenchmarkFrontmatterTags/TOML:1-4 9231 2912 -68.45%
BenchmarkFrontmatterTags/TOML:11-4 19808 5184 -73.83%
BenchmarkFrontmatterTags/TOML:21-4 31200 7536 -75.85%
```
See #3541
Updates #3464
Diffstat (limited to 'hugolib')
-rw-r--r-- | hugolib/menu_old_test.go | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/hugolib/menu_old_test.go b/hugolib/menu_old_test.go index 704a6f70a..036c855ad 100644 --- a/hugolib/menu_old_test.go +++ b/hugolib/menu_old_test.go @@ -25,7 +25,7 @@ import ( "path/filepath" - toml "github.com/pelletier/go-toml" + "github.com/BurntSushi/toml" "github.com/spf13/hugo/source" "github.com/stretchr/testify/assert" "github.com/stretchr/testify/require" @@ -634,12 +634,7 @@ func setupMenuTests(t *testing.T, pageSources []source.ByteSource, configKeyValu } func tomlToMap(s string) (map[string]interface{}, error) { - tree, err := toml.Load(s) - - if err != nil { - return nil, err - } - - return tree.ToMap(), nil - + var data = make(map[string]interface{}) + _, err := toml.Decode(s, &data) + return data, err } |