diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-09-23 22:16:57 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-09-23 22:16:57 -0500 |
commit | 2ba804e6db52637fbe23fbecfd0e65b4db1a49e2 (patch) | |
tree | 3ee26fc1116d7c4e50dd0385640ad6c8d613f43c /python/tidy/servo_tidy/tidy.py | |
parent | 28ebab43987b0f9829a03a3c043e621cae2db1fe (diff) | |
parent | 0ac212afb18c98ea929368e9ccf21101e7a8921c (diff) | |
download | servo-2ba804e6db52637fbe23fbecfd0e65b4db1a49e2.tar.gz servo-2ba804e6db52637fbe23fbecfd0e65b4db1a49e2.zip |
Auto merge of #13361 - Mylainos:issue-13350, r=Wafflespeanut
Report tidy error for space after (
<!-- Please describe your changes on the following line: -->
Add report in tidy for code which have a space after the `(` like `some_function( argument)`
---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix #13350 (github issue number if applicable).
<!-- Either: -->
- [X] There are tests for these changes OR
- [ ] These changes do not require tests because _____
<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13361)
<!-- Reviewable:end -->
Diffstat (limited to 'python/tidy/servo_tidy/tidy.py')
-rw-r--r-- | python/tidy/servo_tidy/tidy.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/python/tidy/servo_tidy/tidy.py b/python/tidy/servo_tidy/tidy.py index 081352f8a43..4b02f9dbf07 100644 --- a/python/tidy/servo_tidy/tidy.py +++ b/python/tidy/servo_tidy/tidy.py @@ -483,6 +483,7 @@ def check_rust(file_name, lines): (r"\{[A-Za-z0-9_]+\};", "use statement contains braces for single import", lambda match, line: line.startswith('use ')), (r"^\s*else {", "else braces should be on the same line", no_filter), + (r"[^$ ]\([ \t]", "extra space after (", no_filter), ] for pattern, message, filter_func in regex_rules: |