search

mumble-voip / grumble

Alternative Mumble server
http://mumble.info/grumble
Other
273 stars 88 forks source link

Don't try to close the TCP listener, since it was already closed when the TLS listener was closed #43

Closed olabiniV2 closed 4 years ago

olabiniV2 commented 4 years ago

Superseded by #45.

davidebeatrici commented 4 years ago

There are unrelated changes, could you split them in different commits, please?

I can also do that, if you prefer.

olabiniV2 commented 4 years ago

Sorry about that - I'll submit a new pull request. My environment automatically reformats after gofmt guidelines, and since the grumble code base in some cases diverges, that's why the extra changes happened.

davidebeatrici commented 4 years ago

No problem.

You could also keep this pull request and force-push.