diff options
| author | Esteban Küber <esteban@kuber.com.ar> | 2019-07-25 16:26:33 -0700 |
|---|---|---|
| committer | Esteban Küber <esteban@kuber.com.ar> | 2019-07-26 12:18:28 -0700 |
| commit | 6263eb438def79c393e372eac1237b4d4f97772f (patch) | |
| tree | 95acfb9ca42d359a21307f5b8f182860609ea65f /src | |
| parent | eeb1bd18dc172c2ba80c116d711214c2cdac3e32 (diff) | |
| download | rust-6263eb438def79c393e372eac1237b4d4f97772f.tar.gz rust-6263eb438def79c393e372eac1237b4d4f97772f.zip | |
ignore-tidy-trailing-newlines
Diffstat (limited to 'src')
| -rw-r--r-- | src/test/ui/parser/issue-62973.rs | 2 | ||||
| -rw-r--r-- | src/tools/tidy/src/style.rs | 16 |
2 files changed, 15 insertions, 3 deletions
diff --git a/src/test/ui/parser/issue-62973.rs b/src/test/ui/parser/issue-62973.rs index c04608e0550..18bc51e7ba7 100644 --- a/src/test/ui/parser/issue-62973.rs +++ b/src/test/ui/parser/issue-62973.rs @@ -1,4 +1,4 @@ -// ignore-tidy-end-whitespace +// ignore-tidy-trailing-newlines // error-pattern: aborting due to 6 previous errors fn main() {} diff --git a/src/tools/tidy/src/style.rs b/src/tools/tidy/src/style.rs index 4a159d926b7..6a0d530e236 100644 --- a/src/tools/tidy/src/style.rs +++ b/src/tools/tidy/src/style.rs @@ -152,6 +152,8 @@ pub fn check(path: &Path, bad: &mut bool) { let mut skip_file_length = contains_ignore_directive(can_contain, &contents, "filelength"); let mut skip_end_whitespace = contains_ignore_directive(can_contain, &contents, "end-whitespace"); + let mut skip_trailing_newlines = + contains_ignore_directive(can_contain, &contents, "trailing-newlines"); let mut skip_copyright = contains_ignore_directive(can_contain, &contents, "copyright"); let mut leading_new_lines = false; let mut trailing_new_lines = 0; @@ -214,10 +216,17 @@ pub fn check(path: &Path, bad: &mut bool) { if leading_new_lines { tidy_error!(bad, "{}: leading newline", file.display()); } + let mut err = |msg: &str| { + tidy_error!(bad, "{}: {}", file.display(), msg); + }; match trailing_new_lines { - 0 => tidy_error!(bad, "{}: missing trailing newline", file.display()), + 0 => suppressible_tidy_err!(err, skip_trailing_newlines, "missing trailing newline"), 1 => {} - n => tidy_error!(bad, "{}: too many trailing newlines ({})", file.display(), n), + n => suppressible_tidy_err!( + err, + skip_trailing_newlines, + &format!("too many trailing newlines ({})", n) + ), }; if lines > LINES { let mut err = |_| { @@ -247,6 +256,9 @@ pub fn check(path: &Path, bad: &mut bool) { if let Directive::Ignore(false) = skip_end_whitespace { tidy_error!(bad, "{}: ignoring trailing whitespace unnecessarily", file.display()); } + if let Directive::Ignore(false) = skip_trailing_newlines { + tidy_error!(bad, "{}: ignoring trailing newlines unnecessarily", file.display()); + } if let Directive::Ignore(false) = skip_copyright { tidy_error!(bad, "{}: ignoring copyright unnecessarily", file.display()); } |
