Fix onInterrupt impl (#1744)

src/commands/registry.ts

@@ -1,24 +1,28 @@
-import { EmacsCommand } from ".";
+import { EmacsCommand, ITextEditorInterruptionHandler, isTextEditorInterruptionHandler } from ".";
 
 export class EmacsCommandRegistry {
   private commands: Map<string, EmacsCommand>;
+  private interruptionHandlers: ITextEditorInterruptionHandler[];
 
   constructor() {
     this.commands = new Map();
+    this.interruptionHandlers = [];
   }
 
   public register(command: EmacsCommand): void {
     this.commands.set(command.id, command);
+    if (isTextEditorInterruptionHandler(command)) {
+      this.interruptionHandlers.push(command);
+    }
   }
 
   public get(commandName: string): EmacsCommand | undefined {
     return this.commands.get(commandName);
   }
 
-  public forEach(
-    callbackfn: (value: EmacsCommand, key: string, map: Map<string, EmacsCommand>) => void,
-    thisArg?: unknown,
-  ): void {
-    this.commands.forEach(callbackfn, thisArg);
+  public onInterrupt(): void {
+    for (const handler of this.interruptionHandlers) {
+      handler.onDidInterruptTextEditor();
+    }
   }
 }

src/emulator.ts

@@ -1,6 +1,5 @@
 import * as vscode from "vscode";
 import { Selection, TextEditor } from "vscode";
-import { isTextEditorInterruptionHandler } from "./commands";
 import { AddSelectionToNextFindMatch, AddSelectionToPreviousFindMatch } from "./commands/add-selection-to-find-match";
 import * as CaseCommands from "./commands/case";
 import { DeleteBlankLines } from "./commands/delete-blank-lines";
@@ -611,12 +610,7 @@ export class EmacsEmulator implements IEmacsController, vscode.Disposable {
   }
 
   private onDidInterruptTextEditor() {
-    this.commandRegistry.forEach((command) => {
-      if (isTextEditorInterruptionHandler(command)) {
-        // TODO: Cache the array of IEmacsCommandInterrupted instances
-        command.onDidInterruptTextEditor();
-      }
-    });
+    this.commandRegistry.onInterrupt();
   }
 
   public saveRegister(arg: string): void {