From 5410cdea2925fbdd8882709990f8e508e06f103c Mon Sep 17 00:00:00 2001
From: uz <uz@b7a2c559-68d2-44c3-8de9-860c34a00d81>
Date: Sun, 4 Nov 2012 12:35:20 +0000
Subject: [PATCH] Add line info for macro parameters.

git-svn-id: svn://svn.cc65.org/cc65/trunk@5903 b7a2c559-68d2-44c3-8de9-860c34a00d81
---
 src/ca65/macro.c | 48 +++++++++++++++++++++++++++++++++---------------
 1 file changed, 33 insertions(+), 15 deletions(-)

diff --git a/src/ca65/macro.c b/src/ca65/macro.c
index 99c6acce7..719e76f0a 100644
--- a/src/ca65/macro.c
+++ b/src/ca65/macro.c
@@ -132,6 +132,7 @@ struct MacExp {
     TokNode**  	Params;	  	/* List of actual parameters */
     TokNode*   	ParamExp;	/* Node for expanding parameters */
     LineInfo*   LI;             /* Line info for the expansion */
+    LineInfo*   ParamLI;        /* Line info for parameter expansion */
 };
 
 /* Maximum number of nested macro expansions */
@@ -310,6 +311,7 @@ static MacExp* NewMacExp (Macro* M)
     }
     E->ParamExp	        = 0;
     E->LI               = 0;
+    E->ParamLI          = 0;
 
     /* Mark the macro as expanding */
     ++M->Expansions;
@@ -344,6 +346,9 @@ static void FreeMacExp (MacExp* E)
     xfree (E->Params);
 
     /* Free the additional line info */
+    if (E->ParamLI) {
+        EndLine (E->ParamLI);
+    }
     if (E->LI) {
         EndLine (E->LI);
     }
@@ -441,20 +446,20 @@ void MacDef (unsigned Style)
 
      	    /* Insert the struct into the list, checking for duplicate idents */
      	    if (M->ParamCount == 0) {
-     		M->Params = I;
+     	    	M->Params = I;
      	    } else {
-     		IdDesc* List = M->Params;
-     		while (1) {
-     		    if (SB_Compare (&List->Id, &CurTok.SVal) == 0) {
-     		   	Error ("Duplicate symbol `%m%p'", &CurTok.SVal);
-     		    }
-     		    if (List->Next == 0) {
-     			break;
-     		    } else {
-     			List = List->Next;
-     		    }
-     		}
-     		List->Next = I;
+     	    	IdDesc* List = M->Params;
+     	    	while (1) {
+     	    	    if (SB_Compare (&List->Id, &CurTok.SVal) == 0) {
+     	    	   	Error ("Duplicate symbol `%m%p'", &CurTok.SVal);
+     	    	    }
+     	    	    if (List->Next == 0) {
+     	    		break;
+     	    	    } else {
+     	    		List = List->Next;
+     	    	    }
+     	    	}
+     	    	List->Next = I;
      	    }
      	    ++M->ParamCount;
 
@@ -513,7 +518,7 @@ void MacDef (unsigned Style)
      	    	IdDesc* I;
 
      	  	/* Skip .local or comma */
-       		NextTok ();
+       	    	NextTok ();
 
      		/* Need an identifer */
      		if (CurTok.Tok != TOK_IDENT && CurTok.Tok != TOK_LOCAL_IDENT) {
@@ -544,7 +549,7 @@ void MacDef (unsigned Style)
      	/* Create a token node for the current token */
      	N = NewTokNode ();
 
-     	/* If the token is an ident, check if it is a local parameter */
+        /* If the token is an identifier, check if it is a local parameter */
      	if (CurTok.Tok == TOK_IDENT) {
      	    unsigned Count = 0;
      	    IdDesc* I = M->Params;
@@ -648,11 +653,24 @@ ExpandParam:
        	/* Ok, use token from parameter list */
        	TokSet (Mac->ParamExp);
 
+        /* Create new line info for this parameter token */
+        if (Mac->ParamLI) {
+            EndLine (Mac->ParamLI);
+        }
+        Mac->ParamLI = StartLine (&CurTok.Pos, LI_TYPE_MACPARAM, Mac->MacExpansions);
+
        	/* Set pointer to next token */
        	Mac->ParamExp = Mac->ParamExp->Next;
 
        	/* Done */
        	return 1;
+
+    } else if (Mac->ParamLI) {
+
+        /* There's still line info open from the parameter expansion - end it */
+        EndLine (Mac->ParamLI);
+        Mac->ParamLI = 0;
+
     }
 
     /* We're not expanding macro parameters. Check if we have tokens left from
-- 
2.39.5