lint: use clippy instead of hand-written no_printlns/no_dbgs

magistau · magistau · commit a2ab1dc8d89f · 2025-08-05T16:43:26.000+03:00
diff --git a/.clippy.toml b/.clippy.toml
@@ -0,0 +1 @@
+allow-print-in-tests = true
diff --git a/Cargo.toml b/Cargo.toml
@@ -169,5 +169,13 @@ name = "uiua"
 [workspace]
 members = ["site", "tests_ffi", "pad/editor", "parser"]
 
+[workspace.lints.clippy]
+print_stdout = "warn"
+print_stderr = "allow"
+dbg_macro = "warn"
+
+[lints]
+workspace = true
+
 [profile.dev]
 incremental = true
diff --git a/pad/editor/.clippy.toml b/pad/editor/.clippy.toml
@@ -0,0 +1 @@
+../../.clippy.toml
diff --git a/pad/editor/Cargo.toml b/pad/editor/Cargo.toml
@@ -6,6 +6,9 @@ edition = "2021"
 [lib]
 crate-type = ["cdylib", "rlib"]
 
+[lints]
+workspace = true
+
 [dependencies]
 leptos = { version = "0.6.10", features = ["csr"] }
 js-sys = "0.3.69"
diff --git a/parser/.clippy.toml b/parser/.clippy.toml
@@ -0,0 +1 @@
+../.clippy.toml
diff --git a/parser/Cargo.toml b/parser/Cargo.toml
@@ -10,6 +10,9 @@ repository = "https://github.com/uiua-lang/uiua"
 rust-version = "1.82.0"
 version = "0.17.0-dev.1"
 
+[lints]
+workspace = true
+
 [dependencies]
 bytemuck = {version = "1.17", features = ["must_cast", "derive", "extern_crate_alloc"]}
 colored = "2"
diff --git a/site/.clippy.toml b/site/.clippy.toml
@@ -0,0 +1 @@
+../.clippy.toml
diff --git a/site/Cargo.toml b/site/Cargo.toml
@@ -3,6 +3,9 @@ edition = "2021"
 name = "site"
 version = "0.1.0"
 
+[lints]
+workspace = true
+
 [dependencies]
 base64 = "0.22.0"
 comrak = "0.39.0"
diff --git a/src/compile/mod.rs b/src/compile/mod.rs
@@ -2610,6 +2610,7 @@ impl Compiler {
         let inputs = self.asm.inputs.clone();
         self.emit_diagnostic_impl(Diagnostic::new(message.into(), span, kind, inputs));
     }
