Fix possibly-used-before-assignment violations

Author: webknjaz

cheroot/server.py

@@ -161,6 +161,8 @@ def get(self, *args, **kwargs):
         IS_UID_GID_RESOLVABLE = False
         grp, pwd = None, None
     import struct
+else:
+    grp, pwd, struct = None, None, None
 
 
 if IS_WINDOWS and hasattr(socket, 'AF_INET6'):
@@ -249,6 +251,7 @@ def __call__(self, rfile, hdict=None):  # noqa: C901  # FIXME
         if hdict is None:
             hdict = {}
 
+        hname = None
         while True:
             line = rfile.readline()
             if not line:

cheroot/test/test_ssl.py

@@ -730,16 +730,18 @@ def test_http_over_https_error(
             104,
             'Connection reset by peer',
         )
-    if IS_MACOS:
+    elif IS_MACOS:
         expected_error_code, expected_error_text = (
             54,
             'Connection reset by peer',
         )
-    if IS_WINDOWS:
+    elif IS_WINDOWS:
         expected_error_code, expected_error_text = (
             10054,
             'An existing connection was forcibly closed by the remote host',
         )
+    else:
+        assert False  # pragma: no cover
 
     underlying_error = ssl_err.value.args[0].args[-1]
     err_text = str(underlying_error)

cheroot/workers/threadpool.py

@@ -367,6 +367,8 @@ def stop(self, timeout=5):
 
         if timeout is not None:
             endtime = time.time() + timeout
+        else:
+            endtime = float('inf')
 
         # Must shut down threads here so the code that calls
         # this method can know when all threads are stopped.