Simplify code
authorChristian Muehlhaeuser <muesli@gmail.com>
Fri, 2 Aug 2019 14:37:28 +0000 (14:37 +0000)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Sat, 10 Aug 2019 18:10:32 +0000 (20:10 +0200)
- Use bytes.Equal instead of bytes.Compare
- Omit range's value where it's unused

helpers/content_test.go
tpl/tplimpl/template.go

index 04e9466d0f531d4ed37ca1e4f4a79f7cd1da286d..011d606e6aded70b5f5714e1af26df6b2eb5d709 100644 (file)
@@ -45,7 +45,7 @@ func TestTrimShortHTML(t *testing.T) {
        c := newTestContentSpec()
        for i, test := range tests {
                output := c.TrimShortHTML(test.input)
-               if bytes.Compare(test.output, output) != 0 {
+               if !bytes.Equal(test.output, output) {
                        t.Errorf("Test %d failed. Expected %q got %q", i, test.output, output)
                }
        }
index cce867ac2bab9c5097d8936ef3a844d854240c46..35407722a0cf06ad49f40df79d34dc55779391cf 100644 (file)
@@ -512,7 +512,7 @@ func (t *htmlTemplates) addTemplateIn(tt *template.Template, name, tpl string) (
                return nil, err
        }
 
-       for k, _ := range c.notFound {
+       for k := range c.notFound {
                t.transformNotFound[k] = true
        }
 
@@ -580,7 +580,7 @@ func (t *textTemplates) addTemplateIn(tt *texttemplate.Template, name, tpl strin
                return nil, err
        }
 
-       for k, _ := range c.notFound {
+       for k := range c.notFound {
                t.transformNotFound[k] = true
        }
 
@@ -637,7 +637,7 @@ func (t *templateHandler) postTransform() error {
                        return templT.Tree
                }, t.text.transformNotFound},
        } {
-               for name, _ := range s.transformNotFound {
+               for name := range s.transformNotFound {
                        templ := s.lookup(name)
                        if templ != nil {
                                _, err := applyTemplateTransformers(templateUndefined, templ, s.lookup)