chore: simplify again

ldez · ldez · commit 9f1f2d196d6a · 2024-03-22T17:47:45.000+01:00
diff --git a/contextcheck.go b/contextcheck.go
@@ -584,13 +584,11 @@ func (r *runner) checkFuncWithCtx(f *ssa.Function, tp entryType) {
 
 			key := ff.RelString(nil)
 			res, ok := r.getValue(key, ff)
-			if ok {
-				if !res.Valid {
-					if instr.Pos().IsValid() {
-						r.Reportf(instr, "Function `%s` should pass the context parameter", strings.Join(reverse(res.Funcs), "->"))
-					} else {
-						r.Reportf(ff, "Function `%s` should pass the context parameter", strings.Join(reverse(res.Funcs), "->"))
-					}
+			if ok && !res.Valid {
+				if instr.Pos().IsValid() {
+					r.Reportf(instr, "Function `%s` should pass the context parameter", strings.Join(reverse(res.Funcs), "->"))
+				} else {
+					r.Reportf(ff, "Function `%s` should pass the context parameter", strings.Join(reverse(res.Funcs), "->"))
 				}
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -584,13 +584,11 @@ func (r runner) checkFuncWithCtx(f ssa.Function, tp entryType) {`
`584`	`584`
`585`	`585`	`key := ff.RelString(nil)`
`586`	`586`	`res, ok := r.getValue(key, ff)`
`587`		`- if ok {`
`588`		`- if !res.Valid {`
`589`		`- if instr.Pos().IsValid() {`
`590`		- r.Reportf(instr, "Function `%s` should pass the context parameter", strings.Join(reverse(res.Funcs), "->"))
`591`		`- } else {`
`592`		- r.Reportf(ff, "Function `%s` should pass the context parameter", strings.Join(reverse(res.Funcs), "->"))
`593`		`- }`
	`587`	`+ if ok && !res.Valid {`
	`588`	`+ if instr.Pos().IsValid() {`
	`589`	+ r.Reportf(instr, "Function `%s` should pass the context parameter", strings.Join(reverse(res.Funcs), "->"))
	`590`	`+ } else {`
	`591`	+ r.Reportf(ff, "Function `%s` should pass the context parameter", strings.Join(reverse(res.Funcs), "->"))
`594`	`592`	`}`
`595`	`593`	`}`
`596`	`594`	`}`