X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-psi;a=blobdiff_plain;f=src%2Fcpp_tokiter.c;h=57645ec0db2ffccdd906116b29382777547c321e;hp=51daaa9bb794a850e3ae215b152471a941255a62;hb=e48a77547645d7985d11880f463d811bbe10da6a;hpb=c101a4de2983791d5892e60d765b73b6b3ccf6f8

diff --git a/src/cpp_tokiter.c b/src/cpp_tokiter.c
index 51daaa9..57645ec 100644
--- a/src/cpp_tokiter.c
+++ b/src/cpp_tokiter.c
@@ -233,7 +233,8 @@ bool psi_cpp_tokiter_defined(struct psi_cpp *cpp)
 	return false;
 }
 
-void psi_cpp_tokiter_expand_tokens(struct psi_cpp *cpp, struct psi_plist *tokens)
+static size_t psi_cpp_tokiter_expand_tokens(struct psi_cpp *cpp,
+		struct psi_token *target, struct psi_plist *tokens)
 {
 	if (tokens && psi_plist_count(tokens)) {
 		size_t i = 0, n = 0;
@@ -255,17 +256,22 @@ void psi_cpp_tokiter_expand_tokens(struct psi_cpp *cpp, struct psi_plist *tokens
 				continue;
 			}
 
-			if (paste && n > 0 && exp_tokens[n - 1] &&
-					(new_tok = psi_token_cat(NULL, 2, exp_tokens[n - 1], tok))) {
-				free(exp_tokens[n - 1]);
+			if (paste && n > 0 && exp_tokens[n - 1]) {
+				struct psi_token *tmp_tok, *old_tok = exp_tokens[n - 1];
+
+				tmp_tok = psi_token_init(old_tok->type, "", 0,
+						target->col, target->line, target->file?:"");
+
+				new_tok = psi_token_cat(NULL, 3, tmp_tok, old_tok, tok);
+				free(old_tok);
+				free(tmp_tok);
+
 				exp_tokens[n - 1] = new_tok;
 			} else {
-				struct psi_token *cpy = psi_token_copy(tok);
+				new_tok = psi_token_init(stringify ? PSI_T_QUOTED_STRING : tok->type,
+						tok->text, tok->size, target->col, target->line, target->file?:"");
 
-				if (stringify) {
-					cpy->type = PSI_T_QUOTED_STRING;
-				}
-				exp_tokens[n++] = cpy;
+				exp_tokens[n++] = new_tok;
 			}
 
 #if PSI_CPP_DEBUG
@@ -278,6 +284,10 @@ void psi_cpp_tokiter_expand_tokens(struct psi_cpp *cpp, struct psi_plist *tokens
 		}
 		psi_cpp_tokiter_ins_range(cpp, psi_cpp_tokiter_index(cpp), n, (void *) exp_tokens);
 		free(exp_tokens);
+
+		return n;
+	} else {
+		return 0;
 	}
 }
 
@@ -310,10 +320,6 @@ static struct psi_plist **psi_cpp_tokiter_read_call_tokens(
 
 	arg_tokens[0] = psi_plist_init(NULL);
 
-	/* free macro name token on success */
-	tok = psi_cpp_tokiter_current(cpp);
-	free_tokens = psi_plist_add(free_tokens, &tok);
-
 	/* next token must be a LPAREN for a macro call */
 	psi_cpp_tokiter_next(cpp);
 	tok = psi_cpp_tokiter_current(cpp);
@@ -374,7 +380,8 @@ fail:
 }
 
 static void psi_cpp_tokiter_expand_call_tokens(struct psi_cpp *cpp,
-		struct psi_cpp_macro_decl *macro, struct psi_plist **arg_tokens_list)
+		struct psi_token *target, struct psi_cpp_macro_decl *macro,
+		struct psi_plist **arg_tokens_list)
 {
 	size_t i;
 	struct psi_token *tok;
@@ -404,12 +411,12 @@ static void psi_cpp_tokiter_expand_call_tokens(struct psi_cpp *cpp,
 		}
 	}
 
-	psi_cpp_tokiter_expand_tokens(cpp, tokens);
+	psi_cpp_tokiter_expand_tokens(cpp, target, tokens);
 	psi_plist_free(tokens);
 }
 
 static bool psi_cpp_tokiter_expand_call(struct psi_cpp *cpp,
-		struct psi_cpp_macro_decl *macro)
+		struct psi_token *target, struct psi_cpp_macro_decl *macro)
 {
 	/* function-like macro
 	 * #define FOO(a,b) a>b // macro->sig == {a, b}, macro->tokens = {a, >, b}
@@ -430,15 +437,50 @@ static bool psi_cpp_tokiter_expand_call(struct psi_cpp *cpp,
 	psi_cpp_tokiter_seek(cpp, start);
 
 	/* insert and expand macro tokens */
-	psi_cpp_tokiter_expand_call_tokens(cpp, macro, arg_tokens_list);
+	psi_cpp_tokiter_expand_call_tokens(cpp, target, macro, arg_tokens_list);
 	psi_cpp_tokiter_free_call_tokens(arg_tokens_list, psi_plist_count(macro->sig), true);
 
 	/* back to where we took off */
 	psi_cpp_tokiter_seek(cpp, start);
+
+	free(target);
+	++cpp->expanded;
+	return true;
+}
+
+static bool psi_cpp_tokiter_expand_def(struct psi_cpp *cpp,
+		struct psi_token *target, struct psi_cpp_macro_decl *macro)
+{
+	size_t index = psi_cpp_tokiter_index(cpp);
+
+	/* delete current token from stream */
+	psi_cpp_tokiter_del_cur(cpp, false);
+
+	if (index != psi_cpp_tokiter_index(cpp)) {
+		/* might have been last token */
+		psi_cpp_tokiter_next(cpp);
+	}
+	/* replace with tokens from macro */
+	psi_cpp_tokiter_expand_tokens(cpp, target, macro->tokens);
+
+	free(target);
 	++cpp->expanded;
 	return true;
 }
 
+static inline int psi_cpp_tokiter_expand_cmp(struct psi_token *t,
+		struct psi_cpp_macro_decl *m)
+{
+	if (psi_plist_count(m->tokens) == 1) {
+		struct psi_token *r;
+
+		psi_plist_get(m->tokens, 0, &r);
+
+		return strcmp(r->text, t->text);
+	}
+	return -1;
+}
+
 bool psi_cpp_tokiter_expand(struct psi_cpp *cpp)
 {
 	if (psi_cpp_tokiter_valid(cpp)) {
@@ -455,22 +497,9 @@ bool psi_cpp_tokiter_expand(struct psi_cpp *cpp)
 				psi_token_dump(2, current);
 #endif
 				if (macro->sig) {
-					return psi_cpp_tokiter_expand_call(cpp, macro);
-				} else {
-					size_t index = psi_cpp_tokiter_index(cpp);
-
-					/* delete current token from stream */
-					psi_cpp_tokiter_del_cur(cpp, true);
-
-					if (index != psi_cpp_tokiter_index(cpp)) {
-						/* might have been last token */
-						psi_cpp_tokiter_next(cpp);
-					}
-					/* replace with tokens from macro */
-					psi_cpp_tokiter_expand_tokens(cpp, macro->tokens);
-
-					++cpp->expanded;
-					return true;
+					return psi_cpp_tokiter_expand_call(cpp, current, macro);
+				} else if (psi_cpp_tokiter_expand_cmp(current, macro)) {
+					return psi_cpp_tokiter_expand_def(cpp, current, macro);
 				}
 			}
 		}