Issue #11727: Antoine and Arfrever don't like "0 < timeout" style (regrtest.py)

diff --git a/Lib/test/regrtest.py b/Lib/test/regrtest.py
index f14b89d62237b08df2c2d84a6fe1561c2dd2e5f6..8e9fe656c2abf1c7c13c0313f07ec5ae3a511197 100755 (executable)
--- a/Lib/test/regrtest.py
+++ b/Lib/test/regrtest.py
@@ -818,7 +818,8 @@ def runtest(test, verbose, quiet,
     support.verbose = verbose  # Tell tests to be moderately quiet
     if use_resources is not None:
         support.use_resources = use_resources
-    if timeout is not None and timeout > 0:
+    use_timeout = (timeout is not None and timeout > 0)
+    if use_timeout:
         faulthandler.dump_tracebacks_later(timeout, exit=True)
     try:
         result = runtest_inner(test, verbose, quiet, huntrleaks, debug)
@@ -830,7 +831,7 @@ def runtest(test, verbose, quiet,
             runtest(test, True, quiet, huntrleaks, debug, timeout=timeout)
         return result
     finally:
-        if timeout and 0 < timeout:
+        if use_timeout:
             faulthandler.cancel_dump_tracebacks_later()
         cleanup_test_droppings(test, verbose)