summaryrefslogtreecommitdiff
path: root/ext/web/10_filereader.js
diff options
context:
space:
mode:
authorMarcos Casagrande <marcoscvp90@gmail.com>2022-10-24 20:27:22 +0200
committerGitHub <noreply@github.com>2022-10-24 20:27:22 +0200
commitac5fcf626a77db7795f7ab2b4f15e4ecb3270171 (patch)
tree1135cc4a5b4fed75a3f17bb0e3c12cd11c4abf43 /ext/web/10_filereader.js
parent7a65b8e8dae8660f0e40130ba5a63f6c10d358c6 (diff)
perf(ext/web): add op_encode_binary_string (#16352)
Add a new op to use in `reader.readAsBinaryString(blob)`. ``` File API binary string: 400b 35.12 µs/iter (21.93 µs … 3.27 ms) 31.87 µs 131.95 µs 217.63 µs File API binary string: 4kb 46.49 µs/iter (29.36 µs … 4.42 ms) 42.5 µs 122.48 µs 155.1 µs File API binary string: 2.2mb 4.17 ms/iter (1.75 ms … 8.54 ms) 5.48 ms 7.39 ms 8.54 ms ``` **main** ``` benchmark time (avg) (min … max) p75 p99 p995 --------------------------------------------------------------------- ----------------------------- File API binary string: 400b 56.17 µs/iter (43.09 µs … 784.52 µs) 49.6 µs 177.18 µs 241.23 µs File API binary string: 4kb 277.2 µs/iter (240.29 µs … 1.84 ms) 269.87 µs 649.79 µs 774.46 µs File API binary string: 2.2mb 180.03 ms/iter (173.32 ms … 194.35 ms) 182.54 ms 194.35 ms 194.35 ms ``` It can also handle bigger files, when encoding a 200mb file, main crashes with OOM ``` <--- Last few GCs ---> [132677:0x560504676550] 5012 ms: Scavenge 417.3 (434.6) -> 401.8 (434.6) MB, 0.1 / 0.0 ms (average mu = 0.824, current mu = 0.825) allocation failure; [132677:0x560504676550] 5038 ms: Scavenge 417.3 (434.6) -> 401.8 (434.6) MB, 0.1 / 0.0 ms (average mu = 0.824, current mu = 0.825) allocation failure; [132677:0x560504676550] 5064 ms: Scavenge 417.3 (434.6) -> 401.8 (434.6) MB, 0.1 / 0.0 ms (average mu = 0.824, current mu = 0.825) allocation failure; ```
Diffstat (limited to 'ext/web/10_filereader.js')
-rw-r--r--ext/web/10_filereader.js12
1 files changed, 2 insertions, 10 deletions
diff --git a/ext/web/10_filereader.js b/ext/web/10_filereader.js
index 8a76b2e0f..49f4babe1 100644
--- a/ext/web/10_filereader.js
+++ b/ext/web/10_filereader.js
@@ -13,6 +13,7 @@
"use strict";
((window) => {
+ const core = window.Deno.core;
const webidl = window.__bootstrap.webidl;
const { forgivingBase64Encode } = window.__bootstrap.infra;
const { ProgressEvent } = window.__bootstrap.event;
@@ -21,8 +22,6 @@
const { parseMimeType } = window.__bootstrap.mimesniff;
const { DOMException } = window.__bootstrap.domException;
const {
- ArrayPrototypeJoin,
- ArrayPrototypeMap,
ArrayPrototypePush,
ArrayPrototypeReduce,
FunctionPrototypeCall,
@@ -33,7 +32,6 @@
ObjectPrototypeIsPrototypeOf,
queueMicrotask,
SafeArrayIterator,
- StringFromCodePoint,
Symbol,
TypedArrayPrototypeSet,
TypeError,
@@ -170,13 +168,7 @@
break;
}
case "BinaryString":
- this[result] = ArrayPrototypeJoin(
- ArrayPrototypeMap(
- [...new Uint8Array(bytes.buffer)],
- (v) => StringFromCodePoint(v),
- ),
- "",
- );
+ this[result] = core.ops.op_encode_binary_string(bytes);
break;
case "Text": {
let decoder = undefined;