Moved to bool event lookup vs bitwise checks

Author: justinwilaby

.github/workflows/ci.yaml

@@ -45,6 +45,9 @@ jobs:
       - name: Build and test
         run: npm run lint && npm run build && npm run coverage
 
+      - name: Benchmark
+        run: npm run benchmark
+
       - name: Cache Node.js modules
         uses: actions/cache@v2
         with:

lib/cjs/index.js

@@ -24,6 +24,7 @@
     "build:wasm": "cargo build --target wasm32-unknown-unknown --release && npm run wasm-opt",
     "wasm-opt": "wasm-opt -O4 target/wasm32-unknown-unknown/release/sax_wasm.wasm -o lib/sax-wasm.wasm",
     "build": "(npm run build:esm && npm run build:cjs) && npm run build:wasm && tsc --emitDeclarationOnly",
+    "build:dev": "(npm run build:esm -- --sourceMap && npm run build:cjs -- --sourceMap ) && npm run build:wasm && tsc --emitDeclarationOnly",
     "build:esm": "tsc --module esnext --outDir lib/esm --target esnext",
     "build:cjs": "tsc --module commonjs --outDir lib/cjs --target esnext",
     "benchmark": "node src/js/__test__/benchmark.mjs",

lib/cjs/index.js.map

@@ -3,7 +3,7 @@ import { URL } from 'url';
 import { resolve } from 'path';
 import { Buffer } from 'buffer';
 
-import { SAXParser, SaxEventType } from '../../../lib/cjs/index.js';
+import { SAXParser, SaxEventType } from '../../../lib/esm/index.js';
 
 import nodeXml from 'node-xml';
 import expat from 'node-expat';

lib/cjs/saxWasm.js

@@ -132,10 +132,10 @@ impl GraphemeClusters<'_> {
     /// assert!(gc.take_until_ascii(&[b'!']).is_none());
     ///
     /// // Handle broken surrogate at the end
-    /// let bytes_with_surrogate = "🐉 hello, world!\xF0".as_bytes();
-    /// let mut gc_with_surrogate = GraphemeClusters::new(bytes_with_surrogate);
+    /// let bytes = "hello, world!🐉🐉🐉".as_bytes();
+    /// let mut gc_with_surrogate = GraphemeClusters::new(&bytes[..14]);
     /// if let Some(result) = gc_with_surrogate.take_until_ascii(&[b'!']) {
-    ///     assert_eq!(result.0, "🐉 hello, world");
+    ///     assert_eq!(result.0, "hello, world");
     ///     assert_eq!(result.1, 0);
     ///     assert_eq!(result.2, 12);
     /// }
@@ -166,8 +166,8 @@ impl GraphemeClusters<'_> {
             cursor += len;
         }
 
-        if cursor >= byte_len {
-          cursor = byte_len;
+        if cursor > byte_len {
+          cursor =  byte_len - (cursor - byte_len);
         }
         // Nothing to take
         if start >= cursor {