From 4e657c370c0d643364e389dbc8cfd11c1800d540 Mon Sep 17 00:00:00 2001
From: Linus Groh <mail@linusgroh.de>
Date: Thu, 21 May 2020 13:41:54 +0100
Subject: LibJS: Make Interpreter::call() this_value a required argument

Right now the default is an empty value, which we accidentally exposed
in set{Interval,Timeout}() by not providing a custom this value, which
should't happen at all. Let's just make it a required argument instead.
---
 Libraries/LibJS/Interpreter.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'Libraries')

diff --git a/Libraries/LibJS/Interpreter.h b/Libraries/LibJS/Interpreter.h
index 7f26cd58fe..5757e92fc7 100644
--- a/Libraries/LibJS/Interpreter.h
+++ b/Libraries/LibJS/Interpreter.h
@@ -104,7 +104,7 @@ public:
     void enter_scope(const ScopeNode&, ArgumentVector, ScopeType);
     void exit_scope(const ScopeNode&);
 
-    Value call(Function&, Value this_value = {}, Optional<MarkedValueList> arguments = {});
+    Value call(Function&, Value this_value, Optional<MarkedValueList> arguments = {});
     Value construct(Function&, Function& new_target, Optional<MarkedValueList> arguments = {});
 
     CallFrame& push_call_frame()
-- 
cgit v1.2.3