Skip to content

Commit 3038502

Browse files
committed
build: No tidy for test files. Closes #2271
1 parent 3f3339e commit 3038502

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

mk/tests.mk

+1
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,7 @@ ALL_HS := $(filter-out $(S)src/rt/vg/valgrind.h \
120120
tidy:
121121
@$(call E, check: formatting)
122122
$(Q)find $(S)src -name '*.r[sc]' \
123+
| grep '^$(S)src/test' -v \
123124
| xargs -n 10 python $(S)src/etc/tidy.py
124125
$(Q)find $(S)src/etc -name '*.py' \
125126
| xargs -n 10 python $(S)src/etc/tidy.py

src/etc/tidy.py

+1-3
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,7 @@ def report_err(s):
3232
report_err("trailing whitespace")
3333
line_len = len(line)-2 if autocrlf else len(line)-1
3434

35-
# Along long lines if they are part of an expected error message
36-
# in a test, which is denoted with "//!":
37-
if line_len > cols and "//!" not in line:
35+
if line_len > cols:
3836
report_err("line longer than %d chars" % cols)
3937
except UnicodeDecodeError, e:
4038
report_err("UTF-8 decoding error " + str(e))

0 commit comments

Comments
 (0)