size: smaller utf16 my moving encodeCharcodes to /platform

Author: ChALkeR

fallback/latin1.js

@@ -3,6 +3,7 @@ import {
   nativeDecoder,
   nativeDecoderLatin1,
   nativeBuffer,
+  encodeCharcodes,
   isHermes,
   isDeno,
   isLE,
@@ -107,25 +108,6 @@ export const decodeAscii = nativeBuffer
 
 /* eslint-disable @exodus/mutable/no-param-reassign-prop-only */
 
-export const encodeCharcodes = isHermes
-  ? (str, arr) => {
-      const length = str.length
-      if (length > 64) {
-        const at = str.charCodeAt.bind(str) // faster on strings from ~64 chars on Hermes, but can be 10x slower on e.g. JSC
-        for (let i = 0; i < length; i++) arr[i] = at(i)
-      } else {
-        for (let i = 0; i < length; i++) arr[i] = str.charCodeAt(i)
-      }
-
-      return arr
-    }
-  : (str, arr) => {
-      const length = str.length
-      // Can be optimized with unrolling, but this is not used on non-Hermes atm
-      for (let i = 0; i < length; i++) arr[i] = str.charCodeAt(i)
-      return arr
-    }
-
 export function encodeAsciiPrefix(x, s) {
   let i = 0
   for (const len3 = s.length - 3; i < len3; i += 4) {

fallback/platform.browser.js

@@ -1,5 +1,7 @@
 import { decodePartAddition as decodePart } from './platform.native.js'
 
+export { encodeCharcodesPure as encodeCharcodes } from './platform.native.js'
+
 export const nativeBuffer = null
 export const isHermes = false
 export const isDeno = false

fallback/platform.native.js

@@ -95,3 +95,28 @@ export function decode2string(arr, start, end, m) {
 
   return decodePart(arr, start, end, m)
 }
+
+/* eslint-disable @exodus/mutable/no-param-reassign-prop-only */
+
+function encodeCharcodesHermes(str, arr) {
+  const length = str.length
+  if (length > 64) {
+    const at = str.charCodeAt.bind(str) // faster on strings from ~64 chars on Hermes, but can be 10x slower on e.g. JSC
+    for (let i = 0; i < length; i++) arr[i] = at(i)
+  } else {
+    for (let i = 0; i < length; i++) arr[i] = str.charCodeAt(i)
+  }
+
+  return arr
+}
+
+export function encodeCharcodesPure(str, arr) {
+  const length = str.length
+  // Can be optimized with unrolling, but this is not used on non-Hermes atm
+  for (let i = 0; i < length; i++) arr[i] = str.charCodeAt(i)
+  return arr
+}
+
+/* eslint-enable @exodus/mutable/no-param-reassign-prop-only */
+
+export const encodeCharcodes = isHermes ? encodeCharcodesHermes : encodeCharcodesPure

fallback/utf16.js

@@ -1,6 +1,6 @@
-import { decodeUCS2, encodeCharcodes } from './latin1.js'
+import { decodeUCS2 } from './latin1.js'
 import { assertU8, E_STRING, E_STRICT_UNICODE } from './_utils.js'
-import { nativeDecoder, isLE } from './platform.js'
+import { nativeDecoder, isLE, encodeCharcodes } from './platform.js'
 
 export const E_STRICT = 'Input is not well-formed utf16'
 const isWellFormedStr = String.prototype.isWellFormed