From e5fbf21fee84c1bae373885cb418dbbfff00cd80 Mon Sep 17 00:00:00 2001 From: Ian Douglas Scott Date: Wed, 2 Jul 2025 18:46:01 -0700 Subject: [PATCH] Remove debug prints that probably shouldn't have been merged Also add a comment to the one place `eprintln!` is used to explain why it's appropriate. --- src/backend/winit.rs | 2 +- src/shell/layout/tiling/mod.rs | 2 +- src/shell/mod.rs | 1 - src/utils/rlimit.rs | 1 + 4 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/backend/winit.rs b/src/backend/winit.rs index e365c51f..9563a6f6 100644 --- a/src/backend/winit.rs +++ b/src/backend/winit.rs @@ -113,7 +113,7 @@ impl WinitState { .get::>() .unwrap() .borrow_mut(); - if dbg!(config.mode.0) != dbg!((size.w, size.h)) { + if config.mode.0 != (size.w, size.h) { if !test_only { config.mode = ((size.w, size.h), None); } diff --git a/src/shell/layout/tiling/mod.rs b/src/shell/layout/tiling/mod.rs index 3d6f3516..78b73bc7 100644 --- a/src/shell/layout/tiling/mod.rs +++ b/src/shell/layout/tiling/mod.rs @@ -2692,7 +2692,7 @@ impl TilingLayout { data.add_window(bound_idx); bound_idx }; - tree.make_nth_sibling(&new_id, dbg!(idx)).unwrap(); + tree.make_nth_sibling(&new_id, idx).unwrap(); *window.tiling_node_id.lock().unwrap() = Some(new_id); window } diff --git a/src/shell/mod.rs b/src/shell/mod.rs index 97ab7287..cd36f5b2 100644 --- a/src/shell/mod.rs +++ b/src/shell/mod.rs @@ -740,7 +740,6 @@ impl Workspaces { // If this is the first output added, create workspaces for pinned workspaces from config for pinned in std::mem::take(&mut self.persisted_workspaces) { - tracing::error!("pinned workspace: {:?}", pinned); let workspace = create_workspace_from_pinned( &pinned, workspace_state, diff --git a/src/utils/rlimit.rs b/src/utils/rlimit.rs index 9d3e1c09..a5e9066e 100644 --- a/src/utils/rlimit.rs +++ b/src/utils/rlimit.rs @@ -25,6 +25,7 @@ pub fn restore_nofile_limit() { }; if let Err(err) = setrlimit(Resource::Nofile, limits) { + // Use `eprintln!` instead of `tracing` since this is used in `pre_exec`. eprintln!("Failed to restore nofile soft limit: {:?}", err); } }