Do not use time.clock for Py3 in runtest

time.clock is deprecated since Python 3.3 and will be removed in
Python 3.8.  3.7 started issuing DeprecationWarning, which
fails about 16 tests in AppVeyor CI (and running manually),
since the warning appears in stderr stream and so real vs expected
does not match any longer.  Arguably the tests could be fixed to
do a different check (contains rather than exactly-equal), but
a change needs to be made anyway.

use time.perf_counter as the time function for Python 3.  This
works for Windows and non-Windows; however since this function
did not exist in 2.7 (added for 3.3), the Py2 case is left, which
selects between time.clock and time.time depending on OS.

Note this addresses runtest.py, but not code in bench/, which
need attention also.  Suggest that be a separate change since
it's not failing tests.

Signed-off-by: Mats Wichmann <mats@linux.com>

1 files changed, 8 insertions, 4 deletions

diff --git a/runtest.py b/runtest.py
index 293e4ca..1cac45b 100755
--- a/runtest.py
+++ b/runtest.py
@@ -95,11 +95,12 @@ import time
 import threading
 try:                        # python3
     from queue import Queue
+    PY3=True
 except ImportError as e:    # python2
     from Queue import Queue
+    PY3=False
 import subprocess
 
-
 cwd = os.getcwd()
 
 baseline = 0
@@ -770,10 +771,13 @@ os.environ["python_executable"] = python
 # but time.time() does a better job on Linux systems, so let that be
 # the non-Windows default.
 
-if sys.platform == 'win32':
-    time_func = time.clock
+if PY3:
+    time_func = time.perf_counter
 else:
-    time_func = time.time
+    if sys.platform == 'win32':
+        time_func = time.clock
+    else:
+        time_func = time.time
 
 if print_times:
     print_time_func = lambda fmt, time: sys.stdout.write(fmt % time)