From 29f57b7899dcb032624e1bec54e883723bb1851c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tobias=20Nie=C3=9Fen?= <tniessen@tnie.de>
Date: Wed, 14 Sep 2022 23:44:31 +0200
Subject: [PATCH] src: avoid copy when creating Blob

PR-URL: https://github.com/nodejs/node/pull/44616
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
---
 src/node_blob.cc |  8 +++-----
 src/node_blob.h  | 11 ++++-------
 2 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/src/node_blob.cc b/src/node_blob.cc
index b319a74ebaedbf..aba95e47f353ad 100644
--- a/src/node_blob.cc
+++ b/src/node_blob.cc
@@ -71,11 +71,9 @@ bool Blob::HasInstance(Environment* env, v8::Local<v8::Value> object) {
   return GetConstructorTemplate(env)->HasInstance(object);
 }
 
-BaseObjectPtr<Blob> Blob::Create(
-    Environment* env,
-    const std::vector<BlobEntry> store,
-    size_t length) {
-
+BaseObjectPtr<Blob> Blob::Create(Environment* env,
+                                 const std::vector<BlobEntry>& store,
+                                 size_t length) {
   HandleScope scope(env->isolate());
 
   Local<Function> ctor;
diff --git a/src/node_blob.h b/src/node_blob.h
index 61f4da655c93cd..a1f7293bfff58f 100644
--- a/src/node_blob.h
+++ b/src/node_blob.h
@@ -45,16 +45,13 @@ class Blob : public BaseObject {
   static v8::Local<v8::FunctionTemplate> GetConstructorTemplate(
       Environment* env);
 
-  static BaseObjectPtr<Blob> Create(
-      Environment* env,
-      const std::vector<BlobEntry> store,
-      size_t length);
+  static BaseObjectPtr<Blob> Create(Environment* env,
+                                    const std::vector<BlobEntry>& store,
+                                    size_t length);
 
   static bool HasInstance(Environment* env, v8::Local<v8::Value> object);
 
-  const std::vector<BlobEntry> entries() const {
-    return store_;
-  }
+  const std::vector<BlobEntry>& entries() const { return store_; }
 
   void MemoryInfo(MemoryTracker* tracker) const override;
   SET_MEMORY_INFO_NAME(Blob)