Fix review comments

.gitignore

@@ -1,4 +1,3 @@
-.idea
 golinter
 revive
 vendor

rule/file-header.go

@@ -1,7 +1,6 @@
 package rule
 
 import (
-	"go/ast"
 	"regexp"
 
 	"github.com/mgechev/revive/lint"
@@ -10,68 +9,33 @@ import (
 // FileHeaderRule lints given else constructs.
 type FileHeaderRule struct{}
 
-// Apply applies the rule to given file.
-func (r *FileHeaderRule) Apply(file *lint.File, arguments lint.Arguments) []lint.Failure {
-	var failures []lint.Failure
-
-	header, ok := arguments[0].(string)
-	if !ok {
-		panic("Invalid argument to the FileHeaderRule")
-	}
-
-	regex, err := regexp.Compile(header)
-	if err != nil {
-		panic(err.Error())
-	}
-
-	fileAst := file.AST
-	walker := lintFileHeader{
-		file:    file,
-		fileAst: fileAst,
-		regex:   regex,
-		onFailure: func(failure lint.Failure) {
-			failures = append(failures, failure)
-		},
-	}
-
-	ast.Walk(walker, fileAst)
-
-	return failures
-}
-
-// Name returns the rule name.
-func (r *FileHeaderRule) Name() string {
-	return "file-header"
-}
-
-type lintFileHeader struct {
-	file      *lint.File
-	fileAst   *ast.File
-	regex     *regexp.Regexp
-	onFailure func(lint.Failure)
-}
-
 var (
 	multiRegexp  = regexp.MustCompile("^/\\*")
 	singleRegexp = regexp.MustCompile("^//")
 )
 
-func (w lintFileHeader) Visit(_ ast.Node) ast.Visitor {
+// Apply applies the rule to given file.
-	failure := lint.Failure{
+func (r *FileHeaderRule) Apply(file *lint.File, arguments lint.Arguments) []lint.Failure {
-		Node:       w.fileAst,
+	header, ok := arguments[0].(string)
-		Confidence: 1,
+	if !ok {
-		Failure:    "the file doesn't have an appropriate header",
+		panic("Invalid argument to the FileHeaderRule")
 	}
 
-	if len(w.fileAst.Comments) == 0 {
+	failure := []lint.Failure{
-		w.onFailure(failure)
+		{
-		return nil
+			Node:       file.AST,
+			Confidence: 1,
+			Failure:    "the file doesn't have an appropriate header",
+		},
 	}
 
-	g := w.fileAst.Comments[0]
+	if len(file.AST.Comments) == 0 {
+		return failure
+	}
+
+	g := file.AST.Comments[0]
 	if g == nil {
-		w.onFailure(failure)
+		return failure
-		return nil
 	}
 	comment := ""
 	for _, c := range g.List {
@@ -84,8 +48,18 @@ func (w lintFileHeader) Visit(_ ast.Node) ast.Visitor {
 		comment += text
 	}
 
-	if !w.regex.Match([]byte(comment)) {
+	regex, err := regexp.Compile(header)
-		w.onFailure(failure)
+	if err != nil {
+		panic(err.Error())
+	}
+
+	if !regex.Match([]byte(comment)) {
+		return failure
 	}
 	return nil
 }
+
+// Name returns the rule name.
+func (r *FileHeaderRule) Name() string {
+	return "file-header"
+}

test/file-header_test.go

@@ -37,10 +37,4 @@ func BenchmarkLintFileHeader(b *testing.B) {
 	benchRule(b, "lint-file-header1", &rule.FileHeaderRule{}, &lint.RuleConfig{
 		Arguments: []interface{}{"foobar"},
 	})
-	//var t *testing.T
-	//for i := 0; i <= b.N; i++ {
-	//	testRule(t, "lint-file-header1", &rule.FileHeaderRule{}, &lint.RuleConfig{
-	//		Arguments: []interface{}{"foobar"},
-	//	})
-	//}
 }

test/import-blacklist_test.go

@@ -17,7 +17,10 @@ func TestImportsBlacklist(t *testing.T) {
 
 func BenchmarkImportsBlacklist(b *testing.B) {
 	args := []interface{}{"crypto/md5", "crypto/sha1"}
-	benchRule(b, "imports-blacklist", &rule.ImportsBlacklistRule{}, &lint.RuleConfig{
+	var t *testing.T
-		Arguments: args,
+	for i := 0; i <= b.N; i++ {
-	})
+		testRule(t, "imports-blacklist", &rule.ImportsBlacklistRule{}, &lint.RuleConfig{
+			Arguments: args,
+		})
+	}
 }

test/unused-param_test.go

@@ -11,5 +11,8 @@ func TestUnusedParam(t *testing.T) {
 }
 
 func BenchmarkUnusedParam(b *testing.B) {
-	benchRule(b, "unused-param", &rule.UnusedParamRule{})
+	var t *testing.T
+	for i := 0; i <= b.N; i++ {
+		testRule(t, "unused-param", &rule.UnusedParamRule{})
+	}
 }