[cpclib-asm] Add some more tests

cpclib-asm/src/assembler/mod.rs

@@ -412,7 +412,6 @@ pub struct Env {
     /// Listing of symbols generator
     symbols_output: SymbolOutputGenerator,
 
-    string_warning_done: bool,
     warnings: Vec<AssemblerWarning>,
 
     /// Counter to disable some instruction in rorg stuff
@@ -472,7 +471,6 @@ impl Clone for Env {
             run_options: self.run_options.clone(),
             output_trigger: self.output_trigger.clone(),
             symbols_output: self.symbols_output.clone(),
-            string_warning_done: self.string_warning_done.clone(),
             warnings: self.warnings.clone(),
             nested_rorg: self.nested_rorg.clone(),
             sections: self.sections.clone(),
@@ -551,7 +549,6 @@ impl Default for Env {
 
             crunched_section_state: None,
 
-            string_warning_done: false,
             warnings: Vec::new(),
             nested_rorg: 0,
 

cpclib-asm/src/error.rs

@@ -366,6 +366,13 @@ impl AssemblerError {
         }
     }
 
+    pub fn render(self) -> Self {
+        match &self {
+            Self::AlreadyRenderedError(_) => self,
+            _ => Self::AlreadyRenderedError(self.to_string())
+        }
+    }
+
 
     pub fn format(&self, f: &mut std::fmt::Formatter<'_>, complete: bool) -> std::fmt::Result {
         match self {

cpclib-basm/src/lib.rs

@@ -183,8 +183,7 @@ pub fn parse<'arg>(
 
     let res = crate::parse_z80_with_context_builder(code, builder)
         .map_err(|e| {
-            debug_assert!(e.is_already_rendered());
-            BasmError::from(e)
+            BasmError::from(e.render())
         });
 
     if options.show_progress {

cpclib-basm/tests/asm/good_ld_warning_in_macro.asm

@@ -0,0 +1,9 @@
+
+	macro gen
+		ld hl, de
+	endm
+	
+
+	gen (void)
+	gen(void)
+	gen()

cpclib-basm/tests/asm/good_ld_warning_in_repeat.asm

@@ -0,0 +1,3 @@
+repeat 20 
+	ld hl, de
+endr