using nullable reference is slow, use index pattern again

Author: cometkim

src/core.js

@@ -61,18 +61,18 @@ export function decodeUnicodeData(data, cats = '') {
  * @template {number} [T=number]
  * @param {number} cp
  * @param {CategorizedUnicodeRange<T>[]} ranges
- * @return {CategorizedUnicodeRange<T> | null}
+ * @return {number} index of matched unicode range, or -1 if no match
  */
-export function searchUnicodeRange(cp, ranges) {
+export function findUnicodeRangeIndex(cp, ranges) {
   let len = ranges.length
     , i = 1;
 
   while (i <= len) {
     let range = ranges[i - 1]
       , l = range[0]
       , h = range[1];
-    if (l <= cp && cp <= h) return range;
+    if (l <= cp && cp <= h) return i - 1;
     i = 2 * i + (cp > h ? 1 : 0);
   }
-  return null;
+  return -1;
 }

src/emoji.js

@@ -1,6 +1,6 @@
 // @ts-check
 
-import { searchUnicodeRange } from './core.js';
+import { findUnicodeRangeIndex } from './core.js';
 import {
   emoji_presentation_ranges,
   extended_pictographic_ranges,
@@ -25,7 +25,7 @@ export function isEmoji(cp) {
  * @return boolean
  */
 export function isEmojiPresentation(cp) {
-  return searchUnicodeRange(cp, emoji_presentation_ranges) !== null;
+  return findUnicodeRangeIndex(cp, emoji_presentation_ranges) >= 0;
 }
 
 /**
@@ -35,5 +35,5 @@ export function isEmojiPresentation(cp) {
  * @return boolean
  */
 export function isExtendedPictographic(cp) {
-  return searchUnicodeRange(cp, extended_pictographic_ranges) !== null;
+  return findUnicodeRangeIndex(cp, extended_pictographic_ranges) >= 0;
 }

src/general.js

@@ -1,6 +1,6 @@
 // @ts-check
 
-import { searchUnicodeRange } from './core.js';
+import { findUnicodeRangeIndex } from './core.js';
 import {
   letter_ranges,
   alphabetic_ranges,
@@ -14,7 +14,7 @@ import {
  * @return boolean
  */
 export function isLetter(cp) {
-  return searchUnicodeRange(cp, letter_ranges) !== null;
+  return findUnicodeRangeIndex(cp, letter_ranges) >= 0;
 }
 
 /**
@@ -24,7 +24,7 @@ export function isLetter(cp) {
  * @return boolean
  */
 export function isAlphabetic(cp) {
-  return searchUnicodeRange(cp, alphabetic_ranges) !== null;
+  return findUnicodeRangeIndex(cp, alphabetic_ranges) >= 0;
 }
 
 /**
@@ -34,7 +34,7 @@ export function isAlphabetic(cp) {
  * @return boolean true if 
  */
 export function isNumeric(cp) {
-  return searchUnicodeRange(cp, numeric_ranges) !== null;
+  return findUnicodeRangeIndex(cp, numeric_ranges) >= 0;
 }
 
 /**

src/grapheme.js

@@ -13,7 +13,7 @@
 
 // @ts-check
 
-import { searchUnicodeRange } from './core.js';
+import { findUnicodeRangeIndex } from './core.js';
 import { isBMP } from './utils.js';
 import { GraphemeCategory, grapheme_ranges } from './_grapheme_data.js';
 import { consonant_ranges } from './_incb_data.js';
@@ -45,8 +45,11 @@ export {
  * @return A {@link GraphemeCategoryRange} value if found, or garbage value with {@link GC_Any} category.
  */
 export function searchGraphemeCategory(cp) {
-  let range = searchUnicodeRange(cp, grapheme_ranges);
-  return range || [0, 0, 0 /* GC_Any */];
+  let index = findUnicodeRangeIndex(cp, grapheme_ranges);
+  if (index < 0) {
+    return [0, 0, 0 /* GC_Any */];
+  }
+  return grapheme_ranges[index];
 }
 
 /**
@@ -245,10 +248,11 @@ function cat(cp, cache) {
     // If this char isn't within the cached range, update the cache to the
     // range that includes it.
     if (cp < cache[0] || cp > cache[1]) {
-      let range = searchUnicodeRange(cp, grapheme_ranges);
-      if (!range) {
+      let index = findUnicodeRangeIndex(cp, grapheme_ranges);
+      if (index < 0) {
         return 0;
       }
+      let range = grapheme_ranges[index];
       cache[0] = range[0];
       cache[1] = range[1];
       cache[2] = range[2];
@@ -262,7 +266,7 @@ function cat(cp, cache) {
  * @return {boolean}
  */
 function isIndicConjunctCosonant(cp) {
-  return searchUnicodeRange(cp, consonant_ranges) !== null;
+  return findUnicodeRangeIndex(cp, consonant_ranges) >= 0;
 }
 
 /**