fix: Make sure "event" can be passed around. (#309)

Author: Jason3S

src/vscode/EventEmitter.test.ts

@@ -7,8 +7,9 @@ describe('EventEmitter', () => {
         const emitter = new EventEmitter<string>();
         const listener = jest.fn();
         const listener2 = jest.fn();
+        const event = emitter.event;
         emitter.event(listener);
-        emitter.event(listener2);
+        event(listener2);
         emitter.fire('foo');
         expect(listener).toHaveBeenCalledWith('foo');
         expect(listener2).toHaveBeenCalledWith('foo');

src/vscode/EventEmitter.ts

@@ -12,7 +12,7 @@ export class EventEmitter<T> implements vscode.EventEmitter<T> {
     /**
      * The event listeners can subscribe to.
      */
-    event(listener: (e: T) => any, thisArgs?: any, disposables?: Disposable[]): Disposable {
+    event = (listener: (e: T) => any, thisArgs?: any, disposables?: Disposable[]): Disposable => {
         const fn = thisArgs ? listener.bind(thisArgs) : listener;
         this.#listeners.add(fn);
         const disposable = {
@@ -26,30 +26,30 @@ export class EventEmitter<T> implements vscode.EventEmitter<T> {
         }
 
         return disposable;
-    }
+    };
 
     /**
      * Notify all subscribers of the {@link EventEmitter.event event}. Failure
      * of one or more listener will not fail this function call.
      *
      * @param data The event object.
      */
-    fire(data: T): void {
+    fire = (data: T): void => {
         for (const listener of this.#listeners) {
             try {
                 listener(data);
             } catch {
                 // ignore
             }
         }
-    }
+    };
 
     /**
      * Dispose this object and free resources.
      */
-    dispose(): void {
+    dispose = (): void => {
         this.#listeners.clear();
-    }
+    };
 
     constructor() {}
 }