You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Your code doesn't work the way you think it does. Arguments to deferred function calls are evaluated at the defer statement. You are always passing false to undoThingsUnless. Try passing &ok instead.
When you went to open this issue, did you see an issue template called "False positive"? Did you ignore it, and why? (E.g., are you not familiar with the term "false positive"?) I ask because I created that issue template for the sake of reducing spurious reports like this one — and yet here we are. Can you think of a better name for the issue template? I'd appreciate your feedback :)
on some functions, to make error handling cleaner, I put a sentinel on defer to take care of undoing incomplete "transactions". it looks like this:
actual code example: https://github.com/darvaza-proxy/darvaza/blob/master/shared/net/bind/listen.go#L114 or https://github.com/darvaza-proxy/darvaza/blob/master/shared/net/bind/bind.go#L165
I don't know the version, but I just noticed it on
goreportcard
. https://goreportcard.com/report/github.com/darvaza-proxy/darvaza/sharedThe text was updated successfully, but these errors were encountered: