From: Josh Soref <2119212+jsoref@users.noreply.github.com> Date: Thu, 13 Nov 2025 17:52:19 +0000 (-0500) Subject: Return void X-Git-Tag: rec-5.4.0-alpha1~80^2~1 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=726586bcb09090c1dbfec804e161ac54a78b5310;p=thirdparty%2Fpdns.git Return void And omit `;` as python does not really believe in them Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com> --- 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')