From 3626a7fd6d2c4792c24144e958fffc38987e0c1b Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Wed, 29 Dec 2021 00:07:52 +0100 Subject: [PATCH] LibJS: Remove unused FunctionEnvironment this value getter/setter The this value is only supposed to be set via the BindThisValue and accessed via the GetThisBinding AOs, so exposing a direct getter/setter would only lead to potentially non-spec-compliant behavior down the line. --- Userland/Libraries/LibJS/Runtime/FunctionEnvironment.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/Userland/Libraries/LibJS/Runtime/FunctionEnvironment.h b/Userland/Libraries/LibJS/Runtime/FunctionEnvironment.h index b2574e324d..7487ed689c 100644 --- a/Userland/Libraries/LibJS/Runtime/FunctionEnvironment.h +++ b/Userland/Libraries/LibJS/Runtime/FunctionEnvironment.h @@ -25,9 +25,6 @@ public: explicit FunctionEnvironment(Environment* parent_scope); virtual ~FunctionEnvironment() override; - Value this_value() const { return m_this_value; } - void set_this_value(Value value) { m_this_value = value; } - ThisBindingStatus this_binding_status() const { return m_this_binding_status; } void set_this_binding_status(ThisBindingStatus status) { m_this_binding_status = status; }