From 50d0f879f7298a90d530a153a729f905257075f7 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Tue, 3 May 2022 22:51:36 +0200 Subject: [PATCH] LibJS: Remove bogus FIXME in ArgumentsObject::internal_get() Not sure what happened here, but the comment does not match the spec, and so the FIXME I added is not actually an issue. --- Userland/Libraries/LibJS/Runtime/ArgumentsObject.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Userland/Libraries/LibJS/Runtime/ArgumentsObject.cpp b/Userland/Libraries/LibJS/Runtime/ArgumentsObject.cpp index 3a8be5b28c..f9b6d0eb90 100644 --- a/Userland/Libraries/LibJS/Runtime/ArgumentsObject.cpp +++ b/Userland/Libraries/LibJS/Runtime/ArgumentsObject.cpp @@ -41,8 +41,7 @@ ThrowCompletionOr ArgumentsObject::internal_get(PropertyKey const& proper // 3. If isMapped is false, then if (!is_mapped) { - // a. Return ! OrdinaryGet(args, P, Receiver). - // FIXME: Using MUST here breaks one test in test262 (spec issue). + // a. Return ? OrdinaryGet(args, P, Receiver). return Object::internal_get(property_key, receiver); }