Re: [go-nuts] defer func() { _ = resp.Body.Close() }()

2017-08-16 Thread Jakob Borg
As far as I'm concerned, `defer resp.Body.Close()` is perfectly cromulent and there's no need to for error checking contortions to satisfy the lint tool. If we're talking about a http.Request I doubt that the close can ever fail (I haven't checked; but it doesn't seem like something that would m

[go-nuts] defer func() { _ = resp.Body.Close() }()

2017-08-16 Thread Gert
To pass errcheck I need to do something like defer func() { _ = resp.Body.Close() }() instead of defer resp.Body.Close() Is this something the errcheck tool can figure out to mark as valid instead or does the errcheck tool need help from the compiler so the second case is also ok? -- You r