Skip to content

Commit

Permalink
Avoid using destructuring in tco-fn-realm.js
Browse files Browse the repository at this point in the history
  • Loading branch information
tannal authored and ptomato committed Oct 10, 2024
1 parent ab809f8 commit 954d637
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -27,13 +27,15 @@ assert.sameValue(Array.prototype.toString.call({ join: {} }), "[object Object]")

let revokeOnGet = false;
const proxyTarget = [];
const { proxy, revoke } = Proxy.revocable(proxyTarget, {
var proxyObj = Proxy.revocable(proxyTarget, {
get: (target, key, receiver) => {
if (revokeOnGet)
revoke();
return Reflect.get(target, key, receiver);
},
});
var proxy = proxyObj.proxy;
var revoke = proxyObj.revoke;

proxyTarget.join = undefined;
assert.sameValue(Array.prototype.toString.call(proxy), "[object Array]");
Expand Down
4 changes: 3 additions & 1 deletion test/built-ins/Proxy/revocable/tco-fn-realm.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,9 @@ features: [Proxy, tail-call-optimization]
var other = $262.createRealm();
var F = other.evalScript(`
(function() {
var { proxy, revoke } = Proxy.revocable(function() {}, {});
var proxyObj = Proxy.revocable(function() {}, {});
var proxy = proxyObj.proxy;
var revoke = proxyObj.revoke;
revoke();
return proxy();
})
Expand Down

0 comments on commit 954d637

Please sign in to comment.