Remove a TODO

Author: dtcaciuc

errcheck/analyzer.go

@@ -64,10 +64,6 @@ func runAnalyzer(pass *analysis.Pass) (interface{}, error) {
 
 		ast.Walk(v, f)
 
-		// TODO have visitor report all facts and then sort out
-		// the ones we need to report based on flags. This can
-		// likely simplify the visitor code.
-
 		for _, err := range v.errors {
 			pass.Report(analysis.Diagnostic{
 				Pos:     token.Pos(int(f.Pos()) + err.Pos.Offset),

errcheck/errcheck.go

@@ -512,7 +512,7 @@ func (v *visitor) isRecover(call *ast.CallExpr) bool {
 	return false
 }
 
-// TODO collect token.Pos and then convert them to UncheckedErrors
+// TODO (dtcaciuc) collect token.Pos and then convert them to UncheckedErrors
 // after visitor is done running. This will allow to integrate more cleanly
 // with analyzer so that we don't have to convert Position back to Pos.
 func (v *visitor) addErrorAtPosition(position token.Pos, call *ast.CallExpr) {