Merge pull request #50 from Ph0enixKM/panic-when-out-of-bounds

Remove panic when error happens out of bounds of file

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "heraclitus-compiler"
-version = "1.5.9"
+version = "1.6.0"
 edition = "2021"
 description = "Compiler frontend for developing great programming languages"
 license = "MIT"

README.md

@@ -45,6 +45,10 @@ let tokens = cc.tokenize()?;
 
 # Change log 🚀
 
+## Version 1.6.0
+### Fix:
+- Heraclitus no longer panics when error happens out of bounds of file
+
 ## Version 1.5.9
 ### Fix:
 - Escaping escape key now treats it as a character

src/compiling/failing/logger.rs

@@ -128,6 +128,9 @@ impl Logger {
         let (row, col, len) = self.get_row_col_len();
         let max_pad = self.get_max_pad_size(code.len());
         let index = index as i32 + offset as i32;
+        if code.get(index as usize).is_none() {
+            return String::new();
+        }
         let row = row as i32 + offset as i32;
         let code = code[index as usize].clone();
         let line = format!("{row}").pad_to_width(max_pad);
@@ -208,7 +211,7 @@ mod test {
 
     use crate::prelude::{PositionInfo, MessageType};
     #[allow(unused_variables)]
-    
+
     #[test]
     fn test_displayer() {
         let code = vec![
@@ -229,4 +232,21 @@ mod test {
             .path()
             .snippet(Some(code));
     }
-}
+
+    #[test]
+    fn test_end_of_line_displayer() {
+        let code = vec![
+            "hello"
+        ].join("\n");
+        // Uncomment to see the error message
+        sleep(Duration::from_secs(1));
+        let trace = [
+            PositionInfo::at_pos(Some("/path/to/foo".to_string()), (2, 6), 1)
+        ];
+        super::Logger::new(MessageType::Error, &trace)
+            .header(MessageType::Error)
+            .text(Some(format!("Cannot call function \"foobar\" on a number")))
+            .path()
+            .snippet(Some(code));
+    }
+}