checkers/analyzer: remove concurrency from go/analysis analyzer

checkers/analyzer/analyzer.go

@@ -2,8 +2,6 @@
 package analyzer
 
 import (
-	"runtime"
-
 	"github.com/go-critic/go-critic/linter"
 
 	"golang.org/x/tools/go/analysis"
@@ -23,12 +21,11 @@ var Analyzer = &analysis.Analyzer{
 var DisableCache = false
 
 var (
-	flagGoVersion   string
-	flagEnable      string
-	flagDisable     string
-	flagEnableAll   bool
-	flagDebugInit   bool
-	flagConcurrency int
+	flagGoVersion string
+	flagEnable    string
+	flagDisable   string
+	flagEnableAll bool
+	flagDebugInit bool
 )
 
 var (
@@ -50,8 +47,6 @@ func init() {
 		`comma-separated list of checkers to be disabled. Can include #tags`)
 	Analyzer.Flags.StringVar(&flagGoVersion, "go", "",
 		`select the Go version to target. Leave as string for the latest`)
-	Analyzer.Flags.IntVar(&flagConcurrency, "concurrency", runtime.GOMAXPROCS(0),
-		`how many checks to run concurrently (defaults to runtime.GOMAXPROCS(0))`)
 
 	for _, info := range registeredCheckers {
 		for pname, param := range info.Params {

checkers/analyzer/run.go

@@ -39,44 +39,19 @@ func runAnalyzer(pass *analysis.Pass) (interface{}, error) {
 		return nil, err
 	}
 
-	doneCh := make(chan struct{})
-	diagCh := make(chan analysis.Diagnostic)
-	go func() {
-		for diag := range diagCh {
-			pass.Report(diag)
-		}
-		close(doneCh)
-	}()
-
-	sema := make(chan struct{}, flagConcurrency)
-	var wg sync.WaitGroup
-	wg.Add(len(pass.Files))
-
 	for _, f := range pass.Files {
 		f := f
 		filename := filepath.Base(pass.Fset.Position(f.Pos()).Filename)
 		ctx.SetFileInfo(filename, f)
 
-		sema <- struct{}{}
-		go func() {
-			defer func() {
-				wg.Done()
-				<-sema
-			}()
-
-			for _, c := range checkers {
-				warnings := c.Check(f)
-				for _, warning := range warnings {
-					diagCh <- asDiag(c, warning)
-				}
+		for _, c := range checkers {
+			warnings := c.Check(f)
+			for _, warning := range warnings {
+				pass.Report(asDiag(c, warning))
 			}
-		}()
+		}
 	}
 
-	wg.Wait()
-	close(diagCh)
-	<-doneCh
-
 	return nil, nil
 }