This is an archive of the discontinued LLVM Phabricator instance.

Differential D94792

Fix blank line errors
ClosedPublic

Authored by thopre on Jan 15 2021, 9:11 AM.

Details

Reviewers
tnfchris
cmatthews
Summary

Fix the following Flake8 errors:

  • E301 (expected 1 blank line, found 0)
  • E303 (too many blank lines)

Diff Detail

Event Timeline

thopre requested review of this revision.Jan 15 2021, 9:11 AM
thopre created this revision.
Harbormaster completed remote builds in B85361: Diff 316983.Jan 15 2021, 9:11 AM
tnfchris accepted this revision.Jan 15 2021, 9:16 AM
This revision is now accepted and ready to land.Jan 15 2021, 9:16 AM
thopre closed this revision.Jan 15 2021, 9:41 AM

Revision Contents

PathSize
tests/
SharedInputs/
FakeCompilers/
2 lines
lnttool/
4 lines

Diff 316983

tests/SharedInputs/FakeCompilers/fakecompiler.py

Show First 20 Lines • Show All 101 Lines • ▼ Show 20 Lines""", file=sys.stderr)
print("""\ print("""\
"%s" "-cc1" "-E" ... more boring stuff here ...""" % ( "%s" "-cc1" "-E" ... more boring stuff here ...""" % (
g_program,), file=sys.stderr) g_program,), file=sys.stderr)
# Monorepo clang build # Monorepo clang build
class Clang_monorepo(LLVMCompiler): class Clang_monorepo(LLVMCompiler):
compiler_name = "clang-monorepo" compiler_name = "clang-monorepo"
def print_verbose_info(self): def print_verbose_info(self):
print("""\ print("""\
clang version 1.2.3 (ssh://something.com/llvm-project.git 597522d740374f093a089a2acbec5b20466b2f34) clang version 1.2.3 (ssh://something.com/llvm-project.git 597522d740374f093a089a2acbec5b20466b2f34)
Target: arm-apple-darwin11.4.0 Target: arm-apple-darwin11.4.0
Thread model: posix Thread model: posix
InstalledDir: /home/foo/bin InstalledDir: /home/foo/bin
""", file=sys.stderr) """, file=sys.stderr)
print("""\ print("""\
"%s" "-cc1" "-E" ... more boring stuff here ...""" % ( "%s" "-cc1" "-E" ... more boring stuff here ...""" % (
g_program,), file=sys.stderr) g_program,), file=sys.stderr)
# Monorepo clang build with some extra stuff after the version string # Monorepo clang build with some extra stuff after the version string
class Clang_monorepo2(LLVMCompiler): class Clang_monorepo2(LLVMCompiler):
compiler_name = "clang-monorepo2" compiler_name = "clang-monorepo2"
def print_verbose_info(self): def print_verbose_info(self):
print("""\ print("""\
clang version 1.2.3 (ssh://something.com/llvm-project.git 597522d740374f093a089a2acbec5b20466b2f34) (extra) (stuff) (here) clang version 1.2.3 (ssh://something.com/llvm-project.git 597522d740374f093a089a2acbec5b20466b2f34) (extra) (stuff) (here)
Thread model: posix Thread model: posix
InstalledDir: /home/foo/bin InstalledDir: /home/foo/bin
""", file=sys.stderr) """, file=sys.stderr)
print("""\ print("""\
"%s" "-cc1" "-E" ... more boring stuff here ...""" % ( "%s" "-cc1" "-E" ... more boring stuff here ...""" % (
▲ Show 20 Lines • Show All 89 Lines • Show Last 20 Lines

tests/lnttool/email_tools.py

# Testing for the LNT email commands module. # Testing for the LNT email commands module.
# #
# create temporary instance # create temporary instance
# Cleanup temporary directory in case one remained from a previous run - also # Cleanup temporary directory in case one remained from a previous run - also
# see PR9904. # see PR9904.
# RUN: rm -rf %t.instance # RUN: rm -rf %t.instance
# RUN: python %{shared_inputs}/create_temp_instance.py \ # RUN: python %{shared_inputs}/create_temp_instance.py \
# RUN: %s %{shared_inputs}/SmallInstance %t.instance # RUN: %s %{shared_inputs}/SmallInstance %t.instance
# RUN: lnt send-run-comparison --dry-run --to some@address.com \ # RUN: lnt send-run-comparison --dry-run --to some@address.com \
# RUN: --from some.other@address.com \ # RUN: --from some.other@address.com \
# RUN: --host localhost %t.instance 1 2 | FileCheck %s --check-prefix CHECK0 # RUN: --host localhost %t.instance 1 2 | FileCheck %s --check-prefix CHECK0
# #
# CHECK0: From: some.other@address.com # CHECK0: From: some.other@address.com
# CHECK0: To: some@address.com # CHECK0: To: some@address.com
# CHECK0: Subject: localhost__clang_DEV__x86_64 test results # CHECK0: Subject: localhost__clang_DEV__x86_64 test results
# CHECK0: === text/plain report # CHECK0: === text/plain report
Show All 12 Lines
# CHECK0: === html report # CHECK0: === html report
# CHECK0: <html> # CHECK0: <html>
# CHECK0: <head> # CHECK0: <head>
# CHECK0: <title>localhost__clang_DEV__x86_64 test results</title> # CHECK0: <title>localhost__clang_DEV__x86_64 test results</title>
# CHECK0: </head> # CHECK0: </head>
# ... # ...
# CHECK0: </html> # CHECK0: </html>
# RUN: lnt send-daily-report --dry-run --from some.other@address.com \ # RUN: lnt send-daily-report --dry-run --from some.other@address.com \
# RUN: --host localhost --testsuite nts --filter-machine-regex=machine.? \ # RUN: --host localhost --testsuite nts --filter-machine-regex=machine.? \
# RUN: %t.instance some@address.com | FileCheck %s --check-prefix CHECK1 # RUN: %t.instance some@address.com | FileCheck %s --check-prefix CHECK1
# #
# CHECK1: From: some.other@address.com # CHECK1: From: some.other@address.com
# CHECK1: To: some@address.com # CHECK1: To: some@address.com
# CHECK1: Subject: Daily Report: 2012-04-11 # CHECK1: Subject: Daily Report: 2012-04-11
# CHECK1: === html report # CHECK1: === html report
# CHECK1: <html> # CHECK1: <html>
# CHECK1: <head> # CHECK1: <head>
# CHECK1: <title>Daily Report: 2012-04-11</title> # CHECK1: <title>Daily Report: 2012-04-11</title>
# CHECK1: </head> # CHECK1: </head>
# Make sure we see css inlined into the tags. # Make sure we see css inlined into the tags.
# CHECK1: <body style="color:#000000; background-color:#ffffff; font-family: Helvetica, sans-serif; font-size:9pt"> # CHECK1: <body style="color:#000000; background-color:#ffffff; font-family: Helvetica, sans-serif; font-size:9pt">
# CHECK1: <p>An error was encountered while producing the daily report: no runs to display in selected date range.</p> # CHECK1: <p>An error was encountered while producing the daily report: no runs to display in selected date range.</p>
# CHECK1: </html> # CHECK1: </html>