Reworked imports

src/convert/mod.rs

@@ -1,9 +1,10 @@
-use crate::convert::char::{CharConverter, CharSample};
+mod char;
-use crate::convert::color::ColorConverter;
+mod color;
-use crossterm::style::Color;
 
-pub mod char;
+pub use crate::convert::char::*;
-pub mod color;
+pub use crate::convert::color::*;
+
+use crossterm::style::Color;
 
 #[cfg_attr(test, mockall::automock)]
 pub trait Converter {
@@ -30,8 +31,8 @@ impl<T1: CharConverter, T2: ColorConverter> Converter for ConverterImpl<T1, T2>
 #[cfg(test)]
 mod test {
     use super::*;
-    use crate::convert::char::MockCharConverter;
+    use crate::convert::MockCharConverter;
-    use crate::convert::color::MockColorConverter;
+    use crate::convert::MockColorConverter;
     use mockall::predicate::*;
 
     #[test]

src/main.rs

@@ -1,11 +1,20 @@
 pub mod convert;
-pub mod error;
 pub mod pattern;
-pub mod printer;
+
+mod error;
+mod printer;
 mod renderer;
-pub mod term;
+mod term;
+mod timer;
 mod vec;
 
-fn main() -> Result<(), error::Error> {
+pub use error::*;
+pub use printer::*;
+pub use renderer::*;
+pub use term::*;
+pub use timer::*;
+pub use vec::*;
+
+fn main() -> Result<(), Error> {
     Ok(())
 }

src/pattern/circle.rs

@@ -1,5 +1,5 @@
-use crate::pattern::{Config, Pattern, PatternFactory};
+use crate::pattern::*;
-use crate::vec::Vector;
+use crate::Vector;
 
 #[derive(Default)]
 pub struct CircleFactory;

src/pattern/line.rs

@@ -1,5 +1,5 @@
-use crate::pattern::{Config, Pattern, PatternFactory};
+use crate::pattern::*;
-use crate::vec::Vector;
+use crate::Vector;
 
 #[derive(Default)]
 pub struct LineFactory;

src/pattern/mod.rs

@@ -1,9 +1,14 @@
-pub mod circle;
+mod circle;
-pub mod line;
+mod line;
-pub mod rhombus;
+mod rhombus;
-pub mod wheel;
+mod wheel;
 
-use crate::vec::Vector;
+pub use circle::*;
+pub use line::*;
+pub use rhombus::*;
+pub use wheel::*;
+
+use crate::Vector;
 
 #[derive(Copy, Clone, PartialEq, Debug)]
 pub struct Config {

src/pattern/rhombus.rs

@@ -1,5 +1,5 @@
-use crate::pattern::{Config, Pattern, PatternFactory};
+use crate::pattern::*;
-use crate::vec::Vector;
+use crate::Vector;
 
 #[derive(Default)]
 pub struct RhombusFactory;
@@ -42,7 +42,7 @@ mod test {
     fn name() {
         assert_eq!("rhombus", RhombusFactory::default().name());
     }
-    
+
     #[test]
     fn sample() {
         let config = Config {

src/pattern/wheel.rs

@@ -1,5 +1,5 @@
-use crate::pattern::{Config, Pattern, PatternFactory};
+use crate::pattern::*;
-use crate::vec::Vector;
+use crate::Vector;
 use std::f32::consts::PI;
 
 #[derive(Default)]

src/printer.rs

@@ -1,5 +1,5 @@
-use crate::error::Error;
+use crate::Error;
-use crate::term::Terminal;
+use crate::Terminal;
 use crossterm::cursor::*;
 use crossterm::style::*;
 use crossterm::terminal::*;
@@ -96,7 +96,7 @@ impl<T: Terminal> Printer for PrinterImpl<T> {
 #[cfg(test)]
 mod test {
     use super::*;
-    use crate::term::MockTerminal;
+    use crate::MockTerminal;
     use mockall::predicate::eq;
 
     #[test]

src/renderer.rs

@@ -1,9 +1,8 @@
-use crate::convert::char::CharSample;
+use crate::convert::{CharSample, Converter};
-use crate::convert::Converter;
+use crate::pattern::*;
-use crate::error::Error;
+use crate::Error;
-use crate::pattern::{Config, Sampler, SamplerFactory};
+use crate::Printer;
-use crate::printer::Printer;
+use crate::Vector;
-use crate::vec::Vector;
 
 #[cfg_attr(test, mockall::automock)]
 pub trait Renderer {
@@ -69,7 +68,7 @@ mod test {
     use crate::convert::MockConverter;
     use crate::pattern::MockSampler;
     use crate::pattern::MockSamplerFactory;
-    use crate::printer::MockPrinter;
+    use crate::MockPrinter;
     use crossterm::style::Color;
     use mockall::predicate::eq;
     use mockall::Sequence;

src/term.rs

@@ -1,4 +1,4 @@
-use crate::error::Error;
+use crate::Error;
 use crossterm::{Command, QueueableCommand};
 use std::io::Write;
 

src/timer.rs

@@ -1,5 +1,5 @@
-use crate::error::Error;
+use crate::Error;
-use crate::renderer::Renderer;
+use crate::Renderer;
 use std::thread;
 use std::time::{Duration, Instant};
 
@@ -58,7 +58,7 @@ impl<T: Clock> Timer<T> {
 #[cfg(test)]
 mod test {
     use super::*;
-    use crate::renderer::MockRenderer;
+    use crate::MockRenderer;
     use mockall::predicate::eq;
     use mockall::Sequence;
 

src/vec.rs

@@ -1,6 +1,4 @@
-use std::ops::Sub;
+#[derive(Copy, Clone, PartialEq, Debug, Default, derive_more::Sub)]
-
-#[derive(Copy, Clone, PartialEq, Debug, Default)]
 pub struct Vector {
     pub x: f32,
     pub y: f32,
@@ -36,14 +34,6 @@ impl Vector {
     }
 }
 
-impl Sub for Vector {
-    type Output = Vector;
-
-    fn sub(self, rhs: Self) -> Self::Output {
-        Self::new(self.x - rhs.x, self.y - rhs.y)
-    }
-}
-
 #[cfg(test)]
 mod test {
     use super::*;
@@ -80,12 +70,4 @@ mod test {
     fn angle() {
         assert_abs_diff_eq!(-1.5, Vector::new(2.0, -20.0).angle(), epsilon = 0.1);
     }
-
-    #[test]
-    fn sub() {
-        assert_eq!(
-            Vector::new(-5.0, 10.0),
-            Vector::new(3.0, 16.0) - Vector::new(8.0, 6.0)
-        );
-    }
 }