false negative, possibly due to an else statement

[Josh Bleecher Snyder]

This should be safe to transform:

			if err := dump(w, n, rec); err != nil {
				return err
			} else {
				fmt.Fprintln(w)
			}

to:

			try(dump(w, n, rec)
			fmt.Fprintln(w)

(The key restriction is that err does not appear in the else block.

[Robert Griesemer]

Indeed. And if the assignment is using = instead of :=, it should be ok to transform even if err appears in the else block. Both these assume there are no aliases to err.

[codenoid]

should we close

[Robert Griesemer]

@codenoid Yes. Thanks for reminding me.

Closing this issue as "wontfix" since we have abandoned the try proposal.