Fixing logging bug in wrap.rs

src/wrap.rs

@@ -8,6 +8,8 @@ use log::Level::{Error, Info};
 
 const WRAP: usize = 80;
 
+// TODO add warning about long verbatim lines
+
 pub fn needs_wrap(file: &str) -> bool {
     file.lines().any(|l| l.len() > WRAP)
 }
@@ -78,6 +80,15 @@ fn wrap_line(
             }
             None => {
                 can_wrap = false;
+                record_log(
+                    logs,
+                    Error,
+                    pass,
+                    filename.to_string(),
+                    Some(linum),
+                    Some(line.to_string()),
+                    "Line cannot be wrapped.".to_string(),
+                );
             }
         }
     }
@@ -125,22 +136,5 @@ pub fn wrap(
         }
     }
 
-    if needs_wrap(&new_file) {
-        for (linum, line) in new_file.lines().enumerate() {
-            if line_needs_wrap(line) {
-                // TODO check how this works with verbatim and ignore
-                record_log(
-                    logs,
-                    Error,
-                    pass,
-                    filename.to_string(),
-                    Some(linum),
-                    Some(line.to_string()),
-                    "Line cannot be wrapped.".to_string(),
-                );
-            }
-        }
-    }
-
     new_file
 }

tests/ignore_in.tex

@@ -2,7 +2,7 @@
 
 \begin{document}
 
-Lines which end with the ignore keyword are not indented or wrapped % tex-fmt: skip
+Lines which end with the ignore keyword are not indented or wrapped even if they are long % tex-fmt: skip
 
 % tex-fmt: off
 It is also possible to ignore blocks

tests/ignore_out.tex

@@ -2,7 +2,7 @@
 
 \begin{document}
 
-Lines which end with the ignore keyword are not indented or wrapped % tex-fmt: skip
+Lines which end with the ignore keyword are not indented or wrapped even if they are long % tex-fmt: skip
 
 % tex-fmt: off
 It is also possible to ignore blocks