]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
timeit: remove --clock and --time options
authorVictor Stinner <victor.stinner@gmail.com>
Tue, 18 Oct 2016 15:18:21 +0000 (17:18 +0200)
committerVictor Stinner <victor.stinner@gmail.com>
Tue, 18 Oct 2016 15:18:21 +0000 (17:18 +0200)
Issue #28240: timeit: remove -c/--clock and -t/--time command line options
which were deprecated since Python 3.3.

Doc/library/timeit.rst
Lib/test/test_timeit.py
Lib/timeit.py
Misc/NEWS

index 5bae33bc8f4b7663a2e396f909791f13f5aeca62..bad7194f0910c6fba77d7e6369e2636680af8d28 100644 (file)
@@ -197,7 +197,7 @@ Command-Line Interface
 
 When called as a program from the command line, the following form is used::
 
-   python -m timeit [-n N] [-r N] [-u U] [-s S] [-t] [-c] [-h] [statement ...]
+   python -m timeit [-n N] [-r N] [-u U] [-s S] [-h] [statement ...]
 
 Where the following options are understood:
 
@@ -222,20 +222,12 @@ Where the following options are understood:
 
    .. versionadded:: 3.3
 
-.. cmdoption:: -t, --time
-
-   use :func:`time.time` (deprecated)
-
 .. cmdoption:: -u, --unit=U
 
     specify a time unit for timer output; can select usec, msec, or sec
 
    .. versionadded:: 3.5
 
-.. cmdoption:: -c, --clock
-
-   use :func:`time.clock` (deprecated)
-
 .. cmdoption:: -v, --verbose
 
    print raw timing results; repeat for more digits precision
index 7a2be53a08470e74d8bc46e729f200e9d6957415..cf4eec9719bc648681825f805212927d6f81b2fa 100644 (file)
@@ -293,18 +293,6 @@ class TestTimeit(unittest.TestCase):
         # the help text, but since it's there, check for it.
         self.assertEqual(s, timeit.__doc__ + ' ')
 
-    def test_main_using_time(self):
-        fake_timer = FakeTimer()
-        s = self.run_main(switches=['-t'], timer=fake_timer)
-        self.assertEqual(s, self.MAIN_DEFAULT_OUTPUT)
-        self.assertIs(fake_timer.saved_timer, time.time)
-
-    def test_main_using_clock(self):
-        fake_timer = FakeTimer()
-        s = self.run_main(switches=['-c'], timer=fake_timer)
-        self.assertEqual(s, self.MAIN_DEFAULT_OUTPUT)
-        self.assertIs(fake_timer.saved_timer, time.clock)
-
     def test_main_verbose(self):
         s = self.run_main(switches=['-v'])
         self.assertEqual(s, dedent("""\
index d40998c77d1a8fde12145b3af986a034a47bb86a..3395b36071cc127b95a4b3887375d329c60fdd45 100644 (file)
@@ -9,7 +9,7 @@ the Python Cookbook, published by O'Reilly.
 Library usage: see the Timer class.
 
 Command line usage:
-    python timeit.py [-n N] [-r N] [-s S] [-t] [-c] [-p] [-h] [--] [statement]
+    python timeit.py [-n N] [-r N] [-s S] [-p] [-h] [--] [statement]
 
 Options:
   -n/--number N: how many times to execute 'statement' (default: see below)
@@ -17,8 +17,6 @@ Options:
   -s/--setup S: statement to be executed once initially (default 'pass').
                 Execution time of this setup statement is NOT timed.
   -p/--process: use time.process_time() (default is time.perf_counter())
-  -t/--time: use time.time() (deprecated)
-  -c/--clock: use time.clock() (deprecated)
   -v/--verbose: print raw timing results; repeat for more digits precision
   -u/--unit: set the output time unit (usec, msec, or sec)
   -h/--help: print this usage message and exit
@@ -291,10 +289,6 @@ def main(args=None, *, _wrap_timer=None):
             repeat = int(a)
             if repeat <= 0:
                 repeat = 1
-        if o in ("-t", "--time"):
-            timer = time.time
-        if o in ("-c", "--clock"):
-            timer = time.clock
         if o in ("-p", "--process"):
             timer = time.process_time
         if o in ("-v", "--verbose"):
index 37c2c2b49aa4205cd33e67adb569de3813aa854c..a46c317d22698cc153f781042460c8268f87931e 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -88,6 +88,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #28240: timeit: remove ``-c/--clock`` and ``-t/--time`` command line
+  options which were deprecated since Python 3.3.
+
 - Issue #28240: timeit now repeats the benchmarks 5 times instead of only 3
   to make benchmarks more reliable.