refactor: move iso-8859-8-i to fallback

Author: ChALkeR

fallback/single-byte.js

@@ -4,9 +4,10 @@ import { decode2string } from './_utils.js'
 
 export const E_STRICT = 'Input is not well-formed for this encoding'
 const xUserDefined = 'x-user-defined'
+const iso8i = 'iso-8859-8-i'
 
 export const assertEncoding = (encoding) => {
-  if (Object.hasOwn(encodings, encoding) || encoding === xUserDefined) return
+  if (Object.hasOwn(encodings, encoding) || encoding === xUserDefined || encoding === iso8i) return
   throw new RangeError('Unsupported encoding')
 }
 
@@ -28,6 +29,7 @@ const decoders = new Map()
 // Used only on Node.js, no reason to optimize for anything else
 // E.g. avoiding .from and filling zero-initialized arr manually is faster on Hermes, but we avoid this codepath on Hermes completely
 export function encodingMapper(encoding) {
+  if (encoding === iso8i) encoding = 'iso-8859-8'
   const cached = mappers.get(encoding)
   if (cached) return cached
 
@@ -62,6 +64,7 @@ export function encodingMapper(encoding) {
 }
 
 export function encodingDecoder(encoding) {
+  if (encoding === iso8i) encoding = 'iso-8859-8'
   const cached = decoders.get(encoding)
   if (cached) return cached
 

single-byte.js

@@ -29,7 +29,6 @@ function shouldUseNative(enc) {
 }
 
 export function createSinglebyteDecoder(encoding, loose = false) {
-  if (encoding === 'iso-8859-8-i') encoding = 'iso-8859-8'
   assertEncoding(encoding)
 
   if (canDecoders && shouldUseNative(encoding)) {

single-byte.node.js

@@ -24,7 +24,6 @@ function latin1Prefix(arr, start) {
 }
 
 export function createSinglebyteDecoder(encoding, loose = false) {
-  if (encoding === 'iso-8859-8-i') encoding = 'iso-8859-8'
   const latin1path = encoding === 'windows-1252'
   if (isDeno) {
     const jsDecoder = encodingDecoder(encoding) // asserts