From 6f7d023408a31cd4b3a7fa32d654ed868dc76e01 Mon Sep 17 00:00:00 2001 From: Simon From Jakobsen Date: Tue, 22 Oct 2024 11:23:39 +0000 Subject: [PATCH] Fixes in code --- compiler/chapter_4.md | 13 ++++++++----- compiler/chapter_5.md | 9 +++++---- 2 files changed, 13 insertions(+), 9 deletions(-) diff --git a/compiler/chapter_4.md b/compiler/chapter_4.md index 52349dd..f5e37c5 100644 --- a/compiler/chapter_4.md +++ b/compiler/chapter_4.md @@ -269,10 +269,11 @@ type FnDef = { params: Param[], body: Expr, id: number, + syms: Syms, }; ``` -The parameters are needed, so that we can verify when calling, that we call with the correct amount of arguments. The body is the AST expression to be evaluated. And an identifier, so that we can refer to the definition by its id `fnDefId`. +The parameters are needed, so that we can verify when calling, that we call with the correct amount of arguments. The body is the AST expression to be evaluated. An identifier so that we can refer to the definition by its id `fnDefId`. And a symbol table, at the time and place of the definition, as opposed to the callers time and place. ```ts class Evaluator { @@ -479,7 +480,7 @@ class Evaluator { const fnDef = this.fnDefs[subject.fnDefId]; if (fnDef.params.length !== args.length) throw new Error("incorrect amount of arguments in call to function"); - let fnScopeSyms = new Syms(this.root); + let fnScopeSyms = new Syms(fnDef.syms); for (const [i, param] of fnDef.params.entries()) { fnScopeSyms.define(param.ident, { value: args[i], pos: param.pos }); } @@ -496,7 +497,7 @@ class Evaluator { } ``` -The first thing we do is evaluate the subject expression of the call (`subject(...args)`). If that yields a value, we continue. Then we evaluate each of the arguments in order. If evaluation of an argument doesn't yield a value, we return immediately. Then, if the subject evaluated to a builtin value, we call `executeBuiltin`, which we will define later, with the builtin name, call arguments and symbol table. Otherwise, we assert that the subject value is a function and that a function definition with the id exists. We then check that the correct amount of arguments are passed. Then, we make a new symbol table with the root table as parent, which will be the called function's symbols. We assign each argument value to the corresponding parameter name, dictated by argument order. We then evaluate the function body. Finally, we check that the control flow results in either a value, which we simply return, or a return flow, which we convert to a value. +The first thing we do is evaluate the subject expression of the call (`subject(...args)`). If that yields a value, we continue. Then we evaluate each of the arguments in order. If evaluation of an argument doesn't yield a value, we return immediately. Then, if the subject evaluated to a builtin value, we call `executeBuiltin`, which we will define later, with the builtin name, call arguments and symbol table. Otherwise, we assert that the subject value is a function and that a function definition with the id exists. We then check that the correct amount of arguments are passed. Then, we make a new symbol table with the function definition's symbol table as parent, which will be the called function's symbols. We assign each argument value to the corresponding parameter name, dictated by argument order. We then evaluate the function body. Finally, we check that the control flow results in either a value, which we simply return, or a return flow, which we convert to a value. ### 4.5.7 Unary expressions @@ -887,7 +888,7 @@ class Evaluator { } const id = this.fnDefs.length; - this.fnDefs.push({ params, body, id }); + this.fnDefs.push({ params, body, id, syms }); syms.define(stmt.kind.ident, { value: { type: "fn", fnDefId: id }, pos: stmt.pos }); return flowValue({ type: "null" }); } @@ -1069,7 +1070,7 @@ class Evaluator { for (const arg of args.slice(1)) { if (!msg.includes("{}")) throw new Error("incorrect arguments"); - msg.replace("{}", valueToString(arg)); + msg = msg.replace("{}", valueToString(arg)); } console.log(msg); return flowValue({ type: "null" }); @@ -1183,6 +1184,8 @@ class Evaluator { defineBuiltin(this.root, "string_len"); defineBuiltin(this.root, "println"); defineBuiltin(this.root, "exit"); + defineBuiltin(this.root, "to_string"); + defineBuiltin(this.root, "string_to_int"); } // ... } diff --git a/compiler/chapter_5.md b/compiler/chapter_5.md index 08c992c..3f5181a 100644 --- a/compiler/chapter_5.md +++ b/compiler/chapter_5.md @@ -189,7 +189,7 @@ fn parser_parse_expr(self) { if parser_done(self) { println("expected expr, got end-of-file"); exit(1); - } else if == parser_current(self) "+" { + } else if == parser_current(self).type "+" { parser_step(self); let left = parser_parse_expr(self); let right = parser_parse_expr(self); @@ -198,7 +198,7 @@ fn parser_parse_expr(self) { node["left"] = left; node["right"] = right; node - } else if == parser_current(self) "*" { + } else if == parser_current(self).type "*" { parser_step(self); let left = parser_parse_expr(self); let right = parser_parse_expr(self); @@ -207,9 +207,10 @@ fn parser_parse_expr(self) { node["left"] = left; node["right"] = right; node - } else if == parser_current(self) "int" { - let value = this.current().value; + } else if == parser_current(self).type "int" { + let value = parser_current(self).value; parser_step(self); + let node = struct(); node["type"] = "int"; node["value"] = value; node