A61 Fix: Handle Warnings

Cargo.toml

@@ -6,6 +6,6 @@ edition = "2021"
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
 [dependencies]
-heraclitus-compiler = "1.5.1"
+heraclitus-compiler = "1.5.2"
 similar-string = "1.4.2"
 colored = "2.0.0"

src/cli/cli_interface.rs

@@ -43,7 +43,10 @@ impl CLI {
             match self.flags.get_flag("-e").unwrap().value.clone() {
                 Some(code) => {
                     match AmberCompiler::new(code, None).compile() {
-                        Ok(code) => AmberCompiler::execute(code, self.flags.get_args()),
+                        Ok((code, messages)) => {
+                            messages.iter().for_each(|m| m.show());
+                            AmberCompiler::execute(code, self.flags.get_args())
+                        },
                         Err(err) => {
                             err.show();
                             std::process::exit(1);
@@ -64,7 +67,8 @@ impl CLI {
             match self.read_file(input.clone()) {
                 Ok(code) => {
                     match AmberCompiler::new(code, Some(input)).compile() {
-                        Ok(code) => {
+                        Ok((code, messages)) => {
+                            messages.iter().for_each(|m| m.show());
                             // Save to the output file
                             if self.args.len() >= 3 {
                                 Self::save_to_file(self.args[2].clone(), code)

src/compiler.rs

@@ -77,10 +77,10 @@ impl AmberCompiler {
         block.translate(&mut meta)
     }
 
-    pub fn compile(&self) -> Result<String, Message> {
+    pub fn compile(&self) -> Result<(String, Vec<Message>), Message> {
         self.tokenize()
             .and_then(|tokens| self.parse(tokens))
-            .map(|(block, meta)| self.translate(block, meta))
+            .map(|(block, meta)| (self.translate(block, meta.clone()), meta.messages))
     }
 
     pub fn execute(code: String, flags: &[String]) {
@@ -90,7 +90,7 @@ impl AmberCompiler {
 
     #[allow(dead_code)]
     pub fn test_eval(&self) -> Result<String, Message> {
-        self.compile().map_or_else(Err, |code| {
+        self.compile().map_or_else(Err, |(code, _)| {
             let child = Command::new("/bin/bash")
                 .arg("-c").arg::<&str>(code.as_ref())
                 .output().unwrap();

src/modules/block.rs

@@ -5,7 +5,7 @@ use super::statement::stmt::Statement;
 
 #[derive(Debug, Clone)]
 pub struct Block {
-    statements: Vec<Statement>
+    pub statements: Vec<Statement>
 }
 
 impl Block {

src/modules/condition/ifcond.rs

@@ -17,7 +17,9 @@ impl IfCondition {
         let is_not_if_chain = matches!(statement.value.as_ref().unwrap(), StatementType::IfCondition(_) | StatementType::IfChain(_));
         if is_not_if_chain {
             // TODO: [A34] Add a comment pointing to the website documentation
-            return error!(meta, tok, "You should use if-chain instead of nested if else statements")
+            let message = Message::new_warn_at_token(meta, tok)
+                .message("You should use if-chain instead of nested if else statements");
+            meta.add_message(message);
         }
         Ok(())
     }
@@ -56,7 +58,14 @@ impl SyntaxModule<ParserMetadata> for IfCondition {
             match token(meta, "{") {
                 Ok(_) => {
                     let mut false_block = Box::new(Block::new());
+                    let tok = meta.get_current_token();
                     syntax(meta, &mut *false_block)?;
+                    // Check if the statement is using if chain syntax sugar
+                    if false_block.statements.len() == 1 {
+                        if let Some(statement) = false_block.statements.first() {
+                            self.prevent_not_using_if_chain(meta, statement, tok)?;
+                        }
+                    }
                     self.false_block = Some(false_block);
                     token(meta, "}")?;
                 }

src/utils/metadata/parser.rs

@@ -14,7 +14,8 @@ pub struct ParserMetadata {
     pub trace: Vec<PositionInfo>,
     pub import_history: ImportHistory,
     pub loop_ctx: bool,
-    pub function_ctx: bool
+    pub function_ctx: bool,
+    pub messages: Vec<Message>
 }
 
 impl ParserMetadata {
@@ -25,6 +26,10 @@ impl ParserMetadata {
     pub fn pop_trace(&mut self) -> Option<PositionInfo> {
         self.trace.pop()
     }
+
+    pub fn add_message(&mut self, message: Message) {
+        self.messages.push(message);
+    }
 }
 
 impl Metadata for ParserMetadata {
@@ -40,7 +45,8 @@ impl Metadata for ParserMetadata {
             trace: Vec::new(),
             import_history: ImportHistory::new(path),
             loop_ctx: false,
-            function_ctx: false
+            function_ctx: false,
+            messages: Vec::new(),
         }
     }