Skip to content

Commit ad4f63c

Browse files
committed
tpl: Remove some unreached code branches
1 parent 5320231 commit ad4f63c

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
@@ -738,16 +738,8 @@ func (t *TemplateStore) UnusedTemplates() []*TemplInfo {
738738
// Skip inline partials and internal templates.
739739
continue
740740
}
741-
if vv.noBaseOf {
742-
if vv.executionCounter.Load() == 0 {
743-
unused = append(unused, vv)
744-
}
745-
} else {
746-
for vvv := range vv.BaseVariantsSeq() {
747-
if vvv.Template.executionCounter.Load() == 0 {
748-
unused = append(unused, vvv.Template)
749-
}
750-
}
741+
if vv.executionCounter.Load() == 0 {
742+
unused = append(unused, vv)
751743
}
752744
}
753745

@@ -1543,6 +1535,9 @@ func (s *TemplateStore) resolveOutputFormatAndOrMediaType(ofs, mns string) (outp
15431535
return outputFormat, mediaType
15441536
}
15451537

1538+
// templates iterates over all templates in the store.
1539+
// Note that for templates with one or more base templates applied,
1540+
// we will yield the variants, e.g. the templates that's actually in use.
15461541
func (s *TemplateStore) templates() iter.Seq[*TemplInfo] {
15471542
return func(yield func(*TemplInfo) bool) {
15481543
for _, v := range s.treeMain.All() {
@@ -1697,31 +1692,14 @@ func (s *TemplateStore) transformTemplates() error {
16971692
if vv.category == CategoryBaseof {
16981693
continue
16991694
}
1700-
if !vv.noBaseOf {
1701-
// TODO(bep) I don't think this branch is ever called.
1702-
for vvv := range vv.BaseVariantsSeq() {
1703-
tctx, err := applyTemplateTransformers(vvv.Template, lookup)
1704-
if err != nil {
1705-
return err
1706-
}
1707-
1708-
for name, node := range tctx.deferNodes {
1709-
if err := s.addDeferredTemplate(vvv.Overlay, name, node); err != nil {
1710-
return err
1711-
}
1712-
}
1713-
}
1714-
} else {
1715-
tctx, err := applyTemplateTransformers(vv, lookup)
1716-
if err != nil {
1695+
tctx, err := applyTemplateTransformers(vv, lookup)
1696+
if err != nil {
1697+
return err
1698+
}
1699+
for name, node := range tctx.deferNodes {
1700+
if err := s.addDeferredTemplate(vv, name, node); err != nil {
17171701
return err
17181702
}
1719-
1720-
for name, node := range tctx.deferNodes {
1721-
if err := s.addDeferredTemplate(vv, name, node); err != nil {
1722-
return err
1723-
}
1724-
}
17251703
}
17261704
}
17271705

0 commit comments

Comments
 (0)