defer func() { _ = resp.Body.Close() }()

[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?

[Gert]

Never mind the tool is right I could print the error or something like that, I assumed you couldn't do anything useful with the error anyway in a defer

[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 make sense). More generally, Close() failing on something read-only is not something I'd worry much about. If it happens systematically there's something seriously wrong with the system and you'll get errors from Open() or similar when you run out of file descriptors...

//jb

> --
> You received this message because you are subscribed to the Google Groups "golang-nuts" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to golang-nuts...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

[Eric Johnson]

For the truly paranoid, perhaps write a helper function "EnsureClosed()" or something like that, which will do the close, but perhaps also appropriately log a failure to close.

[dja...@gmail.com]

Useful additional info:
https://mijailovic.net/2017/05/09/error-handling-patterns-in-go/
https://groups.google.com/forum/#!topic/golang-nuts/-eo7navkp10

[Robert Fritzsche]

Hi Gert,

it the -ignore flag an option for you? E.g. errcheck -ignore '[cC]lose' $(GOPACKAGES)

Cheers!

Robert

[Amnon]

Apologies for replying to an old post.
But I am sometimes a bit slow in reading my messages.

I too think that errcheck's is a bit severe in its treatment of deferred calls to Close().

So I forked the code and changed the behaviour to be more permissive.
https://github.com/amnonbc/errcheck

Feel free to use it.

- amnon