Remove unused callback from writePacket

Author: retrixe

src/minecraft/connection/index.ts

@@ -28,11 +28,7 @@ export interface ServerConnection extends events.EventEmitter {
   msgSalt?: Buffer
   disconnectReason?: string
 
-  writePacket: (
-    packetId: number,
-    data: Buffer,
-    cb?: ((err?: Error | undefined) => void) | undefined
-  ) => Promise<boolean>
+  writePacket: (packetId: number, data: Buffer) => Promise<boolean>
 
   close: () => void
 

src/minecraft/connection/javascript.ts

@@ -55,17 +55,14 @@ export class JavaScriptServerConnection
     this.options = options
   }
 
-  async writePacket(
-    packetId: number,
-    data: Buffer,
-    cb?: ((err?: Error | undefined) => void) | undefined
-  ): Promise<boolean> {
+  async writePacket(packetId: number, data: Buffer): Promise<boolean> {
     const compressionThreshold = this.compressionThreshold
     const packet = this.compressionEnabled
       ? await makeBaseCompressedPacket(compressionThreshold, packetId, data)
       : makeBasePacket(packetId, data)
     const toWrite = this.aesCipher ? this.aesCipher.update(packet) : packet
-    return this.socket.write(toWrite, cb)
+    // Technically, this completes when cb is called, but we don't care.
+    return this.socket.write(toWrite)
   }
 
   onlyOneCloseCall = false