From b0bc20bdeae770de3726f87d8ea13038491b2e29 Mon Sep 17 00:00:00 2001
From: Robin Haberkorn <robin.haberkorn@googlemail.com>
Date: Thu, 20 Nov 2014 05:29:26 +0100
Subject: simplified attaching errors to a position in a macro

introduced Error::set_coord()
---
 src/parser.cpp | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

(limited to 'src/parser.cpp')

diff --git a/src/parser.cpp b/src/parser.cpp
index 35b6add..98df598 100644
--- a/src/parser.cpp
+++ b/src/parser.cpp
@@ -104,9 +104,7 @@ Execute::step(const gchar *macro, gint stop_pos)
 				throw StdError(error);
 			}
 		} catch (Error &error) {
-			error.pos = macro_pc;
-			String::get_coord(macro, error.pos,
-					  error.line, error.column);
+			error.set_coord(macro, macro_pc);
 			throw; /* forward */
 		}
 		macro_pc++;
@@ -148,7 +146,8 @@ Execute::macro(const gchar *macro, bool locals)
 
 		/*
 		 * Subsequent errors must still be
-		 * attached to this macro invocation.
+		 * attached to this macro invocation
+		 * via Error::set_coord()
 		 */
 		try {
 			if (Goto::skip_label)
@@ -173,9 +172,7 @@ Execute::macro(const gchar *macro, bool locals)
 			if (locals)
 				QRegisters::locals->clear();
 		} catch (Error &error) {
-			error.pos = strlen(macro);
-			String::get_coord(macro, error.pos,
-					  error.line, error.column);
+			error.set_coord(macro, strlen(macro));
 			throw; /* forward */
 		}
 	} catch (...) {
-- 
cgit v1.2.3