Remove some debouncing

Author: pokey

packages/cursorless-engine/src/scopeProviders/ScopeRangeWatcher.ts

@@ -20,13 +20,18 @@ import { ScopeRangeProvider } from "./ScopeRangeProvider";
  */
 export class ScopeRangeWatcher {
   private disposables: Disposable[] = [];
-  private debouncer = new Debouncer(() => this.onChange());
+  private debouncer = new Debouncer(() => this.onChange);
   private listeners: (() => void)[] = [];
 
   constructor(
     languageDefinitions: LanguageDefinitions,
     private scopeRangeProvider: ScopeRangeProvider,
   ) {
+    this.onChange = this.onChange.bind(this);
+    this.onDidChangeScopeRanges = this.onDidChangeScopeRanges.bind(this);
+    this.onDidChangeIterationScopeRanges =
+      this.onDidChangeIterationScopeRanges.bind(this);
+
     this.disposables.push(
       // An Event which fires when the array of visible editors has changed.
       ide().onDidChangeVisibleTextEditors(this.debouncer.run),
@@ -39,13 +44,9 @@ export class ScopeRangeWatcher {
       // dirty-state changes.
       ide().onDidChangeTextDocument(this.debouncer.run),
       ide().onDidChangeTextEditorVisibleRanges(this.debouncer.run),
-      languageDefinitions.onDidChangeDefinition(this.debouncer.run),
+      languageDefinitions.onDidChangeDefinition(this.onChange),
       this.debouncer,
     );
-
-    this.onDidChangeScopeRanges = this.onDidChangeScopeRanges.bind(this);
-    this.onDidChangeIterationScopeRanges =
-      this.onDidChangeIterationScopeRanges.bind(this);
   }
 
   /**

packages/cursorless-engine/src/scopeProviders/ScopeSupportWatcher.ts

@@ -19,14 +19,17 @@ import { ScopeSupportChecker } from "./ScopeSupportChecker";
  */
 export class ScopeSupportWatcher {
   private disposables: Disposable[] = [];
-  private debouncer = new Debouncer(() => this.onChange());
+  private debouncer = new Debouncer(() => this.onChange);
   private listeners: ScopeSupportEventCallback[] = [];
 
   constructor(
     languageDefinitions: LanguageDefinitions,
     private scopeSupportChecker: ScopeSupportChecker,
     private scopeInfoProvider: ScopeInfoProvider,
   ) {
+    this.onChange = this.onChange.bind(this);
+    this.onDidChangeScopeSupport = this.onDidChangeScopeSupport.bind(this);
+
     this.disposables.push(
       // An event that fires when a text document opens
       ide().onDidOpenTextDocument(this.debouncer.run),
@@ -39,11 +42,9 @@ export class ScopeSupportWatcher {
       // dirty-state changes.
       ide().onDidChangeTextDocument(this.debouncer.run),
       languageDefinitions.onDidChangeDefinition(this.debouncer.run),
-      this.scopeInfoProvider.onDidChangeScopeInfo(() => this.onChange()),
+      this.scopeInfoProvider.onDidChangeScopeInfo(this.onChange),
       this.debouncer,
     );
-
-    this.onDidChangeScopeSupport = this.onDidChangeScopeSupport.bind(this);
   }
 
   /**