Fixed bug that created additional lexer action entries.

The subtleties wrt. to object equality vs. reference equality between Java and TypeScript are a constant source of tricky bugs. However, there's not need to always use a OrderedHashMap if the key is a standard type (a type without equals/hashMap methods) as the standard Map already maintains insertion order.

src/analysis/LeftRecursiveRuleAnalyzer.ts

@@ -36,9 +36,9 @@ enum Associativity {
 export class LeftRecursiveRuleAnalyzer extends LeftRecursiveRuleWalker {
 
     public tool: Tool;
-    public binaryAlts = new OrderedHashMap<number, LeftRecursiveRuleAltInfo>();
-    public ternaryAlts = new OrderedHashMap<number, LeftRecursiveRuleAltInfo>();
-    public suffixAlts = new OrderedHashMap<number, LeftRecursiveRuleAltInfo>();
+    public binaryAlts = new Map<number, LeftRecursiveRuleAltInfo>();
+    public ternaryAlts = new Map<number, LeftRecursiveRuleAltInfo>();
+    public suffixAlts = new Map<number, LeftRecursiveRuleAltInfo>();
     public prefixAndOtherAlts = new Array<LeftRecursiveRuleAltInfo>();
 
     /** Pointer to ID node of ^(= ID element) */

src/automata/LexerATNFactory.ts

@@ -7,8 +7,8 @@
 /* eslint-disable jsdoc/require-param, jsdoc/require-returns */
 
 import {
-    ActionTransition, ATN, ATNState, AtomTransition, BasicState, CodePointTransitions, CommonToken, IntervalSet,
-    IntStream, Lexer, LexerAction, LexerChannelAction, LexerCustomAction, LexerModeAction, LexerMoreAction,
+    ActionTransition, ATN, ATNState, AtomTransition, BasicState, CodePointTransitions, CommonToken, HashMap,
+    IntervalSet, IntStream, Lexer, LexerAction, LexerChannelAction, LexerCustomAction, LexerModeAction, LexerMoreAction,
     LexerPopModeAction, LexerPushModeAction, LexerSkipAction, LexerTypeAction, NotSetTransition, SetTransition, Token,
     TokensStartState, Transition
 } from "antlr4ng";
@@ -122,7 +122,7 @@ export class LexerATNFactory extends ParserATNFactory {
     /**
      * Maps from a {@link LexerAction} object to the action index.
      */
-    private actionToIndexMap = new Map<LexerAction, number>();
+    private actionToIndexMap = new HashMap<LexerAction, number>();
 
     private readonly ruleCommands = new Array<string>();
 

src/tool/Grammar.ts

@@ -31,7 +31,6 @@ import { TokenVocabParser } from "../parse/TokenVocabParser.js";
 import { GrammarType } from "../support/GrammarType.js";
 import type { IGrammar, ITool } from "../types.js";
 
-import { OrderedHashMap } from "../misc/OrderedHashMap.js";
 import type { CommonTree } from "../tree/CommonTree.js";
 import { ANTLRMessage } from "./ANTLRMessage.js";
 import { ANTLRToolListener } from "./ANTLRToolListener.js";
@@ -151,7 +150,7 @@ export class Grammar implements IGrammar, AttributeResolver {
      * All rules defined in this specific grammar, not imported. Also does
      *  not include lexical rules if combined.
      */
-    public rules = new OrderedHashMap<string, Rule>();
+    public rules = new Map<string, Rule>();
 
     public indexToRule = new Array<Rule>(); // used to invent rule names for 'keyword', ';', ... (0..n-1)
 

src/tool/LeftRecursiveRule.ts

@@ -115,7 +115,7 @@ export class LeftRecursiveRule extends Rule {
     /** Get -> labels from those alts we deleted for left-recursive rules. */
 
     public override getAltLabels(): Map<string, Array<[number, AltAST]>> | null {
-        const labels = new OrderedHashMap<string, Array<[number, AltAST]>>();
+        const labels = new Map<string, Array<[number, AltAST]>>();
         const normalAltLabels = super.getAltLabels();
         if (normalAltLabels !== null) {
             normalAltLabels.forEach((value, key) => {