gdb/testsuite: avoid assigning lambdas in Python files

Fix flake8 warnings like this one:

    gdb/testsuite/gdb.perf/template-breakpoints.py:35:13: E731 do not assign a lambda expression, use a def

I chose to inline the lambdas in the expressions, since they are simple
enough and only used once each.

Change-Id: I46fac428a95da38f5a6a87e101be4da9fa4acc31
Approved-By: Tom Tromey <tom@tromey.com>

diff --git a/gdb/testsuite/gdb.perf/backtrace.py b/gdb/testsuite/gdb.perf/backtrace.py
index e3d3e91418c1eee9fc1240221a82e63566b64537..a764e4c1b05f10aaf7aa1d768959ecbae7b2d2f7 100644 (file)
--- a/gdb/testsuite/gdb.perf/backtrace.py
+++ b/gdb/testsuite/gdb.perf/backtrace.py
@@ -45,8 +45,6 @@ class BackTrace(perftest.TestCaseWithBasicMeasurements):
             gdb.execute(line_size_command)
             gdb.execute(size_command)
 
-            func = lambda: self._do_test()
-
-            self.measure.measure(func, line_size)
+            self.measure.measure(self._do_test, line_size)
 
             line_size *= 2

diff --git a/gdb/testsuite/gdb.perf/gmonster-null-lookup.py b/gdb/testsuite/gdb.perf/gmonster-null-lookup.py
index 825632329d48c6090caf95e0583edf3f0a805410..e12377180f697bee6aacf9cf695f99b5d47f1cc4 100644 (file)
--- a/gdb/testsuite/gdb.perf/gmonster-null-lookup.py
+++ b/gdb/testsuite/gdb.perf/gmonster-null-lookup.py
@@ -39,6 +39,7 @@ class NullLookup(perftest.TestCaseWithBasicMeasurements):
             iteration = 5
             while iteration > 0:
                 utils.safe_execute("mt flush symbol-cache")
-                func = lambda: utils.safe_execute("p symbol_not_found")
-                self.measure.measure(func, run)
+                self.measure.measure(
+                    lambda: utils.safe_execute("p symbol_not_found"), run
+                )
                 iteration -= 1

diff --git a/gdb/testsuite/gdb.perf/gmonster-print-cerr.py b/gdb/testsuite/gdb.perf/gmonster-print-cerr.py
index 0f201b3e27e30ebbe0e8cf89db6de18e963868e7..de80c039c65cd551d8fbfa3ec0c8bf3827155102 100644 (file)
--- a/gdb/testsuite/gdb.perf/gmonster-print-cerr.py
+++ b/gdb/testsuite/gdb.perf/gmonster-print-cerr.py
@@ -45,6 +45,7 @@ class PrintCerr(perftest.TestCaseWithBasicMeasurements):
             iteration = 5
             while iteration > 0:
                 utils.safe_execute("mt flush symbol-cache")
-                func = lambda: utils.safe_execute("print gm_std::cerr")
-                self.measure.measure(func, run)
+                self.measure.measure(
+                    lambda: utils.safe_execute("print gm_std::cerr"), run
+                )
                 iteration -= 1

diff --git a/gdb/testsuite/gdb.perf/gmonster-ptype-string.py b/gdb/testsuite/gdb.perf/gmonster-ptype-string.py
index 3397b123a1e09dada630e53aee4a5fb5f5360304..0b1938cfd99a4ed3c8d09fb2de1b6836ca93d83e 100644 (file)
--- a/gdb/testsuite/gdb.perf/gmonster-ptype-string.py
+++ b/gdb/testsuite/gdb.perf/gmonster-ptype-string.py
@@ -40,7 +40,10 @@ class GmonsterPtypeString(perftest.TestCaseWithBasicMeasurements):
             iteration = 5
             while iteration > 0:
                 utils.safe_execute("mt flush symbol-cache")
-                func1 = lambda: utils.safe_execute("ptype hello")
-                func = lambda: utils.run_n_times(2, func1)
-                self.measure.measure(func, run)
+                self.measure.measure(
+                    lambda: utils.run_n_times(
+                        2, lambda: utils.safe_execute("ptype hello")
+                    ),
+                    run,
+                )
                 iteration -= 1

diff --git a/gdb/testsuite/gdb.perf/gmonster-runto-main.py b/gdb/testsuite/gdb.perf/gmonster-runto-main.py
index 097c32a1bdd23967b9eab3de47b00776fe4e45bc..f5782ae4a13cf69c93d066d5c9b87f38f61b8e54 100644 (file)
--- a/gdb/testsuite/gdb.perf/gmonster-runto-main.py
+++ b/gdb/testsuite/gdb.perf/gmonster-runto-main.py
@@ -33,6 +33,5 @@ class GmonsterRuntoMain(perftest.TestCaseWithBasicMeasurements):
             utils.select_file(this_run_binfile)
             iteration = 5
             while iteration > 0:
