From a3701e09313695d4a0f6fb0eb7844c1a4befc07a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bj=C3=B8rn=20Erik=20Pedersen?= Date: Tue, 27 Jul 2021 19:07:10 +0200 Subject: Switch to go-toml v2 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit We have been using `go-toml` for language files only. This commit makes it the only TOML library. It's spec compliant and very fast. A benchark building a site with 200 pages with TOML front matter: ```bash name old time/op new time/op delta SiteNew/Regular_TOML_front_matter-16 48.5ms ± 1% 47.1ms ± 1% -2.85% (p=0.029 n=4+4) name old alloc/op new alloc/op delta SiteNew/Regular_TOML_front_matter-16 16.9MB ± 0% 16.7MB ± 0% -1.56% (p=0.029 n=4+4) name old allocs/op new allocs/op delta SiteNew/Regular_TOML_front_matter-16 302k ± 0% 296k ± 0% -2.20% (p=0.029 n=4+4) ``` Note that the front matter unmarshaling is only a small part of building a site, so the above is very good. Fixes #8801 --- common/herrors/file_error_test.go | 1 - common/herrors/line_number_extractors.go | 15 +++++++++++++-- 2 files changed, 13 insertions(+), 3 deletions(-) (limited to 'common/herrors') diff --git a/common/herrors/file_error_test.go b/common/herrors/file_error_test.go index 4b42f8f39..675be94e8 100644 --- a/common/herrors/file_error_test.go +++ b/common/herrors/file_error_test.go @@ -36,7 +36,6 @@ func TestToLineNumberError(t *testing.T) { {errors.New(`template: _default/single.html:4:15: executing "_default/single.html" at <.Titles>: can't evaluate field Titles in type *hugolib.PageOutput`), 0, 4, 15}, {errors.New("parse failed: template: _default/bundle-resource-meta.html:11: unexpected in operand"), 0, 11, 1}, {errors.New(`failed:: template: _default/bundle-resource-meta.html:2:7: executing "main" at <.Titles>`), 0, 2, 7}, - {errors.New("error in front matter: Near line 32 (last key parsed 'title')"), 0, 32, 1}, {errors.New(`failed to load translations: (6, 7): was expecting token =, but got "g" instead`), 0, 6, 7}, } { diff --git a/common/herrors/line_number_extractors.go b/common/herrors/line_number_extractors.go index 13e94614d..3df62bdfe 100644 --- a/common/herrors/line_number_extractors.go +++ b/common/herrors/line_number_extractors.go @@ -16,6 +16,10 @@ package herrors import ( "regexp" "strconv" + + "github.com/pkg/errors" + + "github.com/pelletier/go-toml/v2" ) var lineNumberExtractors = []lineNumberExtractor{ @@ -24,8 +28,7 @@ var lineNumberExtractors = []lineNumberExtractor{ newLineNumberErrHandlerFromRegexp(".*:(\\d+):"), // TOML parse errors - newLineNumberErrHandlerFromRegexp(".*Near line (\\d+)(\\s.*)"), - + tomlLineNumberExtractor, // YAML parse errors newLineNumberErrHandlerFromRegexp("line (\\d+):"), @@ -35,6 +38,14 @@ var lineNumberExtractors = []lineNumberExtractor{ type lineNumberExtractor func(e error) (int, int) +var tomlLineNumberExtractor = func(e error) (int, int) { + e = errors.Cause(e) + if terr, ok := e.(*toml.DecodeError); ok { + return terr.Position() + } + return -1, -1 +} + func newLineNumberErrHandlerFromRegexp(expression string) lineNumberExtractor { re := regexp.MustCompile(expression) return extractLineNo(re) -- cgit v1.2.3