Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-06-06 16:45:39 +00:00
Code Issues Projects Releases Packages Wiki Activity

add tidy test for 'missing space after :' in stucts

Browse source
This commit is contained in:
aeischeid 2016-05-11 16:19:17 -05:00
parent a85db48623
commit fa4665e2a7
3 changed files with 9 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
python/tidy/servo_tidy/tidy.py
View file

@ -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}"

4
python/tidy/servo_tidy_tests/rust_tidy.rs
View file

@ -29,6 +29,10 @@ impl test {
2 => 1,
};
let z = &Vec<T>;
struct Member {
member_name:"Foo"
member_id:5
}
}
fn test_fun2(y : &String, z : &Vec<f32>) -> f32 {

2
python/tidy/servo_tidy_tests/test_tidy.py
View file

@ -66,6 +66,8 @@ class CheckTidiness(unittest.TestCase):
self.assertEqual('missing space before -', errors.next()[2])
self.assertEqual('missing space before *', errors.next()[2])
self.assertEqual('missing space after =>', errors.next()[2])
self.assertEqual('missing space after :', errors.next()[2])
self.assertEqual('missing space after :', errors.next()[2])
self.assertEqual('extra space before :', errors.next()[2])
self.assertEqual('extra space before :', errors.next()[2])
self.assertEqual('use &[T] instead of &Vec<T>', errors.next()[2])