From 8f3a5ba5d83dcbc5038b81806d28a69525a20407 Mon Sep 17 00:00:00 2001
From: Linus Groh <mail@linusgroh.de>
Date: Sat, 4 Sep 2021 17:07:46 +0100
Subject: [PATCH] LibJS: Add Array::create_from() for generic Vector<T>

It relies on a mapper function to convert each T& to a JS::Value. This
allows us to avoid awkward Vector<T> to MarkedValueList conversion at
the call site.
---
 Userland/Libraries/LibJS/Runtime/Array.h | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/Userland/Libraries/LibJS/Runtime/Array.h b/Userland/Libraries/LibJS/Runtime/Array.h
index 0a38828e83..703d16d6d5 100644
--- a/Userland/Libraries/LibJS/Runtime/Array.h
+++ b/Userland/Libraries/LibJS/Runtime/Array.h
@@ -6,6 +6,10 @@
 
 #pragma once
 
+#include <AK/Assertions.h>
+#include <AK/Function.h>
+#include <AK/Vector.h>
+#include <LibJS/Runtime/GlobalObject.h>
 #include <LibJS/Runtime/Object.h>
 
 namespace JS {
@@ -16,6 +20,19 @@ class Array : public Object {
 public:
     static Array* create(GlobalObject&, size_t length, Object* prototype = nullptr);
     static Array* create_from(GlobalObject&, Vector<Value> const&);
+    // Non-standard but equivalent to CreateArrayFromList.
+    template<typename T>
+    static Array* create_from(GlobalObject& global_object, Vector<T>& elements, Function<Value(T&)> map_fn)
+    {
+        auto& vm = global_object.vm();
+        auto values = MarkedValueList { global_object.heap() };
+        values.ensure_capacity(elements.size());
+        for (auto& element : elements) {
+            values.append(map_fn(element));
+            VERIFY(!vm.exception());
+        }
+        return Array::create_from(global_object, values);
+    }
 
     explicit Array(Object& prototype);
     virtual ~Array() override;