Bug fix: wrong upper case letter check

The previous check did not consider non-letters. The current patch considers only latin letters (A..Z) for the check. The current spec does not allow other upper case Unicode letters anyway. Signed-off-by: Mike Lischke <[email protected]>
mike-lischke · Dec 25, 2024 · c0a1771 · c0a1771
1 parent 4bbb636
commit c0a1771
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/Vocabulary.ts b/src/Vocabulary.ts
@@ -79,12 +79,12 @@ export class Vocabulary {
                 continue;
             }
 
-            if (tokenName?.length > 0) {
-                const firstChar = tokenName.charAt(0);
-                if (firstChar === "'") {
+            if (tokenName.length > 0) {
+                const firstChar = tokenName.codePointAt(0)!;
+                if (firstChar === 0x27) { // single-quote
                     symbolicNames[i] = null;
                     continue;
-                } else if (firstChar.toUpperCase() === firstChar) {
+                } else if (firstChar >= 0x41 && firstChar <= 0x5A) { // 'A'..'Z'
                     literalNames[i] = null;
                     continue;
                 }