use String.fromCharCode.apply() in WASM ver.

msgpack · May 15, 2019 · dfe06c3 · dfe06c3
1 parent eea2f04
commit dfe06c3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/benchmark/string.ts b/benchmark/string.ts
@@ -1,6 +1,6 @@
 import { encode, decode } from "../src";
 
-const data = "Hello, 🌏\n".repeat(1000);
+const data = "Hello, 🌏\n".repeat(10000);
 
 // warm up
 const encoded = encode(data);
@@ -9,13 +9,13 @@ decode(encoded);
 // run
 
 console.time("encode");
-for (let i = 0; i < 10000; i++) {
+for (let i = 0; i < 1000; i++) {
   encode(data);
 }
 console.timeEnd("encode");
 
 console.time("decode");
-for (let i = 0; i < 10000; i++) {
+for (let i = 0; i < 1000; i++) {
   decode(encoded);
 }
 console.timeEnd("decode");
diff --git a/wasmModule.ts b/wasmModule.ts
@@ -23,5 +23,5 @@ export function utf8Decode2(bytes: Uint8Array, offset: number, byteLength: numbe
   // console.log([byteLength, outputStart, outputEnd]);
   // console.log(instanceMemory.subarray(0, 10));
   // console.log(utf16array);
-  return String.fromCharCode(...utf16array);
+  return String.fromCharCode.apply(String, utf16array as any);
 }