From 726586bcb09090c1dbfec804e161ac54a78b5310 Mon Sep 17 00:00:00 2001 From: Josh Soref <2119212+jsoref@users.noreply.github.com> Date: Thu, 13 Nov 2025 12:52:19 -0500 Subject: [PATCH] Return void And omit `;` as python does not really believe in them Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com> --- regression-tests.recursor-dnssec/recursortests.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/regression-tests.recursor-dnssec/recursortests.py b/regression-tests.recursor-dnssec/recursortests.py index e195fc1485..e05067cc26 100644 --- a/regression-tests.recursor-dnssec/recursortests.py +++ b/regression-tests.recursor-dnssec/recursortests.py @@ -783,7 +783,7 @@ distributor-threads={threads} def killProcess(cls, p): # Don't try to kill it if it's already dead if p.poll() is not None: - return p + return try: p.terminate() for count in range(100): # tsan can be slow @@ -806,7 +806,7 @@ distributor-threads={threads} @classmethod def tearDownAuth(cls): for _, auth in cls._auths.items(): - cls.killProcess(auth); + cls.killProcess(auth) @classmethod def tearDownRecursor(cls, subdir=None): @@ -827,7 +827,7 @@ distributor-threads={threads} p = cls._recursor for count in range(100): # tsan can be slow if p.poll() is not None: - break; + break time.sleep(0.1) if p.poll() is None: raise AssertionError('Process did not exit on request within 10s') -- 2.47.3