-                func = lambda: utils.runto_main()
-                self.measure.measure(func, run)
+                self.measure.measure(lambda: utils.runto_main(), run)
                 iteration -= 1

diff --git a/gdb/testsuite/gdb.perf/gmonster-select-file.py b/gdb/testsuite/gdb.perf/gmonster-select-file.py
index 05db3624dac0d1255dedefb6bf7067124db42ec3..6ac77858cb49c9cad76cc76cf9bee467a65ccf66 100644 (file)
--- a/gdb/testsuite/gdb.perf/gmonster-select-file.py
+++ b/gdb/testsuite/gdb.perf/gmonster-select-file.py
@@ -36,6 +36,5 @@ class GmonsterSelectFile(perftest.TestCaseWithBasicMeasurements):
             this_run_binfile = "%s-%s" % (self.binfile, utils.convert_spaces(run))
             iteration = 5
             while iteration > 0:
-                func = lambda: self._doit(this_run_binfile)
-                self.measure.measure(func, run)
+                self.measure.measure(lambda: self._doit(this_run_binfile), run)
                 iteration -= 1

diff --git a/gdb/testsuite/gdb.perf/single-step.py b/gdb/testsuite/gdb.perf/single-step.py
index eecf2f982a0dcc8744223bf7002e3675685dace7..27d3e053d512873c39a38ccf652204185b471ed5 100644 (file)
--- a/gdb/testsuite/gdb.perf/single-step.py
+++ b/gdb/testsuite/gdb.perf/single-step.py
@@ -33,5 +33,4 @@ class SingleStep(perftest.TestCaseWithBasicMeasurements):
 
     def execute_test(self):
         for i in range(1, 5):
-            func = lambda: self._run(i * self.step)
-            self.measure.measure(func, i * self.step)
+            self.measure.measure(lambda: self._run(i * self.step), i * self.step)

diff --git a/gdb/testsuite/gdb.perf/skip-command.py b/gdb/testsuite/gdb.perf/skip-command.py
index b344c9c36a35f6e0dc24be0047337822d00e6ed3..5614aebc75d87c9d8e6b118d9587db85d62a26cb 100644 (file)
--- a/gdb/testsuite/gdb.perf/skip-command.py
+++ b/gdb/testsuite/gdb.perf/skip-command.py
@@ -33,5 +33,4 @@ class SkipCommand(perftest.TestCaseWithBasicMeasurements):
 
     def execute_test(self):
         for i in range(1, 5):
-            func = lambda: self._run(i * self.step)
-            self.measure.measure(func, i * self.step)
+            self.measure.measure(lambda: self._run(i * self.step), i * self.step)

diff --git a/gdb/testsuite/gdb.perf/solib.py b/gdb/testsuite/gdb.perf/solib.py
index c440635147b3d6892a71f5fa38f54f5cb7c46c47..1f780146591d5491e649d987bb28fe863db5f571 100644 (file)
--- a/gdb/testsuite/gdb.perf/solib.py
+++ b/gdb/testsuite/gdb.perf/solib.py
@@ -47,9 +47,7 @@ class SolibLoadUnload1(perftest.TestCaseWithBasicMeasurements):
             # but measure differently.
             if self.measure_load:
                 do_test_load = "call do_test_load (%d)" % num
-                func = lambda: gdb.execute(do_test_load)
-
-                self.measure.measure(func, num)
+                self.measure.measure(lambda: gdb.execute(do_test_load), num)
 
                 do_test_unload = "call do_test_unload (%d)" % num
                 gdb.execute(do_test_unload)
@@ -59,9 +57,7 @@ class SolibLoadUnload1(perftest.TestCaseWithBasicMeasurements):
                 gdb.execute(do_test_load)
 
                 do_test_unload = "call do_test_unload (%d)" % num
-                func = lambda: gdb.execute(do_test_unload)
-
-                self.measure.measure(func, num)
+                self.measure.measure(lambda: gdb.execute(do_test_unload), num)
 
             num = num / 2
             iteration -= 1

diff --git a/gdb/testsuite/gdb.perf/template-breakpoints.py b/gdb/testsuite/gdb.perf/template-breakpoints.py
index 82dc5b511a1b3c2e86752f89fccddc2642b03597..ac8b0668d2636edc5062f3922ec768027082f944 100644 (file)
--- a/gdb/testsuite/gdb.perf/template-breakpoints.py
+++ b/gdb/testsuite/gdb.perf/template-breakpoints.py
@@ -32,5 +32,6 @@ class TemplateBreakpoints(perftest.TestCaseWithBasicMeasurements):
 
     def execute_test(self):
         for bpcount in range(1, 10):
-            tfunc = lambda bound_bpcount=bpcount: self._do_test(bound_bpcount)
-            self.measure.measure(tfunc, bpcount)
+            self.measure.measure(
+                lambda bound_bpcount=bpcount: self._do_test(bound_bpcount), bpcount
+            )