summaryrefslogtreecommitdiffstats
path: root/hugolib
diff options
context:
space:
mode:
authorBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>2024-02-23 15:01:19 +0100
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>2024-02-23 16:29:21 +0100
commitbf14d0cb26df901cccea593dfefaabfdc42d01af (patch)
treea01c0662219e04fd2abb6c01bc68f2b6b349f01e /hugolib
parentf521336c8147d00e9caa0b4ba4ab64bc43c69101 (diff)
Speed up GetPage
``` name old time/op new time/op delta GetPage-10 413ns ± 0% 287ns ± 1% -30.47% (p=0.029 n=4+4) GetPageRegular/From_root-10 757ns ± 1% 706ns ± 1% -6.75% (p=0.029 n=4+4) GetPageRegular/Page_relative-10 838ns ± 1% 786ns ± 1% -6.16% (p=0.029 n=4+4) name old alloc/op new alloc/op delta GetPage-10 312B ± 0% 24B ± 0% -92.31% (p=0.029 n=4+4) GetPageRegular/From_root-10 328B ± 0% 200B ± 0% -39.02% (p=0.029 n=4+4) GetPageRegular/Page_relative-10 360B ± 0% 232B ± 0% -35.56% (p=0.029 n=4+4) name old allocs/op new allocs/op delta GetPage-10 8.00 ± 0% 2.00 ± 0% -75.00% (p=0.029 n=4+4) GetPageRegular/From_root-10 7.00 ± 0% 5.00 ± 0% -28.57% (p=0.029 n=4+4) GetPageRegular/Page_relative-10 9.00 ± 0% 7.00 ± 0% -22.22% (p=0.029 n=4+4) ```
Diffstat (limited to 'hugolib')
-rw-r--r--hugolib/pagecollections.go33
1 files changed, 15 insertions, 18 deletions
diff --git a/hugolib/pagecollections.go b/hugolib/pagecollections.go
index fe7c2f1e7..5ed2a28f0 100644
--- a/hugolib/pagecollections.go
+++ b/hugolib/pagecollections.go
@@ -131,22 +131,21 @@ func (c *pageFinder) getContentNode(context page.Page, isReflink bool, ref strin
return c.getContentNodeForRef(context, isReflink, true, inRef, ref)
}
- var refs []string
-
// We are always looking for a content file and having an extension greatly simplifies the code that follows,
// even in the case where the extension does not match this one.
if ref == "/" {
- refs = append(refs, "/_index"+defaultContentExt)
+ if n, err := c.getContentNodeForRef(context, isReflink, false, inRef, "/_index"+defaultContentExt); n != nil || err != nil {
+ return n, err
+ }
} else if strings.HasSuffix(ref, "/index") {
- refs = append(refs, ref+"/index"+defaultContentExt)
- refs = append(refs, ref+defaultContentExt)
+ if n, err := c.getContentNodeForRef(context, isReflink, false, inRef, ref+"/index"+defaultContentExt); n != nil || err != nil {
+ return n, err
+ }
+ if n, err := c.getContentNodeForRef(context, isReflink, false, inRef, ref+defaultContentExt); n != nil || err != nil {
+ return n, err
+ }
} else {
- refs = append(refs, ref+defaultContentExt)
- }
-
- for _, ref := range refs {
- n, err := c.getContentNodeForRef(context, isReflink, false, inRef, ref)
- if n != nil || err != nil {
+ if n, err := c.getContentNodeForRef(context, isReflink, false, inRef, ref+defaultContentExt); n != nil || err != nil {
return n, err
}
}
@@ -175,7 +174,7 @@ func (c *pageFinder) getContentNodeForRef(context page.Page, isReflink, hadExten
rel := path.Join(baseDir, ref)
- relPath := contentPathParser.Parse(files.ComponentFolderContent, rel)
+ relPath, _ := contentPathParser.ParseBaseAndBaseNameNoIdentifier(files.ComponentFolderContent, rel)
n, err := c.getContentNodeFromPath(relPath, ref)
if n != nil || err != nil {
@@ -195,9 +194,9 @@ func (c *pageFinder) getContentNodeForRef(context page.Page, isReflink, hadExten
return nil, nil
}
- refPath := contentPathParser.Parse(files.ComponentFolderContent, ref)
+ relPath, nameNoIdentifier := contentPathParser.ParseBaseAndBaseNameNoIdentifier(files.ComponentFolderContent, ref)
- n, err := c.getContentNodeFromPath(refPath, ref)
+ n, err := c.getContentNodeFromPath(relPath, ref)
if n != nil || err != nil {
return n, err
@@ -221,7 +220,7 @@ func (c *pageFinder) getContentNodeForRef(context page.Page, isReflink, hadExten
return nil, nil
}
- n = c.pageMap.pageReverseIndex.Get(refPath.BaseNameNoIdentifier())
+ n = c.pageMap.pageReverseIndex.Get(nameNoIdentifier)
if n == ambiguousContentNode {
return nil, fmt.Errorf("page reference %q is ambiguous", inRef)
}
@@ -254,9 +253,7 @@ func (c *pageFinder) getContentNodeFromRefReverseLookup(ref string, fi hugofs.Fi
return nil, nil
}
-func (c *pageFinder) getContentNodeFromPath(refPath *paths.Path, ref string) (contentNodeI, error) {
- s := refPath.Base()
-
+func (c *pageFinder) getContentNodeFromPath(s string, ref string) (contentNodeI, error) {
n := c.pageMap.treePages.Get(s)
if n != nil {
return n, nil