Remove obsolete/unused code (#155)

* remove obsolete/unused code

* remove more obsolete/unused code

Author: GLEF1X

doc/conf.py

@@ -12,10 +12,6 @@
 # All configuration values have a default; values that are commented out
 # serve to show the default.
 
-import sys
-import os
-import shlex
-
 # If extensions (or modules to document with autodoc) are in another directory,
 # add these directories to sys.path here. If the directory is relative to the
 # documentation root, use os.path.abspath to make it absolute, like shown here.

doc/examples/bench_time_func.py

@@ -6,7 +6,7 @@ def bench_dict(loops, mydict):
     range_it = range(loops)
     t0 = pyperf.perf_counter()
 
-    for loops in range_it:
+    for _ in range_it:
         mydict['0']
         mydict['100']
         mydict['200']

pyperf/__main__.py

@@ -357,7 +357,7 @@ def group_by_name_ignored(self):
                 yield (suite, ignored)
 
 
-def load_benchmarks(args, name=True):
+def load_benchmarks(args):
     data = Benchmarks()
     data.load_benchmark_suites(args.filenames)
     if getattr(args, 'benchmarks', None):
@@ -681,7 +681,6 @@ def cmd_convert(args):
                       file=sys.stderr)
                 sys.exit(1)
             except TypeError:
-                raise
                 print("ERROR: Metadata %r of benchmark %r is not an integer"
                       % (name, benchmark.get_name()),
                       file=sys.stderr)
@@ -699,7 +698,7 @@ def cmd_convert(args):
 
 
 def cmd_slowest(args):
-    data = load_benchmarks(args, name=False)
+    data = load_benchmarks(args)
     nslowest = args.n
 
     use_title = (data.get_nsuite() > 1)

pyperf/_bench.py

@@ -521,8 +521,7 @@ def _as_json(self, suite_metadata):
         metadata = self._get_common_metadata()
         common_metadata = dict(metadata, **suite_metadata)
 
-        data = {}
-        data['runs'] = [run._as_json(common_metadata) for run in self._runs]
+        data = {'runs': [run._as_json(common_metadata) for run in self._runs]}
         metadata = _exclude_common_metadata(metadata, suite_metadata)
         if metadata:
             data['metadata'] = metadata

pyperf/_cli.py

@@ -87,7 +87,6 @@ def format_run(bench, run_index, run, common_metadata=None, raw=False,
             loops, value = warmup
             raw_value = value * (loops * inner_loops)
             if raw:
-                text = format_value(raw_value)
                 text = ("%s (loops: %s)"
                         % (format_value(raw_value),
                            format_number(loops)))
@@ -273,8 +272,7 @@ def format_stats(bench, lines):
     lines.append('')
 
     # Minimum
-    table = []
-    table.append(("Minimum", bench.format_value(min(values))))
+    table = [("Minimum", bench.format_value(min(values)))]
 
     # Median +- MAD
     median = bench.median()
@@ -382,8 +380,6 @@ def value_bucket(value):
 
         value_width = max([len(bench.format_value(bucket * value_k))
                            for bucket in range(bucket_min, bucket_max + 1)])
-        width = columns - value_width
-
         line = ': %s #' % count_max
         width = columns - (value_width + len(line))
         if not extend:
@@ -517,7 +513,7 @@ def format_result_value(bench):
     return _format_result_value(bench)
 
 
-def format_result(bench, prefix=True):
+def format_result(bench):
     loops = None
     warmups = None
     for run in bench._runs:

pyperf/_collect_metadata.py

@@ -233,7 +233,6 @@ def collect_cpu_freq(metadata, cpus):
                 # Example: "processor 0: version = 00,  identification = [...]"
                 match = re.match(r'^processor ([0-9]+): ', line)
                 if match is None:
-                    raise Exception
                     # unknown /proc/cpuinfo format: silently ignore and exit
                     return
 
@@ -410,9 +409,7 @@ def collect_cpu_metadata(metadata):
 
 
 def collect_metadata(process=True):
-    metadata = {}
-    metadata['perf_version'] = pyperf.__version__
-    metadata['date'] = format_datetime(datetime.datetime.now())
+    metadata = {'perf_version': pyperf.__version__, 'date': format_datetime(datetime.datetime.now())}
 
     collect_system_metadata(metadata)
     collect_cpu_metadata(metadata)

pyperf/_compare.py

@@ -284,9 +284,7 @@ def sort_key(results):
         for item in self.all_results[0]:
             headers.append(item.changed.name)
 
-        all_norm_means = []
-        for column in headers[2:]:
-            all_norm_means.append([])
+        all_norm_means = [[] for _ in range(len(headers[2:]))]
 
         rows = []
         not_significant = []

pyperf/_process_time.py

@@ -60,7 +60,7 @@ def bench_process(loops, args, kw, profile_filename=None):
         temp_profile_filename = tempfile.mktemp()
         args = [args[0], "-m", "cProfile", "-o", temp_profile_filename] + args[1:]
 
-    for loop in range_it:
+    for _ in range_it:
         start_rss = get_max_rss()
 
         proc = subprocess.Popen(args, **kw)
@@ -75,8 +75,6 @@ def bench_process(loops, args, kw, profile_filename=None):
                 os.unlink(temp_profile_filename)
             sys.exit(exitcode)
 
-        proc = None
-
         rss = get_max_rss() - start_rss
         max_rss = max(max_rss, rss)
 

pyperf/_runner.py

@@ -71,7 +71,7 @@ class Runner:
 
     # Default parameters are chosen to have approximatively a run of 0.5 second
     # and so a total duration of 5 seconds by default
-    def __init__(self, values=None, warmups=None, processes=None,
+    def __init__(self, values=None, processes=None,
                  loops=0, min_time=0.1, metadata=None,
                  show_name=True,
                  program_args=None, add_cmdline_args=None,
@@ -485,7 +485,7 @@ def bench_time_func(self, name, time_func, *args, **kwargs):
         if self.args.profile:
             profiler, time_func = profiling_wrapper(time_func)
 
-        def task_func(task, loops):
+        def task_func(_, loops):
             return time_func(loops, *args)
 
         task = WorkerProcessTask(self, name, task_func, metadata)
@@ -514,7 +514,7 @@ def bench_func(self, name, func, *args, **kwargs):
         if self.args.profile:
             profiler, func = profiling_wrapper(func)
 
-        def task_func(task, loops):
+        def task_func(_, loops):
             # use fast local variables
             local_timer = time.perf_counter
             local_func = func
@@ -557,7 +557,7 @@ def bench_async_func(self, name, func, *args, **kwargs):
         if self.args.profile:
             profiler, func = profiling_wrapper(func)
 
-        def task_func(task, loops):
+        def task_func(_, loops):
             if loops != 1:
                 async def main():
                     # use fast local variables

pyperf/_system.py

@@ -213,7 +213,7 @@ def write_msr(self, cpu, reg_num, value):
             fd = os.open(path, os.O_WRONLY)
             try:
                 if hasattr(os, 'pwrite'):
-                    data = os.pwrite(fd, data, reg_num)
+                    os.pwrite(fd, data, reg_num)
                 else:
                     os.lseek(fd, reg_num, os.SEEK_SET)
                     os.write(fd, data)