chore: better struct declarations

Author: Sk7Str1p3

image/src/iterm.rs

@@ -5,25 +5,15 @@ use image::{imageops::FilterType, DynamicImage};
 use std::env;
 use std::io::Cursor;
 
-pub struct ITermBackend {}
+pub struct ITermBackend;
 
 impl ITermBackend {
-    pub fn new() -> Self {
-        ITermBackend {}
-    }
-
     pub fn supported() -> bool {
         let term_program = env::var("TERM_PROGRAM").unwrap_or_else(|_| "".to_string());
         term_program == "iTerm.app"
     }
 }
 
-impl Default for ITermBackend {
-    fn default() -> Self {
-        Self::new()
-    }
-}
-
 impl super::ImageBackend for ITermBackend {
     fn add_image(
         &self,

image/src/kitty.rs

@@ -9,13 +9,9 @@ use libc::{
 use std::io::{stdout, Write};
 use std::time::Instant;
 
-pub struct KittyBackend {}
+pub struct KittyBackend;
 
 impl KittyBackend {
-    pub fn new() -> Self {
-        Self {}
-    }
-
     pub fn supported() -> bool {
         // save terminal attributes and disable canonical input processing mode
         let old_attributes = unsafe {
@@ -75,12 +71,6 @@ impl KittyBackend {
     }
 }
 
-impl Default for KittyBackend {
-    fn default() -> Self {
-        Self::new()
-    }
-}
-
 impl super::ImageBackend for KittyBackend {
     fn add_image(
         &self,

image/src/lib.rs

@@ -22,11 +22,11 @@ pub trait ImageBackend {
 pub fn get_best_backend() -> Option<Box<dyn ImageBackend>> {
     #[cfg(not(windows))]
     if sixel::SixelBackend::supported() {
-        Some(Box::new(sixel::SixelBackend::new()))
+        Some(Box::new(sixel::SixelBackend))
     } else if kitty::KittyBackend::supported() {
-        Some(Box::new(kitty::KittyBackend::new()))
+        Some(Box::new(kitty::KittyBackend))
     } else if iterm::ITermBackend::supported() {
-        Some(Box::new(iterm::ITermBackend::new()))
+        Some(Box::new(iterm::ITermBackend))
     } else {
         None
     }
@@ -39,9 +39,9 @@ pub fn get_best_backend() -> Option<Box<dyn ImageBackend>> {
 pub fn get_image_backend(image_protocol: ImageProtocol) -> Option<Box<dyn ImageBackend>> {
     #[cfg(not(windows))]
     let backend = Some(match image_protocol {
-        ImageProtocol::Kitty => Box::new(kitty::KittyBackend::new()) as Box<dyn ImageBackend>,
-        ImageProtocol::Iterm => Box::new(iterm::ITermBackend::new()) as Box<dyn ImageBackend>,
-        ImageProtocol::Sixel => Box::new(sixel::SixelBackend::new()) as Box<dyn ImageBackend>,
+        ImageProtocol::Kitty => Box::new(kitty::KittyBackend) as Box<dyn ImageBackend>,
+        ImageProtocol::Iterm => Box::new(iterm::ITermBackend) as Box<dyn ImageBackend>,
+        ImageProtocol::Sixel => Box::new(sixel::SixelBackend) as Box<dyn ImageBackend>,
     });
 
     #[cfg(windows)]

image/src/sixel.rs

@@ -12,13 +12,9 @@ use libc::{
 use std::io::{stdout, Write};
 use std::time::Instant;
 
-pub struct SixelBackend {}
+pub struct SixelBackend;
 
 impl SixelBackend {
-    pub fn new() -> Self {
-        Self {}
-    }
-
     pub fn supported() -> bool {
         // save terminal attributes and disable canonical input processing mode
         let old_attributes = unsafe {
@@ -72,12 +68,6 @@ impl SixelBackend {
     }
 }
 
-impl Default for SixelBackend {
-    fn default() -> Self {
-        Self::new()
-    }
-}
-
 impl super::ImageBackend for SixelBackend {
     #[allow(clippy::map_entry)]
     fn add_image(&self, lines: Vec<String>, image: &DynamicImage, colors: usize) -> Result<String> {