Fix whitespaces flake8 errors and warnings

- fix E302 errors (2 blank lines before class/functions)
- fix E305 errors (2 blank lines after class/functions)
- fix W293 warnings (blank line contains whitespace)

Reviewed By: cmatthews

Differential Revision: https://reviews.llvm.org/D94722

Author: Thomas Preud'homme

lnt/server/reporting/analysis.py

@@ -18,6 +18,7 @@
 # Minimal percentage difference that is visible in reports
 MIN_PERCENTAGE_CHANGE = .01
 
+
 def absmin_diff(current, prevs):
     """Min of differences between current sample and all previous samples.
     Given more than one min, use the last one detected which is probably a

lnt/server/reporting/report.py

@@ -8,10 +8,12 @@
 
 OrderAndHistory = namedtuple('OrderAndHistory', ['max_order', 'recent_orders'])
 
+
 def pairs(l):
     """Make an iterable of all pairs of consecutive elements in l."""
     return zip(l[:-1], l[1:])
 
+
 # The hash color palette avoids green and red as these colours are already used
 # in quite a few places to indicate "good" or "bad".
 _hash_color_palette = (
@@ -23,6 +25,7 @@ def pairs(l):
     colorsys.hsv_to_rgb(h=180. / 360, s=0.3, v=0.9999),  # mid cyan
 )
 
+
 def _clamp(v, minVal, maxVal):
     return min(max(v, minVal), maxVal)
 
@@ -55,6 +58,7 @@ def _get_rgb_colors_for_hashes(hash_strings):
             result.append(_toColorString(rgb))
     return result
 
+
 # Helper classes to make the sparkline chart construction easier in the jinja
 # template.
 class RunResult:
@@ -65,6 +69,7 @@ def __init__(self, comparisonResult):
         if self.samples is None:
             self.samples = []
 
+
 class RunResults:
     """
     RunResults contains pre-processed data to easily construct the HTML for
@@ -113,6 +118,7 @@ def complete(self):
             if dr is not None:
                 dr.hash_rgb_color = rgb_colors[i]
 
+
 # Compute static CSS styles for elements. We use the style directly on
 # elements instead of via a stylesheet to support major email clients
 # (like Gmail) which can't deal with embedded style sheets.

lnt/server/reporting/runs.py

@@ -216,6 +216,8 @@ def prioritize_buckets(test_results):
 
 
 BucketEntry = namedtuple('BucketEntry', ['name', 'cr', 'test_id'])
+
+
 def _get_changes_by_type(ts, run_a, run_b, metric_fields, test_names,
                          num_comparison_runs, sri):
     comparison_results = {}

lnt/server/ui/globals.py

@@ -33,9 +33,11 @@ def v4_url_available(*args, **kwargs):
     except Exception:
         return False
 
+
 class fixed_location_response(Response):
     autocorrect_location_header = False
 
+
 def v4_redirect(*args, **kwargs):
     """
     Like redirect but can be used to allow relative URL redirection.
@@ -48,6 +50,7 @@ def v4_redirect(*args, **kwargs):
     """
     return flask.redirect(*args, Response=fixed_location_response, **kwargs)
 
+
 def register(env):
     # Add some normal Python builtins which can be useful in templates.
     env.globals.update(zip=zip)

lnt/server/ui/views.py

@@ -1517,6 +1517,7 @@ def to_key(name):
                            config=config, all_machines=all_machines,
                            all_orders=all_orders, **ts_data(ts))
 
+
 @v4_route("/latest_runs_report")
 def v4_latest_runs_report():
     session = request.session
@@ -1535,6 +1536,7 @@ def v4_latest_runs_report():
                            analysis=lnt.server.reporting.analysis,
                            **ts_data(ts))
 
+
 @db_route("/summary_report")
 def v4_summary_report():
     session = request.session

tests/SharedInputs/FakeCompilers/fakecompiler.py

@@ -118,6 +118,7 @@ def print_verbose_info(self):
  "%s" "-cc1" "-E" ... more boring stuff here ...""" % (
             g_program,), file=sys.stderr)
 
+
 # Monorepo clang build with some extra stuff after the version string
 class Clang_monorepo2(LLVMCompiler):
     compiler_name = "clang-monorepo2"
@@ -131,6 +132,7 @@ def print_verbose_info(self):
  "%s" "-cc1" "-E" ... more boring stuff here ...""" % (
             g_program,), file=sys.stderr)
 
+
 class AppleClang_138_1(LLVMCompiler):
     compiler_name = "apple-clang-138.1"
 

tests/__init__.py

@@ -1,5 +1,6 @@
 import os
 import lit.discovery
 
+
 def test_all():
     return lit.discovery.load_test_suite([os.path.dirname(__file__)])

tests/server/db/Migrations.py

@@ -46,6 +46,7 @@ def sanity_check_instance(instance_path):
         overview = client.get(os.path.join("/", link))
         assert "LNT : %s - Recent Activity" % (name,) in overview.data
 
+
 def check_instance(instance_path, temp_path):
     logging.info("checking instance: %r", instance_path)
 
@@ -66,6 +67,7 @@ def check_instance(instance_path, temp_path):
     # Sanity check that the update instance works correctly.
     sanity_check_instance(instance_temp_path)
 
+
 def main():
     _, temp_path = sys.argv
 
@@ -86,5 +88,6 @@ def main():
         # Otherwise, we have a test instance. Check migration of it.
         check_instance(input_path, temp_path)
 
+
 if __name__ == '__main__':
     main()

tests/server/db/search.py

@@ -105,7 +105,7 @@ def test_default_order(self):
             ('machine3', '6512'),
             ('machine3', '65')
         ])
-        
+
     def test_default_machine(self):
         session = self.session
         ts = self.db.testsuite.get('nts')
@@ -116,6 +116,7 @@ def test_default_machine(self):
             ('machine2', '6512')
         ])
 
+
 if __name__ == '__main__':
     if len(sys.argv) > 1:
         base_path = sys.argv[1]

tests/testing/cPerf.py

@@ -5,7 +5,8 @@
 import os
 import tempfile
 from lnt.testing.profile.perf import LinuxPerfProfile
-    
+
+
 class CPerfTest(unittest.TestCase):
     def setUp(self):
         self.inputs = os.path.join(os.path.dirname(os.path.abspath(__file__)),
@@ -118,7 +119,7 @@ def setUp(self):
                                                     [{},
                                                      4196092,
                                                      u'\td65f03c0 \tret']]}}}}
-        
+
     def _getNm(self, perf_data_fname, non_dynamic=False):
         stub = perf_data_fname.rsplit('.perf_data', 1)[0]
         s = 'python %s/fake-nm.py %s.nm.out' % (self.inputs, stub)
@@ -132,7 +133,7 @@ def _getObjdump(self, perf_data_fname):
 
     def _getInput(self, fname):
         return os.path.join(self.inputs, fname)
-    
+
     def test_check_file(self):
         self.assertTrue(LinuxPerfProfile.checkFile(self._getInput('fib-aarch64.perf_data')))
 
@@ -183,5 +184,6 @@ def test_random_guff2(self):
                 LinuxPerfProfile.deserialize(open(fd.name),
                                              propagateExceptions=True)
 
+
 if __name__ == '__main__':
     unittest.main(argv=[sys.argv[0], ])