Warning and duplication cleanups (#4857)

* cleanup compilation warnings

* consolidate EditScrollbackRaw into EditScrollback

3 files changed, 6 insertions, 0 deletions

diff --git a/default-plugins/fixture-plugin-for-tests/src/main.rs b/default-plugins/fixture-plugin-for-tests/src/main.rs
index 82f1cecc..5663544b 100644
--- a/default-plugins/fixture-plugin-for-tests/src/main.rs
+++ b/default-plugins/fixture-plugin-for-tests/src/main.rs
@@ -2,6 +2,7 @@ use serde::{Deserialize, Serialize};
 use std::collections::BTreeMap;
 #[allow(unused_imports)]
 use std::io::prelude::*;
+#[allow(unused_imports)] // Action is used in non-test code paths (run_action call)
 use zellij_tile::prelude::actions::Action;
 use zellij_tile::prelude::*;
 
@@ -19,6 +20,7 @@ struct State {
     explicit_string_to_render: Option<String>,
 }
 
+#[allow(dead_code)] // used when compiled as wasm plugin, not in native test target
 #[derive(Default, Serialize, Deserialize)]
 struct TestWorker {
     number_of_messages_received: usize,
diff --git a/default-plugins/strider/src/main.rs b/default-plugins/strider/src/main.rs
index 6357d955..14571532 100644
--- a/default-plugins/strider/src/main.rs
+++ b/default-plugins/strider/src/main.rs
@@ -149,6 +149,8 @@ impl ZellijPlugin for State {
                 self.close_on_selection = true;
             } else {
                 // Filepicker callback mode: send result back to caller.
+                #[allow(unused_variables)]
+                // pipe_id is used inside #[cfg(target_family = "wasm")] block
                 if let PipeSource::Cli(pipe_id) = &pipe_message.source {
                     #[cfg(target_family = "wasm")]
                     block_cli_pipe_input(pipe_id);
diff --git a/default-plugins/strider/src/state.rs b/default-plugins/strider/src/state.rs
index 2ad996f1..7932764a 100644
--- a/default-plugins/strider/src/state.rs
+++ b/default-plugins/strider/src/state.rs
@@ -224,6 +224,8 @@ impl State {
                 #[cfg(target_family = "wasm")]
                 close_self();
             },
+            #[allow(unused_variables)]
+            // pipe_id is used inside #[cfg(target_family = "wasm")] blocks
             Some((PipeSource::Cli(pipe_id), _args)) => {
                 #[cfg(target_family = "wasm")]
                 cli_pipe_output(pipe_id, &selected_path.display().to_string());