+    #[allow(clippy::print_stdout)]
     fn emit_diagnostic_impl(&mut self, diagnostic: Diagnostic) {
         if self.print_diagnostics {
             eprintln!("{}", diagnostic.report());
diff --git a/src/ffi.rs b/src/ffi.rs
@@ -3,9 +3,6 @@ use std::{
     str::FromStr,
 };
 
-#[allow(missing_docs)]
-pub const DEBUG: bool = false;
-
 /// Data for how to send an argument type to `&ffi`
 #[derive(Debug)]
 pub struct FfiArg {
diff --git a/src/lib.rs b/src/lib.rs
@@ -356,43 +356,11 @@ mod tests {
         }
     }
 
-    fn recurse_dirs(dir: &std::path::Path, f: &impl Fn(&std::path::Path)) {
-        for entry in std::fs::read_dir(dir).unwrap() {
-            let entry = entry.unwrap();
-            let path = entry.path();
-            if path.to_string_lossy().contains("target") {
-                continue;
-            }
-            if path.is_dir() {
-                recurse_dirs(&path, f);
-            } else {
-                f(&path);
-            }
-        }
-    }
-
     #[test]
     fn no_dbgs() {
-        recurse_dirs(std::path::Path::new("."), &|path| {
-            if path.extension().is_none_or(|ext| ext != "rs")
-                || path.canonicalize().unwrap()
-                    == std::path::Path::new(file!()).canonicalize().unwrap()
-            {
-                return;
-            }
-            let contents = std::fs::read_to_string(path).unwrap();
-            for line in contents.lines() {
-                if line.contains("dbg!") && !line.trim().starts_with("//") {
-                    panic!("File {} contains a dbg! macro", path.display());
-                }
-            }
-        });
         if crate::compile::invert::DEBUG {
             panic!("compile::invert::DEBUG is true");
         }
-        if crate::ffi::DEBUG {
-            panic!("ffi::DEBUG is true");
-        }
         if crate::compile::optimize::DEBUG {
             panic!("compile::optimize::DEBUG is true");
         }
@@ -401,31 +369,6 @@ mod tests {
         }
     }
 
-    #[test]
-    fn no_printlns() {
-        recurse_dirs(std::path::Path::new("."), &|path| {
-            if path.extension().is_none_or(|ext| ext != "rs")
-                || path.canonicalize().unwrap()
-                    == std::path::Path::new(file!()).canonicalize().unwrap()
-                || path
-                    .components()
-                    .any(|c| c.as_os_str() == "main.rs" || c.as_os_str() == "profile.rs")
-            {
-                return;
-            }
-            let contents = std::fs::read_to_string(path).unwrap();
-            for line in contents.lines() {
-                if line.contains("println!")
-                    && !(line.trim().starts_with("//")
-                        || line.contains("eprintln!")
-                        || line.contains("// Allow println"))
-                {
-                    panic!("File {} contains a println! macro", path.display());
-                }
-            }
-        });
-    }
-
     #[test]
     fn external_bind_before() {
         let mut comp = Compiler::new();
diff --git a/src/main.rs b/src/main.rs
@@ -1,3 +1,4 @@
+#![allow(clippy::print_stdout)]
 #[cfg(not(feature = "binary"))]
 compile_error!("To compile the uiua interpreter binary, you must enable the `binary` feature flag");
 
diff --git a/src/profile.rs b/src/profile.rs
@@ -1,3 +1,4 @@
+#![allow(clippy::print_stdout)]
 #[macro_export]
 #[doc(hidden)]
 macro_rules! profile_function {
diff --git a/src/run.rs b/src/run.rs
@@ -274,6 +274,7 @@ impl Uiua {
         take(&mut self.rt.reports)
     }
     /// Print all pending reports
+    #[allow(clippy::print_stdout)]
     pub fn print_reports(&mut self) {
         for report in self.take_reports() {
             eprintln!("{report}");
@@ -790,6 +791,7 @@ impl Uiua {
                 Ok(())
             }
         };
+        #[allow(clippy::print_stdout)]
         if self.rt.time_instrs {
             let end_time = self.rt.backend.now();
             let padding = self.rt.call_stack.len().saturating_sub(1) * 2;
diff --git a/src/sys/native.rs b/src/sys/native.rs
@@ -663,6 +663,7 @@ impl SysBackend for NativeSys {
     fn allow_thread_spawning(&self) -> bool {
         true
     }
+    #[allow(clippy::print_stdout)]
     #[cfg(all(feature = "terminal_image", feature = "image"))]
     fn show_image(&self, image: image::DynamicImage, _label: Option<&str>) -> Result<(), String> {
         let (_width, _height) = if let Some((w, h)) = terminal_size() {
@@ -1279,6 +1280,7 @@ impl SysBackend for NativeSys {
         NATIVE_SYS.git_paths.insert(url.to_string(), res.clone());
         res
     }
+    #[allow(clippy::print_stdout)]
     fn breakpoint(&self, env: &Uiua) -> Result<bool, String> {
         if !self.output_enabled() {
             return Ok(true);
@@ -1299,6 +1301,7 @@ impl SysBackend for NativeSys {
     }
 }
 
+#[allow(clippy::print_stdout)]
 #[doc(hidden)]
 pub fn print_stack(stack: &[Value], color: bool) {
     #[cfg(feature = "window")]
diff --git a/tests_ffi/.clippy.toml b/tests_ffi/.clippy.toml
@@ -0,0 +1 @@
+../.clippy.toml
diff --git a/tests_ffi/Cargo.toml b/tests_ffi/Cargo.toml
@@ -7,5 +7,8 @@ version = "0.1.0"
 crate-type = ["cdylib"]
 name = "ffi_lib"
 
+[lints]
+workspace = true
+
 [dependencies]
 uiua.path = ".."

Original file line number	Diff line number	Diff line change
`@@ -2610,6 +2610,7 @@ impl Compiler {`
`2610`	`2610`	`let inputs = self.asm.inputs.clone();`
`2611`	`2611`	`self.emit_diagnostic_impl(Diagnostic::new(message.into(), span, kind, inputs));`
`2612`	`2612`	`}`
	`2613`	`+ #[allow(clippy::print_stdout)]`
`2613`	`2614`	`fn emit_diagnostic_impl(&mut self, diagnostic: Diagnostic) {`
`2614`	`2615`	`if self.print_diagnostics {`
`2615`	`2616`	`eprintln!("{}", diagnostic.report());`