Check all arguments, fixes #25

Author: tommy-muehle

checks/argument.go

@@ -86,10 +86,9 @@ func (a *ArgumentAnalyzer) checkCallExpr(expr *ast.CallExpr) {
 			if i == 0 {
 				a.pass.Reportf(x.Pos(), reportMsg, x.Value, ArgumentCheck)
 			} else {
-				// Otherwise check the previous element type
-				switch expr.Args[i-1].(type) {
-				case *ast.ChanType:
-					// When it's not a simple buffered channel, report it
+				// Otherwise check all args
+				switch expr.Args[i].(type) {
+				case *ast.BasicLit:
 					if a.isMagicNumber(x) {
 						a.pass.Reportf(x.Pos(), reportMsg, x.Value, ArgumentCheck)
 					}

testdata/src/argument/argument.go

@@ -80,3 +80,15 @@ func example10() {
 		wg.Done()
 	}()
 }
+
+func foobar(a, b int) {}
+
+func example11() {
+	foobar(0, 3) // want "Magic number: 3"
+}
+
+func foobaz(numbers ...int) {}
+
+func example12() {
+	foobaz(0, 1, 3, 5, 6) // want "Magic number: 3" "Magic number: 5" "Magic number: 6"
+}