Did I find an error in the blog?

[Stephan (Stephan Hradek)]

I’m taking the go-tour and read https://blog.golang.org/defer-panic-and-recover

There I see the example:

src, err := os.Open(srcName)
if err != nil {
    return
}
defer src.Close()

Shouldn’t the defer come before the if?

src, err := os.Open(srcName)
defer src.Close()
if err != nil {
    return
}

[lutzhorn]

No, because src will be nil in this case. Defering Close on it only makes sense if there is no error.

[Stephan (Stephan Hradek)]

Thanks for clarification.