remove redundant cases for allocString & stringLengthUTF8

Author: MaxGraey

src/glue/wasm/i64.ts

@@ -226,7 +226,7 @@ function i64_to_f64(value: i64): f64 {
 }
 
 // @ts-ignore: decorator
-@global
+@global @inline
 function i64_to_string(value: i64, unsigned: bool = false): string {
   return (unsigned ? <u64>value : value).toString();
 }

src/module.ts

@@ -2344,17 +2344,13 @@ function stringLengthUTF8(str: string): usize {
       u = 0x10000 + ((u & 0x3FF) << 10) | (str.charCodeAt(++i) & 0x3FF);
     }
     if (u <= 0x7F) {
-      ++len;
+      len += 1;
     } else if (u <= 0x7FF) {
       len += 2;
     } else if (u <= 0xFFFF) {
       len += 3;
-    } else if (u <= 0x1FFFFF) {
-      len += 4;
-    } else if (u <= 0x3FFFFFF) {
-      len += 5;
     } else {
-      len += 6;
+      len += 4;
     }
   }
   return len;
@@ -2379,21 +2375,9 @@ function allocString(str: string | null): usize {
       binaryen.__i32_store8(idx++, (0xE0 |  (u >>> 12)      ) as u8);
       binaryen.__i32_store8(idx++, (0x80 | ((u >>>  6) & 63)) as u8);
       binaryen.__i32_store8(idx++, (0x80 | ( u         & 63)) as u8);
-    } else if (u <= 0x1FFFFF) {
-      binaryen.__i32_store8(idx++, (0xF0 |  (u >>> 18)      ) as u8);
-      binaryen.__i32_store8(idx++, (0x80 | ((u >>> 12) & 63)) as u8);
-      binaryen.__i32_store8(idx++, (0x80 | ((u >>>  6) & 63)) as u8);
-      binaryen.__i32_store8(idx++, (0x80 | ( u         & 63)) as u8);
-    } else if (u <= 0x3FFFFFF) {
-      binaryen.__i32_store8(idx++, (0xF8 |  (u >>> 24)      ) as u8);
-      binaryen.__i32_store8(idx++, (0x80 | ((u >>> 18) & 63)) as u8);
-      binaryen.__i32_store8(idx++, (0x80 | ((u >>> 12) & 63)) as u8);
-      binaryen.__i32_store8(idx++, (0x80 | ((u >>>  6) & 63)) as u8);
-      binaryen.__i32_store8(idx++, (0x80 | ( u         & 63)) as u8);
     } else {
-      binaryen.__i32_store8(idx++, (0xFC |  (u >>> 30)      ) as u8);
-      binaryen.__i32_store8(idx++, (0x80 | ((u >>> 24) & 63)) as u8);
-      binaryen.__i32_store8(idx++, (0x80 | ((u >>> 18) & 63)) as u8);
+      assert(u < 0x200000, "Invalid Unicode code point during allocString");
+      binaryen.__i32_store8(idx++, (0xF0 |  (u >>> 18)      ) as u8);
       binaryen.__i32_store8(idx++, (0x80 | ((u >>> 12) & 63)) as u8);
       binaryen.__i32_store8(idx++, (0x80 | ((u >>>  6) & 63)) as u8);
       binaryen.__i32_store8(idx++, (0x80 | ( u         & 63)) as u8);