From c499ce82a91c3fb673e86c548d86ff021cc8b4fe Mon Sep 17 00:00:00 2001 From: Raymond Hill Date: Thu, 11 Jul 2019 09:45:53 -0400 Subject: [PATCH] Use `Reflect.construct(t)` rather than `new t()` Using `new` seemed to work but it's maybe semantically better to use `Reflect.construct`. --- assets/resources/scriptlets.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/assets/resources/scriptlets.js b/assets/resources/scriptlets.js index cbd9b9fd1..c4207507c 100644 --- a/assets/resources/scriptlets.js +++ b/assets/resources/scriptlets.js @@ -605,9 +605,9 @@ const uBOSafe = new Map(); // jshint ignore: line apply: function(target, thisArg, args) { if ( isGoodConfig(target, args[1]) === false ) { log(args[1]); - return target.apply(thisArg, args.slice(0, 1)); + return Reflect.apply(target, thisArg, args.slice(0, 1)); } - return target.apply(thisArg, args); + return Reflect.apply(target, thisArg, args); }, }); window[rtcName] = @@ -615,9 +615,9 @@ const uBOSafe = new Map(); // jshint ignore: line construct: function(target, args) { if ( isGoodConfig(target, args[0]) === false ) { log(args[0]); - return new target(); + return Reflect.construct(target); } - return new target(...args); + return Reflect.construct(target, args); } }); })();