refactor(render): transform boxed extensions to references

espanso-render/src/extension/clipboard.rs

@@ -24,18 +24,18 @@ pub trait ClipboardProvider {
   fn get_text(&self) -> Option<String>;
 }
 
-pub struct ClipboardExtension {
+pub struct ClipboardExtension<'a> {
-  provider: Box<dyn ClipboardProvider>,
+  provider: &'a dyn ClipboardProvider,
 }
 
 #[allow(clippy::new_without_default)]
-impl ClipboardExtension {
+impl <'a> ClipboardExtension<'a> {
-  pub fn new(provider: Box<dyn ClipboardProvider>) -> Self {
+  pub fn new(provider: &'a dyn ClipboardProvider) -> Self {
     Self { provider }
   }
 }
 
-impl Extension for ClipboardExtension {
+impl <'a> Extension for ClipboardExtension<'a> {
   fn name(&self) -> &str {
     "clipboard"
   }
@@ -76,7 +76,7 @@ mod tests {
   #[test]
   fn clipboard_works_correctly() {
     let provider = MockClipboardProvider { return_none: false };
-    let extension = ClipboardExtension::new(Box::new(provider));
+    let extension = ClipboardExtension::new(&provider);
 
     assert_eq!(
       extension
@@ -90,7 +90,7 @@ mod tests {
   #[test]
   fn none_clipboard_produces_error() {
     let provider = MockClipboardProvider { return_none: true };
-    let extension = ClipboardExtension::new(Box::new(provider));
+    let extension = ClipboardExtension::new(&provider);
 
     assert!(matches!(
       extension.calculate(&Default::default(), &Default::default(), &Params::new()),

espanso-render/src/extension/script.rs

@@ -179,6 +179,7 @@ pub enum ScriptExtensionError {
 #[cfg(test)]
 mod tests {
   use super::*;
+  #[cfg(not(target_os = "windows"))]
   use crate::Scope;
   use std::iter::FromIterator;
 

espanso-render/src/lib.rs

@@ -31,7 +31,7 @@ pub trait Renderer {
     -> RenderResult;
 }
 
-pub fn create(extensions: Vec<Box<dyn Extension>>) -> impl Renderer {
+pub fn create<'a>(extensions: Vec<&'a dyn Extension>) -> impl Renderer + 'a {
   renderer::DefaultRenderer::new(extensions)
 }
 

espanso-render/src/renderer/mod.rs

@@ -36,12 +36,12 @@ lazy_static! {
   static ref WORD_REGEX: Regex = Regex::new(r"(\w+)").unwrap();
 }
 
-pub(crate) struct DefaultRenderer {
+pub(crate) struct DefaultRenderer<'a> {
-  extensions: HashMap<String, Box<dyn Extension>>,
+  extensions: HashMap<String, &'a dyn Extension>,
 }
 
-impl DefaultRenderer {
+impl <'a> DefaultRenderer<'a> {
-  pub fn new(extensions: Vec<Box<dyn Extension>>) -> Self {
+  pub fn new(extensions: Vec<&'a dyn Extension>) -> Self {
     let extensions = extensions
       .into_iter()
       .map(|ext| (ext.name().to_string(), ext))
@@ -50,7 +50,7 @@ impl DefaultRenderer {
   }
 }
 
-impl Renderer for DefaultRenderer {
+impl <'a> Renderer for DefaultRenderer<'a> {
   fn render(
     &self,
     template: &Template,
@@ -287,7 +287,7 @@ mod tests {
   }
 
   pub fn get_renderer() -> impl Renderer {
-    DefaultRenderer::new(vec![Box::new(MockExtension {})])
+    DefaultRenderer::new(vec![&MockExtension {}])
   }
 
   pub fn template_for_str(str: &str) -> Template {