perf: remove unneeded nullish check
authorJérôme Benoit <jerome.benoit@piment-noir.org>
Wed, 18 Oct 2023 15:32:55 +0000 (17:32 +0200)
committerJérôme Benoit <jerome.benoit@piment-noir.org>
Wed, 18 Oct 2023 15:32:55 +0000 (17:32 +0200)
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
src/pools/abstract-pool.ts
src/worker/abstract-worker.ts

index 645a31e1b1edeed626d47d5d31a2bab23dbeaa83..ef5ebeadd6ad08e3ce6a37eb04aa425280a618af 100644 (file)
@@ -495,10 +495,7 @@ export abstract class AbstractPool<
   private checkMessageWorkerId (message: MessageValue<Data | Response>): void {
     if (message.workerId == null) {
       throw new Error('Worker message received without worker id')
-    } else if (
-      message.workerId != null &&
-      this.getWorkerNodeKeyByWorkerId(message.workerId) === -1
-    ) {
+    } else if (this.getWorkerNodeKeyByWorkerId(message.workerId) === -1) {
       throw new Error(
         `Worker message received from unknown worker '${message.workerId}'`
       )
index e10521fcbc7808eeffeb778aa8dfb13ce0eb680e..431354677c17b4da4f21035ba56b2e09ff98b34e 100644 (file)
@@ -405,7 +405,7 @@ export abstract class AbstractWorker<
   private checkMessageWorkerId (message: MessageValue<Data>): void {
     if (message.workerId == null) {
       throw new Error('Message worker id is not set')
-    } else if (message.workerId != null && message.workerId !== this.id) {
+    } else if (message.workerId !== this.id) {
       throw new Error(
         `Message worker id ${message.workerId} does not match the worker id ${this.id}`
       )