crates/codegen/src/ts.rs

@@ -6,12 +6,6 @@ pub struct Codegen {
     pub emitted: String,
 }
 
-impl Default for Codegen {
-    fn default() -> Self {
-        Self::new()
-    }
-}
-
 impl Codegen {
     pub fn new() -> Self {
         Self { emitted: String::new() }
@@ -69,11 +63,11 @@ impl Codegen {
                     "write_file" => { format!("writeFile({}, {}){}\n", self.gen_ir(&args[0], false), self.gen_ir(&args[1], false), semicolon!()) },
                     "read"       => { format!("read({}){}\n"         , self.gen_ir(&args[0], false), semicolon!()) },
                     "read_file"  => { format!("readFile({}){}\n"     , self.gen_ir(&args[0], false), semicolon!()) },
-                    "emit" => { self.gen_ir(&args[0], false).trim_start_matches('"').trim_end_matches('"').to_string() },
+                    "emit" => { format!("{}", self.gen_ir(&args[0], false).trim_start_matches('"').trim_end_matches('"')) },
-
+                    
                     "get" => { format!("{}[{}]", self.gen_ir(&args[0], false), self.gen_ir(&args[1], false)) },
                     "len" => { format!("{}.length", self.gen_ir(&args[0], false)) },
-
+                    
                     "throw" => { format!("throw new Error({}){}", self.gen_ir(&args[0], false), semicolon!()) },
                     _ => unreachable!(format!("Unknown intrinsic: {}", name)) // Shoul be handled by lowering
                 }
@@ -105,7 +99,7 @@ impl Codegen {
             IRKind::Do { body, .. } => {
                 let mut out = "{\n".to_string();
                 for expr in body {
-                    out.push_str(&self.gen_ir(expr, true));
+                    out.push_str(&self.gen_ir(&expr, true));
                 }
                 out.push_str("}\n");
                 out

crates/diagnostic/src/lib.rs

@@ -15,12 +15,6 @@ pub enum Kind {
     TypecheckError(typecheck::TypecheckError),
 }
 
-impl Default for Diagnostics {
-    fn default() -> Self {
-        Self::new()
-    }
-}
-
 impl Diagnostics {
     pub fn new() -> Self {
         Self {
@@ -132,7 +126,7 @@ impl Diagnostics {
             _ => None,
         });
         let typecheck_error = self.errors.iter().filter_map(|kind| match kind {
-            Kind::TypecheckError(error) => Some(<&typecheck::TypecheckError>::clone(&error)),
+            Kind::TypecheckError(error) => Some(error.clone()),
             _ => None,
         });
         // TODO: so many .iter(), maybe collapse them into one?
@@ -144,12 +138,12 @@ impl Diagnostics {
 
             let report = Report::build(ReportKind::Error, (), span.start)
                 .with_message(
-                    message.to_string()
+                    format!("{}", message)
                 )
                 .with_label(
                     Label::new(span.clone())
                     .with_message(
-                        message.to_string()
+                        format!("{}", message)
                     )
                     .with_color(Color::Red)
                 );

crates/hir/src/lib.rs

@@ -94,7 +94,7 @@ pub struct IR {
     pub span: Range<usize>
 }
 
-#[derive(Debug, Clone)]
+#[derive(Debug)]
 pub struct LoweringError {
     pub span: Range<usize>,
     pub message: String,

crates/lexer/src/lib.rs

@@ -165,10 +165,9 @@ pub fn lexer() -> impl Parser<char, Vec<(Token, Span)>, Error = Simple<char>> {
         .repeated()
 }
 
-#[allow(clippy::type_complexity)]
 pub fn lex(src: String) -> (Option<Vec<(Token, std::ops::Range<usize>)>>, Vec<Simple<char>>) {
     let (tokens, lex_error) = lexer().parse_recovery(src.as_str());
-    (tokens, lex_error)
+    return (tokens, lex_error);
 }
 
 #[cfg(test)]

crates/parser/src/lib.rs

@@ -286,7 +286,7 @@ fn expr_parser() -> impl Parser<Token, Vec<Spanned<Expr>>, Error = Simple<Token>
             .map_with_span(|body, span| {
                 (
                     Expr::Do {
-                        body: (body, span.clone()),
+                        body: (body.clone(), span.clone()),
                     },
                     span,
                 )
@@ -366,14 +366,13 @@ fn expr_parser() -> impl Parser<Token, Vec<Spanned<Expr>>, Error = Simple<Token>
         .then_ignore(end())
 }
 
-#[allow(clippy::type_complexity)] // We are going to use this once anyway, why we need to make a type?
 pub fn parse(tokens: Vec<(Token, std::ops::Range<usize>)>, len: usize) -> (Option<Vec<(Expr, std::ops::Range<usize>)>>, Vec<Simple<Token>>) {
     let (ast, parse_error) = expr_parser().parse_recovery(Stream::from_iter(
         len..len + 1,
         tokens.into_iter(),
     ));
 
-    (ast, parse_error)
+    return (ast, parse_error)
 }
 
 #[cfg(test)]

crates/typecheck/src/lib.rs

@@ -29,7 +29,7 @@ pub struct TypecheckError {
     pub span: std::ops::Range<usize>,
 }
 
-pub fn check(irs: &[IR]) -> Result<(), Vec<TypecheckError>> {
+pub fn check(irs: &Vec<IR>) -> Result<(), Vec<TypecheckError>> {
     let mut errors = Vec::new();
     for ir in irs {
         match &ir.kind {

test.hz

@@ -0,0 +1,3 @@
+fun main : int = do
+    "Hello, World" |> @write();
+end;