From 6b9e2042c37e3a8fce251586dcff71859ca057a6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=A0imon=20Brandner?= <simon.bra.ag@gmail.com>
Date: Tue, 1 Jun 2021 16:28:57 +0200
Subject: [PATCH] Use a Set instead of an Array
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
---
 src/components/structures/CallEventGrouper.ts | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/components/structures/CallEventGrouper.ts b/src/components/structures/CallEventGrouper.ts
index 339b9359c2..267f8edacf 100644
--- a/src/components/structures/CallEventGrouper.ts
+++ b/src/components/structures/CallEventGrouper.ts
@@ -39,7 +39,7 @@ export enum CustomCallState {
 }
 
 export default class CallEventGrouper extends EventEmitter {
-    events: Array<MatrixEvent> = [];
+    events: Set<MatrixEvent> = new Set<MatrixEvent>();
     call: MatrixCall;
     state: CallState | CustomCallState;
 
@@ -50,7 +50,7 @@ export default class CallEventGrouper extends EventEmitter {
     }
 
     private get invite(): MatrixEvent {
-        return this.events.find((event) => event.getType() === EventType.CallInvite);
+        return [...this.events].find((event) => event.getType() === EventType.CallInvite);
     }
 
     public answerCall = () => {
@@ -65,7 +65,7 @@ export default class CallEventGrouper extends EventEmitter {
         defaultDispatcher.dispatch({
             action: 'place_call',
             type: this.isVoice ? CallType.Voice : CallType.Video,
-            room_id: this.events[0]?.getRoomId(),
+            room_id: [...this.events][0]?.getRoomId(),
         });
     }
 
@@ -89,14 +89,14 @@ export default class CallEventGrouper extends EventEmitter {
     }
 
     public getHangupReason(): string | null {
-        return this.events.find((event) => event.getType() === EventType.CallHangup)?.getContent()?.reason;
+        return [...this.events].find((event) => event.getType() === EventType.CallHangup)?.getContent()?.reason;
     }
 
     /**
      * Returns true if there are only events from the other side - we missed the call
      */
     private wasThisCallMissed(): boolean {
-        return !this.events.some((event) => event.sender?.userId === MatrixClientPeg.get().getUserId());
+        return ![...this.events].some((event) => event.sender?.userId === MatrixClientPeg.get().getUserId());
     }
 
     private setCallListeners() {
@@ -108,7 +108,7 @@ export default class CallEventGrouper extends EventEmitter {
         if (SUPPORTED_STATES.includes(this.call?.state)) {
             this.state = this.call.state;
         } else {
-            const lastEvent = this.events[this.events.length - 1];
+            const lastEvent = [...this.events][this.events.size - 1];
             const lastEventType = lastEvent.getType();
 
             if (this.wasThisCallMissed()) this.state = CustomCallState.Missed;
@@ -120,7 +120,7 @@ export default class CallEventGrouper extends EventEmitter {
     }
 
     private setCall = () => {
-        const callId = this.events[0].getContent().call_id;
+        const callId = [...this.events][0].getContent().call_id;
         if (!this.call) {
             this.call = CallHandler.sharedInstance().getCallById(callId);
             this.setCallListeners();
@@ -129,7 +129,7 @@ export default class CallEventGrouper extends EventEmitter {
     }
 
     public add(event: MatrixEvent) {
-        this.events.push(event);
+        this.events.add(event);
         this.setState();
     }
 }