Remove explicit default method calls

spencewenski · Jan 8, 2024 · aedf8c1 · aedf8c1
1 parent 460a451
commit aedf8c1
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 8 deletions.
diff --git a/src/renderer/markdown_renderer.rs b/src/renderer/markdown_renderer.rs
@@ -15,7 +15,7 @@ pub struct MarkdownRenderer;
 
 impl MarkdownRenderer {
     pub fn new() -> MarkdownRenderer {
-        MarkdownRenderer::default()
+        MarkdownRenderer
     }
 }
 

diff --git a/src/renderer/mod.rs b/src/renderer/mod.rs
@@ -17,10 +17,10 @@ trait Renderer<I, O> {
 
 pub fn render_resume(resume: &Resume, config: &Config) -> Result<(), String> {
     let renderers: Vec<Box<dyn Renderer<Resume, PathBuf>>> = vec![
-        Box::new(TextRenderer::default()),
-        Box::new(PdfRenderer::default()),
-        Box::new(MarkdownRenderer::default()),
-        Box::new(GitHubRenderer::default()),
+        Box::<TextRenderer>::default(),
+        Box::<PdfRenderer>::default(),
+        Box::<MarkdownRenderer>::default(),
+        Box::<GitHubRenderer>::default(),
     ];
     renderers.iter().try_for_each(|x| -> Result<(), String> {
         x.render(resume, config)?;

diff --git a/src/renderer/pdf_renderer.rs b/src/renderer/pdf_renderer.rs
@@ -17,7 +17,7 @@ pub struct PdfRenderer;
 
 impl PdfRenderer {
     pub fn new() -> PdfRenderer {
-        PdfRenderer::default()
+        PdfRenderer
     }
 }
 

diff --git a/src/renderer/text_renderer.rs b/src/renderer/text_renderer.rs
@@ -8,15 +8,14 @@ use crate::util::{
     cover_letter_file_name, date_string, split_string_across_lines, time_range_string,
     write_string_to_file, FooterText,
 };
-use std::ops::Deref;
 use std::path::PathBuf;
 
 #[derive(Default)]
 pub struct TextRenderer;
 
 impl TextRenderer {
     pub fn new() -> TextRenderer {
-        TextRenderer::default()
+        TextRenderer
     }
 }