Be more specific on unused objects

It was downright lazy from my end to not inspect the line of the object
that was being unused. Fix this by trying to fetch the bundle's node.

This right now applies to variable definitions. Objects like macro
arguments are pending to be done.

Signed-off-by: Miquel Sabaté Solà <mssola@mssola.com>

1 files changed, 7 insertions, 2 deletions

diff --git a/lib/xixanta/src/assembler.rs b/lib/xixanta/src/assembler.rs
index 6ad2294..bc74d9e 100644
--- a/lib/xixanta/src/assembler.rs
+++ b/lib/xixanta/src/assembler.rs
@@ -1230,15 +1230,20 @@ impl<'a> Assembler<'a> {
                             global: true,
                         });
                     } else {
+                        let (line, global) = match &bundle.node {
+                            Some(n) => (n.value.line, false),
+                            None => (0, true),
+                        };
+
                         // Otherwise, we cannot exactly pin down whether this is
                         // harmless or not. Hence, just issue a warning and let
                         // the programmer decide on this.
                         self.warnings.push(Error {
-                            line: 0,
+                            line,
                             message: format!("{} '{full_name}' is unused", bundle.object_type),
                             source,
                             expanded_from: self.macro_context.clone(),
-                            global: true,
+                            global,
                         });
                     }
                     continue;