about summary refs log tree commit diff
path: root/src
diff options
context:
space:
mode:
authorEsteban Küber <esteban@kuber.com.ar>2019-07-25 16:26:33 -0700
committerEsteban Küber <esteban@kuber.com.ar>2019-07-26 12:18:28 -0700
commit6263eb438def79c393e372eac1237b4d4f97772f (patch)
tree95acfb9ca42d359a21307f5b8f182860609ea65f /src
parenteeb1bd18dc172c2ba80c116d711214c2cdac3e32 (diff)
downloadrust-6263eb438def79c393e372eac1237b4d4f97772f.tar.gz
rust-6263eb438def79c393e372eac1237b4d4f97772f.zip
ignore-tidy-trailing-newlines
Diffstat (limited to 'src')
-rw-r--r--src/test/ui/parser/issue-62973.rs2
-rw-r--r--src/tools/tidy/src/style.rs16
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());
         }