aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authordaurnimator <quae@daurnimator.com>2017-05-05 14:41:50 +1000
committerdaurnimator <quae@daurnimator.com>2017-05-05 15:44:21 +1000
commit4c8de564f08e122e011c5f5ecaa96bea595603ed (patch)
treecbfd0ffc066c6663e869439a0523188468a89d49 /src
parent4a265e29565718684ce30a7e90263da0ba71a0c1 (diff)
downloadfengari-4c8de564f08e122e011c5f5ecaa96bea595603ed.tar.gz
fengari-4c8de564f08e122e011c5f5ecaa96bea595603ed.tar.bz2
fengari-4c8de564f08e122e011c5f5ecaa96bea595603ed.zip
src/lfunc.js: Setting a closed upvalue should change a TValue, not create a new one
Diffstat (limited to 'src')
-rw-r--r--src/lfunc.js18
1 files changed, 11 insertions, 7 deletions
diff --git a/src/lfunc.js b/src/lfunc.js
index 544b87c..db623b5 100644
--- a/src/lfunc.js
+++ b/src/lfunc.js
@@ -2,7 +2,9 @@
"use strict";
const assert = require('assert');
+const defs = require('./defs.js');
const lobject = require('./lobject.js');
+const CT = defs.constant_types;
class Proto {
@@ -28,7 +30,7 @@ class UpVal {
constructor(L) {
this.L = L; // Keep track of the thread it comes from
- this.v = null; /* if null, upval is closed, value is in u.value */
+ this.v = null; /* if open: stack index. if closed: null (find value in this.u.value) */
this.refcount = 0;
this.u = {
open: { /* (when open) */
@@ -44,10 +46,11 @@ class UpVal {
}
setval(L, ra) {
- let o = L.stack[ra];
if (this.v !== null) {
- this.L.stack[this.v] = new lobject.TValue(o.type, o.value);
- } else this.u.value = new lobject.TValue(o.type, o.value);
+ this.L.stack[this.v] = L.stack[ra];
+ } else {
+ this.u.value.setfrom(L.stack[ra]);
+ }
}
isopen() {
@@ -91,7 +94,8 @@ const luaF_close = function(L, level) {
assert(uv.isopen());
L.openupval = uv.u.open.next; /* remove from 'open' list */
if (uv.refcount > 0) {
- uv.u.value = L.stack[uv.v];
+ let from = L.stack[uv.v];
+ uv.u.value = new lobject.TValue(from.type, from.value);
uv.v = null;
}
}
@@ -104,8 +108,8 @@ const luaF_initupvals = function(L, cl) {
for (let i = 0; i < cl.nupvalues; i++) {
let uv = new UpVal(L);
uv.refcount = 1;
- uv.u.value = null;
- uv.v = uv.u.value;
+ uv.u.value = new lobject.TValue(CT.LUA_TNIL, null);
+ uv.v = null;
cl.upvals[i] = uv;
}
};