diff options
| author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-05-11 16:27:37 -0700 |
|---|---|---|
| committer | bors-servo <lbergstrom+bors@mozilla.com> | 2016-05-11 16:27:37 -0700 |
| commit | 8c2c0bd964bfbf1a2d7494bd8fbbdd263173ef7d (patch) | |
| tree | 248464a15b87e54a886c10b7b4f49230251ff7bc /python/tidy/servo_tidy/tidy.py | |
| parent | 49fd06089de8357c0d139f6aa623cde55ccd9ef0 (diff) | |
| parent | fa4665e2a74dc324f3e03cd8342e9abb151133c2 (diff) | |
| download | servo-8c2c0bd964bfbf1a2d7494bd8fbbdd263173ef7d.tar.gz servo-8c2c0bd964bfbf1a2d7494bd8fbbdd263173ef7d.zip | |
Auto merge of #10713 - aeischeid:master, r=jdm
add tidy test for space after ":" in stucts
Addresses issue #10702
the tidy self-test doesn't seem to catch the `member_name:"Foo"` line like it should. the regex follows similar pattern in the file and works in my regex tester tool, so not really sure what is going on there.
<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10713)
<!-- Reviewable:end -->
Diffstat (limited to 'python/tidy/servo_tidy/tidy.py')
| -rw-r--r-- | python/tidy/servo_tidy/tidy.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/python/tidy/servo_tidy/tidy.py b/python/tidy/servo_tidy/tidy.py index 2a80cc68dfd..1007fcb0440 100644 --- a/python/tidy/servo_tidy/tidy.py +++ b/python/tidy/servo_tidy/tidy.py @@ -336,7 +336,8 @@ def check_rust(file_name, lines): # get rid of strings and chars because cases like regex expression, keep attributes if not line_is_attribute(line): - line = re.sub(r'"(\\.|[^\\"])*?"|' + r"'(\\.|[^\\'])*?'", '', line) + line = re.sub(r'"(\\.|[^\\"])*?"', '""', line) + line = re.sub(r"'(\\.|[^\\'])*?'", "''", line) # get rid of comments line = re.sub('//.*?$|/\*.*?$|^\*.*?$', '', line) @@ -373,7 +374,7 @@ def check_rust(file_name, lines): (r" :[^:]", "extra space before :", lambda match, line: 'trait ' not in line[:match.start()]), # ignore "crate::mod" and ignore flagging macros like "$t1:expr" - (r"[^:]:[A-Za-z]", "missing space after :", + (r"[^:]:[A-Za-z0-9\"]", "missing space after :", lambda match, line: '$' not in line[:match.end()]), (r"[A-Za-z0-9\)]{", "missing space before {{", no_filter), # ignore cases like "{}", "}`", "}}" and "use::std::{Foo, Bar}" |