Skip to content

Commit 51e5cda

Browse files
committed
tpl: Remove some unreached code branches
1 parent 65c94c7 commit 51e5cda

File tree

1 file changed

+11
-33
lines changed

1 file changed

+11
-33
lines changed

tpl/tplimpl/templatestore.go

+11-33
Original file line numberDiff line numberDiff line change
@@ -720,16 +720,8 @@ func (t *TemplateStore) UnusedTemplates() []*TemplInfo {
720720
// Skip inline partials and internal templates.
721721
continue
722722
}
723-
if vv.noBaseOf {
724-
if vv.executionCounter.Load() == 0 {
725-
unused = append(unused, vv)
726-
}
727-
} else {
728-
for vvv := range vv.BaseVariantsSeq() {
729-
if vvv.Template.executionCounter.Load() == 0 {
730-
unused = append(unused, vvv.Template)
731-
}
732-
}
723+
if vv.executionCounter.Load() == 0 {
724+
unused = append(unused, vv)
733725
}
734726
}
735727

@@ -1559,6 +1551,9 @@ func (s *TemplateStore) resolveOutputFormatAndOrMediaType(ofs, mns string) (outp
15591551
return outputFormat, mediaType
15601552
}
15611553

1554+
// templates iterates over all templates in the store.
1555+
// Note that for templates with one or more base templates applied,
1556+
// we will yield the variants, e.g. the templates that's actually in use.
15621557
func (s *TemplateStore) templates() iter.Seq[*TemplInfo] {
15631558
return func(yield func(*TemplInfo) bool) {
15641559
for _, v := range s.treeMain.All() {
@@ -1724,31 +1719,14 @@ func (s *TemplateStore) transformTemplates() error {
17241719
if vv.category == CategoryBaseof {
17251720
continue
17261721
}
1727-
if !vv.noBaseOf {
1728-
// TODO(bep) I don't think this branch is ever called.
1729-
for vvv := range vv.BaseVariantsSeq() {
1730-
tctx, err := applyTemplateTransformers(vvv.Template, lookup)
1731-
if err != nil {
1732-
return err
1733-
}
1734-
1735-
for name, node := range tctx.deferNodes {
1736-
if err := s.addDeferredTemplate(vvv.Overlay, name, node); err != nil {
1737-
return err
1738-
}
1739-
}
1740-
}
1741-
} else {
1742-
tctx, err := applyTemplateTransformers(vv, lookup)
1743-
if err != nil {
1722+
tctx, err := applyTemplateTransformers(vv, lookup)
1723+
if err != nil {
1724+
return err
1725+
}
1726+
for name, node := range tctx.deferNodes {
1727+
if err := s.addDeferredTemplate(vv, name, node); err != nil {
17441728
return err
17451729
}
1746-
1747-
for name, node := range tctx.deferNodes {
1748-
if err := s.addDeferredTemplate(vv, name, node); err != nil {
1749-
return err
1750-
}
1751-
}
17521730
}
17531731
}
17541732

0 commit comments

Comments
 (0)