From 637ff4b71b2a118c8458881fa2cf74ed7d089fb6 Mon Sep 17 00:00:00 2001 From: Timur Demin Date: Tue, 10 Nov 2020 09:51:03 +0500 Subject: [PATCH] Rid of an unneeded check RenderMarkdown doesn't really need an extra check on whether or not the Markdown content size is more than a specific amount. --- render.go | 32 +++++++++++++++++--------------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/render.go b/render.go index 9737851..8b320cf 100644 --- a/render.go +++ b/render.go @@ -52,22 +52,24 @@ var yamlDelimiter = []byte("---\n") // Only a subset of front matter data parsed by Hugo is included in the // final document. At this point it's just title and date. func RenderMarkdown(md []byte) (geminiText []byte, err error) { - var metadata hugoMetadata - if len(md) > len(yamlDelimiter)*2 { - // only allow front matter at file start - if bytes.Index(md, yamlDelimiter) != 0 { - goto parse - } - blockEnd := bytes.Index(md[len(yamlDelimiter):], yamlDelimiter) - if blockEnd == -1 { - goto parse - } - yamlContent := md[len(yamlDelimiter) : blockEnd+len(yamlDelimiter)] - if err := yaml.Unmarshal(yamlContent, &metadata); err != nil { - return nil, fmt.Errorf("invalid front matter: %w", err) - } - md = md[blockEnd+len(yamlDelimiter)*2:] + var ( + metadata hugoMetadata + blockEnd int + yamlContent []byte + ) + // only allow front matter at file start + if bytes.Index(md, yamlDelimiter) != 0 { + goto parse } + blockEnd = bytes.Index(md[len(yamlDelimiter):], yamlDelimiter) + if blockEnd == -1 { + goto parse + } + yamlContent = md[len(yamlDelimiter) : blockEnd+len(yamlDelimiter)] + if err := yaml.Unmarshal(yamlContent, &metadata); err != nil { + return nil, fmt.Errorf("invalid front matter: %w", err) + } + md = md[blockEnd+len(yamlDelimiter)*2:] parse: ast := markdown.Parse(md, parser.NewWithExtensions(parser.CommonExtensions)) var geminiContent []byte