diff --git a/hugolib/pageCache.go b/hugolib/pageCache.go index 2ac584920..15598a4f0 100644 --- a/hugolib/pageCache.go +++ b/hugolib/pageCache.go @@ -27,7 +27,7 @@ func (entry pageCacheEntry) matches(pageLists []Pages) bool { return false } for i, p := range pageLists { - if !fastEqualPages(p, entry.in[i]) { + if !pagesEqual(p, entry.in[i]) { return false } } @@ -103,10 +103,8 @@ func (c *pageCache) getP(key string, apply func(p *Pages), pageLists ...Pages) ( } -// "fast" as in: we do not compare every element for big slices, but that is -// good enough for our use cases. -// TODO(bep) there is a similar method in pagination.go. DRY. -func fastEqualPages(p1, p2 Pages) bool { +// pagesEqual returns whether p1 and p2 are equal. +func pagesEqual(p1, p2 Pages) bool { if p1 == nil && p2 == nil { return true } @@ -123,13 +121,7 @@ func fastEqualPages(p1, p2 Pages) bool { return true } - step := 1 - - if len(p1) >= 50 { - step = len(p1) / 10 - } - - for i := 0; i < len(p1); i += step { + for i := 0; i < len(p1); i++ { if p1[i] != p2[i] { return false } diff --git a/hugolib/pageCache_test.go b/hugolib/pageCache_test.go index 52a7f4494..48f595f86 100644 --- a/hugolib/pageCache_test.go +++ b/hugolib/pageCache_test.go @@ -57,8 +57,8 @@ func TestPageCache(t *testing.T) { l1.Unlock() p2, c2 := c1.get("k1", nil, p) assert.True(t, c2) - assert.True(t, fastEqualPages(p, p2)) - assert.True(t, fastEqualPages(p, pages)) + assert.True(t, pagesEqual(p, p2)) + assert.True(t, pagesEqual(p, pages)) assert.NotNil(t, p) l2.Lock() diff --git a/hugolib/pageSort_test.go b/hugolib/pageSort_test.go index 02fa8dcc9..f842ecd78 100644 --- a/hugolib/pageSort_test.go +++ b/hugolib/pageSort_test.go @@ -114,7 +114,7 @@ func TestPageSortReverse(t *testing.T) { assert.Equal(t, 9, p2[0].fuzzyWordCount) assert.Equal(t, 0, p2[9].fuzzyWordCount) // cached - assert.True(t, fastEqualPages(p2, p1.Reverse())) + assert.True(t, pagesEqual(p2, p1.Reverse())) } func TestPageSortByParam(t *testing.T) { diff --git a/hugolib/pages_related.go b/hugolib/pages_related.go index 665019bb4..2881a45e6 100644 --- a/hugolib/pages_related.go +++ b/hugolib/pages_related.go @@ -154,7 +154,7 @@ func newSearchIndexHandler(cfg related.Config) *relatedDocsHandler { // This assumes that a lock has been acquired. func (s *relatedDocsHandler) getIndex(p Pages) *related.InvertedIndex { for _, ci := range s.postingLists { - if fastEqualPages(p, ci.p) { + if pagesEqual(p, ci.p) { return ci.postingList } }