From ff2d1376a1456d5c06e208eedd9dbc65e71e81b4 Mon Sep 17 00:00:00 2001 From: Robin Haberkorn Date: Wed, 30 Oct 2024 13:54:28 +0300 Subject: fixed invalid memory access when executing the F< command (but only when jumping to the beginning of the macro) * I am not sure whether this feature is really that useful... * teco_machine_main_t::macro_pc is now pointing to the __next__ character to execute, therefore it's easier to manipulate by flow control commands. Also, it can now be unsigned (gsize) like all other program counters. * Detected thanks to running the testsuite under Valgrind. --- src/parser.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/parser.h') diff --git a/src/parser.h b/src/parser.h index 066896f..659e9f9 100644 --- a/src/parser.h +++ b/src/parser.h @@ -447,8 +447,8 @@ typedef enum { struct teco_machine_main_t { teco_machine_t parent; - /* signed because it is sometimes set to -1 for flow control */ - gssize macro_pc; + /** Program counter, ie. pointer to the next character in the current macro frame */ + gsize macro_pc; /** * Aliases bitfield with an integer. -- cgit v1.2.3