Browse code

Just pass an error code to uxn_halt.

Andrew Alderwick authored on 05/01/2022 13:06:22
Showing 3 changed files
... ...
@@ -13,11 +13,17 @@ THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
13 13
 WITH REGARD TO THIS SOFTWARE.
14 14
 */
15 15
 
16
-static const char *errors[] = {"underflow", "overflow", "division by zero"};
16
+static const char *errors[] = {
17
+	"Working-stack underflow",
18
+	"Return-stack underflow",
19
+	"Working-stack overflow",
20
+	"Return-stack overflow",
21
+	"Working-stack division by zero",
22
+	"Return-stack division by zero"};
17 23
 
18 24
 int
19
-uxn_halt(Uxn *u, Uint8 error, char *name, Uint16 addr)
25
+uxn_halt(Uxn *u, Uint8 error, Uint16 addr)
20 26
 {
21
-	fprintf(stderr, "Halted: %s %s#%04x, at 0x%04x\n", name, errors[error - 1], u->ram[addr], addr);
27
+	fprintf(stderr, "Halted: %s#%04x, at 0x%04x\n", errors[error], u->ram[addr], addr);
22 28
 	return 0;
23 29
 }
... ...
@@ -20,8 +20,8 @@ WITH REGARD TO THIS SOFTWARE.
20 20
 #define PUSH8(s, x) { if(s->ptr == 0xff) { errcode = 2; goto err; } s->dat[s->ptr++] = (x); }
21 21
 #define PUSH16(s, x) { if((j = s->ptr) >= 0xfe) { errcode = 2; goto err; } k = (x); s->dat[j] = k >> 8; s->dat[j + 1] = k; s->ptr = j + 2; }
22 22
 #define PUSH(s, x) { if(bs) { PUSH16(s, (x)) } else { PUSH8(s, (x)) } }
23
-#define POP8(o) { if(!(j = *sp)) { errcode = 1; goto err; } o = (Uint16)src->dat[--j]; *sp = j; }
24
-#define POP16(o) { if((j = *sp) <= 1) { errcode = 1; goto err; } o = src->dat[j - 1]; o += src->dat[j - 2] << 8; *sp = j - 2; }
23
+#define POP8(o) { if(!(j = *sp)) { errcode = 0; goto err; } o = (Uint16)src->dat[--j]; *sp = j; }
24
+#define POP16(o) { if((j = *sp) <= 1) { errcode = 0; goto err; } o = src->dat[j - 1]; o += src->dat[j - 2] << 8; *sp = j - 2; }
25 25
 #define POP(o) { if(bs) { POP16(o) } else { POP8(o) } }
26 26
 #define POKE(x, y) { if(bs) { u->ram[(x)] = (y) >> 8; u->ram[(x) + 1] = (y); } else { u->ram[(x)] = y; } }
27 27
 #define PEEK16(o, x) { o = (u->ram[(x)] << 8) + u->ram[(x) + 1]; }
... ...
@@ -89,7 +89,7 @@ uxn_eval(Uxn *u, Uint16 pc)
89 89
 		case 0x18: /* ADD */ POP(a) POP(b) PUSH(src, b + a) break;
90 90
 		case 0x19: /* SUB */ POP(a) POP(b) PUSH(src, b - a) break;
91 91
 		case 0x1a: /* MUL */ POP(a) POP(b) PUSH(src, (Uint32)b * a) break;
92
-		case 0x1b: /* DIV */ POP(a) POP(b) if(a == 0) { errcode = 3; goto err; } PUSH(src, b / a) break;
92
+		case 0x1b: /* DIV */ POP(a) POP(b) if(a == 0) { errcode = 4; goto err; } PUSH(src, b / a) break;
93 93
 		case 0x1c: /* AND */ POP(a) POP(b) PUSH(src, b & a) break;
94 94
 		case 0x1d: /* ORA */ POP(a) POP(b) PUSH(src, b | a) break;
95 95
 		case 0x1e: /* EOR */ POP(a) POP(b) PUSH(src, b ^ a) break;
... ...
@@ -99,10 +99,10 @@ uxn_eval(Uxn *u, Uint16 pc)
99 99
 	return 1;
100 100
 
101 101
 err:
102
-	if ((instr & 0x40 ? 1 : 0) ^ (errcode == 2 && ((instr & 0x1f) == 0x0e || (instr & 0x1f) == 0x0f)))
103
-		return uxn_halt(u, errcode, "Return-stack", pc - 1);
104
-	else
105
-		return uxn_halt(u, errcode, "Working-stack", pc - 1);
102
+	/* set 1 in errcode if it involved the return stack instead of the working stack */
103
+	/*        (stack overflow & (       instruction opcode was STH or JSR        )) ^ Return Mode */
104
+	errcode |= ((errcode >> 1 & instr >> 1 & instr >> 2 & instr >> 3 & ~instr >> 4) ^ instr >> 6) & 1;
105
+	return uxn_halt(u, errcode, pc - 1);
106 106
 }
107 107
 
108 108
 /* clang-format on */
... ...
@@ -45,5 +45,5 @@ typedef struct Uxn {
45 45
 
46 46
 int uxn_boot(Uxn *c, Uint8 *memory);
47 47
 int uxn_eval(Uxn *u, Uint16 pc);
48
-int uxn_halt(Uxn *u, Uint8 error, char *name, Uint16 addr);
48
+int uxn_halt(Uxn *u, Uint8 error, Uint16 addr);
49 49
 Device *uxn_port(Uxn *u, Uint8 id, Uint8 (*deifn)(Device *, Uint8), void (*deofn)(Device *, Uint8));